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

Skip to content

[ExpressionLanguage] add XOR operator #34329

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

Merged
merged 1 commit into from
Nov 12, 2019
Merged

Conversation

ottaviano
Copy link
Contributor

Q A
Branch? 4.4
Bug fix? no
New feature? yes
Deprecations? no
Tickets ~
License MIT
Doc PR ~

👋
little PR for adding the best logical operator: XOR

@nicolas-grekas
Copy link
Member

Thank you @ottaviano.

nicolas-grekas added a commit that referenced this pull request Nov 12, 2019
This PR was merged into the 4.4 branch.

Discussion
----------

[ExpressionLanguage] add XOR operator

| Q             | A
| ------------- | ---
| Branch?       | 4.4
| Bug fix?      | no
| New feature?  | yes
| Deprecations? | no
| Tickets       | ~
| License       | MIT
| Doc PR        | ~

👋
little PR for adding the best logical operator: `XOR`

Commits
-------

46fe917 [ExpressionLanguage] add XOR operator
@nicolas-grekas nicolas-grekas merged commit 46fe917 into symfony:4.4 Nov 12, 2019
@ottaviano ottaviano deleted the exp-xor branch November 12, 2019 14:17
@fabpot
Copy link
Member

fabpot commented Nov 12, 2019

I'm against this addition. I made a conscious decision to not add such an operator to ExpressionLanguage.

nicolas-grekas added a commit to nicolas-grekas/symfony that referenced this pull request Nov 12, 2019
…ottaviano)"

This reverts commit d1d4bc8, reversing
changes made to f63976f.
nicolas-grekas added a commit that referenced this pull request Nov 12, 2019
…erator (ottaviano)" (nicolas-grekas)

This PR was merged into the 4.4 branch.

Discussion
----------

Revert "feature #34329 [ExpressionLanguage] add XOR operator (ottaviano)"

| Q             | A
| ------------- | ---
| Branch?       | 4.4
| Bug fix?      | no
| New feature?  | yes
| Deprecations? | no
| Tickets       | -
| License       | MIT
| Doc PR        | -

This reverts commit d1d4bc8, reversing
changes made to f63976f.

Commits
-------

adb716d Revert "feature #34329 [ExpressionLanguage] add XOR operator (ottaviano)"
nicolas-grekas added a commit that referenced this pull request Nov 12, 2019
* 4.4:
  Revert "feature #34329 [ExpressionLanguage] add XOR operator (ottaviano)"
  Allow \Throwable $previous everywhere
@ottaviano
Copy link
Contributor Author

ottaviano commented Nov 12, 2019

@fabpot, ah, I did not find any issue/discussion on this. It would be nice to know the reason. thanks

This was referenced Nov 12, 2019
hultberg pushed a commit to hultberg/symfony that referenced this pull request Sep 17, 2021
* 4.4:
  Revert "feature symfony#34329 [ExpressionLanguage] add XOR operator (ottaviano)"
  Allow \Throwable $previous everywhere
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.

5 participants