? Pending

User tests: Successful: Unsuccessful:

avatar johansundell
johansundell
12 Sep 2017

Pull Request for Issue # .

Summary of Changes

Because php does not support class names with a - character editors-xtd can't use com_ajax, all I did was to remove the - from the class name.

Testing Instructions

I made a simple editor-xtd plugin to test it with https://gist.github.com/johansundell/eb505037336a6c6965283f502dac4558

Expected result

Actual result

Documentation Changes Required

avatar joomla-cms-bot joomla-cms-bot - change - 12 Sep 2017
Category Libraries
avatar johansundell johansundell - open - 12 Sep 2017
avatar johansundell johansundell - change - 12 Sep 2017
Status New Pending
avatar laoneo
laoneo - comment - 12 Sep 2017

We have that change in the 4.0 branch since the new media manager merge, so we should be safe here https://github.com/joomla/joomla-cms/blame/4.0-dev/libraries/src/Plugin/PluginHelper.php#L245.

avatar johansundell johansundell - change - 12 Sep 2017
Labels Added: ?
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 29 Oct 2017

@johansundell can you please provide more Information how to test?

avatar Quy
Quy - comment - 30 Apr 2018

@laoneo I think PR #17907 addresses the same issue. Which is better?

avatar brianteeman
brianteeman - comment - 13 May 2018

I will test this during the week

avatar wilsonge wilsonge - change - 13 May 2018
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2018-05-13 21:34:26
Closed_By wilsonge
avatar wilsonge wilsonge - close - 13 May 2018
avatar wilsonge wilsonge - merge - 13 May 2018

Add a Comment

Login with GitHub to post a comment