Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[DependencyInjection] Add autowiring capabilities #15613

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 19 commits into from
Closed

[DependencyInjection] Add autowiring capabilities #15613

Changes from 1 commit
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
9828da2
[DependencyInjection] Add autowiring capabilities
dunglas Aug 24, 2015
195f6c0
Fix CS
dunglas Aug 25, 2015
8d23c9d
Remove unused argument
dunglas Aug 25, 2015
27f4de8
Fix CS
dunglas Aug 25, 2015
5575ec1
Fix issues raised by @stof
dunglas Aug 25, 2015
edce23d
Better RunTime exception error.
dunglas Aug 26, 2015
07d475e
Fix CS
dunglas Aug 26, 2015
afe009a
Flag to enable/disable autowiring. no_autowiring tag.
dunglas Aug 27, 2015
7ab5b08
Fix some tests
dunglas Aug 27, 2015
a7ca3f2
Fix @meyerbaptiste comments
dunglas Aug 28, 2015
57c494c
Move the autowiring parameter to a setter
dunglas Aug 28, 2015
e642aa8
Fix SecurityBundle test
dunglas Aug 28, 2015
d00c7e3
Optin. Removed config flag.
dunglas Aug 28, 2015
55bb42c
Fix CS
dunglas Aug 30, 2015
7806487
[DependencyInjection] Rename types to autowiring_types
dunglas Sep 24, 2015
585616d
[DependencyInjection] Add the new 'autowire' syntax.
dunglas Sep 28, 2015
c717c69
Fix tests.
dunglas Sep 28, 2015
3b7f553
Fix @fabpot and @stof comments.
dunglas Oct 2, 2015
936a16a
Typehint against a non-existing class
dunglas Oct 2, 2015
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
Fix @meyerbaptiste comments
  • Loading branch information
@dunglas
dunglas committed Oct 1, 2015
commit a7ca3f231bc426d1f21b86e3e65c37e948b6ee9e
14 changes: 7 additions & 7 deletions src/Symfony/Component/DependencyInjection/Compiler/AutowiringPass.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -119,14 +119,14 @@ private function populateAvailableTypes()
}

/**
* Populates the of available types for a given definition.
* Populates the list of available types for a given definition.
*
* @param string $id
* @param Definition $definition
*/
private function populateAvailableType($id, Definition $definition)
{
if (!($class = $definition->getClass())) {
if (!$definition->getClass()) {
return;
}

Expand Down Expand Up @@ -172,19 +172,19 @@ private function extractAncestors($id, \ReflectionClass $reflectionClass)
}

/**
* Associates if applicable a type and a service id or a class.
* Associates a type and a service id if applicable.
*
* @param string $type
* @param string $value A service id or a class name depending of the value of $class
* @param string $id
*/
private function set($type, $value)
private function set($type, $id)
{
if (isset($this->definedTypes[$type]) || isset($this->notGuessableTypes[$type])) {
return;
}

if (isset($this->types[$type])) {
if ($this->types[$type] === $value) {
if ($this->types[$type] === $id) {
return;
}

Expand All @@ -194,7 +194,7 @@ private function set($type, $value)
return;
}

$this->types[$type] = $value;
$this->types[$type] = $id;
}

/**
Expand Down