Project

General

Profile

Anomalie #25956

Décocher toutes les urls d'une app fait planter l'enregistrement

Added by Renaud Dussol over 1 year ago. Updated over 1 year ago.

Status:
Fermé
Priority:
Haut
Assigned To:
Target version:
Start date:
11/20/2018
Due date:
% Done:

100%

Distribution:

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

Revision dba00a98 (diff)
Added by Renaud Dussol over 1 year ago

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 1 year 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 1 year 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 1 year ago

  • Status changed from Nouveau to Résolu
  • % Done changed from 0 to 100

#4 Updated by Arnaud FORNEROT over 1 year ago

  • Status changed from Résolu to Fermé

Also available in: Atom PDF