? Pending

User tests: Successful: Unsuccessful:

avatar roland-d
roland-d
30 Jul 2018

Pull Request for Issue #21300

Summary of Changes

Use the CMSObject::class instead.

Testing Instructions

See Issue #21300 for test instructions.

Expected result

Page loads still

Actual result

Page loads

Documentation Changes Required

None

avatar roland-d roland-d - open - 30 Jul 2018
avatar roland-d roland-d - change - 30 Jul 2018
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 30 Jul 2018
Category Libraries
avatar franz-wohlkoenig franz-wohlkoenig - test_item - 30 Jul 2018 - Tested successfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 30 Jul 2018

I have tested this item successfully on d72a089


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

avatar brianteeman
brianteeman - comment - 30 Jul 2018

@roland-d i think there are similar improvements that can be made to

$classname = 'JRouter' . ucfirst($client);

avatar mbabker
mbabker - comment - 30 Jul 2018

In that specific case, no. The ::class magic constant refers to a fully qualified class name, in that specific case you're concatenating a string to build a class name.

In that case you should do __NAMESPACE__ . '\' . ucfirst($client) . 'Router' to transition that line of code from the global class name to the namespaced code (__NAMESPACE__ being a magic constant since PHP 5.3 to get the namespace of the current file, since we're referencing another class in the same namespace that works well here).

avatar wilsonge wilsonge - change - 30 Jul 2018
Labels Added: ?
avatar wilsonge wilsonge - change - 30 Jul 2018
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2018-07-30 13:32:54
Closed_By wilsonge
avatar wilsonge wilsonge - close - 30 Jul 2018
avatar wilsonge wilsonge - merge - 30 Jul 2018

Add a Comment

Login with GitHub to post a comment