enregistrer le nom du user qui crée l'objet

Version imprimable

09/12/2010, 16h35
jfsenechal

enregistrer le nom du user qui crée l'objet
bonjour,

j'ai un champ user_add varchar

je souhaite simplement enregistrer automatiquement le nom du user connecté

je pensais faire une surchage de la methode
Code:

1 2 3 4 5 public function save(Doctrine_Connection $conn = null) { $this->setUserAdd(); parent::save(); }
comment accéder au nom du user ?

Code:

$this->getUser()->getUsername();

ne marche pas

merci

j'essaie aussi un surchangeant dans actions.class

Code:

1
2
3
4
5
6
7
8
9
public function executeNew(sfWebRequest $request) {
 
        $this->form = $this->configuration->getForm();
        $objet = $this->form->getObject();
 
        $objet->setUserAdd($this->getUser()->getUsername());
 
        $this->travaux = $objet;
    }

mais mon champ reste vide

09/12/2010, 17h36
Michel Rotta
Nous avons ici trois problèmes distinct à régler :

La question de l'enregistrement du nom de l'utilisateur dans l'objet user de la session. Comment ce connecte ton utilisateur ? Quel est le processus de validation ? Comment est alimenté le sfUser ?

La question de la récupération. Que tu partes du form ou de l'objet du modèle, tu dois pouvoir accéder à ton objet user. Pour ce faire :
sfContext::getCurrent()->getUser()...

La dernière question, où placer le code d'enregistrement de l'information. Il y a trois réponses, les trois sont bonnes :
- Lors de la sauvegarde de l'enregistrement, et à chaque enregistrement, d'où que viennent la demande d'enregistrement, on récupère et on sauve.
- Dans le form, à chaque demande d'enregistrement de ce form, et uniquement de ce form, qui que cela soit qui fasse l'enregistrement
- Dans le contrôleur, suivant le cas et d'autre conditions.
Je ne connais pas la théorie métier que tu veux utiliser, à toi de choisir en fonction de ton application.

Question complémentaire. Tu veux stocker le nom de l'utilisateur. Ceci ne me semble pas conforme aux usage, je te conseillerais, plutôt, de stocker l'id de l'utilisateur. A partir de là, tu peux toujours récupérer le nom. Si besoin.

Pour finir j'ai fait ainsi :

Code:

1
2
3
4
5
6
7
8
9
10
 
public function executeNew(sfWebRequest $request) {
 
        $travaux = new Travaux();
        $travaux->setUserAdd($this->getUser()->getGuardUser()->getUsername());
 
        $this->form = $this->configuration->getForm($travaux);
        $this->travaux = $this->form->getObject();
 
    }

En mettant mon champ useradd en hidden

Code:

1
2
 
$this->widgetSchema['user_add'] = new sfWidgetFormInputHidden();

15/12/2010, 13h01
Michel Rotta

La méthode est bonne, quoique.

En effet, tu fais transiter une information que l'utilisateur ne doit pas pouvoir modifier dans un champ hiden qu'il est relativement simple de modifier pour un utilisateur averti.

Je mettrais la donnée, de la même manière, plutôt lors du retour du form ce qui évitera d'envoyer les données pour un allé retour sur le poste utilisateur. La donnée est dans le sfUser, elle y sera aussi au retour du form.

Je souhaitais revenir sur ce topic

Comme l'a dit Rotta, ma méthode n'est pas spécialement mauvais, mais pour une donnée sensible comme ici, ce n'est pas très sécurisé.

Alors j'ai trouvé un bon tuto sur http://prendreuncafe.com/blog/post/2...s-with-Symfony

on se rend compte qu' en faite accéder aux données user depuis un formulaire ce n'est pas si simple que ça

J'ai utilisé la méthode context.load_factories

J'ai surchargé la méthode doSave dans monform.class.php
Code:

1 2 3 4 5 6 7 8 9 10 11 12 protected function doSave($con = null) { if ($this->isNew()) { $user = self::getValidUser(); $username = $user->getGuardUser()->getUsername(); $objet = $this->getObject(); $objet->setUserAdd($username); } parent::doSave($con); }

09/02/2011, 18h32
Michel Rotta

Pour retrouver le user courant tu as plus simple :

Code:

sfConfig::getContext()->getUser()

comme indiqué ci-dessus.

Par curiosité, tu as apportés tes modifications au seins du form, pourquoi ne pas avoir mis ce code dans le modèle, se qui aurait ipso facto enregistré le user a chaque ajout, pas uniquement depuis un form ?
09/02/2011, 19h10
jfsenechal

pt'ite correction ;-)

Citation:

sfConfig::getContext()->getUser()

Code:

sfContext::getInstance()->getUser()

parce que partout où cette solution était donnée, elle est déconseillée
pas propre ?

Citation:

pourquoi ne pas avoir mis ce code dans le modèle

Dans lib/model/doctrine/maClasse.php ?

Parce que je pensais que ces surcharges ne fonctionnaient que dans maClasseForm.php

c'était jamais clairement expliqué dans les forums

j'essaierai demain
10/02/2011, 10h35
Michel Rotta

Taper du code de mémoire n'est jamais bon :oops:

C'est bien ce que je disais (ici ou ailleurs). Dans la théorie MVC c'est pas propre. Mais ça marche parfaitement. Tu obtiens une autre instance sur l'objet user, pas un autre objet pour le même user, donc propre.

Ensuit on veut une séparation absolue (avec de bonnes raisons) entre le M de MVC et le C de MVC. Sauf que les objets du modèle, dans l'implémentation qu'en fait symfony sont a cheval sur le M et le C. Pour moi, ils sont plus proche d'un C spécialisé dans le traitement de la donnée que du pure M qui est essentiellement la couche Doctrine à la quel nous ne toucherons pas. Donc y mettre de l'intelligence pour que les données soient conforme à ce qu'attend l'application est parfaitement viable, même si cela peut un peu choquer les conscience. :ccool:
10/02/2011, 10h41
jfsenechal

Houla j'ai encore eu droit à une réponse à la rotta en pleine forme :P

Bon j'ai déplacé mon dosave() dans lib/model/doctrine/maClasse.php

se passe rien du tout, le dosave est abosulement pas éxécuter, je le remet dans maclasse.form.php :P
10/02/2011, 11h13
Michel Rotta

Normal, doSave() est une méthode qui n'existe que dans les objets form...

Je regarderais du côté de la méthode preInsert() du sfDoctrineRecord qui n'est exécutée qu'à la création de l'enregistrement...