Discussion :

[guice11]

Bonjour,

J'utilise en ce moment le Criteria pour filtrer une collection d'objets. Mais lorsque je souhaite réaliser un orderBy avec 2 champ, seul le premier est pris en compte. Je ne comprends pas.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
$events = new Collections\ArrayCollection($results);
$dateFrom = new \DateTime($date);
$dateTo   = new \DateTime(date('Y-m-d H:i:s', strtotime($date . ' + 1 day')));
 
$criteria = Criteria::create()
	->where(Criteria::expr()->eq('activity', $activity));
 
$criteria->orderBy(array(
	"time" => "ASC",
	"title" => "ASC"
 
));

Je remercie d'avance pour d'éventuelles réponses .

[bilbot]

Et si tu fais ca :

Code :

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

1
2
3
 
$criteria->orderBy(array("time" => "ASC"))
->addOrderBy(array("title" => "ASC"));

est ce que ca marche mieux

je trouve pas dans la doc de doctrine2 http://docs.doctrine-project.org/en/...l#expr-classes la notation que tu as utilisé.

[guice11]

Bonjour,

La documentation que j'ai utilisé est celle ci : http://doctrine-orm.readthedocs.org/...ng-collections

Malheuresement avec un addOrderBy, cela ne marche toujours pas.

[Nico_F]

Est-ce qu'on peut savoir ce que ça veut dire "ça ne marche pas" ? Quel est le résultat attendu, quel est le résultat rendu ?

Donne nous un exemple avec des données.

[guice11]

En fait je souhaite que la collection soit trier par "time", puis ensuite par "title" mais en ce moment, il n'y a que "time" qui est pris en compte.

[arnooo999]

Bonjour,


Ou se trouve ton code ?
1 / dans un repository
-> dans ce cas utilise plutôt le query builder

2 / dans une méthode de ton entité
-> Y a t'il déjà une annotation orderBy sur ton association qui pourrait interférer ? (si oui essaie de l'enlever)
-> dans ton test, le tri se fait t'il
-> au niveau SQL (dans ce cas tu devrais voir la requete généré dans le profiler symfony)
-> au niveau php ? (Si la collection a déjà été chargé)

(je viens de tester cette fonctionnalité de Doctrine que je ne connaissais pas, et j'ai bien mes 2 orderBy dans la requete SQL (SF 2.3 / Doctrine 2.2 )

[guice11]

Mon code se situe dans un controller.

Tu trouveras la fonction du controller ci-dessous :

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
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
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
 
public function resultsAction($page)
{
	$session 	 = $this->getRequest()->getSession();
	$request 	 = $this->container->get('request');
	$searchEvent = $session->get('searchEvent');
 
	if ($request->getMethod() == 'POST'){
		$parameters = array_keys($request->request->all());
		$nameForm = $parameters[0];
 
		if ($nameForm == 'jesors_eventbundle_eventsearchindex'){
			$form = $this->container->get('form.factory')->createBuilder(new EventSearchIndexType())->getForm();
		} elseif($nameForm == 'jesors_eventbundle_eventsearch') {
			$form = $this->container->get('form.factory')->createBuilder(new EventSearchType())->getForm();
		}
 
		$formHandler = new EventHandler($form, $request, $this->getDoctrine()->getManager());
 
		if ($formHandler->process()) {
			$location  = $form['location']->getData();
			$latitude  = $form['latitude']->getData();
			$longitude = $form['longitude']->getData();
			$activity  = $form['activity']->getData();
			$date      = $form['date']->getData();
			$distance  = $form['distance']->getData();
 
			$filter = new GeoDistance('lat_long', array('lat' => $latitude,'lon' => $longitude), $distance);
			$filtered = new Filtered(new MatchAll(), $filter);
			$results = $this->container->get('fos_elastica.finder.website.event')->find($filtered, 15000);
 
			$events = new Collections\ArrayCollection($results);
			$dateFrom = new \DateTime($date);
			$dateTo   = new \DateTime(date('Y-m-d H:i:s', strtotime($date . ' + 1 day')));
 
			$criteria = Criteria::create()
				->where(Criteria::expr()->eq('activity', $activity));
 
			if (!is_null($date)){
				$criteria->andWhere(Criteria::expr()->gte('date', $dateFrom))
						 ->andWhere(Criteria::expr()->lt('date', $dateTo));
			} else {
				$criteria->andWhere(Criteria::expr()->gte('date', new \DateTime(date('Y-m-d H:i:s'))));
			}
 
			$criteria->orderBy(array(
				"date" => "ASC",
				"time" => "ASC"
			));
 
			$session->set('searchEvent', $events->matching($criteria));
			$session->set('searchLocation', $location);
			$session->set('searchDistance', $distance);
 
			$paginator  = $this->get('knp_paginator');
			$pagination = $paginator->paginate(
				$events->matching($criteria),
				$this->get('request')->query->get('page', 1),
				15
			);
		}
	} else {
		$paginator  = $this->get('knp_paginator');
		$pagination = $paginator->paginate(
			$searchEvent,
			$this->get('request')->query->get('page', $page),
			15
		);
	}
 
	return $this->render('JesorsEventBundle:Event:list.html.twig', array(
		'pagination'     => $pagination
	));
}

[Nico_F]

Je réitère la deuxième partie de ma réponse :

Donne nous un exemple avec des données.

Si tu fais un tri par time (à la seconde près) il est fort probable que tu n'aies pas deux valeurs avec le même 'time' exactement.
Le deuxième tri, sur ton autre paramètre, ne s'applique que si tu as des valeurs strictement identiques dans le premier cas.
Dans un cas à la seconde près, l'ordre à la seconde aura lieu, et à moins que tu aies plusieurs valeurs avec un time égal, il n'y aura pas besoin de faire un deuxième tri.

Avant d'accuser un problème avec le code, assure toi que la réflexion est bonne

[guice11]

En fait là ça me sort comme valeurs :

Entité 1:
Heure : 15:35
Date:05/09/2014

Entité 2:
Heure : 13:30
Date:05/09/2014

Entité 3:
Heure : 19:45
Date:05/09/2014

La date est bonne, mais l'heure n'est pas ordonnée.

[Nico_F]

Il n'y a rien d'ordonné là : l'heure n'est pas ordonnée et les dates sont toutes les mêmes.

Utilise au moins des données significatives quand tu veux faites des tests.

Autrement, un Datetime n'aurait-il pas été plus pratique ?