Probleme récupération données avec foreach

[kiradeus]

Bonjour à tous,

Voila j'ai décidé de faire un filtre sur mon site de cocktails.

Pour ce faire, à chaque clic d'une catégorie dans mon filtre, ça exécute un code javascript qui fait appel à chaque fois à la fonction filter() qui se trouve dans mon CocktailsController.php. Voici son contenu :

Code :

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
function filter(){
		if(isset($_POST['categories'])){
			$term_id = $_POST['categories'];
			foreach($term_id as $k => $v){
				$termR = $this->Cocktail->TermR->find('list', array(
					'fields' => array('ref_id'),
					'conditions' => array('TermR.term_id' => $v)
				));
				foreach($termR as $w => $x){
					$d['cocktails'] = $this->Cocktail->find('all', array('conditions' => array('Cocktail.id' => $x)));
					$this->set($d);
				}
			}
		}
		else{
			$this->paginate = array('Cocktail' => array('limit' => 10));
			$cocktails = $this->Paginate('Cocktail', array('Cocktail.online' => 1, 'Cocktail.created <= NOW()'));
			$this->set($d);
		}
	}

Et mon filter.ctp correspondant :

Code :

1
2
3
4
5
6
<?php if(!empty($cocktails)): ?>
	<?php foreach($cocktails as $k => $v): $v = current($v); ?>
		<?php echo $v['name']; ?>
		<a href="<?php echo $this->Html->url($v['link']); ?>" class="btn primary">Consulter</a><br>
	<?php endforeach; ?>
<?php endif; ?>

Le problème c'est que dès qu'il y a plusieurs cocktails à retourner, et bien seul le dernier résultat est pris en compte. En fait en faisant des debug je me suis vite aperçu qu'à chaque appelle de la fonction filter(), le foreach écrasait mes anciennes données à chaque fois. Seul le dernier résultat est pris en compte donc.

Je comprend très bien le problème donc mais je ne vois pas du tout comment faire pour y remédier.
Je m'en remet donc à vous car je sèche depuis déjà plusieurs jours sans trouver la solution.
J'espère que j'ai donner tout les éléments nécessaires pour que vous puissiez bien comprendre mon problème.

Merci en tous les cas de votre aide.

[rawsrc]

Bonjour,

Est-ce que tu pourrais essayer avec ceci :

Code php :

$d['cocktails'][] = $this->Cocktail->find('all', array('conditions' => array('Cocktail.id' => $x)));

Et tu n'as pas besoin de ça :

Code :

$v = current($v);

[kiradeus]

Bonjour rawsrc, merci de ta réponse !

J'ai fait comme tu dis et ça marche niquel sauf qu'il y a deux soucis maintenant :

1 - Il y a des doublons. Je m'explique, si un cocktail appartient à la catégorie 'Rhum' et 'Vodka', et que l'on sélectionne ces deux catégories dans le filtre, et bien le cocktail en question va s'afficher deux fois.

2 - Ensuite si j'utilise le $d['cocktails'][] dans le premier cas, et bien il faut que je change également le passage suivant :

Code :

1
2
3
4
5
else{
		$this->paginate = array('Cocktail' => array('limit' => 10));
		$cocktails = $this->Paginate('Cocktail', array('Cocktail.online' => 1, 'Cocktail.created <= NOW()'));
		$this->set($d);
		}

J'ai essayé d'utiliser également le $d['cocktails'][] ici mais seul le dernier cocktail de ma base de données s'affiche. Comprend pas trop...

Voici mon nouveau code :

Code :

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
function filter(){
		if(isset($_POST['categories'])){
			$term_id = $_POST['categories'];
			foreach($term_id as $k => $v){
				$termR = $this->Cocktail->TermR->find('list', array(
					'fields' => array('ref_id'),
					'conditions' => array('TermR.term_id' => $v)
				));
				foreach($termR as $w => $x){
					$d['cocktails'][] = $this->Cocktail->find('all', array(
						'conditions' => array('Cocktail.id' => $x)
					));
					$this->set($d);
				}
			}
		}
		else{
			$d['cocktails'][] = $this->Cocktail->find('all', array(
				'conditions' => array('Cocktail.online' => 1, 'Cocktail.created <= NOW()')
			));
			$this->set($d);
		}
	}

Et :

Code :

1
2
3
4
5
6
<?php if(!empty($cocktails)): ?>
	<?php foreach($cocktails as $k => $v): $v = current($v); ?>
		<?php echo $v['Cocktail']['name']; ?>
		<a href="<?php echo $this->Html->url($v['Cocktail']['link']); ?>" class="btn primary">Consulter</a><br>
	<?php endforeach; ?>
<?php endif; ?>

Je sais j'utilise encore le current mais si je l'enlève ça ne marche pas.
Et encore merci à toi car tu m'as déjà beaucoup fait avancé !

[rawsrc]

Envoyé par kiradeus

1 - Il y a des doublons. Je m'explique, si un cocktail appartient à la catégorie 'Rhum' et 'Vodka', et que l'on sélectionne ces deux catégories dans le filtre, et bien le cocktail en question va s'afficher deux fois.

Ça relève du SQL et pas du PHP.

Ensuite j'ai du mal à comprendre pourquoi tu ne récupères pas en une seule passe tes données ? Tu fais des foreach en pagaille.
Au vu de ton code, je pense que tu fais de l'héritage fonctionnel -> pas bien.
Regardes ici

Enfin, ta pagination semble exotique, je ne te cache pas que je ne comprend pas tout...