Feeling Lucky
?
[#13240] - Simplify some ternary operations using elvis operator and remove unnecessary parentheses in libraries/legacy
- Fixed in Code Base
- 13 Jun 2017
- Medium
- Build: staging
- # 13240
- Diff
- frankmayer:elvis-and-unneeded-parentheses-in-libraries-legacy
- Success continuous-integration/drone/pr the build was successful Details
- Success continuous-integration/travis-ci/pr The Travis CI build passed Details
- Success JTracker/HumanTestResults Human Test Results: 2 Successful 0 Failed. Details
- Success continuous-integration/appveyor/pr AppVeyor build succeeded Details
User tests: Successful: Unsuccessful:
Summary of Changes
- Simplify some ternary operations using elvis operator and remove unnecessary parentheses in libraries/legacy
This PR is part of a set to try to separate some of the changes done in one of my previous batch PR's for the libraries/legacy directory, which is still on hold (#12220).
Once the new set is merged it will hopefully reduce the changes in that PR, so it can be reviewed easier and finally be merged.
The changes in this PR should be fairly easy to review. In hope that this will get merged quickly. ;)
Testing Instructions
None, should not change behavior
Documentation Changes Required
None.
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Libraries |
| Title |
|
||||||
| Title |
|
||||||
| Labels |
Added:
?
|
||
Quy
- comment
- 1 Jun 2017
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/13240.
andrepereiradasilva
- comment
- 1 Jun 2017
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/13240.
| Status | Pending | ⇒ | Ready to Commit |
franz-wohlkoenig
- comment
- 1 Jun 2017
RTC after two successful tests.
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2017-06-13 13:10:00 |
| Closed_By | ⇒ | rdeutz |
Conflicts resolved. Pls check and merge.