From 6ed30a6a5e789779549fee2972581ca0884a2e42 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Jo=CC=88rn=20Zaefferer?= Date: Tue, 6 Oct 2015 15:19:32 +0200 Subject: [PATCH] Controlgroup: Convert to onevar, fix guard-clause/else, rm tmp var --- ui/widgets/controlgroup.js | 34 ++++++++++++++++++---------------- 1 file changed, 18 insertions(+), 16 deletions(-) diff --git a/ui/widgets/controlgroup.js b/ui/widgets/controlgroup.js index d82efcd12..793874df5 100644 --- a/ui/widgets/controlgroup.js +++ b/ui/widgets/controlgroup.js @@ -69,13 +69,15 @@ return $.widget( "ui.controlgroup", { // First we iterate over each of the items options $.each( this.options.items, function( widget, selector ) { - var widgets, labels, - options = {}; + var labels; + var options = {}; // Make sure the widget actually exists and has a selector set if ( !$.fn[ widget ] || !selector ) { return; - } else if ( selector && widget === "controlgroupLabel" ) { + } + + if ( widget === "controlgroupLabel" ) { labels = that.element.find( selector ); labels.each( function() { $( this ).contents().wrapAll( "" ); @@ -92,19 +94,19 @@ return $.widget( "ui.controlgroup", { } // Find instances of this widget inside controlgroup and init them - widgets = that.element.find( selector )[ widget ]( options ); - - widgets.each( function() { - var element = $( this ); - - // Store an instance of the controlgroup to be able to reference - // from the outermost element for changing options and refresh - var widgetElement = element[ widget ]( "widget" ); - $.data( widgetElement[ 0 ], "ui-controlgroup-data", - element[ widget ]( "instance" ) ); - - childWidgets.push( widgetElement[ 0 ] ); - } ); + that.element + .find( selector )[ widget ]( options ) + .each( function() { + var element = $( this ); + + // Store an instance of the controlgroup to be able to reference + // from the outermost element for changing options and refresh + var widgetElement = element[ widget ]( "widget" ); + $.data( widgetElement[ 0 ], "ui-controlgroup-data", + element[ widget ]( "instance" ) ); + + childWidgets.push( widgetElement[ 0 ] ); + } ); } ); this.childWidgets = $( $.unique( childWidgets ) ); -- 2.39.5