Anomalie #25956
Décocher toutes les urls d'une app fait planter l'enregistrement
100%
Description
il y a un nouveau menu quand on décoche une url d'une app qui propose soit de la mettre en orpheline soit de sélectionner une appli
le problème est que si on clique sur décocher tout (le bouton en haut à gauche) ce menu n'apparait pas et du coup le post[url_id] n'est pas renseigné
- soit on ne fait pas apparaître le menu si on décoche tout (et tout va en orpheline)
- soit on le fait apparaitre pour toutes les urls (avec par défaut orpheline)
J'essaie de régler ça, il faudrait corriger avant la publication
Associated revisions
Fixes #25956 : Décocher toutes les urls d'une app fait planter l'enregistrement - Problème d'accolade fermante du foreach
History
#1 Updated by Renaud Dussol over 4 years ago
Il y a d'autres problèmes
$post[url_id] n'est pas assigné, ce qui génère un warning... je ne sais pas si cela fait planter ou pas...
ET SURTOUT : l'accolade fermante du foreach (correctif sur mon oubli précédent) est placé" trop loin, ce qui fait que les redirects ne se font pas (le tavbleau $ids est vide donc pas de foreach, d'ailleurs
Ce foreach devrait se terminer plus tôt (je pense juste avant le \R::storeAll($urlToStore);)
Il y a un autre truc qui est embêtant c'est que dans ce cas on ne peut pas modifier les urls car le foreach ne se fera pas comme url_id est vide...
mais bon ce n'est pas trop grave car cette case sert seulement à "vider" rapidement une appli, en général on ne s’amuse pas à modifier les urls puis à les retirer toutes de l'appli
#2 Updated by Renaud Dussol over 4 years ago
en fait tout le pb vient de la correction sur la } qui manquait
c'était bcp plus haut (dans // Si Icon en FS on la modifie)
la } fermante du foreach était bien présente (et elle est bien plus haut)
je corrige
#3 Updated by Renaud Dussol over 4 years ago
- Status changed from Nouveau to Résolu
- % Done changed from 0 to 100
Appliqué par commit dba00a98791ef637b5f82c792e88fa383fcc64b5.
#4 Updated by Arnaud FORNEROT over 4 years ago
- Status changed from Résolu to Fermé