Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[SeoBundle] Avoid using "and" operator, using && #468

Merged
merged 1 commit into from
Jun 2, 2015
Merged

[SeoBundle] Avoid using "and" operator, using && #468

merged 1 commit into from
Jun 2, 2015

Conversation

denbatte
Copy link
Contributor

@denbatte denbatte commented Jun 2, 2015

The and operator does not have the same precedence as &&.
This could lead to unexpected behavior, use && instead.

Q A
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Fixed tickets #467

This could lead to unexpected behavior, use && instead.
@denbatte denbatte modified the milestones: 3.1.2, 3.2.0 Jun 2, 2015
@denbatte denbatte assigned arneruy and denbatte and unassigned arneruy Jun 2, 2015
roderik pushed a commit that referenced this pull request Jun 2, 2015
[SeoBundle] Avoid using "and" operator, using &&
@roderik roderik merged commit 7fdd398 into Kunstmaan:master Jun 2, 2015
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants