diff options
author | Pavel Zuna <pzuna@redhat.com> | 2010-03-24 11:56:39 +0100 |
---|---|---|
committer | Rob Crittenden <rcritten@redhat.com> | 2010-03-26 16:56:47 -0400 |
commit | c7a35f95c5f1835c131797124f95f22968fbf8d8 (patch) | |
tree | e446562eb867a2c14fcc5ac76a5a1fe6c15c2047 /ipalib/cli.py | |
parent | 4a61ff681c73f7994d885ee2638d8378c11931c1 (diff) | |
download | freeipa-c7a35f95c5f1835c131797124f95f22968fbf8d8.tar.gz freeipa-c7a35f95c5f1835c131797124f95f22968fbf8d8.tar.xz freeipa-c7a35f95c5f1835c131797124f95f22968fbf8d8.zip |
Fix output for commands that do not return entries.
I also changed the default value of the print_all argument in
textui.print_entry from False to True. It think it makes more sense this
way, because:
1) if order is None, it will still print something
2) if order is not None, it will print what's in order first and then the
rest
3) commands that care about the print_all argument have to set it in any
case, those that don't care usually want to print everything
Diffstat (limited to 'ipalib/cli.py')
-rw-r--r-- | ipalib/cli.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/ipalib/cli.py b/ipalib/cli.py index db4a08e85..81269cd45 100644 --- a/ipalib/cli.py +++ b/ipalib/cli.py @@ -315,7 +315,7 @@ class textui(backend.Backend): for attr in sorted(entry): print_attr(attr) - def print_entries(self, entries, order=None, labels=None, print_all=False, format='%s: %s', indent=1): + def print_entries(self, entries, order=None, labels=None, print_all=True, format='%s: %s', indent=1): assert isinstance(entries, (list, tuple)) first = True for entry in entries: @@ -324,7 +324,7 @@ class textui(backend.Backend): first = False self.print_entry(entry, order, labels, print_all, format, indent) - def print_entry(self, entry, order=None, labels=None, print_all=False, format='%s: %s', indent=1): + def print_entry(self, entry, order=None, labels=None, print_all=True, format='%s: %s', indent=1): """ """ if isinstance(entry, (list, tuple)): |