Discussion :

[cli16]

Bonjour à tous,


J'ai un module CRUD sur Symfony 1.4 avec Doctrine "Fréquentation" sur lequel les utilisateurs saisissent les visites mensuelles dans leur structure.
Mon problème est que, actuellement le template list de ce module affiche toutes les fréquentations tout utilisateur confondus.
Ce que je voudrai faire c'est :
- pour les administrateurs : toutes les fréquentations visibles dans la list
- pour les non-admin : seulement les fréquentations de l'utilisateur connecté

J'ai cherché dans les classes du module où modifier la requête, mais avec la notion de filtre qui se greffe là-dessus je sais pas trop comment résoudre mon problème.

L'authentification est gérée avec sfDoctrineGuard.

Quelqu'un aurait-il une idée svp ?


Merci d'avance,

cli16 *- *-

[jejeman]

Pas sur que tu puisse faire ça directement...
J'ai écrit un truc dans le genre pour une déclaration de ticket d'intervention.
Et dans mon action class, si le user n'a pas le credential de gestion (administrateur), je rajoute une clause where pour filtrer que ses propres déclaration...

[vincent.le]

Si ça peut t'aider, tu peux filtrer tes requêtes dans ce style là :

Code :

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

1
2
3
4
$q = Doctrine_Query::create()
    ->from('Fichier f')
    ->where('f.user_id = ?' , $this->getUser()->getGuardUser()->getId() );
    $this->fichiers = $q->execute();

Et je sais que tu peux tester si l'utilisateur est admin ou pas, cherche dans la doc.

Avec ça, tu devrait y arriver je pense.

[Michel Rotta]

Quant tu dis module CRUD, c'est un généré ou un module admin ?

La meilleur chose est de créer une méthode dans l'objet de la table, qui va te retourner les données de l'utilisateur courant, sauf s'il est admin.

Le DQL de vincent.le est "presque" bon, vu qu'il lui manque le test sur le fait d'être admin ou pas et qu'il utilise une méthode pour récupérer l'Id de l'utilisateur qui me fait dire qu'il est dans le contrôleur et pas dans le modèle...

Après, dans le module admin, tu précises que tu utilise cette méthode pour récupérer les données.

Si pas de module admin, idem, mais dans la route globale du module.

Il peut aussi être intéressant de mettre une vérification sur l'ouverture de l'enregistrement, pour éviter quelqu'un qui essaye des numéros "au pif".

Pour le module admin, même combat, il faut une méthode dans l'objet du modèle.

Pour le module plus classique, on peut faire le teste une fois l'enregistrement récupéré, si l'id du user est pas bon, forward404...

[cli16]

Bonjour,

Merci pour vos réponses.
J'avais déjà essayé de mettre la requête comme vous me l'avez proposé mais ça ne fonctionnait pas.
Je ne sais pas si le code si dessous est correct ?

Dans actions.class.php :

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
  protected function getPager()
  {
    $pager = $this->configuration->getPager('Frequentation');
 
    if (sfContext::getInstance()->getUser()->hasCredential('admin')) {
        $pager->setQuery($this->buildQuery());
    } else {
      $q = Doctrine_Query::create()
      ->from('Frequentation')
      ->where('user_id=?', sfContext::getInstance()->getUser()->getGuardUser()->getId())
      ->orderBy('periode');
      $pager->setQuery($q);
    }
 
    $pager->setPage($this->getPage());
    $pager->init();
 
    return $pager;
  }

Quand je me connecte en admin c'est tout bon (normal car rien ne change...!) par contre avec un compte non admin j'obtiens une erreur sur les filtres : Notice: Undefined variable: filters.
Je ne sais pas comment gérer ces filtres sans la fonction buildQuery et avec ma propre requête.

Pour Michel : c'est bien un module généré par symfony.

Merci pour vos éclaircissements !

cli16 *-*-

[Michel Rotta]

Ceci ne répond pas à ma dernière question.

Est-ce un module administration avec un generator.yml où un module CRUD que tu modifies par après ?

[cli16]

C'est un module d'administration avec generator.yml.

Merci,


cli16 *-*-

[Michel Rotta]

Il n'est pas nécessaire (ni souhaitable) de modifier le fichier action.class.php

Tu va créé une méthode tel que je l'ai expliqué plus haut.

Ensuite tu vas modifier ton fichier generator.yml pour lui dire que la méthode à utiliser pour récupérer les enregistrements est une nouvelle méthode.

Code :

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

1
2
3
4
 
config:
  list:
    peer_method: MaMethodePourRecupererLesBonnesDonnees

Et tu as le résultat voulu.

Plus simple, non ?

[cli16]

Bonjour,

Parfait, merci pour les explications.

cli16 *-*-