oui de mémoire je crois que j'ai déjà corrigé ça ...
en fait le sujet du mail ne supporte pas les entités html et donc on avait et on a probablement encore quelques petite erreurs de ce genre à corriger

il faudra que je re- regarde celà ...
ça y es celui la est corrigé je vais le corriger sur le labo aussi et tu me diras ... si ça persiste

Citation : phr
Je ne saisis pas la raison du Pb.
Il faudrait reproduire l'incident qui n'a rien à voir à priori avec une injection.
Ce serait un soucis dans le form et des contrôles dans l'admin.

 Message édité par : phr / 08-02-2022 11:11

Voici la copie d’écran d’admin et remarques qu’il n’y a plus de point dans afficher les derniers ajouts alors que j’ai ajouté false dans le fichier config

Y’a un truc

oui il y a un problème avec les boutons radio quand il sont à non il renvoient false .... mais quand ça revient par le fichier config vu qu'on a écrit false sans protection et bien la variable est false ... et ne correspond pas au formulaire .... c'est à priori valable pour tout les radio de la série .....

dans

/modules/npds_galerie/admin/adm_func.php

tu remplace les lignes correspondantes par ça ....
(j'ai corrigé dans le labo)

   $content.= "// Personnalisation de l'affichage\n";

   $content.= "\$view_alea = \"$viewalea\";\n";

   $content.= "\$view_last = \"$viewlast\";\n";

   $content.= "\$aff_vote = \"$vote\";\n";

   $content.= "\$aff_comm = \"$comm\";\n\n";

   $content.= "// Autorisations pour les anonymes\n";

   $content.= "\$vote_anon = \"$votano\";\n";

   $content.= "\$comm_anon = \"$comano\";\n";

   $content.= "\$post_anon = \"$postano\";\n\n";

   $content.= "// Notification admin par email de la proposition\n";

   $content.= "\$notif_admin = \"$notifadmin\";\n\n";

Citation : jpb
dans

/modules/npds_galerie/admin/adm_func.php

tu remplace les lignes correspondantes par ça ....
(j'ai corrigé dans le labo)

   $content.= "// Personnalisation de l'affichage\n";

   $content.= "\$view_alea = \"$viewalea\";\n";

   $content.= "\$view_last = \"$viewlast\";\n";

   $content.= "\$aff_vote = \"$vote\";\n";

   $content.= "\$aff_comm = \"$comm\";\n\n";

   $content.= "// Autorisations pour les anonymes\n";

   $content.= "\$vote_anon = \"$votano\";\n";

   $content.= "\$comm_anon = \"$comano\";\n";

   $content.= "\$post_anon = \"$postano\";\n\n";

   $content.= "// Notification admin par email de la proposition\n";

   $content.= "\$notif_admin = \"$notifadmin\";\n\n";

patrick RICHARD

t'inquiètes, je voulais juste le signaler pour corriger si ce n'était pas déjà fait

donc cool et merci

oui j'ai corrigé et updaté la version sur github... (car c'était une grosse bétise qu'on avait pas vu ....)

mais il faut que tu appliques la correction sinon tes config ne fonctionneront pas ....

heu je viens de le faire,
mais,
même si je valide avec des modifications l'affichage reviens toujours à OUI ?

si je remets l'ancienne version elle conserve les choix effectuées mais n'affiche pas le selecteur

oui tu as raison ... j'ai pas assez modifié .....

Citation : jpb
oui tu as raison ... j'ai pas assez modifié .....

patrick RICHARD

😂

ça saoule j'en ai plein la tête lol ...
bon je vais regarder ça à tête reposée car cette modif en entraine d'autres dans d'autres fichier quand on utilise ces valeurs ! .......