Feeling Lucky
?
[#13216] - Various changes in modules
- Fixed in Code Base
- 16 Dec 2016
- Medium
- Build: staging
- # 13216
- Diff
- frankmayer:various-changes-in-modules
- Success continuous-integration/drone 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
User tests: Successful: Unsuccessful:
Summary of Changes
- Simplify ternary operation with elvis operator
- Remove unnecessary parentheses
- Remove redundant variables
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 modules, which is still on hold (#12229).
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 are few and very 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 | ⇒ | Modules Front End |
| Labels |
Added:
?
|
||
ggppdk
- comment
- 16 Dec 2016
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/13216.
| Status | Pending | ⇒ | Ready to Commit |
jeckodevelopment
- comment
- 16 Dec 2016
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/13216.
| Milestone |
Added: |
||
rdeutz
- | Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2016-12-16 14:22:06 |
| Closed_By | ⇒ | rdeutz | |
| Labels |
Added:
?
|
||
cpfeifer
-
I have tested this item✅ successfully on 1fbe4be
Code review.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/13216.