[Struts] Identifier le submit cliqué dans l'ActionForm

Salut tout le monde,

Voilà j'ai un problème sur un formulaire qui contient plusieurs submit qui on le même libellé, voici un exemple :

Citation:

---

<html:submit property="action" onclick="return verifEditLibFonction();"><bean:message key="btn.editLibelleFonction"/></html:submit>

<html:submit property="action" onclick="return verifAddFonction();"><bean:message key="btn.addLibelleFonction"/></html:submit>

---

considérons ces 2 submit dans la jsp, et ceci dans le ressourcesproperties :

Code:

---

1 2	btn.editLibelleFonction=Appliquer btn.addLibelleFonction=Appliquer

---

ce que je voudrais faire c'est une verif dans le validate() de l'ActionForm, mais le problème c'est que quelque soit le submit cliqué, ceci renverra la même valeur :

Code:

---

request.getParameter("action")

---

à savoir ca retournera le String "Appliquer" mais ca me dira pas sur quel submit on a cliqué.

Une solution serait de donner des libellés différents à chacuns de mes submit, mais j'ai 3 fois "Ajouter" et 3 fois "Appliquer" alors bon faut presque un dictionnaire des synonymes si je fais ca !

merci bcp

Sinon tu peux utiliser javascript.

Change ton input type="sumbit" en button et sur l'evenement onClick (ou autre ?) tu appelles une fonction javascript qui remplie une variable from (le bouton que tu as cliqué) et qui submit le formulaire.

Ex :

Code:

---

1 2 3 4 5 6 7 8

function submitMonForm(from) { var form = document.candidatureForm; form.action = 'tonAction.do'; form.from = from; form.submit(); }

---

PS : je l'ecris a l'arrache, j'ai pas testé hein ;)

Edit : pas besoin de modifier le bouton submit, tu appelles ta fonction javascript dans le onsubmit de ton formulaire (l'attribut doit exister dans le tag du form) et tu n'es pas non plus obligé de modifier ton action non plus dans ton javascript.

Par contre comme le dis Vanilla il faut ajouter la propriété from dans ton ActionForm ainsi que de le déclarer en hidden dans ta JSP

Je sais pas trop si c'est compatible avec le LookupDispatchAction que j'utilise, parce que le paramètre action doit prendre la valeur de la clé que je donne dans le html:submit...

je peux aussi mettre un champ caché que je rempli avant le submit :

Code:

---

1 2 3

<html:submit property="action" onclick="return verifEditLibFonction();"><bean:message key="btn.editLibelleFonction"/></html:submit>   <html:submit property="action" onclick="return verifAddFonction();"><bean:message key="btn.addLibelleFonction"/></html:submit>

---

et dans chacune de mes fonctions javascript, une fois les vérifs faites, je place une valeur drapeau dans ce champ caché, puis je récupère ce champ avec un request.getParameter() non?

J'ai édité le message d'avant.

ce que j'ai testé entre temps fonctionne, à savoir ca :

mettre le hidden dans le form

Code:

---

<html:hidden property="origineSubmit"/>

---

dans les fonctions javascript qui vérifient la saisie, j'ai ca avant de retourne true :

Code:

---

document.forms[0].origineSubmit.value="ajouterFonction";

---

ou

Code:

---

document.forms[0].origineSubmit.value="editerFonction";

---

puis quand j'arrive dans l'ActionForm je teste comme cela dans le validate() :

Code:

---

1 2 3 4 5 6 7 8

// TRAITEMENT DES FONCTIONS/METIERs if (origineSubmit.equals("editerFonction")){ System.out.println("éditer fonction"); }   if (origineSubmit.equals("ajouterFonction")){ System.out.println("add fonction"); }

---

voilà bon par contre pour éviter de zapper la mise en place du paramètre faut que je mette ma vérif sur chaque submit à la fois dans le onclick et le on keypress sinon si l'utilisateur tape entrée ca ne prend pas en compte l'origine du submit :

Code:

---

1 2 3

<html:submit property="action" onkeypress="return verifEditLibFonction();" onclick="return verifEditLibFonction();"> <bean:message key="btn.editLibelleFonction"/> </html:submit>

---

je craque ca marche pas, si on valide avec entrée et qu'aucun submit n'a le focus, c'est le premier submit de la page qui est envoyé.

si quelqu'un a un tuyaux il me sortirai une grosse épine du pied parce là j'ai plus d'idées !

Oui donc retour au début :)

Utilises un input type="button" avec une methode onclick de type

Code:

---

1 2 3

<html:button onclick="taFonction('ici')"/>

---

et dans ta fonction tu fais ce que j'ai dis plus haut.

le problème c'est que je n'ai pas tout compris dans ton code :

Code:

---

1 2 3 4 5 6

function submitMonForm(from) { var form = document.candidatureForm; form.action = 'tonAction.do'; form.from = from; form.submit(); }

---

ligne 1 : tu recupere le formulaire html dans form
ligne 2 : tu place la valeur 'tonAction.do' dans un champ hidden
ligne 3 : tu place l'origine de l'appel dans un autre champ hidden "from"
et tu submit

ce qui me dérange c'est ca : form.action = 'tonAction.do'; je vois pas trop quoi mettre sachant que moi derrière j'ai une Action de type LookupDispatchAction avec cette methode

Code:

---

1 2 3 4 5 6

protected Map getKeyMethodMap() { Map map = new HashMap(); map.put("btn.editLibelleFonction", "editLibelleFonction"); map.put("btn.addLibelleFonction", "addLibelleFonction"); return map; }

---

qui fait la correspondance entre le submit cliqué et une methode propre à ce submit.

Essaie en remplaçant :

Code:

---

form.action = 'tonAction.do';

---

par :

Code:

---

1 2	form.action = 'tonAction.do?action=Appliquer';

---

non c'est pas bon parce que j'ai plusieurs "Appliquer" dans ressourcesproperties, et l'Action attend une key...

c'est la galère ce truc bizarre que ca ai pas été prévu dans Struts

Bonjour,

pour en revenir au problème de la touche entrée, tu as sans doute une action "par défaut". Pourquoi ne pas adapter le code suivant :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26

// handle an event generated when a user press "enter" from an input form element function handleReturnKeyPressed(evt) { // character code for the key which has been pressed var characterCode;   if (evt.which) { characterCode = evt.which; } else if (evt.keyCode && evt.srcElement) { // IE specific characterCode = evt.keyCode; } else { // unhandled browser return true; }   // ascii 13 is the character code for the 'return' key if (characterCode && characterCode == 13) { var actionElement = document.getElementById('defaultAction'); if (actionElement) { actionElement.click(); } return false; }   return true; }

---

par exemple sur une zone de saisie :

Code:

---

onkeypress="handleReturnKeyPressed(event);"

---

oui, j'y ai pensé, ca résoud le problème, mais il faut se mettre ca sur tous les champs du form et à la main.

Sinon faudrait simplement mettre une fonction javascript qui intercepte toute touche entrée sur la page et qui l'annule, mais bon je trouvais pas ca très propre et surtout si le javascript est désactivé c'est foutu !

Citation:

---

Envoyé par vallica

non c'est pas bon parce que j'ai plusieurs "Appliquer" dans ressourcesproperties, et l'Action attend une key...

---

Je ne pense pas.
Pour moi, la valeur du paramètre passée à l'Action est le nom de la méthode à exécuter, donc la valeur affectée à la clé dans le fichier properties, donc dans ton cas Appliquer.
Avec en plus le champ hidden from que t'as proposé yolepro, ça devrait fonctionner.

oui mais à ce moment là, vu que dans l'Action j'ai ça :

Code:

---

1 2 3 4 5 6 7

<div style="text-align: left;">protected Map getKeyMethodMap() { Map map = new HashMap(); map.put("btn.editLibelleFonction", "editLibelleFonction"); map.put("btn.addLibelleFonction", "addLibelleFonction"); return map; }</div>

---

je dois passer dans le paramètre action des valeurs du style "editLibelleFonction" ou "addLibelleFonction" qui sont les vrais noms des méthodes qu'il faudra executer, parce que si je passe "Appliquer" comment Struts va savoir de quel appliquer on parle?

Citation:

---

Envoyé par vallica

oui, j'y ai pensé, ca résoud le problème, mais il faut se mettre ca sur tous les champs du form et à la main.

---

Tu peux toujours étendre les taglibs et l'ajouter :wink:

Citation:

---

Envoyé par vallica

Sinon faudrait simplement mettre une fonction javascript qui intercepte toute touche entrée sur la page et qui l'annule, mais bon je trouvais pas ca très propre et surtout si le javascript est désactivé c'est foutu !

---

Sincèrement, ton appli elle fonctionne complètement si on désacrive le js ?

Citation:

---

Sincèrement, ton appli elle fonctionne complètement si on désacrive le js ?

---

j'en doute.... prévoir quelques exceptions :lol:

mais c'est justement pour ca que je veux valider en second lieu avec le validate()

Citation:

---

Envoyé par vallica

oui mais à ce moment là, vu que dans l'Action j'ai ça :

Code:

---

1 2 3 4 5 6 7

<div style="text-align: left;">protected Map getKeyMethodMap() { Map map = new HashMap(); map.put("btn.editLibelleFonction", "editLibelleFonction"); map.put("btn.addLibelleFonction", "addLibelleFonction"); return map; }</div>

---

je dois passer dans le paramètre action des valeurs du style "editLibelleFonction" ou "addLibelleFonction" qui sont les vrais noms des méthodes qu'il faudra executer, parce que si je passe "Appliquer" comment Struts va savoir de quel appliquer on parle?

---

Effectivement, ce n'est pas Appliquer mais editLibelleFonction ou addLibelleFonction.
Je croyais que Appliquer était le nom de ta méthode.:oops:
As-tu fait un test ?

non je ne peux pas le faire ce soir mais je vais me débrouiller pour le faire demain.

en attendant merci bcp pour votre aide, et je viendrais vous embeter d'ici peu je pense...

passez une bonne soirée ;)

Citation:

---

Envoyé par c_nvy

Je ne pense pas.
Pour moi, la valeur du paramètre passée à l'Action est le nom de la méthode à exécuter, donc la valeur affectée à la clé dans le fichier properties, donc dans ton cas Appliquer.
Avec en plus le champ hidden from que t'as proposé yolepro, ça devrait fonctionner.

---

En fait, je me suis trompée en disant que c'était la valeur affectée à la clé parce que dans mon cas, cette valeur est égale au nom de la méthode à exécuter, contrairement à toi.
Par contre, il est vrai que la valeur du paramètre passée à l'action est le nom de la méthode.

Il faut donc faire :

Code:

---

1 2	form.action = 'tonAction.do?action=addLibelleFonction';

---

Bonne soirée à toi aussi.;)

Salut tout le monde,

Je viens de me remettre sur ce problème depuis 20mn, et j'ai voulu essayer ce que tu disais c_nvy, mais avant voici ce que j'ai fait :

- j'ai changé via le fichier ressource.properties tous les labels de mes submit, ce qui fait qu'aucun n'a le même label :

Code:

---

1 2 3 4 5 6

btn.editLibelleFonction=Modifier ce libellé fonction btn.addLibelleFonction=Ajouter cette fonction btn.editLibelleService=Modifier ce libellé service btn.addLibelleService=Ajouter ce service btn.editLibelleStatut=Modifier ce libellé statut btn.addLibelleService=Ajouter ce service

---

Et voici ce que je fais dans le validate()

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35

public ActionErrors validate(ActionMapping mapping, HttpServletRequest request) { ActionErrors errors = new ActionErrors();   // VERIFICATION DU FORMULAIRE DES STATUTS CIVIL   if (request.getParameter("action").equals("Ajouter ce statut")){ System.out.println("ajout statut"); }   if (request.getParameter("action").equals("Modifier ce libellé statut")){ System.out.println("modif statut"); }   // VERIFICATION DU FORMULAIRE DES FONCTIONS/METIERS   if (request.getParameter("action").equals("Ajouter cette fonction")){ System.out.println("ajout fct"); }   if (request.getParameter("action").equals("Modifier ce libellé fonction")){ System.out.println("modif fct"); }   // VERIFICATION DU FORMULAIRE DES SERVICES if (request.getParameter("action").equals("Ajouter ce service")){ System.out.println("ajout service"); }   if (request.getParameter("action").equals("Modifier ce libellé service")){ System.out.println("modif serv"); }   errors.add("name", new ActionMessage("error.name.required")); return errors; }

---

Et dans la console j'ai bien les println qui correspondent au submit que j'ai cliqué, ce qui veut dire que qu'à ce stade là, ce n'est pas encore le nom de la méthode qui est passé, mais la value du submit cliqué...

Donc bon en ayant des libellés submit uniques ca passe, j'arrive à trouver d'ou vient le clic, mais c'est pas super quoi :?

Salut vallica ;)
Mea culpa. J'ai vraiment raconté n'importe quoi Vendredi...:oops:
La valeur passée en paramètre est donc le libellé du bouton submit défini dans le fichier properties.
Désolée que ça ne convienne pas.:oops:

et oui, ce n'est qu'après qu'il interroge la map et qu'il fait la correspondance, et à mon avis ca aurait même posé problème à ce niveau là si des boutons avaient le même label !

donc je reste avec cette solution, encore merci à tous ceux qui on donné un coup de main ;)

Tu as essayer en appelant la methode javascript que je t'ai donnée mais en enlevant

form.action = ....

Ton action est déjà renseigné, je t'ai dis que j'avais fait un copié collé a l'arrache.

La seul chose que tu as besoin de savoir en javascript c'est :
- Quel bouton tu as cliqué (d'ou le taMethode(from) )
- rajouter la propriété from dans ton Form et en hidden
- Tu soumets le formulaire en javascript (d'ou le form.submit();).

Le reste on s'en fou,

Il te faut ensuite récupérer le hidden from dans ta méthode validate pour savoir d'ou tu viens.

Edit : C'est quoi ta solution finalement?

Salut,

Non, je n'ai pas essayé ce que tu m'a dit parce que j'etais pas convaincu que Struts allait tout comprendre, c'est à dire que lui pour une LookupDispatchAction, il veut que les submit aient un propriété action qui se remplit du libellé du bouton lors du clic. alors je me disais que si je virais ca, lui ne saurait plus où il en est.

De toute façon, si j'avais gardé des submit avec le même libellé, ca n'aurait pas fonctionné lorsque struts aurait regardé la map de l'Action, il aurait pas su choisir entre 2 méthodes à la clé identique.

Donc la solution qui marche dans mon cas, ca a été de donner un nom différent à chaque submit :

Code:

---

1 2 3 4 5 6

btn.editLibelleFonction=Modifier ce libellé fonction btn.addLibelleFonction=Ajouter cette fonction btn.editLibelleService=Modifier ce libellé service btn.addLibelleService=Ajouter ce service btn.editLibelleStatut=Modifier ce libellé statut btn.addLibelleStatut=Ajouter ce statut

---

ce qui me permet de tester comme suit dans le validate :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29

// VERIFICATION DU FORMULAIRE DES STATUTS CIVIL   if (request.getParameter("action").equals("Ajouter ce statut")){ if (this.libStatutAdd.equals("")) errors.add("statut", new ActionMessage("error.options.statut.libAjoutVide")); }   if (request.getParameter("action").equals("Modifier ce libellé statut")){ if (this.noStatut.equals("0") || this.noStatut.equals("")) errors.add("statut", new ActionMessage("error.options.statut.pasChoisi"));   if (this.libStatutEdit.equals("")) errors.add("statut", new ActionMessage("error.options.statut.libEditVide")); }   // VERIFICATION DU FORMULAIRE DES FONCTIONS/METIERS   if (request.getParameter("action").equals("Ajouter cette fonction")){ if (this.libFonctionAdd.equals("")) errors.add("fct", new ActionMessage("error.options.fct.libFctVide")); }   if (request.getParameter("action").equals("Modifier ce libellé fonction")){ if (this.libFonctionEdit.equals("")) errors.add("fct", new ActionMessage("error.options.fct.libFctVide"));   if (this.codeFonction.equals("") || this.codeFonction.equals("0")) errors.add("fct", new ActionMessage("error.options.fct.pasChoisi")); }

---

et d'executer comme suit dans l'Action :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52

protected Map getKeyMethodMap() { Map map = new HashMap(); map.put("btn.editLibelleFonction", "editLibelleFonction"); map.put("btn.addLibelleFonction", "addLibelleFonction");   map.put("btn.editLibelleService", "editLibelleService"); map.put("btn.addLibelleService", "addLibelleService");   map.put("btn.editLibelleStatut", "editLibelleStatut"); map.put("btn.addLibelleStatut", "addLibelleStatut");   return map; }   /** Action called on Add button click      */ public ActionForward addLibelleFonction(ActionMapping mapping, ActionForm form, HttpServletRequest request, HttpServletResponse response) throws java.lang.Exception { OptionsForm f = (OptionsForm) form; FonctionAgentBean fct = new FonctionAgentBean(); fct.setLibelleFonction(f.getLibFonctionAdd()); fct.setGradeFonction(f.getGradeChoisi2());   ActionMessages messages=new ActionMessages(); if (new FonctionAgentDAO().addFonctionAgent(fct)>0) messages.add("msg",new ActionMessage("msg.options.fct.ajoutOk")); else messages.add("msg",new ActionMessage("msg.options.fct.ajoutEchec"));   this.saveMessages(request,messages);   return mapping.findForward("retourOptions"); }   /** Action called on Edit button click      */ public ActionForward editLibelleFonction(ActionMapping mapping, ActionForm form, HttpServletRequest request, HttpServletResponse response) { OptionsForm f = (OptionsForm) form; FonctionAgentBean fct = new FonctionAgentBean(); fct.setLibelleFonction(f.getLibFonctionEdit()); fct.setLibelleFonction(f.getLibFonctionEdit()); fct.setCodeFonction(f.getCodeFonction());   ActionMessages messages=new ActionMessages(); if (new FonctionAgentDAO().updateFonctionAgent(fct)>0) messages.add("msg",new ActionMessage("msg.options.fct.editOk")); else messages.add("msg",new ActionMessage("msg.options.echec"));   this.saveMessages(request,messages);   return mapping.findForward("retourOptions"); }

---