Produits Entr'ouvert » w.c.s.

Garder en plus un attribut caché "manuellement édité", qui serait posé lorsque cette boite de dialogue est utilisée, pour bloquer la mise à jour automatique.

En fait à la reflexion mon problème actuel est un peu différent : j'ai des URL codées par ailleurs (dans combo, mais ça pourrait être un CMS externe). Je ne veux pas que des URLs changent quand je migre un formulaire vers une autre plateforme. Or, un risque existe.

Finalement, je voudrais que l'URL ne change jamais. Sauf si on le demande explicitement.

Finalement, je voudrais que l'URL ne change jamais. Sauf si on le demande explicitement.

Je propose alors une évolution de la boite de changement de titre d'un formulaire, qu'elle devienne :

Titre du formulaire :
 [ Demande d'acte de naissance ......... ]

Identifiant du formulaire :
 [ demande-d-acte-de-naissance ......... ]

Si à l'ouverture de la boite de dialogue, le slug == slugify(titre), alors il serait mis à jour automatiquement selon ce que l'utilisateur met dans le titre. (c'est par exemple important pour qu'après avoir dupliqué un formulaire on ne se trouve pas avec "demande-d-acte-de-naissance-copie" comme slug de la demande d'acte de mariage).

Si des demandes existent, le champ slug serait en lecture seule, avec une note d'information + un bouton pour l'éditer quand même, genre :

 (i) L'identifiant du formulaire ne peut pas être modifié car des demandes existent.
     [autoriser les modifications malgré tout]

(attention comme quoi urlname est aussi utilisé pour le stockage pickle)

0001 :

new_url_name = simplify(self.name, space='-') : le space est '-' par défaut, pas utile de le rappeler.

Il manque un changed=True dans le migrate après self.internal_identifier = self.url_name

Au fait, ce migrate est bien fait aussi lors d'une lecture d'un objet pickle ? J'en suis pas si sûr à cause des ignore_migrations, et je me dis : si un gars veut renommer ou overrider un formdef existant, qui n'aurait donc pas encore de internal_identifier, ça pourrait poser des soucis ? (bien relou à tester). Je dis peut-être un peu n'imp.

Dans wcs/admin/forms.py lors de l'import je pense qu'il faut laisser « formdef.url_name = None # a new one will be set in .store() » (ligne 1555) sinon on pourrait importer un formulaire qui a le même url_name qu'un existant, et boum. (et sans doute ajouter un test pour vérifier ça)

0002:

Au niveau de :

disabled_url_name = bool(self.formdef.data_class().count())

je pense qu'en mode SQL au moins, on pourrait ne pas compter les anonymisés (ni même les brouillons peut-être)... mais ça peut être dans un autre ticket à venir.

(Voilà déjà fournée 1 ; j'ai pas super trop bien tout relu vraiment encore)

Nouvelle relecture, j'ai l'impression que tout est ok. Ca pourrait être poussé, joué sur la dev (y compris branche django), et w.c.s. tagué rapidement pour profiter de 10 jours de recette.