Ce site est optimisé pour être consulté depuis un navigateur moderne dans lequel JavaScript est activé.

Formulaire front-office renvoie '0' sans erreur

juliedeco47

Bonjour à tous,

Je suis confrontée à un souci assez frustrant sur ma boutique PrestaShop 1.7.4.3. Quand je soumets un formulaire depuis le front-office via mon contrôleur question.php (qui fait partie du module cfrompro), la page affiche parfois juste un '0' et rien d'autre. Pas de message d'erreur, pas de succès, rien. Le formulaire n'est pas envoyé, et je me retrouve bloquée sans savoir pourquoi.

Ce contrôleur est censé gérer une requête AJAX pour un questionnaire produit. Le souci, c'est que ce die('0') rend le débogage impossible, surtout quand la vérification du secure_key échoue. Je ne comprends pas ce qui se passe et cela arrive aléatoirement.

Voici le fragment de code concerné dans cfrompro/controllers/front/question.php:

function initContent()
    {
        if (!Module::isEnabled('cfrompro') || Tools::getValue('action') != 'sendQuestion' || Tools::getValue('secure_key') != $this->module->secure_key) {
            die('0');
        }
        $customerMail = trim((stri

Est-ce que quelqu'un aurait une idée de comment mieux gérer cette sortie pour avoir des messages plus explicites, ou pourquoi j'obtiens ce '0' pur et simple ?

oceane_75

Salut @juliedeco47,

Je connais bien ce genre de problème avec PrestaShop 1.7 et les contrôleurs de module. Le die('0') est une pratique un peu archaïque et très peu informative, surtout pour des requêtes AJAX. Le fait que ça renvoie un '0' sec veut souvent dire qu'une condition n'est pas remplie, et ton module coupe court à l'exécution.

Quelle est la version exacte de ton PrestaShop ? Et est-ce que tu es sûr que tous les paramètres action et secure_key sont toujours correctement passés dans ta requête AJAX ? Tu devrais ouvrir la console de ton navigateur (onglet Réseau) pour voir la requête et la réponse brute.

pierre75

Exactement ce que dit @oceane_75. Ce die('0') est le pire ami du développeur ! Pour une requête AJAX, il faut toujours renvoyer une réponse structurée (du JSON par exemple) même en cas d'erreur, et surtout ne pas laisser le contrôleur rendre le template complet du thème, ce qui peut créer des '0' ou du HTML inattendu.

Dans ton ModuleFrontController, as-tu bien pensé à définir $this->ajax = true; quelque part, par exemple dans ton initContent() ou postProcess() ? Sans ça, PrestaShop essaie de construire une page complète, ce qui n'est pas ce que tu veux pour une requête AJAX. Ça pourrait expliquer les sorties bizarres, y compris des morceaux de HTML en plus de ton '0'.

Milo

Bonjour @juliedeco47,

Le problème que vous rencontrez est un classique avec les ModuleFrontController et les requêtes AJAX sur PrestaShop 1.7.4.3. Le die('0') est effectivement problématique car il interrompt brutalement l'exécution sans donner de feedback utile au client JavaScript qui a initié la requête.

Pour gérer correctement une requête AJAX dans un ModuleFrontController, il faut deux choses :

Indiquer à PrestaShop que la requête est de type AJAX en définissant $this->ajax = true; dans votre méthode initContent() ou postProcess(). Cela empêche PrestaShop de charger le template complet du thème.
Retourner des données au format JSON (ou autre format structuré) et utiliser exit; pour stopper l'exécution après l'envoi de la réponse, au lieu d'un simple die('0').

Pour votre extrait de code, la modification suivante permettrait de renvoyer une réponse JSON explicite en cas d'échec de la vérification du secure_key:

function initContent()
    {
        if (!Module::isEnabled('cfrompro') || Tools::getValue('action') != 'sendQuestion' || Tools::getValue('secure_key') != $this->module->secure_key) {
            header('Content-Type: application/json');
            echo json_encode(['success' => false, 'message' => 'Accès non autorisé ou paramètres manquants.']);
            exit;
        }
        $customerMail = trim((stri

N'oubliez pas d'ajouter également $this->ajax = true; au début de votre initContent() ou dans postProcess() pour être sûr que le contrôleur se comporte comme un contrôleur AJAX pur. Cela devrait résoudre les sorties '0' ou les fragments HTML non désirés.