<html>
<head>
<meta content="text/html; charset=windows-1252"
http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<div class="moz-cite-prefix">On 29/01/15 17:10, Gabe Alford wrote:<br>
</div>
<blockquote
cite="mid:CAGLxfGwQ6yfD=Am2msQV71uvji=GgAF5NAOKZCh46sMet-sHgg@mail.gmail.com"
type="cite">
<div dir="ltr">
<div>
<div>Hello,<br>
<br>
</div>
Fix for <a moz-do-not-send="true"
href="https://fedorahosted.org/freeipa/ticket/4872">https://fedorahosted.org/freeipa/ticket/4872</a><br>
<br>
</div>
Thanks,<br>
<br>
Gabe<br>
</div>
<br>
<fieldset class="mimeAttachmentHeader"></fieldset>
<br>
<pre wrap="">_______________________________________________
Freeipa-devel mailing list
<a class="moz-txt-link-abbreviated" href="mailto:Freeipa-devel@redhat.com">Freeipa-devel@redhat.com</a>
<a class="moz-txt-link-freetext" href="https://www.redhat.com/mailman/listinfo/freeipa-devel">https://www.redhat.com/mailman/listinfo/freeipa-devel</a></pre>
</blockquote>
Thank you for your patch.<br>
<br>
IMO, would be better to use flag, alwaysask for ipapermright,
instead of creating new callback:<br>
<br>
StrEnum(<br>
'ipapermright*',<br>
cli_name='right',<br>
deprecated_cli_aliases={'permissions'},<br>
label=_('Granted rights'),<br>
doc=_('Rights to grant '<br>
'(read, search, compare, write, add, delete,
all)'),<br>
values=(u'read', u'search', u'compare',<br>
u'write', u'add', u'delete', u'all'),<br>
+ alwaysask=True,<br>
),<br>
<br>
This change requires to generate new API.txt<br>
<br>
please run ./makeapi and increment API version in VERSION file.<br>
<br>
Thank you in advance :-)<br>
<br>
Martin^2<br>
<pre class="moz-signature" cols="72">--
Martin Basti</pre>
</body>
</html>