? ? ? Pending

User tests: Successful: Unsuccessful:

avatar zero-24
zero-24
14 Aug 2017

Summary of Changes

Fix inline comment and remove blank lines that are not needed

Testing Instructions

review and drone passes as no production code is changed

Documentation Changes Required

none

avatar joomla-cms-bot joomla-cms-bot - change - 14 Aug 2017
Category Administration com_finder
avatar zero-24 zero-24 - open - 14 Aug 2017
avatar zero-24 zero-24 - change - 14 Aug 2017
Status New Pending
avatar Quy Quy - test_item - 17 Aug 2017 - Tested successfully
avatar Quy
Quy - comment - 17 Aug 2017

I have tested this item successfully on 3e9979b

Code review


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

avatar zero-24 zero-24 - change - 17 Aug 2017
Labels Added: ? ?
avatar akritianand akritianand - test_item - 19 Aug 2017 - Tested successfully
avatar akritianand
akritianand - comment - 19 Aug 2017

I have tested this item successfully on fe7d4ff


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

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 19 Aug 2017

@Quy can you please retest?

avatar Quy Quy - test_item - 20 Aug 2017 - Tested successfully
avatar Quy
Quy - comment - 20 Aug 2017

I have tested this item successfully on 4719042

Code review


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

avatar franz-wohlkoenig franz-wohlkoenig - change - 20 Aug 2017
Status Pending Ready to Commit
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 20 Aug 2017

thanks @Quy
RTC after two successful tests.

avatar mbabker mbabker - change - 20 Aug 2017
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-08-20 15:55:48
Closed_By mbabker
Labels Added: ?
avatar mbabker mbabker - close - 20 Aug 2017
avatar mbabker mbabker - merge - 20 Aug 2017

Add a Comment

Login with GitHub to post a comment