? Success

User tests: Successful: Unsuccessful:

avatar digitalgarage
digitalgarage
11 Mar 2015

Hi Folks,

Prior to this commit on IE 10 trying to publish / unpublish a plugin in the Plugin Manager by clicking the tick or cross box would not work.

The IE 10 developer tools console would throw SCRIPT87: Invalid argument error in core.js. (see screenshot) This only affected IE 10, other browsers were fine.

ie-10-core-js-error

This fixes that by changing the document.adminForm.fireEvent from submit to onsubmit as per HTML DOM Event Object list here(http://www.w3schools.com/jsref/dom_obj_event.asp)

To test, WITHOUT patch go to Plugin Manager and click on the cross box next to a plugin to publish. IE 10 developer tools should show error message as per screenshot and the plugin publish state will not change. Install patch and repeat. With patch installed the plugin should be correctly enabled / disabled.

avatar digitalgarage digitalgarage - open - 11 Mar 2015
avatar joomla-cms-bot joomla-cms-bot - change - 11 Mar 2015
Labels Added: ?
avatar n9iels
n9iels - comment - 11 Mar 2015

Looks like this problem only occurs in IE10, I can't reproduce it for IE11

avatar wilsonge
wilsonge - comment - 11 Mar 2015

#5914 This should have been already fixed by this PR in 3.4. Can you ensure you've tested this bug against 3.4 please :)

avatar digitalgarage
digitalgarage - comment - 11 Mar 2015

@n9iels yes it was only IE10. IE11 was fine.

@wilsonge that fixes the same problem further up in core-uncompressed.js. In 3.4 the toolbar was fixed by #5914 on line 30. ...But.... the toggle buttons to publish / unpublish were using almost identical code on line 427 - 429 of core-uncompressed.js that weren't updated.

Copy of current lines 427 - 429 below from core-uncompressed.js. This is taken from branch staging checked out this morning. This PR replicates the fix used at line 30 (ie. to change the submit to onsubmit') but applies it to 427-429.

if (typeof document.adminForm.fireEvent == "function") {
    document.adminForm.fireEvent('submit');
}

Eric.

avatar Fedik
Fedik - comment - 11 Mar 2015

@wilsonge looks like somewhere still in use deprecated function submitform(pressbutton)
I missed it in #5914

avatar wilsonge wilsonge - change - 11 Mar 2015
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2015-03-11 10:06:31
avatar wilsonge wilsonge - close - 11 Mar 2015
avatar wilsonge wilsonge - reference | - 11 Mar 15
avatar wilsonge wilsonge - merge - 11 Mar 2015
avatar wilsonge wilsonge - close - 11 Mar 2015
avatar wilsonge wilsonge - change - 11 Mar 2015
Milestone Added:
avatar wilsonge
wilsonge - comment - 11 Mar 2015

My bad :) Didn't notice it was different lines. I blame it on reviewing stuff before I had morning coffee

avatar digitalgarage
digitalgarage - comment - 11 Mar 2015

No worries.... Coffee is the only thing that keeps my brain ticking over some days!

avatar digitalgarage digitalgarage - head_ref_deleted - 11 Mar 2015

Add a Comment

Login with GitHub to post a comment