Bug Tracker

Ticket #922 (closed bug: fixed)

Opened 8 years ago

Last modified 8 years ago

Fix Strict Warnings

Reported by: jamie{dot}mason{at}b Owned by:
Priority: minor Milestone: 1.1.3
Component: core Version: 1.1.2
Keywords: strict Cc:
Blocking: Blocked by:

Description

Very trivial, but it would be nice to fix the small strict warnings in jQuery which get in the way when looking for any of your own.

Error: t.substring is not a function Line: 1

----------

Warning: anonymous function does not always return a value Line: 1, Column: 225 Source code: (c[j].apply(this,args)===false){event.preventDefault();event.stopPropagation();returnValue=false}}if(jQuery.browser.msie)event.target=event.preventDefault=event.stopPropagation=event.handler=event.data=null;return returnValue},fix:function(event){if(!even

----------

Warning: anonymous function does not always return a value Line: 1, Column: 86 Source code:

lt=="even"&&num%2==0&&num>1&&cur==elem
result=="odd"&&num%2==1&&cur==elem)return cur}},sibling:function(n,elem){var r=[];for(;n;n=n.nextSibling){if(n.nodeType==1&&(!elemn!=elem))r.push(n)}return r}});jQuery.event={add:function(element,type,handler,data

----------

Warning: anonymous function does not always return a value Line: 1, Column: 53 Source code:

._resort)m=jQuery.expr[m[1]]._resort(m);return false}});if(m[1]==":"&&m[2]=="not")r=jQuery.filter(m[3],r,true).r;else if(m[1]=="."){var re=new RegExp("(|\s)"+m[2]+"(\s|$)");r=jQuery.grep(r,function(e){return re.test(e.className
"")},not)}else{var f=jQ

----------

Warning: anonymous function does not always return a value Line: 1, Column: 164 Source code: on(e){return rec.test(e.className)});if(m[1]=="#"&&ret.length==1){var tmp=r;r=[];jQuery.each(tmp,function(){if(this.getAttribute("id")==m[2]){r=[this];return false}})}ret=r}t=t.replace(re2,"")}}if(t){var val=jQuery.filter(t,r);ret=r=val.r;t=jQuery.trim(va

----------

Warning: test for equality (==) mistyped as assignment (=)? Line: 1, Column: 53 Source code: 1){target=this;a=0}var prop;while(prop=arguments[a++])for(var i in prop)target[i]=prop[i];return target};jQuery.extend({noConflict:function(){if(jQuery._$)$=jQuery._$;return jQuery},isFunction:function(fn){return!!fn&&typeof fn!="string"&&typeof fn[0]=="u

Change History

comment:1 Changed 8 years ago by john

  • Status changed from new to closed
  • Priority changed from trivial to minor
  • Version set to 1.1.2
  • Milestone set to 1.1.3
  • need set to Review
  • Resolution set to fixed

Fixed in SVN rev [1817].

Note: See TracTickets for help on using tickets.