https://dev.entrouvert.org/https://dev.entrouvert.org/favicon.ico?15861920342019-10-08T13:29:48ZRedmine Entr’ouvertAuthentic 2 - Development #36783: Remonter toutes les OUs liés aux utilisateurs ayant accès à un servicehttps://dev.entrouvert.org/issues/36783?journal_id=1956292019-10-08T13:29:48ZBenjamin Dauvergne
<ul><li><strong>Assigné à</strong> mis à <i>Benjamin Dauvergne</i></li></ul> Authentic 2 - Development #36783: Remonter toutes les OUs liés aux utilisateurs ayant accès à un servicehttps://dev.entrouvert.org/issues/36783?journal_id=1956332019-10-08T13:32:24ZBenjamin Dauvergne
<ul></ul><p>Ça annule et remplace <a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Development: Login, ajouter un setting pour pré-selectionner l'OU du service appelant (Fermé)" href="https://dev.entrouvert.org/issues/35213">#35213</a>.</p> Authentic 2 - Development #36783: Remonter toutes les OUs liés aux utilisateurs ayant accès à un servicehttps://dev.entrouvert.org/issues/36783?journal_id=1962432019-10-10T17:27:16ZBenjamin Dauvergne
<ul><li><strong>Fichier</strong> <a href="/attachments/38048">0001-authenticators-add-easy-accesible-OU-based-on-servic.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/38048/0001-authenticators-add-easy-accesible-OU-based-on-servic.patch">0001-authenticators-add-easy-accesible-OU-based-on-servic.patch</a> ajouté</li><li><strong>Statut</strong> changé de <i>Nouveau</i> à <i>Solution proposée</i></li><li><strong>Patch proposed</strong> changé de <i>Non</i> à <i>Oui</i></li></ul><p>J'en ai profité pour corriger le placement du choix vide qui se retrouvait coincé entre les OUs préférées et les autres.</p>
<p>Le comportement est désormais : par défaut mettre en avant les dernières OUs sélectionnées, puis les OU adaptées au service, si pas de dernière OU sélectionnée c'est la première OU adaptée qui est pré-sélectionnée.</p> Authentic 2 - Development #36783: Remonter toutes les OUs liés aux utilisateurs ayant accès à un servicehttps://dev.entrouvert.org/issues/36783?journal_id=2046032019-12-04T10:53:35ZPaul Marillonnet
<ul><li><strong>Statut</strong> changé de <i>Solution proposée</i> à <i>Solution validée</i></li></ul><p>Dans <code>get_service_ous</code>:<br /><pre><code class="python syntaxhl"> <span class="k">for</span> <span class="n">ou_id</span><span class="p">,</span> <span class="n">count</span> <span class="ow">in</span> <span class="n">qs</span><span class="p">:</span>
<span class="k">if</span> <span class="ow">not</span> <span class="n">ou_id</span><span class="p">:</span>
<span class="k">continue</span>
</code></pre></p>
<p>Question d'affichage je pense qu'on peut aussi <code>continuer</code> pour les OUs pour lesquelles <code>count</code> vaut zéro.</p>
<p>Pour le reste c'est bon pour moi.</p> Authentic 2 - Development #36783: Remonter toutes les OUs liés aux utilisateurs ayant accès à un servicehttps://dev.entrouvert.org/issues/36783?journal_id=2048782019-12-05T11:35:59ZBenjamin Dauvergne
<ul></ul><p>Paul Marillonnet a écrit :</p>
<blockquote>
<p>Question d'affichage je pense qu'on peut aussi <code>continuer</code> pour les OUs pour lesquelles <code>count</code> vaut zéro.</p>
</blockquote>
<p>Ce n'est pas possible, si ou_id est non NULL alors count vaut au moins 1, sinon il ne serait pas dans le résultat du GROUP BY, sur un values+annotate(x=aggregate(y)), Django fait implicitement un GROUPBY. (en gros ça fait <pre>SELECT ou_id, COUNT(ou_id) FROM auth_user GROUP BY ou_id</pre>).</p>
<blockquote>
<p>Pour le reste c'est bon pour moi.</p>
</blockquote>
<p>Ok.</p> Authentic 2 - Development #36783: Remonter toutes les OUs liés aux utilisateurs ayant accès à un servicehttps://dev.entrouvert.org/issues/36783?journal_id=2048792019-12-05T11:36:13ZBenjamin Dauvergne
<ul><li><strong>Statut</strong> changé de <i>Solution validée</i> à <i>Résolu (à déployer)</i></li></ul><pre>commit 94f4ec8c7d0abb44a3869681f95473014dabbb85
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date: Tue Oct 8 16:46:34 2019 +0200
authenticators: add easy accesible OU based on service's ACL (#36783)
It replaces changes from #35213. OU are added after OU remembered
through cookies; they are ordered based on their user subset's count
(how many of their users can access the targeted service).</pre> Authentic 2 - Development #36783: Remonter toutes les OUs liés aux utilisateurs ayant accès à un servicehttps://dev.entrouvert.org/issues/36783?journal_id=2049252019-12-05T14:15:10ZFrédéric Pétersfpeters@entrouvert.com
<ul><li><strong>Statut</strong> changé de <i>Résolu (à déployer)</i> à <i>Solution déployée</i></li></ul>