Skip to main content

Search and Top Navigation

#9565 closed bug (notabug)

Opened September 20, 2013 04:02PM UTC

Closed September 20, 2013 05:29PM UTC

error using _superApply() in IE8

Reported by: sarah Owned by: sarah
Priority: minor Milestone: none
Component: ui.widget Version: 1.10.3
Keywords: Cc:
Blocked by: Blocking:

Using jQuery 1.9.1 and jQuery UI 1.10.3 on IE8 or a later IE browser set to IE8 Browser and Document Modes. The call to _superApply() came from code inheriting jQuery UI widgets, but the error occurs in the jQuery UI code. The error marks line 86 of the jquery.ui.widget.js file (as displayed on github). The actual error states that 'Function.prototype.apply: argument is null or undefined'.


The code that causes the error:

_superApply = function( args ) {
   return base.prototype[ prop ].apply( this, args );

My quick solution that does not seem to cause problems anywhere else:

_superApply = function( args ) {
   if(args==undefined){return base.prototype[ prop ].apply( this);}
   else{return base.prototype[ prop ].apply( this, args );}
Attachments (0)
Change History (5)

Changed September 20, 2013 04:04PM UTC by scottgonzalez comment:1

owner: → sarah
status: newpending

We'll definitely need a reduced test case showing the problem before we make any changes to this.

Changed September 20, 2013 04:14PM UTC by sarah comment:2

status: pendingnew

Replying to [comment:1 scott.gonzalez]:

We'll definitely need a reduced test case showing the problem before we make any changes to this.

I've edited the example.

Changed September 20, 2013 04:26PM UTC by jzaefferer comment:3

status: newpending

In your fiddle you're calling _superApply without arguments, but this method has a required argument:

Is there any reason why you're not calling this._superApply(arguments) or just this._super()?

Changed September 20, 2013 05:24PM UTC by sarah comment:4

status: pendingnew

I do not recall why I made that decision. It could have been that I was following an erroneous (or outdated) example or that it was included along with a few other changes to fix bugs. Either way, your suggestion works.


Changed September 20, 2013 05:29PM UTC by scottgonzalez comment:5

resolution: → notabug
status: newclosed