MineWeb / MineWebCMS

🚀 A French Minecraft CMS since 2015 (used by +1k websites)
https://mineweb.org
GNU General Public License v3.0
93 stars 41 forks source link

Fix form data #305

Closed StanByes closed 2 years ago

StanByes commented 2 years ago

Le _Token et le xss du formulaire étaient détéctés dans la boucle d'initialisation rank/permissions donc il ajoutait dans le tableau une clé vide ('') pour ces deux champs.

nivcoo commented 2 years ago

ça marche quand tu enlève une permission aussi ?

StanByes commented 2 years ago

Non je viens de tester, je fais la modification tout de suite !

nivcoo commented 2 years ago

Change par

            if(is_array($checked))
                continue;
StanByes commented 2 years ago

Ca fonctionne presque tout le temps. Si tu essayes d'enlever toutes les permissions d'un rank ça ne fonctionnera pas

nivcoo commented 2 years ago

Normalement ce que j'ai fait fonctionne

StanByes commented 2 years ago

Bah ça fonctionne pour enlever des permissions mais tu peux pas enlever toutes les perms d'un grade car sinon le premier tableau ne stocke pas en clé l'id du rank et ne MAJ pas la table

nivcoo commented 2 years ago

Ouais je corrige ça

StanByes commented 2 years ago

Ok ;)