search

OpenEdition / lodel

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

Warning PHP depuis la dernière mise à jour #138

Open nahuelange opened 1 year ago

nahuelange commented 1 year ago

Bonjour,

Depuis la dernière mise à jour de sécurité, un warning PHP est soulevé systématiquement à l'édition d'une entité.

https://github.com/OpenEdition/lodel/blob/master/lodel/scripts/context.php#L253 La valeur de $v est un tableau, preg_match ne peut pas fonctionner sur les tableaux.

@lnprieto @jcsouplet

brrd commented 1 year ago

@lnprieto peut-on espérer un correctif dans les prochaines semaines ?

Pour info, voici le message d'erreur : 

PHP Error (Warning) in file 'chemin/vers/lodel/lodel/scripts/context.php' on line 253 : 
preg_match() expects parameter 2 to be string, array given

Warnings seem to appear on this page. You may go on anyway by following this link. Please report the problem to help us to improve Lodel.
lnprieto commented 1 year ago

C'est en cours de validation, cela devrait être fait très vite

Le mer. 18 janv. 2023 à 12:30, Thomas Brouard @.***> a écrit :

@lnprieto https://github.com/lnprieto peut-on espérer un correctif dans les prochaines semaines ?

Pour info, voici le message d'erreur :

PHP Error (Warning) in file 'chemin/vers/lodel/lodel/scripts/context.php' on line 253 :

preg_match() expects parameter 2 to be string, array given

Warnings seem to appear on this page. You may go on anyway by following this link. Please report the problem to help us to improve Lodel.

— Reply to this email directly, view it on GitHub https://github.com/OpenEdition/lodel/issues/138#issuecomment-1386905274, or unsubscribe https://github.com/notifications/unsubscribe-auth/AG3E6JTDEUFMDSXPDJX5JJTWS7H4FANCNFSM6AAAAAATS26XVE . You are receiving this because you were mentioned.Message ID: @.***>

brrd commented 1 year ago

@lnprieto merci

nahuelange commented 1 year ago

Bonjour, Malgré le patch, le problème persiste. La récursivité des array n'ayant pas été prise en compte. Prévoyez-vous de déployer un nouveau correctif ?

brrd commented 6 months ago

FTR, ce problème est toujours d'actualité et assez gênant à l'utilisation sous environnement de développement.

image

Le code teste bien le type array mais ne prends pas en compte le fait qu'un array peut lui-même contenir un array.

Voir ligne 256 : https://github.com/OpenEdition/lodel/commit/a921de20e3f5b2ce5064c1cc19340f5dbeeffcc5#diff-c5c7a83d86838b3c576d3a2ecdd448de07b095aacc256888e43621bc49267467