THELIA Forum

Bonjour,

En pleine création d'un module qui doit permettre de rajouter facilement une balise meta noindex dans certaines de nos pages, je me retrouve bloqué. J'ai crée un hook dans ce module pour vérifier si l'url de la page actuellement visitée faisait partie de celle qu'on ne veut pas indexer pour le référencement (auquel cas la méthode getIndex() retourne 0).

namespace Indexation\Hook;

use Indexation\Model\IndexationQuery;
use Thelia\Core\Event\Hook\HookRenderEvent;
use Thelia\Core\Hook\BaseHook;

class Front extends BaseHook {
    public function onNoindexDisplay(HookRenderEvent $event) {
        $pageURL = $this->getRequest()->getPathInfo();

        $indexation = IndexationQuery::create()->findByUrl($pageURL);
        if(null !== $currentURL && null !== $pageURL){
            foreach($indexation->getUrl() as $url){
                if($url == $pageURL){
                    $event->add('<meta name="robots" content="noindex">');
                }
            }
        }
    }

}

Et dans le fichier config.xml de ce module, j'ai rajouté ceci :

    <hooks>
        <hook id="indexation.hook.front" class="Indexation/Hook/Front" scope="request">
            <tag name="hook.event_listener" event="noindex.display" />
        </hook>
    </hooks>

Je ne sais pas encore ce que va donner ce hook parce que le site (FO comme BO) est inaccessible. J'ai cette erreur que je ne comprends pas :
InvalidArgumentException in RegisterHookListenersPass.php line 81:
Hook class "Indexation/Hook/Front" must extends class "Thelia\Core\Hook\BaseHook".

J'ai pourtant bien un extends vers BaseHook, l'erreur doit être ailleurs.
Quelqu'un avec un regard extérieur peut me dire ce qui ne va pas avec ma classe, s'il-vous-plaît ?

PS : j'ai bien vidé les caches par suppression du contenu du dossier cache, et les logs Thelia ne sont pas parlants du tout.

Last edited by HeishPi (26-04-2019 08:00:15)

Effectivement, c'est le genre d'erreur bête dont je ne me serais jamais aperçu seul. merci Roadster !

Maintenant, j'ai droit à l'erreur "Hook noindex.display is unknown"

Sur un autre topic, je t'ai vu expliquer ceci :

J'ai complètement supprimé mon module, l'ai réinstallé et je lui ai remis le hook, le schema.xml et le config.xml que j'avais afin qu'il prenne le hook en compte, le tout avant de générer le model, et pourtant j'ai toujours la même erreur...

Hum, j'obtiens la même erreur mais dans l'invite de commande cette fois

PHP Fatal error:  Uncaught exception 'InvalidArgumentException' with message 'Hook noindex.display is unknown.' in blablabla

Rooh, je me suis précipité dans la partie Hook de la Doc sans même m'apercevoir que y avait un onglet "Create your own hook". Je me fatigue, parfois.
Donc c'est fait ! J'ai écrit ça dans la classe principale, à la racine du module :

namespace Indexation;

use Thelia\Core\Template\TemplateDefinition;
use Thelia\Module\BaseModule;

class Indexation extends BaseModule
{
    /** @var string */
    const DOMAIN_NAME = 'indexation';

    /*
     * You may now override BaseModuleInterface methods, such as:
     * install, destroy, preActivation, postActivation, preDeactivation, postDeactivation
     *
     * Have fun !
     */

    public function getHooks()
    {
        return array(
            // Only register the title in the default language
            array(
                "type" => TemplateDefinition::BACK_OFFICE,
                "code" => "noindex.display",
                "title" => "IndexationHook",
                "description" => "Permet de rajouter la balise noindex",
            ),
            array(
                "active" => true
            )
        );
    }
}

J'ai désactivé/réactivé mon module, le hook apparaît bien dans la table "hook" de la BDD, et pourtant j'ai toujours l'erreur Hook noindex.display is unknown (mais seulement en mode développement !).
J'ai essayé de faire un hook:clean mais la console m'a demandé si je voulais supprimer tous les hook de mon module. Si je dis Yes, j'obtiens à nouveau la même erreur dans l'invite de commande.

Est-ce que j'aurais du supprimer ce que j'avais déjà mis dans config.xml et dans le hook ?

Non non, ça ne marche ni en prod ni en dév mais ça ne m'étonne pas trop vu que je n'ai jamais pu vérifier si le contenu de mon hook est fonctionnel. Ce qu'il y a c'est que je n'obtiens cette page d'erreur QUE en mode dév !

Aaaaaaaah, effectivement ! Je n'ai plus d'erreur maintenant, merci beaucoup Roadster !

Purge les hooks avant de changer le type

Qu'est-ce que je peux faire différemment pour contrer ça ?

Par ailleurs, est-ce que c'est vraiment grave d'utiliser un hook backOffice sur un template frontOffice ?

Ah, j'aurai du commencé par là. J'ai refait la manip mais j'ai forcé le changement en base, je lui ai filé le bon type en manipulant directement les données et là ça fonctionne parfaitement. Le hook a même très bien rempli son rôle !

Merci encore Roadster, ton aide fait gagner un temps précieux aux 3/4 du forum

Si jamais quelqu'un passe par là et cherche à faire la même chose (bloquer l'indexation de certaines pages), voilà la tronche du hook que j'utilise :

public function onNoindexDisplay(HookRenderEvent $event) {
        $actualPage = (isset($_SERVER['HTTPS']) && $_SERVER['HTTPS'] === 'on' ? "https" : "http") . "://$_SERVER[HTTP_HOST]$_SERVER[REQUEST_URI]";

        $indexationTab = IndexationQuery::create()->find();
        if(null !== $indexationTab && null !== $actualPage){
            foreach($indexationTab as $url){
                if($url->getUrl() == $actualPage && $url->getIndex() == 0){
                    $event->add('<meta name="robots" content="noindex">');
                }
            }
        }
    }

J'ai une table Indexation dans ma BDD qui contient toutes les url possibles de mon site, avec pour chacune un booléen (ici appelé index) qui dit si on veut l'indexer (1) ou non (0). C'est géré en backOffice pour que le client final puisse décider quelle page il veut soustraire au référencement.