Skip to content
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.

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

[FrameworkBundle] AddExpressionLanguageProvidersPass #12847

Closed
maarekj opened this issue Dec 4, 2014 · 2 comments
Closed

[FrameworkBundle] AddExpressionLanguageProvidersPass #12847

maarekj opened this issue Dec 4, 2014 · 2 comments

Comments

@maarekj
Copy link
Contributor

maarekj commented Dec 4, 2014

Hi,
In the class AddExpressionLanguageProvidersPass, this code return always false:

$container->hasDefinition("router")

Meaning that the tag routing.expression_language_provider is never treated.

If I replace that code with this code it works:

 $container->has("router")

In fact "router" is alias not a definition.

@jakzal
Copy link
Contributor

jakzal commented Dec 13, 2014

@maarekj would you mind submitting a PR?

@maarekj
Copy link
Contributor Author

maarekj commented Dec 15, 2014

It's done, it's my first pull request on symfony.

@jakzal jakzal closed this as completed in fd97cef Jan 8, 2015
fabpot added a commit that referenced this issue Jan 8, 2015
…ases (maarekj, jakzal)

This PR was merged into the 2.7 branch.

Discussion
----------

Make that AddExpressionLanguageProviderPass works with aliases

| Q             | A
| ------------- | ---
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | #12847
| License       | MIT
| Doc PR        |

Replaces #12982 by @maarekj (includes his commit).

Commits
-------

16a22cd [FrameworkBundle] Add a test case for service aliases used with AddExpressionLanguageProviderPass.
fd97cef [FrameworkBundle] fixed #12847 AddExpressionLanguageProviderPass
fabpot added a commit that referenced this issue Jan 9, 2015
* 2.7: (22 commits)
  [DependencyInjection] deprecated synchronized services
  [FrameworkBundle] adds legacy tests for deprecated configuration keys.
  [TwigBundle] adds legacy tests for deprecated configuration keys.
  [PropertyAccessor] Added test to allow null value for a array
  [Yaml] Fixed #10597: Improved Yaml directive parsing
  [Validator] always use the lazy loading metadata factory
  [Validator] removed usage of deprecated getMessageParameters() and getMessagePluralization() in unit tests
  [Validator] fixed deprecation notices for BuildViolation() calls in constraints
  [Validator] fixed usage of deprecate Validator features
  [Validator] removed obsolete code
  removed the Validator BC layer for PHP < 5.3.9
  removed code for PHP < 5.3.9
  bumped min PHP version to 5.3.9
  fixed deprecation summary and missing error_reporting() in tests
  [Form] Fixed check of violation constraint #12792
  [FrameworkBundle] adds deprecation notice on framework.csrf_protection.field_name configuration key.
  [TwigBundle] adds missing deprecation notice for the twig.form.resources configuration key.
  [FrameworkBundle] avoid using deprecated classes for reflection
  [FrameworkBundle] Add a test case for service aliases used with AddExpressionLanguageProviderPass.
  [FrameworkBundle] fixed #12847 AddExpressionLanguageProviderPass
  ...

Conflicts:
	.travis.yml
	composer.json
	src/Symfony/Bridge/Doctrine/composer.json
	src/Symfony/Bridge/Monolog/composer.json
	src/Symfony/Bridge/Propel1/composer.json
	src/Symfony/Bridge/ProxyManager/composer.json
	src/Symfony/Bridge/Swiftmailer/composer.json
	src/Symfony/Bridge/Twig/composer.json
	src/Symfony/Bundle/DebugBundle/composer.json
	src/Symfony/Bundle/FrameworkBundle/DependencyInjection/Configuration.php
	src/Symfony/Bundle/FrameworkBundle/Tests/DependencyInjection/FrameworkExtensionTest.php
	src/Symfony/Bundle/FrameworkBundle/composer.json
	src/Symfony/Bundle/SecurityBundle/composer.json
	src/Symfony/Bundle/TwigBundle/composer.json
	src/Symfony/Bundle/WebProfilerBundle/composer.json
	src/Symfony/Component/BrowserKit/composer.json
	src/Symfony/Component/ClassLoader/composer.json
	src/Symfony/Component/Config/composer.json
	src/Symfony/Component/Console/composer.json
	src/Symfony/Component/CssSelector/composer.json
	src/Symfony/Component/Debug/composer.json
	src/Symfony/Component/DependencyInjection/composer.json
	src/Symfony/Component/DomCrawler/composer.json
	src/Symfony/Component/EventDispatcher/composer.json
	src/Symfony/Component/ExpressionLanguage/composer.json
	src/Symfony/Component/Filesystem/composer.json
	src/Symfony/Component/Finder/composer.json
	src/Symfony/Component/Form/Extension/Core/DataTransformer/DateTimeToStringTransformer.php
	src/Symfony/Component/Form/Tests/Extension/Core/DataTransformer/DateTimeToStringTransformerTest.php
	src/Symfony/Component/Form/composer.json
	src/Symfony/Component/HttpFoundation/composer.json
	src/Symfony/Component/HttpKernel/DataCollector/DumpDataCollector.php
	src/Symfony/Component/HttpKernel/composer.json
	src/Symfony/Component/Intl/DateFormatter/IntlDateFormatter.php
	src/Symfony/Component/Intl/Tests/DateFormatter/AbstractIntlDateFormatterTest.php
	src/Symfony/Component/Intl/Tests/DateFormatter/IntlDateFormatterTest.php
	src/Symfony/Component/Intl/composer.json
	src/Symfony/Component/Locale/composer.json
	src/Symfony/Component/OptionsResolver/composer.json
	src/Symfony/Component/Process/composer.json
	src/Symfony/Component/PropertyAccess/composer.json
	src/Symfony/Component/Routing/composer.json
	src/Symfony/Component/Security/Acl/composer.json
	src/Symfony/Component/Security/Core/composer.json
	src/Symfony/Component/Security/Csrf/composer.json
	src/Symfony/Component/Security/Http/composer.json
	src/Symfony/Component/Security/composer.json
	src/Symfony/Component/Serializer/composer.json
	src/Symfony/Component/Stopwatch/composer.json
	src/Symfony/Component/Templating/composer.json
	src/Symfony/Component/Translation/composer.json
	src/Symfony/Component/Validator/Tests/Validator/Abstract2Dot5ApiTest.php
	src/Symfony/Component/Validator/Tests/ValidatorBuilderTest.php
	src/Symfony/Component/Validator/composer.json
	src/Symfony/Component/VarDumper/composer.json
	src/Symfony/Component/Yaml/composer.json
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants