? ? Success

User tests: Successful: Unsuccessful:

avatar nonumber
nonumber
4 Mar 2015

This series of Pull Requests improve some basic code styles.
Mostly if/else syntax improvements, focussing on removing else statements which follow returns (or continues in loops).
Also de-nesting of if structures by adding early returns/elses.

Example:

function foobar($foo, $bar)
{
   if($foo > $bar)
   {
      return $foo;
   } else {
      return $bar;
   }
}

Can be changed to:

function foobar($foo, $bar)
{
   if($foo > $bar)
   {
      return $foo;
   }

   return $bar;
}

PS: This only illustrates the 'issue' with the useless else statements.

Current PRs in this series:

avatar nonumber nonumber - open - 4 Mar 2015
avatar joomla-cms-bot joomla-cms-bot - change - 4 Mar 2015
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - change - 4 Mar 2015
Labels Added: ?
avatar zero-24 zero-24 - change - 6 Mar 2015
Category Code style
avatar nonumber nonumber - change - 7 Mar 2015
Title
Edit Improved syntax: elses after returns/continues [com_categories]
Edit Improved syntax: Reducing use of else statements, adding early returns [com_categories]
avatar nonumber nonumber - change - 8 Mar 2015
Title
Edit Improved syntax: Reducing use of else statements, adding early returns [com_categories]
Improved syntax: Reducing use of else statements, adding early returns [com_categories]
avatar nonumber nonumber - change - 8 Mar 2015
Title
Improved syntax: Reducing use of else statements, adding early returns [com_categories]
[CODE SYNTAX] Reducing use of else statements, adding early returns [com_categories]
avatar nonumber nonumber - change - 8 Mar 2015
Title
[CODE SYNTAX] Reducing use of else statements, adding early returns [com_categories]
[CODE STYLE] Reducing use of else statements, adding early returns [com_categories]
avatar nonumber nonumber - change - 8 Mar 2015
Title
[CODE SYNTAX] Reducing use of else statements, adding early returns [com_categories]
[CODE STYLE] Reducing use of else statements, adding early returns [com_categories]
avatar brianteeman brianteeman - change - 23 Jun 2015
Labels Added: ?
avatar nonumber
nonumber - comment - 21 Jul 2015

If this doesn't find any interest within a week or 2, I'll close this PR and create new PRs with smaller fixes (one fix per PR).

avatar Duke3D
Duke3D - comment - 24 Jul 2015

Patches successfully applied.

No @test instruction so did a superficial scan of the diff for functions likely to be affected. Exercised Category Manager in admin by adding, removing, and editing categories. Categories List view filters and ordering worked as expected. Tested category filters in article manager and changing categories in an article. Executed a batch change of category for a group of articles. No changes detected to front end display of category blogs. Leaving patch applied to working site that content is being added into - will report if anything comes up. Nice work on the cleanup!!!!!!!!!


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

avatar Duke3D Duke3D - test_item - 24 Jul 2015 - Tested successfully
avatar nonumber
nonumber - comment - 10 Aug 2015

Due to lack of interest (read: not enough testing), this is now closed.

avatar nonumber nonumber - close - 10 Aug 2015
avatar nonumber nonumber - change - 10 Aug 2015
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2015-08-10 18:22:55
Closed_By nonumber
avatar nonumber nonumber - close - 10 Aug 2015
avatar nonumber nonumber - head_ref_deleted - 27 Oct 2015

Add a Comment

Login with GitHub to post a comment