Bug #13726
contrib/nancypoll: utiliser ApiError dans
Statut:
Fermé
Priorité:
Normal
Assigné à:
Josué Kouka
Version cible:
-
Début:
24 octobre 2016
Echéance:
% réalisé:
100%
Temps estimé:
Patch proposed:
Oui
Planning:
Description
thomas@zepo ~/dev/passerelle/passerelle/contrib/nancypoll master $ git grep raise
models.py: raise ParameterError('All parameters are required')
models.py: raise ParameterError('Invalid street no value')
models.py: raise FileContentError('No content found')
models.py: raise ObjectDoesNotExist('Polling Station Not Found')
à remplacer par des APIError
Fichiers
Révisions associées
Historique
Mis à jour par Josué Kouka il y a plus de 7 ans
- Fichier 0001-nancy-poll-use-APIError-instead-of-custom-Exceptions.patch 0001-nancy-poll-use-APIError-instead-of-custom-Exceptions.patch ajouté
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a plus de 7 ans
Il faudrait ajouter des assert int(...['err']) != 0 (parce que c'est ça qui compte pour wcs, pas err_desc).
Mis à jour par Josué Kouka il y a plus de 7 ans
Mis à jour par Frédéric Péters il y a plus de 7 ans
J'avais noté int() parce que vraiment, c'est ce que wcs fait, et je ne voudrais pas qu'une régression passe inaperçue ici si jamais "err" se trouvait devenir "0".
Mis à jour par Josué Kouka il y a plus de 7 ans
- Fichier 0001-nancy-poll-use-APIError-instead-of-custom-Exceptions.patch 0001-nancy-poll-use-APIError-instead-of-custom-Exceptions.patch ajouté
Frédéric Péters a écrit :
J'avais noté int() parce que vraiment, c'est ce que wcs fait, et je ne voudrais pas qu'une régression passe inaperçue ici si jamais "err" se trouvait devenir "0".
checked
Mis à jour par Josué Kouka il y a plus de 7 ans
- Statut changé de En cours à Résolu (à déployer)
- % réalisé changé de 0 à 100
Mis à jour par Benjamin Dauvergne il y a presque 6 ans
- Statut changé de Résolu (à déployer) à Fermé
nancy poll: use APIError instead of custom Exceptions (#13726)