Jason Gerard DeRose wrote:
On Wed, 2009-11-25 at 12:05 -0500, Rob Crittenden wrote:
This is purely from reading the patch, I haven't applied and tested it yet.

+primary_key = Output('primary_key', unicode,
+    'The primary key of the deleted entry'

This isn't only for deleted entries, right?

Ah, yeah, that should be made more generic.  This doc message is only
used by developers, though.

This import doesn't seem to be used:
from inspect import getdoc

What is dont_output_for_cli()? Is this an effort to make things work while we're in transition?

Yeah, I just renamed some methods so we can reference how they were
implemented.  Temporary.

You seem to have disabled the raw option in LDAPSearch, was that intentional?

Originally I got the impression we weren't going to keep both --raw and
--all, but this can be changed.

Is cli_name being dropped for label? I'm ok with that but should we remove it from all the plugins?

No, here is how they work:

`cli_name` is used for the optparse names and defaults to Param.name,


`label` is a human readable, translatable string.  It's used in the
webUI, and to prompt show entries on cli, like:

  First name: John Doe

`doc` is human readable help passed to optparse.make_option(help=doc).
It default to the value of the label.  It's used like this:

  --uid=INT        UID (use this option to set it manually)

In the above case the `label` is "UID" (not shown) but the `doc` is this
longer string.

The user plugins provide good examples of how I think these should be

I'll submit a patch later documented these different string uses.


We'll also need to determine what we'll do about all the plugins. The cert plugin, for example, isn't ported to this new return value system and blows up in many places.

There are also some labels missing, such as for fqdn in the host plugin.

These are both quite easy to fix, I think we just need to coordinate things. Perhaps if Pavel and I split up the plugins and fix anything that needs fixing and commit all the patches at one time to avoid any period of breakage.


Freeipa-devel mailing list

Reply via email to