? Pending

User tests: Successful: Unsuccessful:

avatar photodude
photodude
26 Dec 2017

Pull Request for Issue code style fixes.

Summary of Changes

  • Blank line found at start of control structure
  • No blank line found after control structure
  • Expected 1 space before "?"; 0 found
  • Expected 1 space after "="; 2 found
  • Expected 0 spaces before closing bracket; 1 found
  • Please consider an empty line before the if statement;
  • Please consider an empty line before the switch statement
  • Please consider an empty line before the foreach statement
  • Line indented incorrectly
  • Please consider an empty line before the return statement;
  • New line between unrelated comments

Automatically fixed with Joomla code standards 2.0.0 PHPCS2-alpha2 fixers

None of the manual only fixes have been applied

Testing Instructions

Merge by code review

Expected result

code style has been applied as listed above, old code style testing on drone does not error.

Actual result

code style had not been applied. Autofixers from the Joomla code standards 2.0.0 PHPCS2 alpha2 were used to implement fixable code style

Documentation Changes Required

none

avatar photodude photodude - open - 26 Dec 2017
avatar photodude photodude - change - 26 Dec 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 26 Dec 2017
Category Front End com_config com_contact com_content com_finder com_newsfeeds com_tags com_users
avatar photodude photodude - change - 26 Dec 2017
The description was changed
avatar photodude photodude - edited - 26 Dec 2017
avatar Quy Quy - test_item - 26 Dec 2017 - Tested successfully
avatar Quy
Quy - comment - 26 Dec 2017

I have tested this item successfully on 58ab8be


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

avatar wilsonge wilsonge - change - 27 Dec 2017
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-12-27 20:30:32
Closed_By wilsonge
Labels Added: ?
avatar wilsonge wilsonge - close - 27 Dec 2017
avatar wilsonge wilsonge - merge - 27 Dec 2017

Add a Comment

Login with GitHub to post a comment