? Pending

User tests: Successful: Unsuccessful:

avatar laoneo
laoneo
18 May 2018

Pull Request for Issue #20435.

Summary of Changes

Sends the onBeforeExecute correctly in the applications.

Testing Instructions

Enable the language filter plugin.

Expected result

No exception is thrown.

Actual result

Exception
Argument 1 passed to PlgSystemLanguageFilter::onBeforeExecute() must be an instance of Joomla\CMS\Event\BeforeExecuteEvent, instance of Joomla\Event\Event given
is thrown.

avatar laoneo laoneo - open - 18 May 2018
avatar laoneo laoneo - change - 18 May 2018
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 18 May 2018
Category Libraries
avatar laoneo laoneo - change - 18 May 2018
The description was changed
avatar laoneo laoneo - edited - 18 May 2018
avatar infograf768
infograf768 - comment - 18 May 2018

I have tested this item successfully on 3fbe042


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

avatar infograf768 infograf768 - test_item - 18 May 2018 - Tested successfully
avatar laoneo laoneo - change - 18 May 2018
Labels Added: ?
avatar infograf768
infograf768 - comment - 19 May 2018

@mbabker @wilsonge This one is rather urgent if we want to test associations and solve the multilingual issues

avatar roland-d roland-d - test_item - 19 May 2018 - Tested successfully
avatar roland-d
roland-d - comment - 19 May 2018

I have tested this item successfully on ac6b179

Before the fix after I activated the plugin it was impossible to access Joomla, after the fix everything worked as expected.


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

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 19 May 2018

@infograf768 can you please retest?

avatar rdeutz rdeutz - close - 19 May 2018
avatar rdeutz rdeutz - merge - 19 May 2018
avatar rdeutz rdeutz - change - 19 May 2018
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2018-05-19 09:42:49
Closed_By rdeutz

Add a Comment

Login with GitHub to post a comment