Skip to content

Conversation

maarekj
Copy link
Contributor

@maarekj maarekj commented Dec 15, 2014

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

@stof
Copy link
Member

stof commented Jan 4, 2015

Please add a test to avoid regressions

@maarekj
Copy link
Contributor Author

maarekj commented Jan 8, 2015

I do not know what to test.

@jakzal
Copy link
Contributor

jakzal commented Jan 8, 2015

@maarekj I wrote test cases for you. See #13336. Thanks for the contribution!

@jakzal jakzal closed this Jan 8, 2015
fabpot added a commit that referenced this pull request 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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants