Discussion :

[Mordri]

Bonjour,

voila j'ai une table (fiche) qui est créée par quelqu'un et peu être modifier par d'autres personnes,

j'ai un champ created_by_id pour savoir qui la créée et un champ updated_by_id pour savoir qui est le dernier à l'avoir modifier

j'utilise le plugin sfGuardSecurityUser sur mon module backend et donc j'aimerai récupérer la personne connecté pour remplir ces champs

je ne sais pas comment transmettre l'information de l'id récupéré dans le backend vers le modele doctrine (pour modifier la fonction save)

par ailleurs je ne sais pas comment changer cette fonction proporement.

pour l'instant dans save j'ai :

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
19
20
21
public function save(Doctrine_Connection $conn = null)
  {
    $conn = $conn ? $conn : JobeetJobTable::getConnection(); //vu sur différents sites et d'autres ne l'utilise pas, mais je n'ai vu aucune explication sur l'utilité 
    $conn->beginTransaction(); //idem
    try
    {
      if($this->isNew())
        $this->setCreatedById('1'); // j'ai mis 1 pour l'instant mais c'est cette valeur qui doit être modifiée
      else $this->setUpdatedById('1'); //idem
 
      $ret = parent::save($conn);
      $conn->commit(); // pareil que pour la première ligne je ne sais pas trop pourquoi l'utiliser
 
      return $ret;
    }
    catch (Exception $e) //idem
    {
      $conn->rollBack();
      throw $e;
    }
  }

J'ai aussi un autre problème qui va m'arriver bientot, je souhaite garder un historique des modifications de mes fiches (donc creations d'une table presque identique pour stocker les anciennes données) => avant de passer à symfony j'utilisai un trigger directement dans la bd, symfony permet t'il de créer des trigger? si non je le ferai manuellement (mais du coup à refaire à chaque fois que j'utilise "build")

[Michel Rotta]

A priori ton modèle est viable. On est bien d'accord que tu travailles sur le modèle et pas le form.

Pour les questions implicites :

La $conn peut ne pas être initialisée à ce niveau. Je ne pense pas que tes modifications nécessitent d'être absolument dans la transaction. Dans tous les cas, si la méthode save est appelée par un form, tu est déjà dans une transaction. donc tu peux économiser le code qui initialise $conn.

Personnellement je mettrais systématiquement le code pour initialiser update.

Dans l'objet record tu as un endroit qui me semble plus indiquer pour ce traitement, c'est la méthode preSave().

Ton code peut alors devenir :

Code :

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

1
2
3
4
5
6
7
 
public function preSave(Doctrine_Connection $conn = null)
{
    $this->setUpdatedById( sfContext::getInstance()->getUser()->getObject()->getId() );
    if($this->isNew()) $this->setCreatedById($this->getUpdateById); 
 
}

Plus léger non ?

Pour l'historique, ce sera plus lourd (quoique).

symfony/doctrine ne gère pas les triger. La rançon de la portabilité entre différentes bases.

Tu peux mettre du code dans le preSave() pour stocker tes modifications. Tu dois pouvoir assez faciliement générer une chaine du genre "table truc, modifié le x/x/x par : Durant {nom: de:truc a:machin} et mettre le tous dans une table générique (je pense qu'il doit être possible d'envoyer ton objet à un setter de cette table qui pourra, alors, automatiquement générer la chaine sans connaitre l'objet au départ, à vérifier).

Tu as un behavior qui permet de gérer des versions d'un enregistrement dans une seul et même table.

[Mordri]

merci pour ta réponse je test ça tout de suite

Je travaille bien dans le modèle.

Tu as un behavior qui permet de gérer des versions d'un enregistrement dans une seul et même table.

Je ne connais pas ce que c'est peux tu m'en expliquer un peu plus ?

[Mordri]

j'ai eu quelques soucis avec ce que tu m'as proposé mais j'ai corrigé

Code :

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

1
2
3
4
5
public function preSave($conn = null) // la fonction prend une variable appelé $event quand je laisse Doctrine_Connection ça me fait une erreur d'incompatibilité alors je l'ai retiré et c'est nickel maintenant ;)
{
    $this->setUpdatedById( sfContext::getInstance()->getUser()->getId());//getObject n'existe pas, mais j'ai accès à l'id directement
    if($this->isNew()) $this->setCreatedById($this->getUpdatedById()); 
}

pour avoir mon id j'avais défini ça y'a un petit moment :

Code :

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

1
2
3
4
5
6
7
class myUser extends sfGuardSecurityUser
{
  public function getId()
  {
    return $this->getAttribute('user_id', null, 'sfGuardSecurityUser');
  }
}

[Michel Rotta]

Ce n'est pas getObject() mais getGuardUser().

Mais ta méthode est très bonne aussi.

Pour le plugin, pas mieux que pour le nom de la méthode il ne fonctionne qu'avec propel.

J'ai a cet occasion trouvé un plugin sfDoctrineActAsSignablePlugin qui fait ce que tu cherches a faire et que je ne connaissait pas.

[Mordri]

effectivement, si j'ai bien compris ce plugin il fait ce que j'ai fait à la main, enfin c'était pas trop compliqué ni trop long vu que c'est pour une seule table, mais je le retiens pour mes développements future

pour l'historisation je verrais plus tard, j'ai déjà mes triggers faits (avec la première version de ce que j'avais fait sans symfony)

[Michel Rotta]

[Mordri]

dsl pour le up mais il y a une petite erreur que je viens de voir et que j'ai résolu tout seul comme un grand

dans /lib/form/votreForm.class.php

Code :

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

1
2
3
4
public function configure()
{
  unset($this['created_by_id'])
}

si y'a pas ça, pendant un update le created_by_id est mis à NULL