<html>
<head>
<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<div class="moz-cite-prefix">On 11/02/15 17:28, Gabe Alford wrote:<br>
</div>
<blockquote
cite="mid:CAGLxfGw2Pj6w8sLXpmwhrAgqKReb296e9W31FKhxLhyj3-yMag@mail.gmail.com"
type="cite">
<div dir="ltr">Oops. My mistake. Corrected patch attached.<br>
<div>
<div class="gmail_extra"><br>
<div class="gmail_quote">On Wed, Feb 11, 2015 at 8:59 AM,
Martin Basti <span dir="ltr"><<a
moz-do-not-send="true" href="mailto:mbasti@redhat.com"
target="_blank">mbasti@redhat.com</a>></span>
wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0
.8ex;border-left:1px #ccc solid;padding-left:1ex">
<div bgcolor="#FFFFFF" text="#000000">
<div>Sorry, alwaysask didnt work.<br>
It was asking for rights during permission-mod.<br>
<br>
I replaced alwaysask with flag "ask_create".<br>
Sorry for late catch.<br>
<br>
Updated patch attached.<br>
<br>
PS: your name+email is missing in commit message, is
it on purpose? And time wasn't correct in previous
patch.
<div>
<div><br>
<br>
<br>
On 11/02/15 15:06, Gabe Alford wrote:<br>
</div>
</div>
</div>
<div>
<div>
<blockquote type="cite">
<div dir="ltr">Good point. I personally was not
aware of all that the API can do. Thanks
Martin^2! Updated patch attached.<br>
<div>
<div class="gmail_extra"><br>
<div class="gmail_quote">On Tue, Feb 10,
2015 at 11:42 AM, Martin Basti <span
dir="ltr"><<a
moz-do-not-send="true"
href="mailto:mbasti@redhat.com"
target="_blank">mbasti@redhat.com</a>></span>
wrote:<br>
<blockquote class="gmail_quote"
style="margin:0px 0px 0px
0.8ex;border-left:1px solid
rgb(204,204,204);padding-left:1ex">
<div bgcolor="#FFFFFF" text="#000000">
<div>
<div>
<div>On 29/01/15 17:10, Gabe
Alford wrote:<br>
</div>
</div>
</div>
<blockquote type="cite">
<div>
<div>
<div dir="ltr">
<div>
<div>Hello,<br>
<br>
</div>
Fix for <a
moz-do-not-send="true"
href="https://fedorahosted.org/freeipa/ticket/4872"
target="_blank">https://fedorahosted.org/freeipa/ticket/4872</a><br>
<br>
</div>
Thanks,<br>
<br>
Gabe<br>
</div>
<br>
<fieldset></fieldset>
<br>
</div>
</div>
<pre>_______________________________________________
Freeipa-devel mailing list
<a moz-do-not-send="true" href="mailto:Freeipa-devel@redhat.com" target="_blank">Freeipa-devel@redhat.com</a>
<a moz-do-not-send="true" href="https://www.redhat.com/mailman/listinfo/freeipa-devel" target="_blank">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<span><font color="#888888"><br>
<pre cols="72">--
Martin Basti</pre>
</font></span></div>
</blockquote>
</div>
<br>
</div>
</div>
</div>
</blockquote>
<br>
<br>
</div>
</div>
<span><font color="#888888">
<pre cols="72">--
Martin Basti</pre>
</font></span></div>
</blockquote>
</div>
<br>
</div>
</div>
</div>
</blockquote>
Thank you<br>
<br>
ACK<br>
<br>
<pre class="moz-signature" cols="72">--
Martin Basti</pre>
</body>
</html>