On 6.11.2013 13:28, Petr Viktorin wrote:
Hello Honza,
This is a simple enough patch, but I'd like you to check if it's
consistent with your vision of the framework.


I used self._raw here deliberately, so that calling repr() on an LDAPEntry does not change its internal state.

I agree that using self._raw alone is insufficient, but I'd like to keep the "no changes" behavior, perhaps using something like this:

    data = dict(self._raw)
    data.update(self._nice)
    return '%s(%r, %r)' % (type(self).__name__, self._dn, data)

Honza

--
Jan Cholasta

_______________________________________________
Freeipa-devel mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/freeipa-devel

Reply via email to