Petit Bug sur le Composant de Recherche

[Fly06_Fr]

Bonjour,

Pour info, je pense avoir trouvé un petit bug sur com_search.

En effectuant une recherche sur une J2.5Fr fraîchement installée avec les options de recherche par défaut, en particulier le type de recherche (tous les mots), j'obtiens ceci :

• Recherche de "specifique" (sans accent) : 0 résultat
• Recherche de "joomla" : 52 résultats
• Recherche de "specifique joomla" : 1 résultat

Pour cette dernière recherche, le bon nombre de résultat devrait être 0.

A priori, le problème doit venir du fait que la partie MySql de la recherche est accent-insensitive (du fait de la collation utf8_general_ci) tandis que la partie php de la recherche est accent-sensitive...

[Fly06_Fr]

Re : Petit Bug sur le Composant de Recherche

Après analyse, le problème vient de la méthode checkNoHtml() de la class SearchHelper.

Les requêtes SQL des plugins de recherche retournent les contenus où sont présents les mots clés 'joomla' et 'spécifique' car la collation utf8_general_ci considère la lettre 'e' ses dérivées accentués comme équivalentes. La méthode checkNoHtml() est ensuite appelée pour supprimer des résultats de recherches les contenus où les mots clés sont présents non pas dans du texte mais dans du html (tag, attributs, valeurs d'attributs, javascript et css).

Le problème vient de cette partie du code de la méthode checkNoHtml() :

foreach($terms AS $term) {
    if(JString::stristr($text, $term) !== false) {
        return true;
    }
} 

Ce code retourne true (résultat confirmé) dès qu'un des mots clés saisis est trouvé.

Ce code n'est pas correct lorsque le paramètre 'searchphrase' vaut 'all' (Tous les mots) puisque dans ce cas il faudrait au contraire renvoyer false dès qu'un des mots clés n'est pas trouvé.

Dans le cas d'une recherche sur 'joomla specifique', le mot clé 'specifique' est invalide car non présent (la méthode stristr() de la classe JString considère la lettre 'e' et ses dérivées accentuées comme des lettres différentes) mais la présence du seul mot clé 'joomla' suffit à valider le résultat.

Le problème ne se pose pas uniquement avec les caractères accentués.

Une recherche (dans les articles uniquement) de 'align' retourne aucun résultat alors qu'une recherche de 'joomla align' retourne 19 résultats. Le mot clé 'align' est bien invalide car présent dans certaines balises html (en tant qu'attribut) mais, comme dans l'exemple précédent, la seule présence du mot clé 'joomla' valide le résultat.

Pour corriger ce problème de cohérence, il faudrait réecrire la méthode checkNoHtml() pour qu'elle tienne compte de la valeur du paramètre 'searchphrase'.