On 19.7.2016 09:15, Martin Babinsky wrote:
On 07/18/2016 08:46 AM, Jan Cholasta wrote:

On 11.7.2016 14:18, Martin Babinsky wrote:

Note that you should use .name rather than .__name__ to get plugin
names, otherwise the code won't work with plugins with non-default names.

There currently aren't any Advice plugins with non-default name, but I
would rather fix this now to avoid surprises later.


I didn't realize this when doing the patch, here's the fix for that.

I have attached the original closed ticket to the commit message, should
I create a new ticket for such a small change?

Bump. I'm fine with new ticket or no ticket, but don't use 6044.

Jan Cholasta

Manage your subscription for the Freeipa-devel mailing list:
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

Reply via email to