Changeset 16554

Timestamp:

11/23/2010 08:20:44 PM (13 years ago)

Author:

scribu

Message:

Make filter_count() a closure. See #14936

Location:

trunk/wp-admin/js

Files:

• theme.dev.js (modified) (2 diffs)
• theme.js (modified) (1 diff)

trunk/wp-admin/js/theme.dev.js

@@ -3 +3 @@
 (function($){
     ThemeViewer = function( args ) {
-
-        function filter_count() {
-            var count = $( '#filter-box :checked' ).length;
-            var text  = $( '#filter-click' ).text();
-
-            if ( text.indexOf( '(' ) != -1 )
-                text = text.substr( 0, text.indexOf( '(' ) );
-
-            if ( count == 0 )
-                $( '#filter-click' ).text( text );
-            else
-                $( '#filter-click' ).text( text + ' (' + count + ')' );
-        }
 
         function init() {
@@ -26 +13 @@
 
             $( '#filter-box :checkbox' ).unbind( 'click' ).click( function() {
-                filter_count();
+                var count = $( '#filter-box :checked' ).length,
+                    text  = $( '#filter-click' ).text();
+
+                if ( text.indexOf( '(' ) != -1 )
+                    text = text.substr( 0, text.indexOf( '(' ) );
+
+                if ( count == 0 )
+                    $( '#filter-click' ).text( text );
+                else
+                    $( '#filter-click' ).text( text + ' (' + count + ')' );
             });
 

trunk/wp-admin/js/theme.js

@@ -1 @@
-var ThemeViewer;(function(a){ThemeViewer=function(b){function d(){var f=a("#filter-box :checked").length;var g=a("#filter-click").text();if(g.indexOf("(")!=-1){g=g.substr(0,g.indexOf("("))}if(f==0){a("#filter-click").text(g)}else{a("#filter-click").text(g+" ("+f+")")}}function e(){a("#filter-click, #mini-filter-click").unbind("click").click(function(){a("#filter-click").toggleClass("current");a("#filter-box").slideToggle();a("#current-theme").slideToggle(300);return false});a("#filter-box :checkbox").unbind("click").click(function(){d()});a("#filter-box :submit").unbind("click").click(function(){var f=[];a("#filter-box :checked").each(function(){f.push(a(this).val())});listTable.update_rows({features:f},true,function(){a("#filter-click").toggleClass("current");a("#filter-box").slideToggle();a("#current-theme").slideToggle(300)});return false})}var c={init:e};return c}})(jQuery);jQuery(document).ready(function(a){theme_viewer=new ThemeViewer();theme_viewer.init()});
+var ThemeViewer;(function(a){ThemeViewer=function(b){function d(){a("#filter-click, #mini-filter-click").unbind("click").click(function(){a("#filter-click").toggleClass("current");a("#filter-box").slideToggle();a("#current-theme").slideToggle(300);return false});a("#filter-box :checkbox").unbind("click").click(function(){var e=a("#filter-box :checked").length,f=a("#filter-click").text();if(f.indexOf("(")!=-1){f=f.substr(0,f.indexOf("("))}if(e==0){a("#filter-click").text(f)}else{a("#filter-click").text(f+" ("+e+")")}});a("#filter-box :submit").unbind("click").click(function(){var e=[];a("#filter-box :checked").each(function(){e.push(a(this).val())});listTable.update_rows({features:e},true,function(){a("#filter-click").toggleClass("current");a("#filter-box").slideToggle();a("#current-theme").slideToggle(300)});return false})}var c={init:d};return c}})(jQuery);jQuery(document).ready(function(a){theme_viewer=new ThemeViewer();theme_viewer.init()});