Joomla! Issue Tracker - CMS

Download
Launch
Feeling Lucky

[#23234] - Fix for Factory reference in JsonDocument ref #21304

? Pending

User tests: Successful: Unsuccessful:

avatar corejoomla
corejoomla
6 Dec 2018

Getting below error when sending ajax requests to json controller.

Fatal error: Cannot use Joomla\CMS\Factory as Factory because the name is already in use in C:\xxx\joomla-cms\libraries\src\Document\JsonDocument

This change will fix this bug.

Pull Request for Issue # .

Summary of Changes

Testing Instructions

Expected result

Actual result

Documentation Changes Required

avatar corejoomla corejoomla - open - 6 Dec 2018
avatar corejoomla corejoomla - change - 6 Dec 2018
Status New ⇒ Pending
avatar joomla-cms-bot joomla-cms-bot - change - 6 Dec 2018
Category ⇒ Libraries
avatar csthomas
csthomas - comment - 6 Dec 2018

Perhaps you can do the same fix for other files in the same namespace, ex:
libraries/src/Document/FeedDocument.php
...

avatar wilsonge wilsonge - change - 6 Dec 2018
Labels Added: ?
avatar corejoomla
corejoomla - comment - 7 Dec 2018

Perhaps you can do the same fix for other files in the same namespace, ex:
libraries/src/Document/FeedDocument.php
...

Thanks. Not just FeedDocument, but also needed the fix for XmlDocument. All three changes are pushed.

avatar wilsonge wilsonge - change - 7 Dec 2018
Status Pending ⇒ Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 ⇒ 2018-12-07 16:23:38
Closed_By ⇒ wilsonge
avatar wilsonge wilsonge - close - 7 Dec 2018
avatar wilsonge wilsonge - merge - 7 Dec 2018
avatar wilsonge
wilsonge - comment - 7 Dec 2018

Thankyou very much!

Add a Comment

Login with GitHub to post a comment