https://dev.entrouvert.org/https://dev.entrouvert.org/favicon.ico?15861920342021-02-19T12:54:08ZRedmine Entr’ouvertAuthentic 2 - Development #51294: ldap: move messages from hard coded to templateshttps://dev.entrouvert.org/issues/51294?journal_id=2888642021-02-19T12:54:08ZLoïc Dachary
<ul><li><strong>Fichier</strong> <a href="/attachments/52099">0001-ldap-move-messages-from-hard-coded-to-templates-5129.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/52099/0001-ldap-move-messages-from-hard-coded-to-templates-5129.patch">0001-ldap-move-messages-from-hard-coded-to-templates-5129.patch</a> ajouté</li></ul><p>Voici un patch (incomplet) qui illustre l'approche. Est-ce que vous pensez que ça va dans la bonne direction ?</p> Authentic 2 - Development #51294: ldap: move messages from hard coded to templateshttps://dev.entrouvert.org/issues/51294?journal_id=2888652021-02-19T12:55:25ZLoïc Dachary
<ul><li><strong>Fichier</strong> <a href="/attachments/52100">0001-ldap-move-messages-from-hard-coded-to-templates-5129.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/52100/0001-ldap-move-messages-from-hard-coded-to-templates-5129.patch">0001-ldap-move-messages-from-hard-coded-to-templates-5129.patch</a> ajouté</li></ul><p>Avec le fichier template qui avait été oublié, pour mieux illustrer (ça passe les tests).</p> Authentic 2 - Development #51294: ldap: move messages from hard coded to templateshttps://dev.entrouvert.org/issues/51294?journal_id=2889692021-02-19T17:33:35ZBenjamin Dauvergne
<ul></ul><p>django.contrib.messages n'accepte pas d'HTML, seulement des chaînes simples, donc ici l'utilisation de templates me semble exagéré, tu peux simplement utiliser un nouveau setting <code>custom_error_message</code> contenant un dico dans LDAP_SETTINGS, ex.:</p>
<pre>
LDAP_SETTINGS = [
{
....
'custom_error_messages': {
'accountLocked': ...
}
}
]
</pre>
<p>à terme ça pourra être remplacé par une interface graphique, quand on pourra configurer un annuaire LDAP directement dans /manage/.</p> Authentic 2 - Development #51294: ldap: move messages from hard coded to templateshttps://dev.entrouvert.org/issues/51294?journal_id=2898892021-02-25T11:53:07ZLoïc Dachary
<ul></ul><p>Je vais suivre ton conseil, merci :-) Une question subsidiaire: est-ce que tu me confirmes qu'il est possible d'avoir des chaînes localisables dans les settings ? Par exemple:</p>
<p>```<br />LDAP_SETTINGS = [
{<br /> ....<br /> 'custom_error_messages': {<br /> 'accountLocked': _('tagada {tsoin}'),<br /> }<br /> }<br />]<br />```</p>
<p>J'imagine que oui mais il n'est pas impossible qu'il y ait un problèmes de poule et d'oeuf et qu'il faille lire le fichier de config pour initialiser la localisation et que ce soit en fait pas possible.</p>
<p>Si ce n'est pas possible d'avoir des traductions dans les settings alors on pourrait peut-être avoir des templates textes (.txt comme il y en a quelque uns) histoire de rappeler qu'il ne faut pas mettre de HTML dedans.</p>
<p>Qu'en dis-tu ?</p> Authentic 2 - Development #51294: ldap: move messages from hard coded to templateshttps://dev.entrouvert.org/issues/51294?journal_id=2912002021-03-04T11:31:11ZLoïc Dachary
<ul><li><strong>Fichier</strong> <a href="/attachments/52466">0001-ldap-move-messages-from-hard-coded-to-templates-5129.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/52466/0001-ldap-move-messages-from-hard-coded-to-templates-5129.patch">0001-ldap-move-messages-from-hard-coded-to-templates-5129.patch</a> ajouté</li></ul><p>Alors j'ai fait un joli patch qui sert juste a rien parce que je n'avais pas compris que les settings devaient être sérialisables en JSON. Je recommence.</p> Authentic 2 - Development #51294: ldap: move messages from hard coded to templateshttps://dev.entrouvert.org/issues/51294?journal_id=2912122021-03-04T11:58:49ZLoïc Dachary
<ul><li><strong>Fichier</strong> <a href="/attachments/52467">0001-ldap-move-messages-from-hard-coded-to-templates-5129.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/52467/0001-ldap-move-messages-from-hard-coded-to-templates-5129.patch">0001-ldap-move-messages-from-hard-coded-to-templates-5129.patch</a> ajouté</li></ul><p>Ce patch ajoute <a href="https://dev.entrouvert.org/issues/51294#note-3" class="external">les messages comme suggéré</a> dans LDAP_AUTH_SETTINGS et les tests passent et c'est internationalisé. Mais il y a un soucis pour les messages qui passent par ngettext. Il y a trois possibilités:</p>
<ul>
<li>les messages qui requiert ngettext sont interdits</li>
<li>hacker un truc qui contourne le problème tout en laissant les messages dans LDAP_AUTH_SETTINGS</li>
<li>mettre les messages dans des templates .txt et profiter du fait que les templates implémentent tout les cas comme il faut</li>
</ul>
<p>J'aurais tendance a choisir les template .txt parce que, même si c'est un peu overkill, c'est carré.</p>
<p>Qu'en dites vous ?</p> Authentic 2 - Development #51294: ldap: move messages from hard coded to templateshttps://dev.entrouvert.org/issues/51294?journal_id=2912132021-03-04T11:59:35ZLoïc Dachary
<ul></ul><blockquote>
<p>même si c'est un peu overkill,</p>
</blockquote>
<p>Je voulais dire "exagéré" et pas "overkill", excuse my english :blush:</p> Authentic 2 - Development #51294: ldap: move messages from hard coded to templateshttps://dev.entrouvert.org/issues/51294?journal_id=2912252021-03-04T13:13:48ZFrédéric Pétersfpeters@entrouvert.com
<ul></ul><blockquote>
<p>Qu'en dites vous ?</p>
</blockquote>
<p>Perso plutôt non; ça casserait ou compliquerait à outrance le "à terme ça pourra être remplacé par une interface graphique".</p> Authentic 2 - Development #51294: ldap: move messages from hard coded to templateshttps://dev.entrouvert.org/issues/51294?journal_id=2912442021-03-04T13:46:40ZLoïc Dachary
<ul></ul><p>Ok, merci pour le retour rapide :-) Je vais opter pour supposer que ngettext n'est pas possible. Ca ne va pas être une catastrophe mondiale et ce sera simple.</p> Authentic 2 - Development #51294: ldap: move messages from hard coded to templateshttps://dev.entrouvert.org/issues/51294?journal_id=2912992021-03-04T16:01:49ZLoïc Dachary
<ul><li><strong>Fichier</strong> <a href="/attachments/52481">0001-ldap-move-messages-from-hard-coded-to-settings-51294.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/52481/0001-ldap-move-messages-from-hard-coded-to-settings-51294.patch">0001-ldap-move-messages-from-hard-coded-to-settings-51294.patch</a> ajouté</li></ul><p>Voici un patch qui est prêt à être revu: il déplace simplement les messages dans LDAP_AUTH_SETTINGS et en permet la traduction. Si les messages sont changés dans settings, il faut faire une opération en plus pour ajouter les traductions, bien sur et c'est la responsabilité de l'admin.</p>
<p>Est-ce que ça pourrait aller ou bien il y a des choses que je n'ai pas vues ?</p> Authentic 2 - Development #51294: ldap: move messages from hard coded to templateshttps://dev.entrouvert.org/issues/51294?journal_id=2940372021-03-18T06:54:53ZLoïc Dachary
<ul></ul><p>Ping ?</p> Authentic 2 - Development #51294: ldap: move messages from hard coded to templateshttps://dev.entrouvert.org/issues/51294?journal_id=2971172021-04-01T16:25:30ZLoïc Dachary
<ul></ul><p>Ping ?</p> Authentic 2 - Development #51294: ldap: move messages from hard coded to templateshttps://dev.entrouvert.org/issues/51294?journal_id=2982142021-04-08T11:34:49ZLoïc Dachary
<ul><li><strong>Fichier</strong> <a href="/attachments/53490">0001-ldap-move-messages-from-hard-coded-to-settings-51294.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/53490/0001-ldap-move-messages-from-hard-coded-to-settings-51294.patch">0001-ldap-move-messages-from-hard-coded-to-settings-51294.patch</a> ajouté</li></ul><p>Rebase sur main, avec résolution les conflits qui empêchait le merge.</p> Authentic 2 - Development #51294: ldap: move messages from hard coded to templateshttps://dev.entrouvert.org/issues/51294?journal_id=3039222021-05-04T09:18:25ZValentin Deniaud
<ul><li><strong>Assigné à</strong> changé de <i>Benjamin Dauvergne</i> à <i>Loïc Dachary</i></li></ul><p>Ça m'a l'air bien, mais attendons <a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Development: ldap: add method to get ppolicy operational attributes (Fermé)" href="https://dev.entrouvert.org/issues/51239">#51239</a> qui touche au même code (d'ailleurs on tolère tout à fait que de tels patches se suivent, genre tu aurais pu soumettre un patch fait pour s'appliquer directement sur <a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Development: ldap: add method to get ppolicy operational attributes (Fermé)" href="https://dev.entrouvert.org/issues/51239">#51239</a>).</p>