un form partiel sur sfGuardUser

Version imprimable

bonjour,
voila j'ai besoin de gérer des groupes d'utilisateurs, après bien des péripéties on a décidé de gerer le groupe au niveau du user (sfGuardUser) et les infos concernant les groupes dans une autre table:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
 
sfGuardUser: 
  actAs: [Timestampable]
  columns:
    first_name: string(255)
    last_name: string(255)
    email_address:
      type: string(255)
      notnull: true
      unique: true
    username:
      type: string(128)
      notnull: true
      unique: true
    algorithm:
      type: string(128)
      default: sha1
      notnull: true
    salt: string(128)
    password: string(128)
    is_active:
      type: boolean
      default: 1
    is_super_admin:
      type: boolean
      default: false
    last_login:
      type: timestamp
    matricule:
      type: integer
      unique: true           
....
    grpe_chq_dej_list_id: 
      type: integer
  indexes:
    is_active_idx:
      fields: [is_active]
  relations:
    Groups:
      class: sfGuardGroup
      local: user_id
      foreign: group_id
      refClass: sfGuardUserGroup
      foreignAlias: Users
    Permissions:
      class: sfGuardPermission
      local: user_id
      foreign: permission_id
      refClass: sfGuardUserPermission
      foreignAlias: Users    
    grpe_chq_dej_list:
      class: GrpeChqDej
      local: grpe_chq_dej_list_id
      foreign: id 
      onDelete: SET NULL

et ma table group:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
 
GrpeChqDej:
  tableName: GrpeChqDej
  columns:
    name:
      type: string(50)
      notnull: true
    primary_user_id:
      type: integer
    secondary_user_id:
      type: integer
    site_id: integer    
  relations:
    primaryUser:
      class: sfGuardUser
      local: primary_user_id
      foreign: id
      onDelete: SET NULL
    secondaryUser:
      class: sfGuardUser
      local: secondary_user_id
      foreign: id
      onDelete: SET NULL
    site:
      class: Site
      local: site_id
      foreign: id
      onDelete: SET NULL

je peut bien entendu gerer ce group via l'interface de sfGuardUser.... Sauf que les personnes gérant ces groupes en sont pas du tout les memes que celles qui gérent le profil.

Donc dois je créer un module a part entier, basé sur sfGuardUser ou bien y a t il moyen de creer simplement un formulaire dans le module GrpeCheqDej ?

19/11/2010, 14h06
Michel Rotta

Il y a plusieurs options.

La première n'est pas mauvaise, sauf que je de dériverais du form de base, pour juste rajouter le champ. Ainsi, si tu modifies un validateur, tu n'as pas à le faire dans tous les forms.

Si non, et si tu rends ton form dans le template sans faire de mise en page élaborée, tu peux aussi envisager de faire un test dans le config du form pour voir si le user courant à les droits (mais pas très propre) et générer le champ si oui.

Tu peux aussi faire, dans ton form, une méthode qui rajoute le champ (widget et validateur) et tester dans le contrôleur si le user courant à les droits, si oui, tu envoies la méthode qui rajoute le champ.

Pas de solution unique, à toi de juger ce qui te convient le mieux.
19/11/2010, 14h29
erictomcat

oui sauf que tu part du form de base et tu rajoute un champ..;
Alors que moi c'est le contraire, je veut juste un form avec le champ en question.

Donc plutôt partir du form original et unset tout les champs sauf celui qui me convient non ?

mais celui ci est gerer via un fichier generator donc je suis un peu paume sur la methode a suivre
19/11/2010, 14h40
Michel Rotta

Le problème c'est que si tu fais cela (soit un form avec un champ après avoir viré les autres) et que tu utilises le générateur, tu vas n'avoir à l'écran que le champ en question, sans savoir à quel utilisateur il correspond (dans la liste, ils y seront), ce qui n'est pas top.

Et mettre des champs en affichage seul avec le générateur, c'est un peu du sport (il faut soit créé en widget spécifique, soit utiliser des partials).

Dans le générateur tu as un paramètre qui permet d'indiquer quel est le form à utiliser pour l'édition et tu peux empêcher la création et la suppression, donc, tu as tous les outils dans la mains.
19/11/2010, 14h46
erictomcat

Citation:

Envoyé par Michel Rotta

....

Dans le générateur tu as un paramètre qui permet d'indiquer quel est le form à utiliser pour l'édition ....

ça me plait bien, donc dans le generator on peut avoir plusieurs form ??

j'ai cherche mais pas trouvé beaucoup d'info la dessus.
19/11/2010, 14h50
Michel Rotta

Tu trouveras des informations ici : http://www.symfony-project.org/refer...dmin-Generator

Théoriquement on doit pouvoir avoir un form pour l'édition et un form pour le news. Il n'y a pas de form pour la liste.

pas bon, on ne peut avoir que 2 form un new et un edit.

Hors j'ai besoin d'avoir un 3eme formulaire avec juste le nom du user et le champ groupe_id_list.. :(

bon j'ai créer un module:

Code:

symfony doctrine:generate-module --non-verbose-templates frontend extrafieldsuser sfGuardUser

comme je vais avoir plus d'un champ mettre a jour comme ça, j'ai créer une action pour ce champ:

Code:

1
2
3
4
5
 public function executeExtraFieldChqDej(sfWebRequest $request)
  {
    $this->forward404Unless($sf_guard_user = Doctrine::getTable('sfGuardUser')->find(array($request->getParameter('id'))), sprintf('Object sf_guard_user does not exist (%s).', $request->getParameter('id')));
    $this->form = new extraFieldChqDejForm($sf_guard_user);
  }

j'ai créer un template:

Code:

1
2
3
<h1>GroupChqDej1</h1>
 
<?php include_partial('extraFieldChqDej', array('form' => $form)) ?>

et un partial

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
<?php use_stylesheets_for_form($form) ?>
<?php use_javascripts_for_form($form) ?>
 
<form action="<?php echo url_for('extrafieldsuser/updateChqDej?id='.$form->getObject()->getId()) ?>" method="post" <?php $form->isMultipart() and print 'enctype="multipart/form-data" ' ?>>
 <table>
    <tfoot>
      <tr>
        <td colspan="2">
          &nbsp;<a href="<?php echo url_for('extrafieldsuser/index') ?>">Back to list</a>       
          <input type="submit" value="Save" />
        </td>
      </tr>
    </tfoot>
    <tbody>
      <?php echo $form ?>
    </tbody>
  </table>
</form>

et une classe:

Code:

1
2
3
4
5
6
7
8
9
class extraFieldChqDejForm extends sfGuardUserForm
{
  public function configure()
  {
  	//unset($this['username']);
  	$this->useFields(array('username','grpe_chq_dej_list_id'));
 
  }
}

j'ai créer un un update specifique:

Code:

1
2
3
4
5
6
7
8
9
10
public function executeUpdateChqDej(sfWebRequest $request)
  {
    $this->forward404Unless($request->isMethod(sfRequest::POST) || $request->isMethod(sfRequest::PUT));
    $this->forward404Unless($sf_guard_user = Doctrine::getTable('sfGuardUser')->find(array($request->getParameter('id'))), sprintf('Object sf_guard_user does not exist (%s).', $request->getParameter('id')));
    $this->form = new extraFieldChqDejForm($sf_guard_user);
 
    $this->processForm($request, $this->form,'extraFieldChqDej');
 
    $this->setTemplate('extraFieldChqDej');
  }

et modifié le process form pour qu'il me recup le bon template:

Code:

1
2
3
4
5
6
7
8
9
10
protected function processForm(sfWebRequest $request, sfForm $form, $template='edit')
  {
    $form->bind($request->getParameter($form->getName()), $request->getFiles($form->getName()));
    if ($form->isValid())
    {
      $sf_guard_user = $form->save();
 
      $this->redirect('extrafieldsuser/'.$template.'?id='.$sf_guard_user->getId());
    }
  }

je pense en avoir fait beaucoup trop, mais ça marche :)

mais bon la je part du user pour aller vers legroupe alors qu'en général on partira du groupe en lui rajoutant un user :aie:

22/11/2010, 18h20
Michel Rotta

Trop de code ? Mais non, ce qu'il faut là où il le faut.

C'est beau symfony quant cela marche avec le bon code, là où il doit être...

Bon, maintenant, l'admin automatique n'est peut-être pas toujours la bonne solution.

As-tu essayés de modifier le form pour mettre une condition sur le type d'utilisateurs dans le configure et pour le modifier à la volée ?
22/11/2010, 23h24
erictomcat

heu j'ai pas tout compris.
En faite, il faut a partir du groupe (donc un champ par défaut égal au groupe) pouvoir sélectionner une nouvelle personne pour l'intégrer au groupe (et le summum, ce serait avec un autocomplete qui ne prend en compte que ceux n'étant pas déjà dans un groupe :)
29/11/2010, 17h24
Michel Rotta

Alors le code ne semble pas le bon.

Il faut probablement générer un CRUD sur le groupe et utiliser le plugin de gestion des embed pour y mettre la liste des membres avec possibilité d'en ajouter. Donc modifier le form du groupe et modifier ou créer le form qui sera embarqué pour gérer la liste déroulante. Il faudra aussi penser à créer une méthode pour récupérer les personnes qui ne sont pas membres du groupe, comme la requête sera lancée dans le configure() du form du groupe, tu ne devrais pas avoir trop de mal pour trouver l'id du groupe à exclure.

Ca ne devrais pas être mortel.

bon je vous soumet ma solution:

une action dans GrpeChqDej

Code:

1
2
3
4
public function executeAddUser(sfWebRequest $request){
		$this->forward404Unless($grpe_chq_dej = Doctrine::getTable('GrpeChqDej')->find(array($request->getParameter('id'))), sprintf('Object grpe_chq_dej does not exist (%s).', $request->getParameter('id')));
		$this->form= new GrpeChqDejAddUserForm(array('groupe'=>$request->getParameter('id')));		
	}

un form:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
 
class GrpeChqDejAddUserForm extends sfForm
{
  public function configure(){
  	sfContext::getInstance()->getConfiguration()->loadHelpers(array('Url'));
 
  	$this->setWidgets(array(
  		'groupe'    => new sfWidgetFormInputText(),
  		'username'  => new sfWidgetFormInputText()
  		));
 
    $this->widgetSchema['username'] = new sfWidgetFormDoctrineJQueryAutocompleter(
      array(
        'model' => "sfGuardUser",
        'url'   => url_for("@ajax_username")
      ), array('size' => 25)
    );
  }
}

et une action derriere:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
 
public function executeAddUserCreate(sfWebRequest $request)
	{
 
		$this->forward404Unless($sfGuardUser = Doctrine::getTable('sfGuardUser')->find(array($request->getPostParameter('username'))), sprintf('Object sf_guard_user does not exist (%s).', $request->getPostParameter('username')));  	
		$sfGuardUser->setGrpe_chq_dej_list_id($request->getPostParameter('groupe'));
		$sfGuardUser->save();
 
		$this->redirect('GrpeChqDej/UserListDisplay?id='.$request->getPostParameter('groupe'));
 
 
	}

pour le filtrage des user, je pense juste créer une liste différente

01/12/2010, 14h33
Michel Rotta

Ton code s'améliore de sujet en sujet :ccool:

Pas pu testé, donc, à la lecture, partiel, du code, cela me semble correcte, mais je ne peux donner plus qu'un sentiment.

Tu veux dire quoi par : "pour le filtrage des user, je pense juste créer une liste différente " ?

en fait j'ai créer une autre route ajax:

Code:

1
2
3
4
 
ajax_username_grpe_chq_dej:
  url: /AjaxUsernameGrpeChqDej
  param: {module: sfGuardUser, action: ajaxUsernameGrpeChqDej}

mes actions:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
 
class sfGuardUserActions extends autoSfGuardUserActions
{
	public function executeAjaxUsernameGrpeChqDej(sfWebRequest $request){
		$where="p.grpe_chq_dej_list_id is null";
 
		return $this->executeAjaxUsername($request,$where);
	}
	public function executeAjaxUsername(sfWebRequest $request,$where=null)
  	{
    	$this->getResponse()->setContentType('application/json');
    	$choices = array();
    	foreach($section = Doctrine::getTable('sfGuardUser')->getUsernameAutocompletion($request->getParameter('q'), $request->getParameter('limit'), $where)->getData() as $p)
    	{
      		$choices[$p->id] = $p->fullname;
    	}
    	if($section != array())
    	{
      		return $this->renderText(json_encode($choices));
    	}
  	}
 
}

sfGuardUserTable:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
 
public function getUsernameAutocompletion($q, $limit,$where=null)
    {
    	$dq = Doctrine_Query::create()
            ->select("p.id, concat(p.last_name,' ',p.first_name) AS fullname")
            ->from("sfGuardUser p")
            ->where("concat(p.last_name,' ',p.first_name) LIKE ?","%".$q."%")
            ->andWhere("p.offrolldate is null")
            ->limit($limit)
            ->orderBy("concat(p.last_name,' ',p.first_name) ASC");
 
        if(!is_null($where)) $dq->where($where);
 
    	return $dq->execute();
	}

Je pense qu'il y moyen de mieux refactorisé mais j'ai pas encore trouve

03/12/2010, 12h52
Michel Rotta

Je ne comprend pas l'intérêt de la méthode executeAjaxUsername(). Soit tu en as besoins ailleur (et c'est alors compréhensible), soit non, dans ce cas, tu peux tout mettre ensemble.

Et si tu utilises deux endroit, je pense qu'il vaudrait mieux faire 3 méthodes, les deux exécutes et une troisième, sur l'objet du modèle pour la partie récupération, tu l'appel alors avec les paramètres pour effectuer la recherche (un objet du modèle n'est pas sensé aller voir dans un objet request).
03/12/2010, 14h00
erictomcat

en faites dans pas mal d'endroit j'ai besoin de mettre un user et vu le nombre, un select n'est pas possible.
Donc j'utilise un champ autocomplete qui prend tout les utilisateurs toujours présent. et que je recupere via cette fonction.
Par contre dans le cas des ChqDej, j'ai besoin de cette liste moins ceux étant déjà dans un groupe.

mais effectivement la fonction particulière ne devrait pas se trouver dans sfGuardUserTable mais plutot dans grpCheqDej je pense.
03/12/2010, 17h30
Michel Rotta

Si tu as besoins de cette liste régulièrement, et, occasionnellement avec ton exception, tu peux factoriser ton code. Il te suffit de lui mettre un paramètre qui renvoie, soit la liste globale, soit la liste particulière. La récupération, à une condition près, doit être pareil.
06/12/2010, 13h51
erictomcat

j'y reflechi mais j'ai pas encore trouve ;)