-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
[Cookbook] [Routing] Article about custom route loaders #2339
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
[Cookbook] [Routing] Article about custom route loaders #2339
Conversation
I noticed the problem with my previous pull request (it was against master) - this is much better... |
Creating a Custom Loader | ||
------------------------ | ||
|
||
To load routes in another way then using annotations, Yaml or XML files, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
then => than ?
Thanks for your feedback! |
with the given resource (``@AcmeDemoBundle/Controller/DemoController.php``) | ||
and type ("annotation") as arguments. When one of the loader returns ``true``, | ||
its method :method:`Symfony\\Component\\Config\\Loader\\LoaderInterface::load` | ||
will be called, and the loader returns a :class:`Symfony\\Component\\Routing\\RouteCollection` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
its method ... gets called in favor of will be called ?
Thanks for the feedback - very good points. I will continue my work on this in a couple of days. |
@e-solving Please don't repeat your question somewhere else when an already given answer in another place is not to your satisfaction. Remember that all people involved in this pull request will see your answer too (just like mine in fact ;)) and that it has no business here. |
I don't repeat the question, my question is why my route new.yml doesn't load the resource... can somebody help me? |
@e-solving there are thousands of people who want to help you, but not here. And yes, you are repeating your question. (4 times to be exact) |
@e-solving do it on matthias blog, or on the forum or on the mailing list for users, those are the channels. |
@wouterj My question is differente isn't how to create a routing.loader class, is that everthing is fine, except when in my news.yml don't load the resource. the problem isn't my custom routing.loader, is why don't load ther resource from the routing, that is all... understand?, @cordoval i did it on matthias blog, but anybody answer... @matthiasnoback please can u see my problem?, can u help me? |
@e-solving we understand your question, but we can't answer it here. The places you asked it now (symfony/symfony and symfony/symfony-docs) are repositories where people talk about the core code and the documentation. This is the place were new features will be discussed. That way, we can seperate these things so people can watch the places they want to contribute. A lot of people get notifications now, because you asked the question here, but all these people aren't interested in helping people, they are interested in working the documentation. If you ask it on the forum, a lot of people will get notified and they love it: They watched the forum because they want to help people. |
@e-solving si tu respondes en el mailing list te podemos ayudar, pero no aca |
I have processed part of your feedback (thanks again!). I was not sure about mentioning other bundles with custom route loaders - is this common practice in the Symfony documentation? Of course some of these bundles are "here to stay", but it adds a bit of instability to this article. |
We use more and more references to other bundles and code. I think it's a great way to show the users some use cases and to show them how to implement this in a real Symfony2 application. |
@wouterj Fine! Just did that. Good luck today by the way! |
I will try, the answer, thanks again @matthiasnoback and @wouterj and good luck too! |
…ader [Cookbook] [Routing] Article about custom route loaders
Hey Matthias! Thanks for adding another very nice entry to the docs! I've proofread this, but made only very minor changes (thanks to everyone else for making the merge so easy). Cheers! |
Thanks for merging! |
I've created a new cookbook article about custom route loaders (see also #526). Feedback is much appreciated.
There is a small difference for version 2.1:
should be changed to
As well as the corresponding
use
at the top.How should I add this to my pull request?