Jointure Best Practises

Version imprimable

Comment faire pour récupérer tous les objets ZuluTree appartenant à un ZuluCategory en une seule requête?

Code:

1
2
3
4
5
6
7
8
 
ZuluTreeCategory:
  columns:
    tree_id:  { type: integer, primary: true }
    category_id: { type: integer, primary: true }
  relations:
    ZuluCategory:  { onDelete: CASCADE, local: category_id, foreign: id, foreignAlias: ZuluCategory}
    ZuluTree: { onDelete: CASCADE, local: tree_id, foreign: id, foreignAlias: ZuluTree }

J'ai essayé de faire ça dans la classe ZuluTreeCategoryTable:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
 
{
 
 
		$q = Doctrine_Query::create()
		->select('j.tree_id,t.*')
		->from('ZuluTreeCategory j')
		->innerJoin('j.ZuluTree t')
		->where('j.category_id = ?', $category->getId());
 
		return $q->execute();
 
	}

Mais, je suppose qu'on peut faire plus simple?

De plus, si je fais un unit test comme ceci:

Code:

1
2
3
4
5
6
7
8
9
 
$category = Doctrine_Core::getTable('ZuluCategory')
	->createQuery()
	->limit(1)
	->fetchOne();
 
 
 
	$t->is(Doctrine::getTable('ZuluTreeCategory')->getTreesForOneCategory($category)->get(0)->getTown(), 'Berlin', 'infos du premier arbre de la category 2');

J'ai l'erreur:

Code:

1
2
 
  Unknown record property / related component "town" on "ZuluTreeCategory"

Merci.:oops:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
 
ZuluCategory:
  relations:
    Trees:
      class: ZuluTree
      refclass: ZuluTreeCategory
      local: category_id
      foreign: tree_id
      foreignAlias: Categories
 
ZuluTreeCategory:
  columns:
    tree_id:  { type: integer, primary: true }
    category_id: { type: integer, primary: true }
  relations:
    ZuluCategory:  { onDelete: CASCADE, local: category_id, foreign: id, foreignAlias: ZuluCategory}
    ZuluTree: { onDelete: CASCADE, local: tree_id, foreign: id, foreignAlias: ZuluTree }

Code:

1
2
 
$category->getTrees();

Effectivement, merci.
Par contre, j'utilise une pagination avec sfDoctrinePager et donc il faudrait que je passe en paramètre la requête et non la doctrine_collection.
Code:

1 2 3 4 5 6 7 8 $this->zulu_trees = Doctrine::getTable('ZuluTree')->getAllQuery(); $this->pager = new sfDoctrinePager('ZuluTree', 8); $this->pager->setQuery( $this->zulu_trees); $this->pager->setPage($request->getParameter('page', 1)); $this->pager->init();
Donc, il y aura un execute() sur la query passée, ce qui marche pas avec $category->getTrees(); :oops:

20/04/2010, 13h54
Invité

D'accord. Dans ce cas le bout de code que tu as mis dans ton post précédent fonctionne non ?
20/04/2010, 14h01
psgman113

Non, car mon test avec getTown plantait.
Je n'arrivais pas à récupérer tous les attributs du ZuluTree.
20/04/2010, 14h25
Invité

D'un côté tu me parles d'un pager qui a donc besoin d'une query, d'un autre côté tu me parles d'un test, qui lui a besoin d'une collection.

Qu'est ce que tu cherches à faire ? Je n'arrive pas à te suivre. :lol:
20/04/2010, 14h48
psgman113

On oublie le test.
Je me concentre sur le pager uniquement qui doit prendre une query et non une collection.

Merci.

Donc si j'ai bien compris, tu as un objet ZuluCategory, et tu veux récupérer les ZuluTree associés et les paginer.

Dans ce cas tu peux simplement faire, dans ZuluTreeTable.class.php
Code:

1 2 3 4 5 6 // ZuluTreeTable.class.php public static function getTreesForCategoryQuery(ZuluCategory $category) { return $q = $this->createQuery('t') ->where('t.category_id = ?', $category->getId()); }
Non ?

20/04/2010, 15h05
psgman113

En fait, l'association entre les catégories et les trees est faite dans la table ZuluTreeCategory qui contient une clé primaire double (tree_id,category_id).

Si je fais à peu près la même méthode que toi dans le ZuluTreeCategory, je vais sûrement récupérer que la liste des tree_id matchant la category que j'ai passé en paramètre.

Mais moi je veux pas récupérer uniquement les id mais tous les attributs de chaque ZuluTree.

Je dois faire ça en une seule query et retourner la query à mon paginer.

Merci

Désolé :aie: j'ai été trop vite.

Oui je comprend bien que tu as besoin de toutes les colonnes de ZuluTree, c'est d'ailleurs pour ça que moi j'ai dit de faire ça dans ZuluTreeTable et non ZuluTreeCategoryTable.

Donc pour poursuivre dans mon idée, peut-être que ceci conviendrait :
Code:

1 2 3 4 5 6 7 // ZuluTreeTable.class.php public static function getTreesForCategoryQuery(ZuluCategory $category) { return $q = $this->createQuery('t') ->innerJoin('t.ZuluTreeCategories tc') ->where('tc.category_id = ?', $category->getId()); }
Par contre il faudrait juste changer un peu ton schema.yml :
Code:

1 2 3 4 5 6 7 8 ZuluTreeCategory: columns: tree_id: { type: integer, primary: true } category_id: { type: integer, primary: true } relations: ZuluCategory: { onDelete: CASCADE, local: category_id, foreign: id, foreignAlias: ZuluTreeCategories} ZuluTree: { onDelete: CASCADE, local: tree_id, foreign: id, foreignAlias: ZuluTreeCategories }
A priori là ça devrait le faire. Sinon je vais me coucher xD

20/04/2010, 15h45
psgman113

Merci, ça marche bien.
Par contre, à moi de jouer pour comprendre comment marche le plugin Paginer pour y intégrer la query car le setQuery n'est pas suffisant pour exécuter la query que tu viens de me dire..

Euh globalement le pager c'est très simple à utiliser:
Code:

1 2 3 4 $this->pager = new sfDoctrinePager('ZuluTree', $nombreParPage); // Instanciation du pager $this->pager->setQuery($q); // On set la query qui va être utilisée $this->pager->setPage($numPage); // On set la page que l'on veut $this->pager->init(); // On initialise le pager
Et puis dans ton template tout bêtement :

Code:

foreach ($pager->getResults() as $zuluTree):