https://dev.entrouvert.org/https://dev.entrouvert.org/favicon.ico?15861920342015-12-15T17:14:46ZRedmine Entr’ouvertw.c.s. - Bug #9329: en fin d'edition, pas de saut vers le statut choisihttps://dev.entrouvert.org/issues/9329?journal_id=386552015-12-15T17:14:46ZThomas Noël
<ul></ul><p>en fait, ça n'a jamais marché, ce n'est pas codé</p> w.c.s. - Bug #9329: en fin d'edition, pas de saut vers le statut choisihttps://dev.entrouvert.org/issues/9329?journal_id=386562015-12-15T17:15:56ZFrédéric Pétersfpeters@entrouvert.com
<ul></ul><p>Quel boulet, ce Fred, déjà en 2006.</p> w.c.s. - Bug #9329: en fin d'edition, pas de saut vers le statut choisihttps://dev.entrouvert.org/issues/9329?journal_id=389122015-12-20T11:34:50ZFrédéric Pétersfpeters@entrouvert.com
<ul><li><strong>Fichier</strong> <a href="/attachments/9120">0001-workflows-implement-status-change-after-editable-act.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/9120/0001-workflows-implement-status-change-after-editable-act.patch">0001-workflows-implement-status-change-after-editable-act.patch</a> ajouté</li><li><strong>Statut</strong> changé de <i>Nouveau</i> à <i>En cours</i></li><li><strong>Patch proposed</strong> changé de <i>Non</i> à <i>Oui</i></li></ul><p>Quatre choses à voir dans ce patch :</p>
<p>- mise en commun du code "wfedit" back et front office (vers wcs/forms/common.py)<br />- conservation de l'id de l'action dans l'URL (.../wfedit-XXX)<br />- suppression du passage d'un argument editing, en faveur d'une série d'attributs, plus clairs (on peut considérer le edit_mode comme inutile, dans la mesure où il peut être déduit des autres, mais je trouve les choses plus claires avec)<br />- exécution du saut vers le nouveau statut</p> w.c.s. - Bug #9329: en fin d'edition, pas de saut vers le statut choisihttps://dev.entrouvert.org/issues/9329?journal_id=390452015-12-23T10:02:49ZThomas Noël
<ul></ul><p>Un joli rangement plutôt agréable.</p>
<p>Ack, avec un détail peu important, cette boucle :</p>
<pre>
for item in wf_status.items:
if not isinstance(item, EditableWorkflowStatusItem):
continue
if item.id != action_id:
continue
if not item.check_auth(self.filled, get_request().user):
continue
</pre>
<p>pourrait plutôt être ça je crois :</p>
<pre>
for item in wf_status.items:
if item.id == action_id:
if not isinstance(item, EditableWorkflowStatusItem):
break
if not item.check_auth(self.filled, get_request().user):
break
...
</pre>
<p>Aussi, pour conjurer le sort, dans les tests, les startswith pourraient tester que les URL ont bien un "wfedit-" vers la fin, avec le nouveau trait d'union [genre <code>.startswith('http://example.net/test/%s/wfedit-')</code>]</p> w.c.s. - Bug #9329: en fin d'edition, pas de saut vers le statut choisihttps://dev.entrouvert.org/issues/9329?journal_id=390472015-12-23T10:26:44ZFrédéric Pétersfpeters@entrouvert.com
<ul><li><strong>Fichier</strong> <a href="/attachments/9140">0001-workflows-implement-status-change-after-editable-act.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/9140/0001-workflows-implement-status-change-after-editable-act.patch">0001-workflows-implement-status-change-after-editable-act.patch</a> ajouté</li></ul><p>J'ai plutôt changé en :</p>
<pre>
for item in wf_status.items:
if item.id != action_id:
continue
if not isinstance(item, EditableWorkflowStatusItem):
break
if not item.check_auth(self.filled, get_request().user):
break
</pre> w.c.s. - Bug #9329: en fin d'edition, pas de saut vers le statut choisihttps://dev.entrouvert.org/issues/9329?journal_id=390492015-12-23T11:07:43ZThomas Noël
<ul></ul><p>Et donc, ack</p> w.c.s. - Bug #9329: en fin d'edition, pas de saut vers le statut choisihttps://dev.entrouvert.org/issues/9329?journal_id=390572015-12-23T13:36:47ZFrédéric Pétersfpeters@entrouvert.com
<ul><li><strong>Statut</strong> changé de <i>En cours</i> à <i>Résolu (à déployer)</i></li></ul><pre>
commit 3259e27de0df733d6eb346bb28f1f5af350b9663
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Sun Dec 20 12:29:48 2015 +0100
workflows: implement status change after "editable" action (#9329)
</pre> w.c.s. - Bug #9329: en fin d'edition, pas de saut vers le statut choisihttps://dev.entrouvert.org/issues/9329?journal_id=395852016-01-06T14:39:56ZFrédéric Pétersfpeters@entrouvert.com
<ul><li><strong>Version cible</strong> mis à <i>v1.26</i></li></ul> w.c.s. - Bug #9329: en fin d'edition, pas de saut vers le statut choisihttps://dev.entrouvert.org/issues/9329?journal_id=396622016-01-07T10:34:50ZThomas Noël
<ul><li><strong>Statut</strong> changé de <i>Résolu (à déployer)</i> à <i>Fermé</i></li></ul>