https://dev.entrouvert.org/https://dev.entrouvert.org/favicon.ico?15861920342011-12-18T23:34:55ZRedmine Entr’ouvertw.c.s. - Development #207: Vérification de la présence de demandes non cloturées à la suppression d'un formulairehttps://dev.entrouvert.org/issues/207?journal_id=28052011-12-18T23:34:55ZThomas Noël
<ul></ul><p>Ok pour vérifier, on peut. Mais que faire ensuite (si la personne clique sur "supprimer le formulaire" et que des formdata sont présents, on lui propose quoi ?)</p> w.c.s. - Development #207: Vérification de la présence de demandes non cloturées à la suppression d'un formulairehttps://dev.entrouvert.org/issues/207?journal_id=45742012-10-01T14:52:40ZVictor Claudetvclaudet@entrouvert.com
<ul><li><strong>Version cible</strong> mis à <i>Au-quotidien 2014.5</i></li></ul> w.c.s. - Development #207: Vérification de la présence de demandes non cloturées à la suppression d'un formulairehttps://dev.entrouvert.org/issues/207?journal_id=157122014-04-08T09:07:42ZThomas Noël
<ul><li><strong>Version cible</strong> <del><i>Au-quotidien 2014.5</i></del> supprimé</li></ul> w.c.s. - Development #207: Vérification de la présence de demandes non cloturées à la suppression d'un formulairehttps://dev.entrouvert.org/issues/207?journal_id=1736932019-05-18T20:07:58ZFrédéric Pétersfpeters@entrouvert.com
<ul><li><strong>Projet</strong> changé de <i>Au quotidien</i> à <i>w.c.s.</i></li></ul> w.c.s. - Development #207: Vérification de la présence de demandes non cloturées à la suppression d'un formulairehttps://dev.entrouvert.org/issues/207?journal_id=2002412019-11-06T22:35:31ZMarie Kuntz
<ul></ul><p>on pourrait alerter et bloquer (empêcher) la suppression</p> w.c.s. - Development #207: Vérification de la présence de demandes non cloturées à la suppression d'un formulairehttps://dev.entrouvert.org/issues/207?journal_id=2502882020-07-24T06:15:21ZFrédéric Pétersfpeters@entrouvert.com
<ul><li><strong>Fichier</strong> <a href="/attachments/46571">0001-admin-don-t-allow-removal-of-form-if-there-are-open-.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/46571/0001-admin-don-t-allow-removal-of-form-if-there-are-open-.patch">0001-admin-don-t-allow-removal-of-form-if-there-are-open-.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> w.c.s. - Development #207: Vérification de la présence de demandes non cloturées à la suppression d'un formulairehttps://dev.entrouvert.org/issues/207?journal_id=2503472020-07-24T09:33:54ZThomas Noël
<ul></ul><p>Quelle est ton idée derrière le check_open_forms_at_deletion=False sur les fiches ?</p> w.c.s. - Development #207: Vérification de la présence de demandes non cloturées à la suppression d'un formulairehttps://dev.entrouvert.org/issues/207?journal_id=2503602020-07-24T10:12:35ZFrédéric Pétersfpeters@entrouvert.com
<ul></ul><p>Que je n'étais pas bien sûr aujourd'hui de vouloir introduire ce comportement de refus de suppression pour les fiches.</p> w.c.s. - Development #207: Vérification de la présence de demandes non cloturées à la suppression d'un formulairehttps://dev.entrouvert.org/issues/207?journal_id=2504092020-07-24T12:55:08ZThomas Noël
<ul></ul><p>Frédéric Péters a écrit :</p>
<blockquote>
<p>Que je n'étais pas bien sûr aujourd'hui de vouloir introduire ce comportement de refus de suppression pour les fiches.</p>
</blockquote>
<p>Je vois dans ce refus, pour ma part, une protection anti-bourdes que j'aime à savoir présente... tu veux pas uniformiser le comportement et on verra si ça gêne vraiment à l'usage ?</p> w.c.s. - Development #207: Vérification de la présence de demandes non cloturées à la suppression d'un formulairehttps://dev.entrouvert.org/issues/207?journal_id=2596622020-09-22T12:52:26ZFrédéric Pétersfpeters@entrouvert.com
<ul><li><strong>Fichier</strong> <a href="/attachments/47989">0001-admin-check-for-contents-before-removal-of-forms-car.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/47989/0001-admin-check-for-contents-before-removal-of-forms-car.patch">0001-admin-check-for-contents-before-removal-of-forms-car.patch</a> ajouté</li></ul><p>Revu ainsi mais ça m'a permis d'imaginer ce qui était réellement mon problème à faire pareil pour les fiches : on interdit la suppression quand il y a des demandes ouvertes, mais pour les fiches on n'a pas vraiment cette notion de "ouvertes". Du coup ce que j'ai fait là c'est pour les fiches, interdire tout bonnement.</p> w.c.s. - Development #207: Vérification de la présence de demandes non cloturées à la suppression d'un formulairehttps://dev.entrouvert.org/issues/207?journal_id=2596922020-09-22T13:57:22ZThomas Noël
<ul></ul><p>Parfait sur le principe.</p>
<p>Mais j'ai l'impression qu'un petit malin pourrait quand même forger une requête avec un submit=blabla qui ferait quand même le delete ? Si oui, ajouter un "if check_count_message: raise AccessForbiddenError(check_count_message)" avant le self.formdef.remove_self() ? (peut-être que je me trompe et qu'un submit bidon arrivera sur un form.has_errors et dans ce cas, c'est validé)</p>
<p>Et au passage, supprimer les commentaires sans objet :<br /><pre>
else:
# XXX: remove form data
#for formdata in FormData.select(FormData.q.formdefID == self.formdef.id):
# formdata.destroySelf()
self.formdef.remove_self()
return redirect('..')
</pre></p> w.c.s. - Development #207: Vérification de la présence de demandes non cloturées à la suppression d'un formulairehttps://dev.entrouvert.org/issues/207?journal_id=2596972020-09-22T14:07:00ZFrédéric Pétersfpeters@entrouvert.com
<ul></ul><p>J'ai poussé en wip/ une branche avec :</p>
<pre>
- if form.get_widget('cancel').parse():
+ if form.get_widget('cancel').parse() or (form.is_submitted() and check_count_message):
</pre> w.c.s. - Development #207: Vérification de la présence de demandes non cloturées à la suppression d'un formulairehttps://dev.entrouvert.org/issues/207?journal_id=2597002020-09-22T14:17:35ZThomas Noël
<ul><li><strong>Statut</strong> changé de <i>Solution proposée</i> à <i>Solution validée</i></li></ul><p>Go (dès que jenkins est vert)</p> w.c.s. - Development #207: Vérification de la présence de demandes non cloturées à la suppression d'un formulairehttps://dev.entrouvert.org/issues/207?journal_id=2597012020-09-22T14:18:27ZFrédéric Pétersfpeters@entrouvert.com
<ul></ul><p>(c'est pour le prochain cycle de toute façon)</p> w.c.s. - Development #207: Vérification de la présence de demandes non cloturées à la suppression d'un formulairehttps://dev.entrouvert.org/issues/207?journal_id=2602772020-09-25T05:18:31ZFrédéric Pétersfpeters@entrouvert.com
<ul><li><strong>Statut</strong> changé de <i>Solution validée</i> à <i>Résolu (à déployer)</i></li></ul><pre>
commit fb1908b2ab589aad77f6d33839bd312047212de6
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Fri Jul 17 14:23:23 2020 +0200
admin: check for contents before removal of forms/cards (#207)
</pre> w.c.s. - Development #207: Vérification de la présence de demandes non cloturées à la suppression d'un formulairehttps://dev.entrouvert.org/issues/207?journal_id=2607312020-09-28T10:16:31ZFré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>