?
avatar zero-24
zero-24
5 Jul 2017

Steps to reproduce the issue

We know that in some cases the new integrated classes are not loaded on update. #16724

Expected result

script run

Actual result

script does break

Additional comments

Maybe we should better do a if class exists check before the call to JExtensionHelper::getCoreExtensions(); in that spezific file? Expecialy as that manifest thing is not that critical when it does not run.

avatar zero-24 zero-24 - open - 5 Jul 2017
avatar joomla-cms-bot joomla-cms-bot - change - 5 Jul 2017
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 5 Jul 2017
avatar richard67
richard67 - comment - 5 Jul 2017

Hmm, good thought.

But I did not experience since long time that problem with new classes coming with the update not being loaded on update.

So I am curious on feedback here.


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

avatar mbabker
mbabker - comment - 5 Jul 2017

It was only a problem historically when supporting core updates through the extension manager. We aren't doing that anymore.

By the time we get to a point where that class gets called for the first time in an update, either the system will be correctly updated and there will be nothing to worry about or there was an issue unpacking the update and the user is going to have other problems anyway.

avatar zero-24
zero-24 - comment - 5 Jul 2017

Ok, i just want to be sure, closing than based on Michaels info. Thanks.

avatar zero-24 zero-24 - change - 5 Jul 2017
Status New Closed
Closed_Date 0000-00-00 00:00:00 2017-07-05 17:38:46
Closed_By zero-24
avatar zero-24 zero-24 - close - 5 Jul 2017
avatar richard67
richard67 - comment - 5 Jul 2017

That's what I had in my memory, too, that the problem was from extension installer being used for core update.

Thanks for confirm.

Add a Comment

Login with GitHub to post a comment