Dec 2, 2013 · Dec 2, 2013 · Dec 2, 2013 · Mar 5, 2014 · Mar 5, 2014 · Mar 5, 2014
diff --git a/book/routing.rst b/book/routing.rst

        return $collection;

 .. note::

   When defining routes, the key (e.g. ``blog_show``) is meaningless.
   Just be sure that it's unique so no other lines override it. In most cases, a
   vendorname_shortbundlename_controllername_actionname pattern
   (e.g. ``acme_blog_show``) is appropriate.

 .. versionadded:: 2.2
    The ``path`` option is new in Symfony2.2, ``pattern`` is used in older
    versions.
Original file line number	Diff line number	Diff line change
Expand Up		@@ -68,6 +68,13 @@ The route is simple:

		return $collection;

		.. note::

		When defining routes, the key (e.g. ``blog_show``) is meaningless.
Copy link Member wouterjDec 2, 2013 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. it's not meaningless, you use it to reference to the route (for instance when generating a route in a template or controller) Copy link ContributorAuthor fatmuemooDec 2, 2013 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. I concur. "Meaningless" is the wrong word. "Arbitrary" would be a better way of describing the route name. Copy link Member weaverryanMar 5, 2014 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. I think we should move this to the last sentence of the next paragraph - so right after`in your controller (keep reading).` - how about: The`blog_show` is the internal name of the route, which just doesn't have any meaning yet and just needs to be unique. Later, you'll use it to generate URLs. I think if we do this (feel free to re-word), then we're in good shape! Copy link Member xabbuhMar 5, 2014 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. We should at least avoid the first person perspective. Copy link Member weaverryanMar 5, 2014 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. Thanks - updated my example above! Copy link Member weaverryanMar 18, 2014 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. @fatmuemoo What do you think about my proposed change above? Can you update your PR? If not, just let me know and someone else can open a new PR with the tweak. Cheers! Copy link ContributorAuthor fatmuemooApr 26, 2014 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. Sounds good to me, thanks Copy link Member weaverryanMay 2, 2014 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. @fatmuemoo :) - ok - but can you also update your pull request with the suggested changes? I can also update it for you if you'd like! Cheers!
		Just be sure that it's unique so no other lines override it. In most cases, a
		vendorname_shortbundlename_controllername_actionname pattern
		(e.g. ``acme_blog_show``) is appropriate.

		.. versionadded:: 2.2
		The ``path`` option is new in Symfony2.2, ``pattern`` is used in older
		versions.
Expand Down