Ticket #27451: 27451.2.patch

src/wp-admin/js/customize-widgets.js

@@ -36 +36 @@
                 transport: 'refresh',
                 params: [],
                 width: null,
-                height: null
+                height: null,
+                search_matched: true
         });
 
         /**
@@ -71 +72 @@
                         // If search is blank, show all themes
                         // Useful for resetting the views when you clean the input
                         if ( this.terms === '' ) {
-                                this.reset( api.Widgets.data.availableWidgets );
+                                this.each( function ( widget ) {
+                                        widget.set( 'search_matched', true );
+                                } );
                         }
-
-                        // Trigger an 'update' event
-                        this.trigger( 'update' );
                 },
 
                 // Performs a search within the collection
                 // @uses RegExp
                 search: function( term ) {
-                        var match, results, haystack;
-
-                        // Start with a full collection
-                        this.reset( api.Widgets.data.availableWidgets, { silent: true } );
+                        var match, haystack;
 
                         // Escape the term string for RegExp meta characters
                         term = term.replace( /[-\/\\^$*+?.()|[\]{}]/g, '\\$&' );
@@ -94 +91 @@
                         term = term.replace( / /g, ')(?=.*' );
                         match = new RegExp( '^(?=.*' + term + ').+', 'i' );
 
-                        results = this.filter( function( data ) {
-                                haystack = _.union( data.get( 'name' ), data.get( 'id' ), data.get( 'description' ) );
-
-                                return match.test( haystack );
-                        });
-
-                        this.reset( results );
+                        this.each( function ( data ) {
+                                haystack = [ data.get( 'name' ), data.get( 'id' ), data.get( 'description' ) ].join( ' ' );
+                                data.set( 'search_matched', match.test( haystack ) );
+                        } );
                 }
         });
         api.Widgets.availableWidgets = new api.Widgets.WidgetCollection( api.Widgets.data.availableWidgets );
@@ -176 +170 @@
 
                         _.bindAll( this, 'close' );
 
-                        this.listenTo( this.collection, 'update', this.updateList );
                         this.listenTo( this.collection, 'change', this.updateList );
 
                         this.updateList();
@@ -222 +215 @@
                         }
                 },
 
-                // Changes visibilty of available widgets
+                // Changes visibility of available widgets
                 updateList: function() {
-                        // First hide all widgets...
-                        this.$el.find( '.widget-tpl' ).hide();
-
-                        // ..and then show only available widgets which could be filtered
                         this.collection.each( function( widget ) {
                                 var widgetTpl = $( '#widget-tpl-' + widget.id );
-                                widgetTpl.toggle( ! widget.get( 'is_disabled' ) );
+                                widgetTpl.toggle( widget.get( 'search_matched' ) && ! widget.get( 'is_disabled' ) );
                                 if ( widget.get( 'is_disabled' ) && widgetTpl.is( this.selected ) ) {
                                         this.selected = null;
                                 }
                         } );
                 },
 
-                // Hightlights a widget
+                // Highlights a widget
                 select: function( widgetTpl ) {
                         this.selected = $( widgetTpl );
                         this.selected.siblings( '.widget-tpl' ).removeClass( 'selected' );
                         this.selected.addClass( 'selected' );
                 },
 
-                // Hightlights a widget on focus
+                // Highlights a widget on focus
                 focus: function( event ) {
                         this.select( $( event.currentTarget ) );
                 },
@@ -1601 +1590 @@
 
                 /**
                  * @param {string} widgetId or an id_base for adding a previously non-existing widget
-                 * @returns {object} widget_form control instance
+                 * @returns {object|false} widget_form control instance, or false on error
                  */
                 addWidget: function( widgetId ) {
                         var self = this, controlHtml, $widget, controlType = 'widget_form', $control, controlConstructor,
@@ -1612 +1601 @@
                                 settingId, isExistingWidget, widgetFormControl, sidebarWidgets, settingArgs;
 
                         if ( ! widget ) {
-                                return;
+                                return false;
                         }
 
                         if ( widgetNumber && ! widget.get( 'is_multi' ) ) {
-                                return;
+                                return false;
                         }
 
                         // Set up new multi widget

src/wp-includes/class-wp-customize-widgets.php

@@ -1150 +1150 @@
                 $added_input_vars = array();
                 if ( ! empty( $_POST['sanitized_widget_setting'] ) ) {
                         $sanitized_widget_setting = json_decode( $this->get_post_value( 'sanitized_widget_setting' ), true );
-                        if ( empty( $sanitized_widget_setting ) ) {
+                        if ( false === $sanitized_widget_setting ) {
                                 $this->stop_capturing_option_updates();
                                 return new WP_Error( 'widget_setting_malformed' );
                         }