TypeError: a.curCSS is not a function (Trac 1.1.1dev-r11469)

[Ryan J Ollos]

The following errors are seen with Trac 1.1.1dev-r11469:

Firefox 16:

[10:46:52.715] TypeError: a.curCSS is not a function @ http://localhost:8000/tracdev/chrome/common/js/jquery-ui.js:5

Chrome 23:

Uncaught TypeError: Object function (a,b){return new p.fn.init(a,b,c)} has no method 'curCSS'

Trac 1.0 has:

• jQuery 1.7.2
• jQuery UI 1.8.21

Trac 1.1.1dev has:

• jQuery 1.8.2 (since ​t:r11412)
• jQuery UI 1.8.21

There is an incompatibility between jQuery 1.8 and jQuery UI < 1.8.22, as described ​here. I'll open a ticket for this in the Trac core, and implement a workaround in the plugin code.

[Ryan J Ollos]

I've filed a ticket in Trac for the jQuery/jQuery UI compatibility issue in ​t:#10976.

[Ryan J Ollos]

(In [12417]) Refs #10688: Implemented workaround for "curCSS is not a function" error when using jQuery UI < 1.8.22 with jQuery 1.8.x.

[Jun Omae]

After r12417, the following error happens with Trac 0.11.7.

Timestamp: 2012-12-09 20:51:02
Error: too much recursion
Source File: http://localhost:3000/chrome/common/js/jquery.js
Line: 11

Applying the following patch, it works on Trac 0.11.7, 0.12.4, 1.0 and trunk (​t:r11479).

adminenumlistplugin/htdocs/adminenumlist.js

@@ -14 +14 @@
     // Trac 1.1.1dev @r11479 provides jQuery UI 1.8.21 and jQuery 1.8.2
     // http://bugs.jquery.com/ticket/11921
     if(!$.isFunction($.fn.curCSS))
-        $.curCSS = $.css;
+        $.fn.curCSS = $.fn.css;
     
     var mouseY, lastY = 0;
     

[Ryan J Ollos]

Thanks, I'll apply that fix. That is interesting though. It must be an issue with an older version of jQuery then? The line in my patch was copied from the ​jQuery UI patch.

[Ryan J Ollos]

In Chrome 23 and the latest Trac trunk, I'm still seeing an error in the JS console after applying the patch:

Uncaught TypeError: Object function (a,b){return new p.fn.init(a,b,c)} has no method 'curCSS' 

The plugin seems to work fine though.

[Ryan J Ollos]

Maybe we have a different issue here though. In the jQuery ticket it was noted that curCSS has been deprecated since jQuery 1.3 and was removed in 1.8. With Trac 0.11.7 we are working with jQuery 1.2.6. The conditional was intended to evaluate to false for jQuery < 1.8, since curCSS should still be available (though deprecated).

[Jun Omae]

Oh, sorry. I mistook. I rework..., the fix needs checking existence of $.curCSS instead of $.fn.curCSS.

	$.cssCSS	$.fn.curCSS	notes
0.11.7	function	undefined	jQuery 1.2.6
0.12.4	function	undefined	jQuery 1.4.4
1.0	function	undefined	jQuery 1.7.2
trunk	undefined	undefined	jQuery 1.8.2

adminenumlistplugin/htdocs/adminenumlist.js

@@ -13 +13 @@
     // Workaround for issue when using jQuery UI < 1.8.22 with jQuery 1.8
     // Trac 1.1.1dev @r11479 provides jQuery UI 1.8.21 and jQuery 1.8.2
     // http://bugs.jquery.com/ticket/11921
-    if(!$.isFunction($.fn.curCSS))
+    if(!$.isFunction($.curCSS))
         $.curCSS = $.css;
     
     var mouseY, lastY = 0;

[Ryan J Ollos]

Thanks, that works for me too. I can extend your list as follows:

	$.curCSS	$.fn.curCSS	notes
0.11.0	function	undefined	jQuery 1.2.3

[Ryan J Ollos]

(In [12423]) Refs #10688: Fixed check for existence of curCSS function. Thanks to Jun Omae for the fix.

[Ryan J Ollos]

In [12463] I had to check for $.fn.prop; checking for $.prop didn't seem to work. The only difference I see between these two cases is the possibility that jQuery might treat a function alias differently than a function.