Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork5.3k
Remove ExpressionLanguage reference for 2.3 version#4548
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
Remove "Using Expressions for Complex Security Restrictions" section using ExpressionLanguage in 2.3 version.ExpressionLanguage is a 2.4 feature.
wouterj commentedNov 29, 2014
I'm 👍 @javiereguiluz I would like to hear your opinion about this |
stof commentedDec 6, 2014
👍 This should indeed not be documented on 2.3 given it is not usable there |
xabbuh commentedDec 6, 2014
We will probably need to take that this is added again into 2.5 and above after the 2.3 branch has been merged up. |
…zon)This PR was merged into the 2.3 branch.Discussion----------Remove ExpressionLanguage reference for 2.3 version| Q | A| ------------- | ---| Doc fix? | yes| New docs? | no| Applies to | 2.3 only| Fixed tickets |Remove "Using Expressions for Complex Security Restrictions" section using ExpressionLanguage in 2.3 version.ExpressionLanguage is a 2.4 feature.Commits-------d1e7334 Remove ExpressionLanguage reference for 2.3 version
weaverryan commentedDec 7, 2014
xabbuh commentedDec 7, 2014
Hi Ryan, looks good as far as I can see. 👍 Thank you! |
xabbuh commentedDec 7, 2014
@weaverryan Sorry, I didn't check carefully enough. One label needs to be added again (see#4607). |
wouterj commentedDec 8, 2014
@weaverryan so we go with different versions of the Best Practices guide? |
weaverryan commentedDec 8, 2014
@wouterj I know we talked about this before, and decided we didn't need it. But I don't see why we should avoid it. For example, imagine in 2.7 we think of some great new way to handle translations and we add it to the best practices. In that case, it could only live in the 2.7 best practices. So I think we should keep them identical,except when a best practice just simply can't be done. The other alternative is to keep them identical, but have a note on features like this: Available only in Symfony 2.4 and higher. |
Remove "Using Expressions for Complex Security Restrictions" section using ExpressionLanguage in 2.3 version.
ExpressionLanguage is a 2.4 feature.