OpenEdition / lodel

Science publishing CMS
GNU General Public License v2.0
50 stars 27 forks source link

Pb class.entities_index.php (reconstruire index de recherche) avec php 7 #95

Open ssonnet opened 5 years ago

ssonnet commented 5 years ago

Bonjour,

Le problème rencontré est au lancement de la fonctionnalité "Moteur de recherche : reconstruire l'index" sous php 7. On obtient les erreurs suivantes (cf. https://github.com/OpenEdition/lodel/issues/91) en boucle :

PHP Error (Warning) in file '/var/www/html/lodel/lodel/scripts/logic/class.entities_index.php' on line 187 : 
preg_replace(): The /e modifier is no longer supported, use preg_replace_callback instead
PHP Error (Warning) in file '/var/www/html/lodel/lodel/scripts/logic/class.entities_index.php' on line 188 : 
preg_replace(): The /e modifier is no longer supported, use preg_replace_callback instead

On nous indique que ce ne sont que des warnings... sauf que concrètement la table search_engine n'est pas alimentée : la recherche est donc inopérationnelle.

Le "coupable" est la méthode _decode_html_entities() dans class.entities_index.php. La méthode en question utilise la fonction preg_replace() avec un modifier "/e", ce qui n'est pas compatible avec php 7. Il faut utiliser preg_replace_callback() à la place. J'ai donc modifié le code comme suit :

protected function _decode_html_entities($text) 
{
    //$text= preg_replace('/&#(\d+);/me',utf8_encode("chr(\\1)"),$text); #decimal notation
        $text = preg_replace_callback(
            '/&#(\d+);/m',
            function ($matches) {
                return utf8_encode(chr($matches[1]));
            },
            $text
        );
    //$text= preg_replace('/&#x([a-f0-9]+);/mei',utf8_encode("chr(0x\\1)"),$text);  #hex notation
        $text = preg_replace_callback(
            '/&#x([a-f0-9]+);/mi',
            function ($matches) {
                return utf8_encode(chr('0x' . $matches[1]));
            },
            $text
        );
    return html_entity_decode($text);
}

Avec ces modifications, ça passe sur mon installation (branche master de Lodel + php 7) : la table search_engine est à présent alimentée. Je me suis permis de rajouter un html_entity_decode() sur le renvoi final au passage...

Je précise que les données sur lesquelles je travaille sont des données provenant initialement d'un site sous Lodel 0.7, migré en 0.9, puis en 1.x...

Est-il envisageable d'apporter ces modifications à votre code ?

Cordialement. Sylvain Sonnet

nahuelange commented 5 years ago

Bonjour, Je pense que la meilleure manière d'intégrer les modifications que tu proposes, c'est de faire des pull requests.