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

Use statement Template #570

Closed
wants to merge 3 commits into from
Closed

Use statement Template #570

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
a9949e1
Use statement Template
SimonVanherweghe Sep 18, 2014
6b7e70d
Extra line between use statement and controller
SimonVanherweghe Sep 25, 2014
97f6d4e
removed spaces
SimonVanherweghe Sep 26, 2014
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 4 additions & 2 deletions tutorial/content-to-controllers.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ Map the ``$routes`` property to contain a collection of all the routes which ref
document::

// src/AcmeBundle/BasicCmsBundle/Document/ContentTrait.php

// ...
trait ContentTrait
{
Expand Down Expand Up @@ -97,8 +97,10 @@ Now create the action in the default controller - you can pass the ``Page``
object and all the ``Posts`` to the view::

// src/Acme/BasicCmsBundle/Controller/DefaultController.php

Copy link
Member

Choose a reason for hiding this comment

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

sorry to say but this should not have unneeded blanks here. can you fix that and then squash your commits?

(i configure my editors to remove all trailing whitespace on save)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Eh, I'm confused: #577 (comment)

Copy link
Member

Choose a reason for hiding this comment

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

You've added spaces on an empty line. You should remove the spaces, but keep the blank line

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Aah ok, I wasn't aware, sorry

// ...
use Sensio\Bundle\FrameworkExtraBundle\Configuration\Template;

class DefaultController extends Controller
{
// ...
Expand Down