Skip to content

Added docs about ArgumentValueResolvers #6438

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.

Sign up for GitHub

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

Closed
wants to merge 17 commits into from
Closed

Added docs about ArgumentValueResolvers #6438

Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
7fc9fe8
Added docs about ArgumentValueResolvers
Apr 6, 2016
7ddf5d6
we>you, php>PHP, fixed directive start & list style
Apr 6, 2016
1a19d2e
Fixed the list indents
Apr 6, 2016
f491199
Added missing use statements to example
Apr 6, 2016
a07fcc3
Processed feedback from @wouterj
Apr 7, 2016
2e41c07
Added some extra info about the DefaultValueResolver
Apr 11, 2016
7cdc96b
Fixed a typo in code example
Apr 11, 2016
7d00d8c
Fixed a wrong tag name
Apr 12, 2016
f9cbe71
Added instanceof check in example
Apr 12, 2016
4c6ed2a
Respecting the line length
Apr 12, 2016
9784406
Fixed the tag name
Apr 14, 2016
e3d1b48
Tried to make the default value resolving more clear
Apr 18, 2016
55a87b5
Minor fixes
Apr 29, 2016
8d30575
Added instance
May 6, 2016
dd225e8
Feedback from PR
May 17, 2016
ca014a6
Fixed sub-title case
May 17, 2016
14d77e1
typehinted > type-hinted
May 17, 2016
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
Prev Previous commit
Next Next commit
Added missing use statements to example
  • Loading branch information
Iltar van der Berg committed Apr 29, 2016
commit f491199c2ae8946ca83232fb12ec71a67b79b178
3 changes: 3 additions & 0 deletions cookbook/controller/argument_value_resolver.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -45,6 +45,9 @@ action::

namespace AppBundle\Controller;

use AppBundle\User;
Copy link
Member

Choose a reason for hiding this comment

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

AppBundle\Entity\User? (I know you're probably not to fond of having this in the Entity namespace, but I think it's the most familar thing for doc readers)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I prefer consistency and familiarity for the developer, so I don't really mind either way. I'll update this to be in the Entity namespace.

use Symfony\Component\HttpFoundation\Response;

class UserController
{
public function indexAction(User $user)
Copy link
Contributor

Choose a reason for hiding this comment

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

I understand correctly comments below, you should add "= null" in the signature, right ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Only when it's optional

Expand Down
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy