Discussion :

[Le Marlou]

Bonjour
Dans mon application, j'ai des JSP sur lesquelles plusieurs boutons sont disponibles. Par exemple, j'ai une JSP avec les boutons "Enregistrer", "Envoyer" et "Supprimer". Le problème est que le bouton "Enregistrer" doit appeler l'Action EnregistrerAction, le bouton "Envoyer" doit appeler l'Action EnvoyerAction, etc... En gros, il doit y avoir une action pour chaque bouton, car les traitements sont complètement différents. L'ennui est que je ne peux pas utiliser le DispacthAction car tous les traitements des boutons d'une JSP seraient dans la même Action et dans mon cas chaque bouton doit appeler sa propre action (je sais c'est chiant mais c'est pas moi qui décide, c'est pour notamment que chaque développeur bosse sur sa propre action et pas plusieurs sur un même action...)
J'ai bien essayé de changer dynamiquement l'attribut action de mon formulaire comme ceci :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
 
<script language="JavaScript">
function changeToSaveAction(){
   document.docForm.action = "/enregistrerAction";
   document.docForm.submit();
}
</script>
... ... ...
<A href="#" onClick="javascript:changeToSaveAction()">
<img src="enregistrer.gif"></a>

et dans mon struts-config, j'ai bien :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
 
<action path="/enregistrerAction"
type="com.myPackage.actions.EnregistrerAction"
name="docForm"
scope="request" />

Mais ça ne marche pas

Qq'un aurait-il une idée. Merci pour votre aide.

[slopera]

A quel moment cela ne fonctionne pas ?
Lors du chargement de votre form ou lors du clic sur l'image ?

Comment cela ne marche pas ?
Message d'erreur ou submit systématique du form de l'action de départ.

[Sherkhan]

Ce que je trouve bizarre, c'est qu'il n'y a pas d'extension a ton action :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

document.docForm.action = "/enregistrerAction";

Moi, j'aurais plus vu :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

document.docForm.action = "/enregistrerAction.do";

Tu mets ".do", ou ".ce que tu veux" suivant le config de ton fichier web.xml

[slopera]

Non Sherkan,
Pour un formulaire, tu mets simplement le path :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
<html:form action="monAction">
...
</html:form>

Par contre, pour y accéder, il faut faire :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

document.monActionForm.action = ...

Mets tu bien le "bon" nom de formulaire pour y accéder dans ta méthode Javascript.

[Le Marlou]

Que je mette un ".do" ou pas je pense que ça ne changera rien, en fait l'Action n'est pas du tout appelée.
j'ai modifié le script comme ceci :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
 
<script language="JavaScript"> 
function changeToSaveAction(){ 
   alert("XXX");
   document.docForm.action = "/enregistrerAction"; 
   alert("XXX");
   document.docForm.submit(); 
} 
</script>

le premier alert s'affiche mais pas le second ! pourtant la syntaxe est bonne. Si je mets en commentaire la ligne document.docForm.action = "...", alors tout se passe bien, les 2 alerts s'affichent et c'est l'action de départ du formulaire qui appelée comme il faut.
Qu'est ce qui peu bien clocher ???

[Le Marlou]

Que je mette un ".do" ou pas je pense que ça ne changera rien, en fait l'Action n'est pas du tout appelée.
j'ai modifié le script comme ceci :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
 
<script language="JavaScript"> 
function changeToSaveAction(){ 
   alert("XXX");
   document.docForm.action = "/enregistrerAction"; 
   alert("XXX");
   document.docForm.submit(); 
} 
</script>

le premier alert s'affiche mais pas le second ! pourtant la syntaxe est bonne. Si je mets en commentaire la ligne document.docForm.action = "...", alors tout se passe bien, les 2 alerts s'affichent et c'est l'action de départ du formulaire qui appelée comme il faut.
Qu'est ce qui peu bien clocher ???

[slopera]

Comment définis tu ton html:form ?

[Sherkhan]

Oui, l'erreur doit se trouver au niveau du formulaire, car si ca, ca ne passe pas :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

document.docForm.action = "/enregistrerAction";

Ca veut dire que le formulaire ne doit pas s'appeler "docForm" ...

Mais sinon, pour appeler tes actions, tu n'utilises pas d'extension (moi j'utilise l'extension .do ...) ???

[slopera]

On met le .do quand on fait un lien :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

<a href="monAction.do">mon lien</a>

mais pas dans le action d'un form

[Le Marlou]

si, le nom du formulaire est correct
sinon pourquoi cette ligne marcherait

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
document.docForm.submit();

et pas celle-ci

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
document.docForm.action = "/enregistrerAction";

????????

[slopera]

Il me semble que si ton second alert ne s'était pas affiché, c'est parce qu'une erreur Javascript s'est déclenchée.
Maintenant, si tu ne veux pas nous dire comment tu déclares ton html:form alors bon courage pour la suite.

Stéphanie

[yolepro]

Moi j'aurais fait une truc du genre :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
 
function submit(){
  var form = document.docForm;
  form.action = 'enregistrerAction.do';
  form.submit();
}

[denisC]

Pour être sur de bien récuperer ton formulaire (tu as peut-être plusieurs formulaire ayant le même nom....), affecte un styleId à celui qui t'interesse, puis tu pourras le récuperer en faisant un:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
document.getElementById('...')

Ensuite, quand tu changes l'attribut action de ton formulaire, il faut bien entendu lui mettre une URL valide, c'est à dire une terminant par un .do

Maintenant, sur le fond, ce que tu essayes de faire est assez hérétique, et ca ne m'etonnes pas que ca ne marche pas. En effet, en struts, lors de la création de ton formulaire, Struts prépare l'ActionForm correspondant. En changeant l'action, tu changes le formulaire, ce qui doit être très perturbant pour toi.

Donc clairement, la seule solution c'est le DispatchAction (voire LookupDispatchAction si tu internationalises). Et pour tes problèmes de management rien ne t'empeche de faire une 'redirection' dans ton Action:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
 
public ActionForward supprimer(ActionMapping mapping,
					 ActionForm form,
					 HttpServletRequest request,
					 HttpServletResponse response) {
	return myClasse1.supprimer(mapping,form,request,response);
	}

Rien ne t'empeche donc de créer chacune de tes Actions dans des classes séparées tout en utilisant un DispatchAction.

[desproges]

Il me semble que le ".do" est quand meme nécessaire si on regarde le html généré d'un page... il est bien présent :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
<form name="SwelAdressesForm" method="post" action="/Adresses.do;jsessionid=5C73D3F801D3DAAFC773DD211799DAE5">

[Le Marlou]

Merci DenisC, je n'avais pas pensé à ça mais je pense que c'est comme cela que je vais faire, je vais créer une classe Action associée à chaque bouton et c'est mon dispatch Action qui appellera la bonne action :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
 
public class EditionAction extends DispatchAction {
   public ActionForward enregistrer(
      ActionMapping mapping, 
      ActionForm form, 
      HttpServletRequest request, 
      HttpServletResponse response) { 
      return EnregistrerAction.execute(mapping,form,request,response); 
   }
   public ActionForward envoyer(
      ActionMapping mapping, 
      ActionForm form, 
      HttpServletRequest request, 
      HttpServletResponse response) { 
      return EnvoyerAction.execute(mapping,form,request,response); 
   }
   ... ... ...
}

[denisC]

Tout a fait, ca me parait être le plus correct

PS: n'oublies pas le Tag [Résolu] 8)