Skip to content
This repository was archived by the owner on Sep 16, 2021. It is now read-only.

Use Route annotations #577

Merged
merged 2 commits into from
Oct 20, 2014
Merged

Use Route annotations #577

merged 2 commits into from
Oct 20, 2014

Conversation

SimonVanherweghe
Copy link
Contributor

Without the use statement, this din't worked

Also I had to set the routing.yml from the bundle to::
AcmeBasicCmsBundle:
resource: "@AcmeBasicCmsBundle/Controller"
type: annotation

Without the use statement, this din't worked

Also I had to set the routing.yml from the bundle to::
    AcmeBasicCmsBundle:
      resource: "@AcmeBasicCmsBundle/Controller"
      type: annotation
@@ -329,6 +329,7 @@ the page action::
// src/Acme/BasicCmsBundle/Controller/DefaultController.php

// ...
use Sensio\Bundle\FrameworkExtraBundle\Configuration\Route;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i think we should have a blank line between the use statement and the class.

@dbu
Copy link
Member

dbu commented Sep 26, 2014

agreed, same problem as #570 - are there more annotations in the tutorial with the same issue?

dbu added a commit that referenced this pull request Oct 20, 2014
@dbu dbu merged commit 05e9d45 into symfony-cmf:master Oct 20, 2014
@dbu
Copy link
Member

dbu commented Oct 20, 2014

thanks a lot for the fix!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants