Fix some code style inconsistencies

This commit is contained in:
Oleg
2013-11-07 21:14:23 +01:00
parent bcae427015
commit 22120cea66
24 changed files with 41 additions and 45 deletions

View File

@@ -102,7 +102,7 @@ function inspectPrefiltersOrTransports( structure, options, originalOptions, jqX
inspected[ dataType ] = true;
jQuery.each( structure[ dataType ] || [], function( _, prefilterOrFactory ) {
var dataTypeOrTransport = prefilterOrFactory( options, originalOptions, jqXHR );
if( typeof dataTypeOrTransport === "string" && !seekingTransport && !inspected[ dataTypeOrTransport ] ) {
if ( typeof dataTypeOrTransport === "string" && !seekingTransport && !inspected[ dataTypeOrTransport ] ) {
options.dataTypes.unshift( dataTypeOrTransport );
inspect( dataTypeOrTransport );
return false;
@@ -135,7 +135,6 @@ function ajaxExtend( target, src ) {
return target;
}
/* Handles responses to an ajax request:
* - finds the right dataType (mediates between content-type and expected dataType)
* - returns the corresponding response
@@ -146,7 +145,7 @@ function ajaxHandleResponses( s, jqXHR, responses ) {
dataTypes = s.dataTypes;
// Remove auto dataType and get content-type in the process
while( dataTypes[ 0 ] === "*" ) {
while ( dataTypes[ 0 ] === "*" ) {
dataTypes.shift();
if ( ct === undefined ) {
ct = s.mimeType || jqXHR.getResponseHeader("Content-Type");
@@ -512,7 +511,7 @@ jQuery.extend({
s.type = options.method || options.type || s.method || s.type;
// Extract dataTypes list
s.dataTypes = jQuery.trim( s.dataType || "*" ).toLowerCase().match( rnotwhite ) || [""];
s.dataTypes = jQuery.trim( s.dataType || "*" ).toLowerCase().match( rnotwhite ) || [ "" ];
// A cross-domain request is in order when we have a protocol:host:port mismatch
if ( s.crossDomain == null ) {
@@ -798,8 +797,8 @@ jQuery.each( [ "get", "post" ], function( i, method ) {
});
// Attach a bunch of functions for handling common AJAX events
jQuery.each( [ "ajaxStart", "ajaxStop", "ajaxComplete", "ajaxError", "ajaxSuccess", "ajaxSend" ], function( i, type ){
jQuery.fn[ type ] = function( fn ){
jQuery.each( [ "ajaxStart", "ajaxStop", "ajaxComplete", "ajaxError", "ajaxSuccess", "ajaxSend" ], function( i, type ) {
jQuery.fn[ type ] = function( fn ) {
return this.on( type, fn );
};
});

View File

@@ -11,7 +11,7 @@ jQuery.parseXML = function( data ) {
try {
if ( window.DOMParser ) { // Standard
tmp = new DOMParser();
xml = tmp.parseFromString( data , "text/xml" );
xml = tmp.parseFromString( data, "text/xml" );
} else { // IE
xml = new ActiveXObject( "Microsoft.XMLDOM" );
xml.async = "false";

View File

@@ -2,7 +2,7 @@ define([
"../var/support"
], function( support ) {
(function () {
(function() {
var a, input, select, opt,
div = document.createElement("div" );

View File

@@ -52,7 +52,7 @@ jQuery.fn.extend({
} else if ( typeof val === "number" ) {
val += "";
} else if ( jQuery.isArray( val ) ) {
val = jQuery.map(val, function ( value ) {
val = jQuery.map( val, function( value ) {
return value == null ? "" : value + "";
});
}

View File

@@ -126,7 +126,7 @@ jQuery.Callbacks = function( options ) {
if ( list ) {
jQuery.each( arguments, function( _, arg ) {
var index;
while( ( index = jQuery.inArray( arg, list, index ) ) > -1 ) {
while ( ( index = jQuery.inArray( arg, list, index ) ) > -1 ) {
list.splice( index, 1 );
// Handle firing indexes
if ( firing ) {

View File

@@ -87,7 +87,6 @@ function completed() {
}
}
jQuery.ready.promise = function( obj ) {
if ( !readyList ) {

View File

@@ -422,7 +422,7 @@ if ( !support.opacity ) {
}
jQuery.cssHooks.marginRight = addGetHookIf( support.reliableMarginRight,
function ( elem, computed ) {
function( elem, computed ) {
if ( computed ) {
// WebKit Bug 13343 - getComputedStyle returns wrong value for margin-right
// Work around by temporarily setting element display to inline-block

View File

@@ -3,7 +3,7 @@ define([
"../var/support"
], function( jQuery, support ) {
(function () {
(function() {
var a, reliableHiddenOffsetsVal, boxSizingVal, boxSizingReliableVal,
pixelPositionVal, reliableMarginRightVal,
div = document.createElement( "div" ),
@@ -34,7 +34,6 @@ define([
// Null elements to avoid leaks in IE.
a = div = null;
jQuery.extend(support, {
reliableHiddenOffsets: function() {
if ( reliableHiddenOffsetsVal != null ) {

View File

@@ -1,3 +1,3 @@
define(function() {
return (/^margin/);
});
});

View File

@@ -2,4 +2,4 @@ define([
"../../var/pnum"
], function( pnum ) {
return new RegExp( "^(" + pnum + ")(?!px)[a-z%]+$", "i" );
});
});

View File

@@ -56,7 +56,7 @@ function isEmptyDataObject( obj ) {
return true;
}
function internalData( elem, name, data, pvt /* Internal Use Only */ ){
function internalData( elem, name, data, pvt /* Internal Use Only */ ) {
if ( !jQuery.acceptData( elem ) ) {
return;
}

View File

@@ -2,7 +2,7 @@ define([
"../var/support"
], function( support ) {
(function () {
(function() {
var div = document.createElement( "div" );
// Execute the test only if not already executed in another module.

View File

@@ -109,9 +109,10 @@ jQuery.extend({
return function( value ) {
contexts[ i ] = this;
values[ i ] = arguments.length > 1 ? slice.call( arguments ) : value;
if( values === progressValues ) {
if ( values === progressValues ) {
deferred.notifyWith( contexts, values );
} else if ( !( --remaining ) ) {
} else if ( !(--remaining) ) {
deferred.resolveWith( contexts, values );
}
};

10
src/effects.js vendored
View File

@@ -21,7 +21,7 @@ var
rrun = /queueHooks$/,
animationPrefilters = [ defaultPrefilter ],
tweeners = {
"*": [function( prop, value ) {
"*": [ function( prop, value ) {
var tween = this.createTween( prop, value ),
target = tween.cur(),
parts = rfxnum.exec( value ),
@@ -68,7 +68,7 @@ var
}
return tween;
}]
} ]
};
// Animations created synchronously will run synchronously
@@ -87,8 +87,8 @@ function genFx( type, includeWidth ) {
// if we include width, step value is 1 to do all cssExpand values,
// if we don't include width, step value is 2 to skip over Left and Right
includeWidth = includeWidth? 1 : 0;
for( ; i < 4 ; i += 2 - includeWidth ) {
includeWidth = includeWidth ? 1 : 0;
for ( ; i < 4 ; i += 2 - includeWidth ) {
which = cssExpand[ i ];
attrs[ "margin" + which ] = attrs[ "padding" + which ] = type;
}
@@ -100,7 +100,6 @@ function genFx( type, includeWidth ) {
return attrs;
}
function createTween( value, prop, animation ) {
var tween,
collection = ( tweeners[ prop ] || [] ).concat( tweeners[ "*" ] ),
@@ -189,7 +188,6 @@ function defaultPrefilter( elem, props, opts ) {
}
}
// show/hide pass
for ( prop in props ) {
value = props[ prop ];

View File

@@ -102,7 +102,7 @@ jQuery.easing = {
return p;
},
swing: function( p ) {
return 0.5 - Math.cos( p*Math.PI ) / 2;
return 0.5 - Math.cos( p * Math.PI ) / 2;
}
};

View File

@@ -3,7 +3,7 @@ define([
"../var/support"
], function( strundefined, support ) {
(function () {
(function() {
var a, shrinkWrapBlocksVal,
div = document.createElement( "div" ),
divReset =
@@ -32,7 +32,7 @@ define([
// Null elements to avoid leaks in IE.
a = div = null;
support.shrinkWrapBlocks = function () {
support.shrinkWrapBlocks = function() {
var body, container, div, containerStyles;
if ( shrinkWrapBlocksVal == null ) {

View File

@@ -79,7 +79,7 @@ jQuery.event = {
}
// Handle multiple events separated by a space
types = ( types || "" ).match( rnotwhite ) || [""];
types = ( types || "" ).match( rnotwhite ) || [ "" ];
t = types.length;
while ( t-- ) {
tmp = rtypenamespace.exec( types[t] ) || [];
@@ -165,7 +165,7 @@ jQuery.event = {
}
// Once for each type.namespace in types; type may be omitted
types = ( types || "" ).match( rnotwhite ) || [""];
types = ( types || "" ).match( rnotwhite ) || [ "" ];
t = types.length;
while ( t-- ) {
tmp = rtypenamespace.exec( types[t] ) || [];

View File

@@ -2,7 +2,7 @@ define([
"../var/support"
], function( support ) {
(function () {
(function() {
var i, eventName,
div = document.createElement( "div" );

View File

@@ -311,7 +311,7 @@ jQuery.extend({
tmp = tmp || safe.appendChild( context.createElement("div") );
// Deserialize a standard representation
tag = ( rtagName.exec( elem ) || ["", ""] )[1].toLowerCase();
tag = (rtagName.exec( elem ) || [ "", "" ])[ 1 ].toLowerCase();
wrap = wrapMap[ tag ] || wrapMap._default;
tmp.innerHTML = wrap[1] + elem.replace( rxhtmlTag, "<$1></$2>" ) + wrap[2];
@@ -556,7 +556,7 @@ jQuery.fn.extend({
dataAndEvents = dataAndEvents == null ? false : dataAndEvents;
deepDataAndEvents = deepDataAndEvents == null ? dataAndEvents : deepDataAndEvents;
return this.map( function () {
return this.map(function() {
return jQuery.clone( this, dataAndEvents, deepDataAndEvents );
});
},
@@ -577,7 +577,7 @@ jQuery.fn.extend({
if ( typeof value === "string" && !rnoInnerhtml.test( value ) &&
( support.htmlSerialize || !rnoshimcache.test( value ) ) &&
( support.leadingWhitespace || !rleadingWhitespace.test( value ) ) &&
!wrapMap[ ( rtagName.exec( value ) || ["", ""] )[1].toLowerCase() ] ) {
!wrapMap[ (rtagName.exec( value ) || [ "", "" ])[ 1 ].toLowerCase() ] ) {
value = value.replace( rxhtmlTag, "<$1></$2>" );

View File

@@ -2,7 +2,7 @@ define([
"../var/support"
], function( support ) {
(function () {
(function() {
var fragment = document.createDocumentFragment(),
div = document.createElement("div"),
input = document.createElement("input");

View File

@@ -44,7 +44,7 @@ jQuery.offset = {
curCSSTop = jQuery.css( elem, "top" );
curCSSLeft = jQuery.css( elem, "left" );
calculatePosition = ( position === "absolute" || position === "fixed" ) &&
jQuery.inArray("auto", [curCSSTop, curCSSLeft]) > -1;
jQuery.inArray("auto", [ curCSSTop, curCSSLeft ] ) > -1;
// need to be able to calculate position if either top or left is auto and position is either absolute or fixed
if ( calculatePosition ) {
@@ -163,7 +163,7 @@ jQuery.fn.extend({
});
// Create scrollLeft and scrollTop methods
jQuery.each( {scrollLeft: "pageXOffset", scrollTop: "pageYOffset"}, function( method, prop ) {
jQuery.each( { scrollLeft: "pageXOffset", scrollTop: "pageYOffset" }, function( method, prop ) {
var top = /Y/.test( prop );
jQuery.fn[ method ] = function( val ) {
@@ -195,7 +195,7 @@ jQuery.each( {scrollLeft: "pageXOffset", scrollTop: "pageYOffset"}, function( me
// rather than make the css module depend on the offset module, we just check for it here
jQuery.each( [ "top", "left" ], function( i, prop ) {
jQuery.cssHooks[ prop ] = addGetHookIf( support.pixelPosition,
function ( elem, computed ) {
function( elem, computed ) {
if ( computed ) {
computed = curCSS( elem, prop );
// if curCSS returns percentage, fallback to offset

View File

@@ -126,7 +126,7 @@ jQuery.fn.extend({
}
type = type || "fx";
while( i-- ) {
while ( i-- ) {
tmp = jQuery._data( elements[ i ], type + "queueHooks" );
if ( tmp && tmp.empty ) {
count++;

View File

@@ -1,7 +1,7 @@
define([
"./core",
"sizzle"
], function ( jQuery, Sizzle ) {
], function( jQuery, Sizzle ) {
jQuery.find = Sizzle;
jQuery.expr = Sizzle.selectors;

View File

@@ -80,25 +80,25 @@ jQuery.fn.extend({
return jQuery.param( this.serializeArray() );
},
serializeArray: function() {
return this.map(function(){
return this.map(function() {
// Can add propHook for "elements" to filter or add form elements
var elements = jQuery.prop( this, "elements" );
return elements ? jQuery.makeArray( elements ) : this;
})
.filter(function(){
.filter(function() {
var type = this.type;
// Use .is(":disabled") so that fieldset[disabled] works
return this.name && !jQuery( this ).is( ":disabled" ) &&
rsubmittable.test( this.nodeName ) && !rsubmitterTypes.test( type ) &&
( this.checked || !rcheckableType.test( type ) );
})
.map(function( i, elem ){
.map(function( i, elem ) {
var val = jQuery( this ).val();
return val == null ?
null :
jQuery.isArray( val ) ?
jQuery.map( val, function( val ){
jQuery.map( val, function( val ) {
return { name: elem.name, value: val.replace( rCRLF, "\r\n" ) };
}) :
{ name: elem.name, value: val.replace( rCRLF, "\r\n" ) };