? Pending

User tests: Successful: Unsuccessful:

avatar PhilETaylor
PhilETaylor
15 Jan 2021

Code review

Add missing includes for docblock type hints

Correct incorrect type hint and include that too (FilterTable)

Remote $tokens from the unset as $tokens is not even set in this method and undefined at this time and place.

avatar PhilETaylor PhilETaylor - open - 15 Jan 2021
avatar PhilETaylor PhilETaylor - change - 15 Jan 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 15 Jan 2021
Category Administration com_finder
avatar PhilETaylor PhilETaylor - change - 15 Jan 2021
Title
add missing and fix wrong includes in com_finder
[4] add missing and fix wrong includes in com_finder, and remove unset on undefined var
avatar PhilETaylor PhilETaylor - edited - 15 Jan 2021
avatar PhilETaylor PhilETaylor - change - 15 Jan 2021
Labels Added: ?
avatar toivo toivo - test_item - 16 Jan 2021 - Tested successfully
avatar toivo
toivo - comment - 16 Jan 2021

I have tested this item successfully on ec614a9

Tested successfully in Beta7-dev of 16 January.


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

avatar gostn gostn - test_item - 16 Jan 2021 - Tested successfully
avatar gostn
gostn - comment - 16 Jan 2021

I have tested this item successfully on ec614a9


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

avatar HLeithner HLeithner - change - 16 Jan 2021
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-01-16 09:00:26
Closed_By HLeithner
avatar HLeithner HLeithner - close - 16 Jan 2021
avatar HLeithner HLeithner - merge - 16 Jan 2021
avatar HLeithner
HLeithner - comment - 16 Jan 2021

Thanks

Add a Comment

Login with GitHub to post a comment