Discussion :

[grochenel]

Bonjour,

j'essayé de modifier une requete servant dans phpbb3.

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
 
$sql_ary = array(
			'SELECT'	=> 'u.user_id, u.user_colour, u.username, u.user_type',
			'FROM'		=> array(USERS_TABLE => 'u'),
			'LEFT_JOIN'	=> array(
				array(
					'FROM'	=> array(SESSIONS_TABLE => 's'),
					'ON'	=> 's.session_user_id = u.user_id',
				),
			),
			'WHERE'		=> 'u.user_lastvisit > ' . (time() - 86400) . ' OR s.session_user_id <> ' . ANONYMOUS,
			'GROUP_BY'	=> 'u.user_id',
			'ORDER_BY'	=> 'u.username',
		);
 
		$result = $db->sql_query($db->sql_build_query('SELECT', $sql_ary));

Je voudrai ajouter une condition pour ne pas lister le nom des bots qui ont un group_id à 6. J'ai essayé celle ci mais ça ne fonctionne pas.

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
 
$sql_ary = array(
			'SELECT'	=> 'u.user_id, u.user_colour, u.username, u.user_type',
			'FROM'		=> array(USERS_TABLE => 'u'),
			'LEFT_JOIN'	=> array(
				array(
					'FROM'	=> array(SESSIONS_TABLE => 's'),
					'ON'	=> 's.session_user_id = u.user_id',
				),
			),
			'WHERE'		=> 'u.user_lastvisit > ' . (time() - 86400) . ' OR s.session_user_id <> ' . ANONYMOUS . ' AND u.group_id <> 6 ',
			'GROUP_BY'	=> 'u.user_id',
			'ORDER_BY'	=> 'u.username',
		);
 
		$result = $db->sql_query($db->sql_build_query('SELECT', $sql_ary));

quel est le probleme ?

Merci

[CinePhil]

mais ça ne fonctionne pas.

Symptômes ?

Elle est horrible cette requête !
Déjà qu'elle est écrite en pseudo SQL et du coup quasi illisible mais en plus :
- Un GROUP BY alors qu'il n'y a pas de fonction de calcul dans le SELECT, c'est pas terrible ! Il vaudrait mieux utiliser DISTINCT. En plus, toutes les colonnes du SELECT devraient figurer dans le GROUP BY !
- Il y a une condition dans le WHERE qui porte sur la table de droite, ce qui revient à transformer la jointure externe gauche en jointure interne !

Si toutes les requêtes de phpBB sont comme ça, on se demande comment ça arrive à fonctionner correctement !

[ericd69]

oui hélas c'est pas très joli ni en php ni en mysql phpbb, heureusement que c'est produit par une équipe de "pros" et qu'ils en sont à la version 3.x hein?

Vu la doc qu'ils donnent j'avoue que c'est pas toujours évident d'adapter leurs requêtes dans leur templates...

C'est vrai que sans l'erreur on peut rien te dire...

[grochenel]

Bonjour,

en réalité je n'ai pas d'erreur affichée. La condition pour ne pas selectionner les bots ne fonctionnent pas.

Code :

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

AND u.group_id <> 6

.

Je vais regarder pour créer une toute nouvelle requete avec jointure d'apres la doc ici et supprimer l'autre.

bien à vous

[ericd69]

Essaye de faire la requête sans passer par système de pseudo code...

Vérifie qu'avec seulement la condition u.group_id = 6 tu obtiens bien les bots

[Yanika_bzh]

cela ne serait pas du a une priorisation des clauses ??

Code :

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

'WHERE ('		=> 'u.user_lastvisit > ' . (time() - 86400) . ' OR s.session_user_id <> ' . ANONYMOUS . ') AND u.group_id <> 6 '

Essayez ...

Bon courage

[grochenel]

bonjour,

grace à votre aide je suis parvenu à une requete fonctionnelle.

Code :

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

1
2
 
'WHERE'		=> ' u.group_id <> 6 AND (u.user_lastvisit > ' . (time() - 86400) . ' OR s.session_user_id <> ' . ANONYMOUS .')' ,

Merci