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

[RFR-ne pas merge] add Route choice type for page form #3

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

Nsy
Copy link
Member

@Nsy Nsy commented Jan 17, 2019

Petite update 11 mois plus tard après tests:

J'ai un probleme car pour le RouteChoiceType.php je me sert de la liste des routes (la même qu'on recupere quand on fait ./bin/console route:debug Or cette liste a les routes custom grâce au CmsRoutesLoader.php (core-bundle opera) qui lui même load les liste qui sont assigné a une page.
Probleme ex:

initial: J'ai une page article qui utilise la route page_article, je retrouve donc cette route dans mon RouteChoiceType et quand je fais route:debug.
step1: je retire cette route de la page.
step2: si je fait route:debug: la route page_article n'est plus là car n'est plus lié a une page (voir CmsRoutesLoader du core bundle) et elle n'existe plus dans mon RouteChoiceType donc je ne peut plus la re-selectionner
step3: je peut pas la reselectionner, la route a disparu :(

@Nsy Nsy changed the title RFR add Route choice type for page form [RFR-ne pas merge] add Route choice type for page form Dec 11, 2019
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

Successfully merging this pull request may close these issues.

2 participants