<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 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 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 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 href="mailto:Freeipa-devel@redhat.com" target="_blank">Freeipa-devel@redhat.com</a>
<a 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>