? ? Pending

User tests: Successful: Unsuccessful:

avatar PhilETaylor
PhilETaylor
11 Oct 2020

Summary of Changes

Return value type is not compatible with declared type in PHPDoc block

I chose to remove the return true instead of update the docs (like I have for these #31060) because this method is ONLY called by

$this->checkTestingSampledata($db);

where it doesn't need a true to be returned and expects just a void return.

Testing Instructions

Code review

avatar PhilETaylor PhilETaylor - open - 11 Oct 2020
avatar PhilETaylor PhilETaylor - change - 11 Oct 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 11 Oct 2020
Category Installation
avatar PhilETaylor PhilETaylor - change - 11 Oct 2020
Title
Return value type is not compatible with declared
[4] Return value type is not compatible with declared
avatar PhilETaylor PhilETaylor - edited - 11 Oct 2020
avatar ChristineWk ChristineWk - test_item - 12 Oct 2020 - Tested successfully
avatar ChristineWk
ChristineWk - comment - 12 Oct 2020

I have tested this item successfully on c35aa5e


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

avatar Quy Quy - test_item - 12 Oct 2020 - Tested successfully
avatar Quy
Quy - comment - 12 Oct 2020

I have tested this item successfully on c35aa5e


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

avatar Quy Quy - change - 12 Oct 2020
The description was changed
Status Pending Ready to Commit
avatar Quy Quy - edited - 12 Oct 2020
avatar Quy
Quy - comment - 12 Oct 2020

RTC


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

avatar richard67 richard67 - change - 15 Oct 2020
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-10-15 16:00:05
Closed_By richard67
Labels Added: ? ?
avatar richard67 richard67 - close - 15 Oct 2020
avatar richard67 richard67 - merge - 15 Oct 2020
avatar richard67
richard67 - comment - 15 Oct 2020

Thanks!

Add a Comment

Login with GitHub to post a comment