Le name du bouton submit empêche la soumission du formulaire en JS #145

Dans inc-saisies-cvt-boutons.html, le name du bouton de soumission est submit :

<button type="submit" class="submit[(#ENV{_etapes}|oui) submit__suivant]" name="submit" value="1">[(#GET{label_valider}|_T_ou_typo)]</button>

Or ce nom empêche de soumettre le formulaire en JS avec la fonction jQuery .submit(), cf. commentaire sur stack :

The NUMBER ONE error is having ANYTHING with the reserved word submit as ID or NAME in your form.

C'est problématique, Spip s'en sert par exemple pour soumettre le formulaire quand on utilise refuser_traiter_formulaire_ajax() dans le traiter.

Bug constaté via Profils/#10

Et le commit qui a introduit ça est donc
5a9efba9db

(parce que bon, ça marchait bien jusqu'ici...)

Et le plugin est donc buggé depuis la version 3.53.2 ?

Ah ! J'ignorais (mais notez que c'était soumis à PR ces modifs). Du coup il faudrait quoi comme name ?

Avant il n'y avait pas de name, l'ajout de l'attribut répondait à un besoin précis ? C'est pour différencier dans le traiter si éventuellement il y a plusieurs boutons de soumission dans le formulaire (valider, annuler, etc.) ?

Dans cas cas il faudrait un nom signifiant que c'est le bouton de soumission par défaut. J'ai pas trop d'idée : spip_submit, defaut_submit, ou autre.

Remarque faudrait la même chose dans tous les formulaires dans ce cas, faudrait peut-être décider dans le core, puisque nicod_ proposait aussi de remplacer les input par des button : spip/spip#4604

Ou alors plus simplement : pas de name sur le bouton de soumission par défaut.
On en ajoute uniquement sur les autres boutons éventuels.

Pour l'instant je trouve aussi que le bouton de soumission par défaut ne devrait pas avoir de name. Normalement il n'y a pas à le distinguer, c'est l'action qui vaut aussi quand on fait "Entrée" dans un champ. C'est seulement quand on fait une autre action particulière qu'il faut la distinguer.

Oui, le name est là pour distinguer d'autres boutons (notamment des boutons d'etapes). Donc oui, prenons une autre convention (et verifions tout de même si le renommage ne casse pas des choses en multietape)

ah bah j'ai pas vu les remarques de @tcharlss et @rastapopoulos . Moi ca me va a priori de pas avoir de name.

Donc on le vire ? Ca me pete mon formulaire la.

Je veux bien m'en occuper parceque j'ai pas mal de formulaires qui font la tête, faut régler ça rapidement :)

@maieul : il faut vérifier que ça casse rien pour les formulaires par étapes, c'est ça ? Il y aurait d'autres choses à vérifier à part ça ?

@tcharlss non rien d'autre à vérifier.

Bon c'est pas un gros changement, mais PR quand même : !146

(bizarre le diff surligne mal le retrait du name)

J'ai mergé la PR et releasé (v4.0.4 + v3.54.9)
Je ferme.