Development #13669
commande suppression des toutes les demandes
0%
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Serghei Mihai il y a plus de 7 ans
- Lié à Autre #13317: Déploiement configurations/données initiales ajouté
Mis à jour par Serghei Mihai il y a plus de 7 ans
- Fichier 0001-empty-forms-command-13669.patch 0001-empty-forms-command-13669.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a plus de 7 ans
Le sujet d'un commit doit être un verbe. Le code dans w.c.s. doit être testé. Il ne faut pas utiliser "forms" comme terme, on pourrait imaginer que ça vide les formdata, alors qu'en fait ça les supprime.
Mis à jour par Serghei Mihai il y a plus de 7 ans
- Fichier 0001-ctl-add-command-to-delete-formdata-13669.patch 0001-ctl-add-command-to-delete-formdata-13669.patch ajouté
Commande renommé en "delete_formdata" et avec un test dans test_formdata.py qui me semble l'endroit approprié pour l'inclure.
Mis à jour par Frédéric Péters il y a plus de 7 ans
Avec ce nom j'imagine la commande comme permettant de supprimer un formdata. Il n'y a aucune espèce d'uniformité au niveau des commandes mais je préfère les tirets aux underscores. Arrêtons-nous sur wipe-data.
Prendre en paramètres les slugs des formdefs à vider. Si pas de slug spécifié, ne rien faire. Sauf si --all est passé.
Mettre les tests dans tests/test_ctl.py.
Aujourd'hui toutes les classes des commandes commencent par Cmd, pas de raison de changer.
formdata = formdef.data_class() C'est bien confusant d'appeler formdata quelque chose qui est en fait la classe. Simplement faire formdef.data_class().wipe().
L'option pour préciser le vhost doit s'appeler --vhost (comme dans export_settings, runscript et trigger_jumps)
publisher.app_dir = os.path.join(publisher.app_dir, ) uh?
Mis à jour par Serghei Mihai il y a plus de 7 ans
- Fichier 0001-ctl-add-command-to-wipe-formdata-13669.patch 0001-ctl-add-command-to-wipe-formdata-13669.patch ajouté
Oops
Mis à jour par Frédéric Péters il y a plus de 7 ans
print >> sys.stderr, 'you must specificy --vhost'
specify.
Mis à jour par Serghei Mihai il y a plus de 7 ans
- Fichier 0001-ctl-add-command-to-wipe-formdata-13669.patch 0001-ctl-add-command-to-wipe-formdata-13669.patch ajouté
Copier-coller tout bête de la commande trigger_jumps: #13758 .
Corrigé.
Mis à jour par Frédéric Péters il y a plus de 7 ans
- Copyright (C) 2005-2010 Entr'ouvert
s/2010/2016/
app_dir = pub.app_dir
Ligne inutile.
Ack après ces deux modifications.
Mis à jour par Serghei Mihai il y a plus de 7 ans
- Statut changé de En cours à Résolu (à déployer)
Fait, Merci.
commit ec729ffc6428a1fd0d29971b4b65da59e4d79706
Author: Serghei Mihai <smihai@entrouvert.com>
Date: Mon Oct 24 16:49:20 2016 +0200
ctl: add command to wipe formdata (#13669)
Mis à jour par Frédéric Péters il y a plus de 7 ans
- Statut changé de Résolu (à déployer) à Fermé
ctl: add command to wipe formdata (#13669)