diff options
author | Petr Viktorin <pviktori@redhat.com> | 2013-11-06 12:40:02 +0100 |
---|---|---|
committer | Petr Viktorin <pviktori@redhat.com> | 2013-11-26 13:19:37 +0100 |
commit | 76c7f24919d30fdd53e4a1cd32880b55c2437ace (patch) | |
tree | 064e420234c9b541f2cb5548ab3fef150c1179f7 | |
parent | f20577ddc4ab40c2365c8abaa703d96019ec4eef (diff) | |
download | freeipa-76c7f24919d30fdd53e4a1cd32880b55c2437ace.tar.gz freeipa-76c7f24919d30fdd53e4a1cd32880b55c2437ace.tar.xz freeipa-76c7f24919d30fdd53e4a1cd32880b55c2437ace.zip |
Improve LDAPEntry.__repr__ for freshly created entries
Creating a LDAPEntry from dict does not set the raw entries,
to display everything we need to combine the underlying data.
https://fedorahosted.org/freeipa/ticket/4015
-rw-r--r-- | ipapython/ipaldap.py | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/ipapython/ipaldap.py b/ipapython/ipaldap.py index 027bfa97..41ae9ec3 100644 --- a/ipapython/ipaldap.py +++ b/ipapython/ipaldap.py @@ -719,7 +719,9 @@ class LDAPEntry(collections.MutableMapping): return self._orig def __repr__(self): - return '%s(%r, %r)' % (type(self).__name__, self._dn, self._raw) + data = dict(self._raw) + data.update((k, v) for k, v in self._nice.iteritems() if v is not None) + return '%s(%r, %r)' % (type(self).__name__, self._dn, data) def copy(self): return LDAPEntry(self) |