On 07/27/2011 11:13 AM, Rob Crittenden wrote:
Martin Kosek wrote:
On Wed, 2011-07-27 at 10:35 -0400, Rob Crittenden wrote:
Martin Kosek wrote:
On Wed, 2011-07-27 at 11:28 +0200, Martin Kosek wrote:
The summary value was set to invalid primary key - automount map
instead of automount key.

https://fedorahosted.org/freeipa/ticket/1524


The other commands' summary may be wrong in some cases as well. Updated
patch fixes summary for all automountkey commands.

Martin

nack

ipalib/plugins/automount.py:926: [E0602, automountkey_show.execute]
Undefined variable 'key'

I think you meant to set result['value'] = options['automountkey']

rob

Ah, that's embarrassing. That happens when one makes "just one small
change before commit". Attaching fixed patch.

Martin

ack

_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel
Before you push:  was it tested against the UI?

_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

Reply via email to