Produits Entr'ouvert » Authentic 2

Frédéric Péters a écrit :

Il me semble que le patch attaché est en fait totalement intégré dans l'autre (#32934) et le patch qui pouvait concerner directement ce ticket, 0002-forms-replaces-RadioSelect-by-Select-of-locked-attri, supprimée; peut-être que l'objectif était l'inverse, supprimer 0001 et garder 0002 ici.

Le 0002, il se cherrypicke sans problème (cfbdd97c91771e402565796bb5b89a1f6d153ba4), le revoilà.

Ouaip mauvais git submit le premier patch c'était celui de #32934, c'est tout comme Fred il dit.

merci :)

Je vois bien que les boutons radio sont transformés en liste quand le champ est passé à "vérifié" et que l'on recharge la page d'édition du profil, cependant j'ai l'impression que dans un cas comme dans l'autre on peut encore modifier la sélection.
(je lance le test avec un point d'arrêt puis je colle le contenu de la réponse dans une page html statique)

Sergei me dis que 'readonly="readonly"' n'a jamais fonctionné que sur les champs texte.
J'ai l'impression que dans les 2 cas on peut verrouiller la sélection en utilisant la "nouvelle" syntaxe :

<<<
readonly="readonly" 
---
disabled="disabled" 
>>>

Sur cette partie-ci supprimée :

        # make sure verified fields are not modified
        for av in models.AttributeValue.objects.with_owner(
                self.instance).filter(verified=True):
            self.cleaned_data[av.attribute.name] = av.to_python()

pour me rassurer sur l'impossibilité qu'un POST altéré touche les données, je propose ce patch supplémentaire sur les tests :

--- a/tests/test_profile.py
+++ b/tests/test_profile.py
@@ -51,7 +51,7 @@ def test_account_edit_view(app, simple_user):
     resp = resp.form.submit().follow()
     assert phone.get_value(simple_user) == '0123456789'

-    resp = app.get(url, status=200)
+    resp = resp1 = app.get(url, status=200)
     resp.form.set('edit-profile-phone', '9876543210')
     resp = resp.form.submit('cancel').follow()
     assert phone.get_value(simple_user) == '0123456789'
@@ -74,6 +74,11 @@ def test_account_edit_view(app, simple_user):
     assert title.get_value(simple_user) == 'Mr'
     assert agreement.get_value(simple_user) is True

+    # check submitting modified data for a locked field doesn't change it
+    resp1.form.set('edit-profile-phone', '9876543210')
+    resp = resp1.form.submit().follow()
+    assert phone.get_value(simple_user) == '0123456789'
+
     phone.disabled = True
     phone.save()
     resp = app.get(url, status=200)

Nicolas Roche a écrit :

A propos de l'intégration du patch ci-dessus,
j'ai l'impression que tu oublies de tenter de modifier la valeur quand tu re-soumets le formulaire :
[...]

old_resp.form conserve les valeurs définie au premier usage, donc je ne pense pas qu'il y ait un souci.

Yep, en effet.

(Pdb) old_resp.form['edit-profile-phone']._value
'1234'