Erreur plutôt que DB fail sur affichage page inexistante

Erreur plutôt que DB fail sur affichage page inexistante was created by lcollong

Posted 2 years 10 months ago #61051
Bonjour,

Ce type d'url : www.mondomaine.fr/fr/?option=com_pagebuilderck&view=page&id=486 permet d'afficher une page en front. C'est celle qui est générée par l'icône "oeil" dans le backend quand on construit une page. Cela permet de se rendre du rendu dans le template. Très pratique.

Mais si l'id de la page n'existe pas, cela provoque une erreur base de données :

Exception Type: JDatabaseExceptionExecuting
File: /home/lactfrwpxp/www/libraries/joomla/database/driver/mysqli.php
Line: 668
Message: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '' at line 1
#0 /home/lactfrwpxp/www/plugins/system/falangdriver/falang_database.php(105): JDatabaseDriverMysqli->execute()
#1 /home/lactfrwpxp/www/administrator/components/com_pagebuilderck/helpers/ckfof.php(185): JFalangDatabase->execute()
#2 /home/lactfrwpxp/www/components/com_pagebuilderck/models/page.php(65): Pagebuilderck\CKFof::dbExecute('UPDATE #__pageb...')
#3 /home/lactfrwpxp/www/components/com_pagebuilderck/views/page/view.html.php(27): PagebuilderckModelPage->getItem(486)
#4 /home/lactfrwpxp/www/administrator/components/com_pagebuilderck/helpers/ckcontroller.php(197): PagebuilderckViewPage->display()

Pas sûr que le problème soit sur PageBuilder. C'est peut-être Falang qui gère mal la situation dons son override (#1). Mais il me semble que le getItem (#3) devrait permettre de sortir "proprement" si la page n'existe pas. Pourquoi est-ce suivi d'un "update" (#2) ?

Notre site est soumis à ne nombreuses tentatives "d'attaques" révélées par adminTools. Il serait préférable de faire un throwError "accès interdit" qui ne donne pas d'explications plutôt qu'un "plantage" DB qui peut donner des "envies" à un pirate (SQL injection)...

Please Log in or Create an account to join the conversation.

Replied by ced1870 on topic Erreur plutôt que DB fail sur affichage page inexistante

Posted 2 years 10 months ago #61052
Salut
merci pour le retour d'infos, je vais regarder ce point
CEd

Joomlack Webmaster and Developer

by ced1870

Please Log in or Create an account to join the conversation.

Replied by ced1870 on topic Erreur plutôt que DB fail sur affichage page inexistante

Posted 2 years 10 months ago #61053
PS : le update c'est pour incrémenter le compteur de visite de page :)

Joomlack Webmaster and Developer

by ced1870

Please Log in or Create an account to join the conversation.

Replied by ced1870 on topic Erreur plutôt que DB fail sur affichage page inexistante

Posted 2 years 10 months ago #61054
essaie en remplacant ce fichier
www.dropbox.com/s/bao2nqv8g6x7otw/page.php?dl=0
dans
components\com_pagebuilderck\models
et dis moi ce que ça donne

Joomlack Webmaster and Developer

by ced1870

Please Log in or Create an account to join the conversation.

Replied by lcollong on topic Erreur plutôt que DB fail sur affichage page inexistante

Posted 2 years 10 months ago #61061
ça c'est du support éclair !
J'ai testé dans la foulée puis reporté en prod.
Impeccable !
Merci.

Please Log in or Create an account to join the conversation.

Time to create page: 0.678 seconds

Fast and powerful creation, customizable and responsive.

Read More

We have 984 guests and no members online