Development #39445
Pouvoir cliquer sur "Annuler" sur le formulaire demandant la "dé-liaison" avec France connect
Début:
31 janvier 2020
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
Une fois connecté avec FC, on peut demander la déliaison du compte.
On arrive sur un formulaire qui nous demande de définir un mot de passe pour le compte local s'il a été créé avec France Connect.
Sur ce formulaire il y a un bouton annuler qui ne sert à rien car il vérifie que le mot de passe a été rempli, ce que je ne veux justement pas faire en cliquant sur annuler.
cf capture
Fichiers
Révisions associées
Historique
Mis à jour par Benjamin Dauvergne il y a environ 4 ans
- Statut changé de Nouveau à Information nécessaire
- Assigné à changé de Benjamin Dauvergne à Stéphane Laget
Je n'arrive pas à reproduire avec un test :
diff --git tests/auth_fc/test_auth_fc.py tests/auth_fc/test_auth_fc.py index 06a47860..ddd960f5 100644 --- tests/auth_fc/test_auth_fc.py +++ tests/auth_fc/test_auth_fc.py @@ -148,6 +148,16 @@ def test_login_simple(app, fc_settings, caplog, hooks, exp): # we must be connected assert app.session['_auth_user_id'] assert models.FcAccount.objects.count() == 1 + + # test unlink cancel case + response = app.get('/accounts/') + response = response.click('Delete link') + import pdb + pdb.set_trace() + response = response.form.submit(name='cancel') + response = response.follow() + + # test unlink submit case response = app.get('/accounts/') response = response.click('Delete link') response.form.set('new_password1', 'ikKL1234')
et le code m'a l'air suffisamment simple pour ne pas y voir de bug :
def post(self, request, *args, **kwargs): if 'cancel' in request.POST: return a2_utils.redirect(request, 'account_management') return super(UnlinkView, self).post(request, *args, **kwargs)
Hmm c'est une validation par le navigateur non ?
Mis à jour par Benjamin Dauvergne il y a environ 4 ans
- Assigné à changé de Stéphane Laget à Benjamin Dauvergne
Mis à jour par Benjamin Dauvergne il y a environ 4 ans
- Fichier 0001-auth2_fc-set-novalidate-on-unlink-cancel-button-3944.patch 0001-auth2_fc-set-novalidate-on-unlink-cancel-button-3944.patch ajouté
- Tracker changé de Bug à Development
- Statut changé de Information nécessaire à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a environ 4 ans
- Statut changé de Solution proposée à Solution validée
Ok, mais #39451.
Mis à jour par Serghei Mihai il y a environ 4 ans
- Statut changé de Solution validée à Solution proposée
Ça doit être plutôt formnovalidate="formnovalidate"
, il me semble.
Mis à jour par Frédéric Péters il y a environ 4 ans
- Statut changé de Solution proposée à Solution validée
Non, preuve par les utilisations actuelles,
src/authentic2/templates/authentic2/accounts_delete_request.html: <button class="cancel-button" name="cancel" formnovalidate>{% trans "Cancel" %}</button> src/authentic2/templates/authentic2/accounts_delete_validation.html: <button class="cancel-button" name="cancel" formnovalidate>{% trans "Cancel" %}</button> src/authentic2/templates/authentic2/accounts_edit.html: <button class="cancel-button" name="cancel" formnovalidate>{% trans "Cancel" %}</button> src/authentic2/templates/authentic2/change_email.html: <button class="cancel-button" name="cancel" formnovalidate>{% trans "Cancel" %}</button> src/authentic2/templates/authentic2/login_password_form.html:<button class="cancel-button" name="cancel" formnovalidate>{% trans 'Cancel' %}</button> ...
Mis à jour par Benjamin Dauvergne il y a environ 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 64abfdca0ce2dd5fd09c9d9d112163e6b7d1fa69 Author: Benjamin Dauvergne <bdauvergne@entrouvert.com> Date: Fri Jan 31 12:05:48 2020 +0100 auth2_fc: set novalidate on unlink cancel button (#39445)
Mis à jour par Frédéric Péters il y a environ 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
auth2_fc: set novalidate on unlink cancel button (#39445)