Produits Entr'ouvert » Combo

On ne peut pas demander aux gens d'écrire du JSON.

Alors une zone texte ou ils saisiront:

nom_champ1:valeur1
nom_champ2:valeur2

Il y a quoi de compliqué à l'idée de proposer de remplir des vrais champs ? Trouver des libellés ?

Les champ différent en fonction du protocole "PESv2" ou "ROLMRE".

Je trouve inutile d'exposer ceux de "ROLMRE" quand on a choisi "PESv2" et vice-versa.

Tu es libre d'ajouter du js.

Entête licence à ajouter en haut des fichiers. (pas allé plus loin que la première ligne du patch).

Pas fan de l'avoir là parce que je compte bien qu'on se dégage les as_p, un jour.

Alors 1), j'allais suggérer de simplement taper ça dans le combo.manager.js général, mais 2) je vois que le nécessaire existe déjà pour que tu puisses simplement préciser un gabarit particulier à ta cellule, via l'attribut manager_form_template sur la classe.

Ça demande quand même sans doute une modification type,

--- a/combo/manager/templates/combo/cell_form.html
+++ b/combo/manager/templates/combo/cell_form.html
@@ -4,7 +4,9 @@
 {% csrf_token %}
 {% if form %}
   <div class="cell-form">
+  {% block cell-edit-form %}
   {{ form.as_p }}
+  {% endblock %}
   </div>
 {% else %}
   <p>{% trans "There are no options for this cell." %}</p>

Pour ne pas devoir dupliquer plein de trucs.

(ou modifier le combo.manager.js, ça reste une option, surtout qu'à continuer dans le patch je vois qu'il est déjà modifié, qu'il y manque juste l'appel à la fonction).

Tant qu'à faire, le code handle_tipi_protocol_fields_display, il pourrait se trouver directement dans le tipi_cell_form.html, et faire directement

var $control_element = $("[name=c{{ cell.get_reference }}-control_protocol]");

(si je lis bien)

Quand je fais enregistrer ça remplace le formulaire sans exécution js et ça change la hauteur et le bouton "enregistrer" n'est plus atteignable.

Parce qu'ensuite c'est alors combo/ajax_cell_form.html qui est appelé, qui fait juste {{form.as_p}}; de là soit on prend la route d'arranger les choses dans combo, mais beaucoup de questions se posent (par exemple, ajouter une possibilité de template parallèle à manager_form_template, qui concernerait que le formulaire ?), soit retour en arrière et taper tout le js dans combo.manager.js, avec dedans la détection automatique des cellules TIPI, qu'il n'y ait pas d'appel explicite à faire.

Façon $('.cell-list').delegate('.cell.tipipaymentcell select', 'click', function() ...

À noter que j'invente ici la présence d'une classe tipipaymentcell, qui n'existe actuellement pas, mais serait un ajout utile. J'ai créé #29201 pour mener ça, je te le prépare tout de suite.

  $('.cell.tipipaymentformcell select').on('change', function() {
        handle_tipi_form($(this));
  });
  $('.cell.tipipaymentformcell select').trigger('change');
  $('.cell.tipipaymentformcell').on('combo:cellform-reloaded', function() {
        $(this).on('change', function() {
            handle_tipi_form($(this).find('select'));
        });
        $(this).find('select').trigger('change');
  });

Je trouve curieux le $(this).on('change', alors qu'ailleurs c'est le select qui est référencé.

Ça marche mais passer de Indigo à ROLMRE déplace le bouton "enregistrer" vers une zone inaccessible.

Je viens de créer #29244 + patch, ça te donnera une fonction compute_max_height à appeler.

Tu peux rebaser maintenant que le compute_... est dans le dépôt ?

Ok, sans doute dernier truc :

+    exer = models.CharField(_('Exer'), max_length=4, blank=True, help_text=_('Default value to be used in form'))
+    idpce = models.CharField(_('IDPCE'), max_length=8, blank=True, help_text=_('Default value to be used in form'))
+    idligne = models.CharField(_('IDLIGNE'), max_length=6, blank=True, help_text=_('Default value to be used in form'))
+    rolrec = models.CharField(_('ROLREC'), max_length=2, blank=True, help_text=_('Default value to be used in form'))
+    roldeb = models.CharField(_('ROLDEB'), max_length=2, blank=True, help_text=_('Default value to be used in form'))
+    roldet = models.CharField(_('ROLDET'), max_length=13, blank=True, help_text=_('Default value to be used in form'))

Un conseil à donner aux traducteurs ?

On ne traduit pas les noms des paramètres. Les connaisseurs TIPI comprendront (ou pas).

Pour le help_text: "Valeur par défaut à utiliser dans le formulaire".

On ne traduit pas les noms des paramètres. Les connaisseurs TIPI comprendront (ou pas).

Alors il ne faut pas marquer pour traduction.