? Success

User tests: Successful: Unsuccessful:

avatar frankmayer
frankmayer
16 Dec 2016

Summary of Changes

  • Inline one-time use variables
  • Removed unnecessary ternary operators
  • Merge unset() calls
  • Use modern day type-casting
  • This can be replaced with type-casting

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.

avatar frankmayer frankmayer - open - 16 Dec 2016
avatar frankmayer frankmayer - change - 16 Dec 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 16 Dec 2016
Category Libraries
avatar RonakParmar
RonakParmar - comment - 16 Dec 2016

I have tested this item successfully on cf060f6

Done code review.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/13241.

avatar RonakParmar RonakParmar - test_item - 16 Dec 2016 - Tested successfully
avatar anibalsanchez
anibalsanchez - comment - 18 Dec 2016

I have tested this item successfully on cf060f6

Test OK


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/13241.

avatar anibalsanchez anibalsanchez - test_item - 18 Dec 2016 - Tested successfully
avatar jeckodevelopment jeckodevelopment - change - 18 Dec 2016
Status Pending Ready to Commit
avatar jeckodevelopment
jeckodevelopment - comment - 18 Dec 2016

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/13241.

avatar jeckodevelopment jeckodevelopment - change - 18 Dec 2016
Milestone Added:
avatar rdeutz rdeutz - change - 18 Dec 2016
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-12-18 20:25:50
Closed_By rdeutz
Labels Added: ?
avatar rdeutz rdeutz - close - 18 Dec 2016
avatar rdeutz rdeutz - merge - 18 Dec 2016
avatar rdeutz rdeutz - reference | aa583c3 - 18 Dec 16
avatar rdeutz rdeutz - merge - 18 Dec 2016
avatar rdeutz rdeutz - close - 18 Dec 2016
avatar cpfeifer cpfeifer - reference | fb3e6ba - 22 Dec 16
avatar frankmayer frankmayer - head_ref_deleted - 25 Dec 2016

Add a Comment

Login with GitHub to post a comment