Requetes Sql introuvable Options

[Guest_asocom_*]

je n'arrive pas a trouver ou est cette requete sql:

CODE

1064 - Erreur de syntaxe près de 'SELECT p.products_image, pd.products_name, p.products_id, p.man' à la ligne 1

select count(p.products_id) as total SELECT p.products_image, pd.products_name, p.products_id, p.manufacturers_id, p.products_price, p.products_tax_class_id, IF(s.status, s.specials_new_products_price, NULL) as specials_new_products_price, IF(s.status, s.specials_new_products_price, p.products_price) as final_price FROM products_description pd, products p left join manufacturers m on p.manufacturers_id = m.manufacturers_id left join specials s on p.products_id = s.products_id, products_to_categories p2c WHERE p.products_status = '1' AND p.products_id = p2c.products_id AND pd.products_id = p2c.products_id AND pd.language_id = '4' AND p2c.categories_id = '5'

c'est lorsque j'essaye d'afficher le listing des produits au niveau des sous categories

Merci d'avance.

[shoprun]

Salut

Cette requête est d'abord construite dans index.php, mais c'est la classe split_page_result qui reconstitue celle ci.
Tout cela pour connaître et pouvoir afficher le nombre de résultat, suivant, précédant, ...

Mais en quoi cette requête te traquasse ?

[Guest_asocom_*]

elle me tracasse parcequ'elle me provoque une erreur

1064 - Erreur de syntaxe près de 'SELECT p.products_image, pd.products_name, p.products_id, p.man' à la ligne 1

select count(p.products_id) as total SELECT p.products_image, pd.products_name, p.products_id, p.manufacturers_id, p.products_price, p.products_tax_class_id, IF(s.status, s.specials_new_products_price, NULL) as specials_new_products_price, IF(s.status, s.specials_new_products_price, p.products_price) as final_price FROM products_description pd, products p left join manufacturers m on p.manufacturers_id = m.manufacturers_id left join specials s on p.products_id = s.products_id, products_to_categories p2c WHERE p.products_status = '1' AND p.products_id = p2c.products_id AND pd.products_id = p2c.products_id AND pd.language_id = '4' AND p2c.categories_id = '5'

[TEP STOP]

et que je n'arrive meme pas a ciblé ou je dois la corriger donc si tu pouvait m'aider je serais tres tres reconnaissant

[fissiaux]

T'as 2 select !!!

select count(p.products_id) as total SELECT p.products_image, p

[shoprun]

Pffff, sorry, j'avais vu.

Bon, ben sais pas trop si c'est cela, de plus plus je me suis jamais trop attardé ce que faisait ce split_page_result, mais il me semble avoir déjà eu ce genre d'erreurs.

Faut totalement éviter les MAJUSCULES dans la requête, du moins pour toutes ces instructions :
select, distinct, from, where, group by, order by, limit.
Et aussi il faut totalement éviter les sauts de lignes.

Bref, ce split_page_result reconstitue la requête en recherchant toutes ces instructions par leur position. D'où le respect de la casse et les sauts de lignes.

Dans ton code je vois le where et le from en majuscule.
Remplace cela pour voir.

[fissiaux]

Non, la casse sur les ordres SQL n'intervient pas dans une requete....

D'ailleurs, sur le site de Mysql, toutes les ordres SQL sont en majuscules.

[shoprun]

Absolument, j'en suis parfaitement d'accord Fissiaux.

Mais je parle très exactement de ce que fait la classe split_page result.php.
Et j'ai remarqué que celle ci reconstitue la requête, justement pour placer un select count(*), en gros pour connaitre le nombre d'enregistrement, placer les boutons suivant, précédent, et de mettre les bons chiffres dans le limit.

Bref, pour cela on est en php, du coup la casse doit être respectée, car comme je l'ai précisé, cette classe recherche par leur position les divers instructions que j'ai cité.
Je viens encore de faire le test, c'est imparable, et cela est parfaitement justifié.

Je suis étonné que tu n'ai pas remarqué cela.
Je dirais plutôt que j'ai mal expliqué le shmilblick.

Ce message a été modifié par shoprun - 13 Aug 2007, 19:59.

[fissiaux]

Sa requete est syntaxiquement fausse....

C'est pas un problème de casse

[fissiaux]

Et pour terminer (de mon côté), il n'a pas une erreur PHP, il a une erreur MYSQL. Donc ton explication de la classe split_page... est erronée.

[Sweet]

Je vais rejoindre Fissiaux voir appuyer son analyse.

select count(p.products_id) as total SELECT p.products_image...

arrivé là, pas d'alternative : la requête part en vrille..

Un mélange de requête ?

Sweet

[fissiaux]

http://dev.mysql.com/doc/refman/5.0/fr/man...onventions.html

En milieu de page :

Les mot réservés SQL ne sont pas sensibles à la casse, et peuvent être écrits en majuscules ou minuscules. Ce manuel utilise les majuscules.

[shoprun]

Que la syntaxe soit fausse, je veux bien (je n'ai pas approfondi le truc), mais je suis (quasi) certain qu'il faudrait d'abord corriger son histoire de from et de where qui est en majuscule (2 seconde chono pour le faire) car sans cela la requête plantera obligatoirement.

Puis comme je viens de dire, je fais à l'instant le test en mettant le from en majuscule, et bien comme c'est bizrrare, j'obtient quasi la même erreur de syntaxe :
select count(p.products_id) as total select p.products_image_pte, ........FROM .........

Troublant non ?

Je ne pense pas car justement, cette classe recherche un from et non un FROM, du coup la requête est déglinguée.

[Sweet]

Que la syntaxe soit fausse, je veux bien (je n'ai pas approfondi le truc),

select count(p.products_id) as total SELECT p.products_image...

Je ne comprends pas .. arrivé ici pourquoi aller plus loin ça ne peut PAS fonctionner...

pour le FROM ou from j'en doute mais je suis curieux et je vais faire l'essai (mais pas sur cette requête)

Sweet

[fissiaux]

T'as raison :

CODE

      $pos_from = strpos($this->sql_query, ' from', 0);

      $pos_group_by = strpos($this->sql_query, ' group by', $pos_from);
      if (($pos_group_by < $pos_to) && ($pos_group_by != false)) $pos_to = $pos_group_by;

      $pos_having = strpos($this->sql_query, ' having', $pos_from);
      if (($pos_having < $pos_to) && ($pos_having != false)) $pos_to = $pos_having;

      $pos_order_by = strpos($this->sql_query, ' order by', $pos_from);
      if (($pos_order_by < $pos_to) && ($pos_order_by != false)) $pos_to = $pos_order_by;

      if (strpos($this->sql_query, 'distinct') || strpos($this->sql_query, 'group by')) {
        $count_string = 'distinct ' . tep_db_input($count_key);
      } else {
        $count_string = tep_db_input($count_key);
      }

extrait de la classe en question.


Mais j'ai raison aussi, sur le principe général !!!

C'est de la mauvaise programmation

[shoprun]

Alors là je suis scotché.

Je persiste à dire que le blème vient du faite que c'est dû à la classes split_page_result.php
Que mysql ne tient pas compte de la casse je le sais déjà.

Je vous invite à faire l'exprérience dans ce index.php par exemple, dans la condition :
elseif ($category_depth == 'products' || isset($_GET['manufacturers_id'])) {

et mettez y des majuscule sur les from ou where, ça plantera à coup sûr.
En tout cas ça belle lurette que j'ai remarqué cela, et j'en ai (en gros) l'explication.

Et c'est bien cela que j'ai remarqué dans sa requête.

[fissiaux]

Relis mon précédent post : je te donnais raison.

Mais tu t'exprimais mal (enfin, c'est mon excuse).

[shoprun]

T'as raison :

Ouf, merci aspro

C'est de la mauvaise programmation

C'est le moins que l'on peu dire.

De plus, je suis certain qu'il y avait moyen de faire cela en acceptant les minuscule et majuscule.
M'enfin, à partir du moment qu'on le sait, on ne ce fait plus prendre.

En tout cas je me suis cassé les dents la dessus dès le début.

[Sweet]

Méa culpa !

En plus pour moi c'est bénéfique

Le pauvre membre à l'origine du message va y perdre son latin

Là je reconnais être allé trop vite.
Bien vu Shoprun !!

Désolé

Sweet

[Gnidhal]

Oooops
j'arrive après la bataille.
Perso j'ai déjà galéré sur cette fonction un jour où je cherchais à différencier les requêtes avec des minuscules ou des majuscules pour pister une erreur et je me suis permis de la modifier comme ça :

CODE

  class splitPageResults {
    function splitPageResults(&$current_page_number, $max_rows_per_page, &$sql_query, &$query_num_rows) {
      if (empty($current_page_number)) $current_page_number = 1;

      $pos_to = strlen($sql_query);
      $pos_from = stripos($sql_query, ' from', 0);

      $pos_group_by = stripos($sql_query, ' group by', $pos_from);
      if (($pos_group_by < $pos_to) && ($pos_group_by != false)) $pos_to = $pos_group_by;

      $pos_having = stripos($sql_query, ' having', $pos_from);
      if (($pos_having < $pos_to) && ($pos_having != false)) $pos_to = $pos_having;

      $pos_order_by = stripos($sql_query, ' order by', $pos_from);
      if (($pos_order_by < $pos_to) && ($pos_order_by != false)) $pos_to = $pos_order_by;

      $reviews_count_query = tep_db_query("select count(*) as total " . substr($sql_query, $pos_from, ($pos_to - $pos_from)));
      $reviews_count = tep_db_fetch_array($reviews_count_query);
      $query_num_rows = $reviews_count['total'];

      $num_pages = ceil($query_num_rows / $max_rows_per_page);
      if ($current_page_number > $num_pages) {
        $current_page_number = $num_pages;
      }
      $offset = ($max_rows_per_page * ($current_page_number - 1));
      $sql_query .= " limit " . max($offset, 0) . ", " . $max_rows_per_page;
    }

car ce problème de casse me les cassait justement !

pour info stripos() est insensible à la casse contrairement à strpos()

[fissiaux]

Je connaissais pas ce bug.....

Et j'en ai pas vraiment trouvé trace sur le forum US !!

Bien vu à vous deux.

[Sweet]

pour info stripos() est insensible à la casse contrairement à strpos()

Adopté !!!

Merci pour ce bout de code !

Sweet

[shoprun]

j'arrive après la bataille.

A bah pas du tout

car ce problème de casse me les cassait justement !

Et bien on est au moins 2.

Et bien très sincèrement, j'allais un de ces jours me retrousser les manches pour rendre tout cela plus conventionnel.
Et bien grâce à toi je crois bien que tu va m'épargner cette peine

Oh merci Gnidhal

[Guest_asocom_*]

Bonjour donc au final je fais quoi?

je remplace la fonction strpos par stripos?
ou je remets des miniscule dans toutes les requetes sql que j'ai faites ?


ps merci a vous tous pour votre aide (sincerement)

asocom

[fissiaux]

Tu remplaces la fonction strpos par stripos....dans un premier temps.

[Guest_asocom_*]

bonjour
pour la permutation entre stripos et strpos ca marche pas j'ai un message d'erreur comme quoi la fonction est indefini.