Discussion :

[Linio]

Bonjour,

Je travaille sur une application web sur laquelle j'ai un comportement problématique.
J'ai une page pour modifier une entité.
Sur cette page selon les droits de l'utilisateur connecté on affiche ou non les éléments qui sont modifiables par lui.
On utilise un objet Form pour stocker toutes ces données (le scope au niveau Struts est session).
J'ai un problème sur certains champs lorsque ceux ci ne sont pas repris dans la page.
Si je tente de faire un affichage de la valeur de mon champ sur ma page, elle a bien la bonne valeur, en revanche lorsque je soumet le formulaire, cette valeur est automatiquement remise à zéro.
Lorsque la donnée est présente dans un input pas de problème.
C'est étrange ça ne me le fait que sur certaines données de types string qui sont gérées par des checkbox.

L'action struts est tout ce qu'il y a de plus classique :

Code :

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

1
2
3
4
5
6
7
8
9
<action  path="/modifierPersonnel"
	    	 type="package.ModifierPersonnelAction"
	   	 name="personnelForm"
	    	 scope="session"
	    	 input=".modifier.personnel" 
	    	 validate="false">
	    	 <forward name="success" path=".afficher.personnel"/>
	    	 <forward name="error" path=".modifier.personnel"/>	        
</action>

Je ne sais pas si quelqu'un a des idées mais je suis preneur, je peux donner plus d'informations au besoin, mais au vu du problème le code ne me semble pas très révélateur.

[aboussaidi]

Bonjour,

Je ne serais rependre à ta problématique pour le moment, (si tu pouvais poster ClassAction, ActionForm, peut être on pourra voir plus clair), MAIS:

Ce que je peux te conseiller :

• il n'est pas tres safe d'envoyer une entité entière avec tous les paramètres et ensuite gérer les droit au niveau de l'affichage, si tu veux suivre réellement le modèle MVC, tu dois éffectuer ce contrôle au niveau de ton controlleur et du coup creer un filtre sur ton entité et envoyer que les données autorisé a cette utilisateur (Dans un modèle MVC: pour la sélection on peut (mais c'est pas recommandé) faire comuniquer la VUE au MODELE, mais jamais en édition)
• Si les données de ton entité sont à afficher dans une seul page pourquoi t'as mis comme scope la session ?

[fardon57]

Il faut toujours faire attention lorsqu'on utilise un scope session et que le formulaire possède des checkbox. ça m'a l'air d'être ton cas .

En général, ce qu'il se passe :

1. Tu envois le formulaire avec la checkbox coché => appel du setter et mise en place de l'actionForm dans la session

2. Tu ré-envois le formulaire avec le checkbox vide => récupération de l'actionForm de la session et .... rien de plus, le checkbox vide n'est pas passé en paramètre ! Donc ton actionForm possèdera toujours l'ancienne valeur, le comportement n'aura pas été changé.

En général, ce qu'il faut faire c'est un override de la méthode reset(ActionMapping mapping, HttpServletRequest request) dans ton actionForm, et là dedans tu réinitialise tes checkbox.


Cf la doc de la méthode reset :

/**
* <p>Can be used to reset bean properties to their default state, as
* needed. This method is called before the properties are repopulated by
* the controller.</p>
*
* <p>The default implementation does nothing. In practice, the only
* properties that need to be reset are those which represent checkboxes
* on a session-scoped form. Otherwise, properties can be given initial
* values where the field is declared. </p>
*
* <p>If the form is stored in session-scope so that values can be
* collected over multiple requests (a "wizard"), you must be very careful
* of which properties, if any, are reset. As mentioned, session-scope
* checkboxes must be reset to false for any page where this property is
* set. This is because the client does not submit a checkbox value when
* it is clear (false). If a session-scoped checkbox is not proactively
* reset, it can never be set to false.</p>
*
* <p>This method is <strong>not</strong> the appropriate place to
* initialize form value for an "update" type page (this should be done in
* a setup Action). You mainly need to worry about setting checkbox values
* to false; most of the time you can leave this method unimplemented.
* </p>
*
* @param mapping The mapping used to select this instance
* @param request The servlet request we are processing
*/

[Linio]

Ouh la la, mais si c'était moi qui avait ne serait-ce qu'écrit une ligne de ce truc, je me serai déjà fait harakiri. J'ai déjà les yeux qui saignent rien qu'à essayer de comprendre comment (pourquoi??) ils ont fait ça comme ça...

@aboussaidi

Je ne peux donc pas suivre un quelconque modèle, et quand bien même je voudrais le faire "à partir de maintenant" ça serait impossible, j'ai du ldapElement dans mes jsp c'est dire si le modèle MVC est oublié depuis longtemps.

Idem pour la session, c'est un truc qui a été décidé à la création de l'appli (là aussi pour des raisons qui me dépassent) et maintenant on paye les pots cassés, parce qu'encore là ça fonctionne vaguement mais ça donne des comportements plus qu'aléatoires sur certaines autres pages (du genre si je vais sur une page et que je reviens en arrière oulala magique j'ai les droits en édition)

@fardon57

En fait ce n'est justement pas exactement un problème de checkbox puisque là j'avais bel et bien vu que Struts ne le gérait pas très bien, en fait les checkbox ne sont affichée (et leur valeur envoyée dans la requête) que dans le cadre où l'utilisateur aurait le droit de modifier cette valeur, dans mon cas à l'inverse on n'affiche rien, et aucune valeur n'est envoyée dans la requête, je m'attends donc (mais c'est là finalement qu'est le coeur de ma question : ai-je bien compris le fonctionnement de Struts??) à avoir la donnée non modifiée dans mon formulaire.

J'ai surchargé la méthode reset mais il ne passe apparemment pas dedans, est-ce que Struts appelle cette méthode à chaque chargement d'action?

En gros ce que je ne comprends pas c'est que j'ai un objet personnelForm qui est en scope session et lorsque je clique sur le lien pour activer l'action de modification l'objet personnelForm a vu une partie de ses valeurs "disparaitre", or le scope est bien session, et pire, d'autres valeurs qui ne sont pas non plus passées par la requêtes sont elles en revanche bien là où elles devraient être.

[OButterlin]

En l'occurrence, regarde le code de l'ActionForm associée à ton action, tu dois avoir une méthode reset(...) qui initialise les valeurs des checkbox.

[c_nvy]

J'ai surchargé la méthode reset mais il ne passe apparemment pas dedans, est-ce que Struts appelle cette méthode à chaque chargement d'action?

En gros ce que je ne comprends pas c'est que j'ai un objet personnelForm qui est en scope session et lorsque je clique sur le lien pour activer l'action de modification l'objet personnelForm a vu une partie de ses valeurs "disparaitre", or le scope est bien session, et pire, d'autres valeurs qui ne sont pas non plus passées par la requêtes sont elles en revanche bien là où elles devraient être.

Un clic sur un lien ne soumet pas le formulaire, ce qui explique pourquoi la méthode reset n'est pas exécutée et pourquoi les valeurs que tu as modifié dans le formulaire sont perdues.

Pour y remédier, lors du clic sur le lien, il faut déclencher une fonction Javascript dans laquelle tu références l'Action et tu soumets le formulaire.

Quelque chose du style :

Code :

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

1
2
3
4
5
function modifAction()
{
  document.forms[0].action="modifAction.do";
  document.forms[0].submit() ;
}

Code :

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

<html:link href="javascript:modifAction();">Modification</html:link>