Joomla! Issue Tracker - CMS

Download
Launch
Feeling Lucky

[#11495] - Code Style Fix for administrator/components/com_checkin/

? Pending

User tests: Successful: Unsuccessful:

avatar photodude
photodude
6 Aug 2016

Pull Request for code style issues.

Summary of Changes

  • Please consider an empty line before the if statement;

Automatically fixed with PHPCS2 fixers

Testing Instructions

Merge by code review

Documentation Changes Required

none

avatar joomla-cms-bot joomla-cms-bot - change - 6 Aug 2016
Category ⇒ Administration Components
avatar photodude photodude - open - 6 Aug 2016
avatar photodude photodude - change - 6 Aug 2016
Status New ⇒ Pending
avatar joomla-cms-bot joomla-cms-bot - change - 6 Aug 2016
Labels Added: ?
avatar rdeutz rdeutz - change - 6 Aug 2016
Milestone Added:
Status Pending ⇒ Ready to Commit
avatar rdeutz
rdeutz - comment - 6 Aug 2016

can be merged on code review

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

avatar joomla-cms-bot joomla-cms-bot - change - 6 Aug 2016
Labels Added: ?
avatar brianteeman brianteeman - change - 6 Aug 2016
Category Administration Components ⇒ Code style
avatar brianteeman brianteeman - change - 6 Aug 2016
Labels
avatar alikon alikon - test_item - 7 Aug 2016 - Tested successfully
avatar alikon
alikon - comment - 7 Aug 2016

I have tested this item ✅ successfully on c60e450

on code review

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

avatar rdeutz rdeutz - alter_testresult - 7 Aug 2016 - deutz: Tested successfully
avatar joomla-cms-bot joomla-cms-bot - change - 7 Aug 2016
Category Code style ⇒ Administration Components Code style
avatar wilsonge wilsonge - change - 7 Aug 2016
Status Ready to Commit ⇒ Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 ⇒ 2016-08-07 18:11:55
Closed_By ⇒ wilsonge
avatar wilsonge wilsonge - close - 7 Aug 2016
avatar wilsonge wilsonge - merge - 7 Aug 2016
avatar joomla-cms-bot joomla-cms-bot - close - 7 Aug 2016
avatar joomla-cms-bot joomla-cms-bot - change - 7 Aug 2016
Labels Removed: ?

Add a Comment

Login with GitHub to post a comment