Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Gestion de l'échappement lors de l'édition d'un thème #400

Open
a3nm opened this issue Apr 12, 2020 · 1 comment
Open

Gestion de l'échappement lors de l'édition d'un thème #400

a3nm opened this issue Apr 12, 2020 · 1 comment

Comments

@a3nm
Copy link

a3nm commented Apr 12, 2020

Bonjour,

Je crois avoir remarqué un petit bug : quand on édite un thème dans l'interface d'administration, avec /admin/parametres_edittpl.php, spécifiquement un fichier CSS, l'échappement des backslashes est mal géré. Par exemple en éditant le fichier theme.css par défaut, le fichier contient initialement "content: '\2630'", et quand on sauvegarde ce qui est enregistré est "content: '2630'" sans le ''. (Pour reproduire : sélectionner "theme.css" dans "Theme edit", faire un changement quelconque, sauvegarder, observer que "\2630" est devenu "2630".

Pour contourner le bug il faut doubler à la main les antislash à chaque fois avant de sauvegarder, mais ce serait mieux si cela pouvait être correctement géré.

Merci !

@bazooka07
Copy link
Collaborator

Bonjour,
Le problème avec "content:` '\2630';" vient de l'appel à la fonction plxUtils::unSlash(...) à la ligne n°43 dans le fichier core/admin/preprend.php. (PluXml version 5.8.3 )

Pour régler provisoirement ce problème, modifier la ligne 43 comme suit :

if(
   $_SERVER['REQUEST_METHOD'] == 'POST' and
   basename($_SERVER['PHP_SELF']) !== 'parametres_edittpl.php'
) {
    $_POST = plxUtils::unSlash($_POST);
}

Pour l'instant, je ne sais pas si l'appel à la méthode plxUtils::unSlash() présente un intérêt.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants