https://dev.entrouvert.org/https://dev.entrouvert.org/favicon.ico?15861920342022-09-29T08:11:04ZRedmine Entr’ouvertw.c.s. - Bug #69699: Vue de traitement, vues personnalisées 'user' et 'all' avec le même slughttps://dev.entrouvert.org/issues/69699?journal_id=4106542022-09-29T08:11:04ZLauréline Guérin
<ul><li><strong>Lié à</strong> <i><a class="issue tracker-2 status-5 priority-4 priority-default closed" href="/issues/68324">Development #68324</a>: SQL: requête sur custom_views exécutée énormément</i> ajouté</li></ul> w.c.s. - Bug #69699: Vue de traitement, vues personnalisées 'user' et 'all' avec le même slughttps://dev.entrouvert.org/issues/69699?journal_id=4106582022-09-29T08:11:23ZLauréline Guérin
<ul><li><strong>Tracker</strong> changé de <i>Development</i> à <i>Bug</i></li></ul> w.c.s. - Bug #69699: Vue de traitement, vues personnalisées 'user' et 'all' avec le même slughttps://dev.entrouvert.org/issues/69699?journal_id=4106612022-09-29T08:12:56ZA. Berriot
<ul></ul><p>Côté DB :</p>
<pre>
ssh wcs.node1.hds.saas.entrouvert
sudo -u wcs wcs-manage shell -d <tenant>
# python shell
from wcs.sql import CustomView
In [21]: CustomView.get_by_slug('toutes')
Out[21]: <CustomView id:349>
In [22]: CustomView.get_by_slug('user-toutes')
None
</pre>
<p>Pour autant que je puisse en juger, il n'y a rien en base pour cette vue.</p> w.c.s. - Bug #69699: Vue de traitement, vues personnalisées 'user' et 'all' avec le même slughttps://dev.entrouvert.org/issues/69699?journal_id=4106632022-09-29T08:16:40ZLauréline Guérin
<ul></ul><p>Le slug en base c'est 'toutes', mais pour la vue user, avec le flag visibility='user'.<br />La méthode get_by_slug renvoie le premier résultat si plusieurs résultats sont trouvés.</p> w.c.s. - Bug #69699: Vue de traitement, vues personnalisées 'user' et 'all' avec le même slughttps://dev.entrouvert.org/issues/69699?journal_id=4106642022-09-29T08:19:02ZA. Berriot
<ul></ul><p>Ok, c'est clair merci !</p> w.c.s. - Bug #69699: Vue de traitement, vues personnalisées 'user' et 'all' avec le même slughttps://dev.entrouvert.org/issues/69699?journal_id=4106772022-09-29T08:59:25ZPierre Ducroquet
<ul></ul><p>Donc il faudrait faire un <code>ORDER BY visibility = 'owner' DESC</code> pour mettre en priorité les vues de l'utilisateur ?</p> w.c.s. - Bug #69699: Vue de traitement, vues personnalisées 'user' et 'all' avec le même slughttps://dev.entrouvert.org/issues/69699?journal_id=4106782022-09-29T09:01:46ZLauréline Guérin
<ul></ul><p>ou corriger le code :)</p> w.c.s. - Bug #69699: Vue de traitement, vues personnalisées 'user' et 'all' avec le même slughttps://dev.entrouvert.org/issues/69699?journal_id=4115802022-10-04T08:44:41ZLauréline Guérin
<ul></ul><p>Pierre Ducroquet a écrit :</p>
<blockquote>
<p>Donc il faudrait faire un <code>ORDER BY visibility = 'owner' DESC</code> pour mettre en priorité les vues de l'utilisateur ?</p>
</blockquote>
<p>Et ça ne serait pas suffisant: car alors on ne pourrait plus activer une vue en visibility 'any', avec le même slug qu'une vue 'owner'</p> w.c.s. - Bug #69699: Vue de traitement, vues personnalisées 'user' et 'all' avec le même slughttps://dev.entrouvert.org/issues/69699?journal_id=4115812022-10-04T08:48:35ZLauréline Guérin
<ul><li><strong>Fichier</strong> <a href="/attachments/70237">0001-custom-views-fix-access-to-owner-any-views-with-same.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/70237/0001-custom-views-fix-access-to-owner-any-views-with-same.patch">0001-custom-views-fix-access-to-owner-any-views-with-same.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>note: une vue avec visibility != owner ne peut pas avoir un slug commençant par <code>user-</code>:<br /><pre>
def ensure_slug(self):1
...
if base_slug.startswith('user-'):
# prevent a slug starting with user- as it's used in URLs
base_slug = 'userx-' + base_slug[5:]
</pre></p> w.c.s. - Bug #69699: Vue de traitement, vues personnalisées 'user' et 'all' avec le même slughttps://dev.entrouvert.org/issues/69699?journal_id=4115832022-10-04T08:48:52ZLauréline Guérin
<ul><li><strong>Assigné à</strong> mis à <i>Lauréline Guérin</i></li></ul> w.c.s. - Bug #69699: Vue de traitement, vues personnalisées 'user' et 'all' avec le même slughttps://dev.entrouvert.org/issues/69699?journal_id=4149672022-10-14T12:54:27ZFrédéric Pétersfpeters@entrouvert.com
<ul><li><strong>Statut</strong> changé de <i>Solution proposée</i> à <i>Solution validée</i></li></ul> w.c.s. - Bug #69699: Vue de traitement, vues personnalisées 'user' et 'all' avec le même slughttps://dev.entrouvert.org/issues/69699?journal_id=4150202022-10-14T14:08:53ZLauréline Guérin
<ul><li><strong>Statut</strong> changé de <i>Solution validée</i> à <i>Résolu (à déployer)</i></li></ul><pre>
commit dfd2902c66134088cabef28934170eff91f16a6a
Author: Lauréline Guérin <zebuline@entrouvert.com>
Date: Tue Oct 4 10:43:28 2022 +0200
custom views: fix access to owner/any views with same slug (#69699)
</pre> w.c.s. - Bug #69699: Vue de traitement, vues personnalisées 'user' et 'all' avec le même slughttps://dev.entrouvert.org/issues/69699?journal_id=4158272022-10-18T10:14:49ZTransition automatique
<ul><li><strong>Statut</strong> changé de <i>Résolu (à déployer)</i> à <i>Solution déployée</i></li></ul> w.c.s. - Bug #69699: Vue de traitement, vues personnalisées 'user' et 'all' avec le même slughttps://dev.entrouvert.org/issues/69699?journal_id=4311752022-12-18T03:41:59ZTransition automatique
<ul></ul><p>Automatic expiration</p>