Conflicting Files ? Failure

User tests: Successful: Unsuccessful:

avatar wilsonge
wilsonge
23 Sep 2019

Summary of Changes

Change typehints + unnamepsaced class usage to namespaced class usage

Testing Instructions

Code reiew

avatar wilsonge wilsonge - open - 23 Sep 2019
avatar wilsonge wilsonge - change - 23 Sep 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 23 Sep 2019
Category Administration com_finder Front End Plugins
avatar wilsonge wilsonge - change - 23 Sep 2019
Title
Fix some typehints in finder plugins to namespaced versions
[4.0] Fix some typehints in finder plugins to namespaced versions
avatar wilsonge wilsonge - edited - 23 Sep 2019
avatar wilsonge wilsonge - change - 24 Sep 2019
Labels Added: ?
avatar SharkyKZ SharkyKZ - test_item - 24 Sep 2019 - Tested successfully
avatar SharkyKZ
SharkyKZ - comment - 24 Sep 2019

I have tested this item successfully on 75d3c08


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

avatar richard67
richard67 - comment - 25 Sep 2019

LGTM. But shouldn't the descriptions not be changed, too, like Quy mentioned? I.e.
@param Table $row A JTable object => @param Table $row A Table object
and so on?

avatar roland-d
roland-d - comment - 1 Aug 2020

@wilsonge Please fix the conflicts.

avatar Hackwar
Hackwar - comment - 16 May 2021

Sorry, my merge didn't turn out well. The use statements at the top of the finder plugins were doubled. I created a replacement PR with #33920

avatar richard67
richard67 - comment - 16 May 2021

@Hackwar @wilsonge So this PR here can be closed?

avatar Hackwar
Hackwar - comment - 17 May 2021

yes

avatar richard67
richard67 - comment - 17 May 2021

Closing in favour of #33920 .

avatar richard67 richard67 - close - 17 May 2021
avatar richard67 richard67 - change - 17 May 2021
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2021-05-17 17:23:54
Closed_By richard67

Add a Comment

Login with GitHub to post a comment