Customize: Improve behavior and extensibility of theme loading and searching.

* Introduce `WP_Customize_Themes_Section::$filter_type`, which has built-in functionality for `local` and `remote` filtering. When this set to `local`, all themes are assumed to be loaded from Ajax when the section is first loaded, and subsequent searching/filtering is applied to the loaded collection of themes within the section. This is how the core "Installed" section behaves - third-party sources with limited numbers of themes may consider leveraging this implementation. When this is set to `remote`, searching and filtering always triggers a new remote query via Ajax. The core "WordPress.org" section uses this approach, as it has over 5000 themes to search.
* Refactor `filterSearch()` to accept a raw term string as input. This enables a feature filter to be used on a section where `filter_type` is `local`.
* Refactor `filter()` on a theme control to check for an array of terms. Also sort the results by the number of matches. Rather than searching for an exact match, this will now search for each word in a search distinctly, allowing things like tags to rank in search results more accurately.
* Split `loadControls()` into two functions for themes section JS: `loadThemes()` to initiate and manage an Ajax request and `loadControls()` to create theme controls based on the results of the Ajax call. If third-party sections need to change the way controls are loaded, such as by using a custom control subclass of `WP_Customize_Theme_Control`, this allows them to use the core logic for managing the Ajax call and only override the actual control-creation process.
* Introduce `customize_load_themes` filter to facilitate loading themes from third-party sources (or modifying the results of the core sections).
* Bring significant improvements to the installed themes search filter.

Props celloexpressions.
Amends [41648].
See #37661.
Fixes #42049.

Built from https://develop.svn.wordpress.org/trunk@41807


git-svn-id: http://core.svn.wordpress.org/trunk@41641 1a063a9b-81f0-0310-95a4-ce76da25c4cd
This commit is contained in:
Weston Ruter 2017-10-10 07:09:52 +00:00
parent 58db3cb54e
commit 7b0dbf49fc
5 changed files with 199 additions and 92 deletions

View File

@ -1717,14 +1717,16 @@
section.closeDetails(); section.closeDetails();
}); });
if ( 'local' === section.params.filter_type ) {
// Filter-search all theme objects loaded in the section. // Filter-search all theme objects loaded in the section.
section.container.on( 'input', '.wp-filter-search-themes', function( event ) { section.container.on( 'input', '.wp-filter-search-themes', function( event ) {
section.filterSearch( event.currentTarget ); section.filterSearch( event.currentTarget.value );
}); });
// Event listeners for remote wporg queries with user-entered terms. } else if ( 'remote' === section.params.filter_type ) {
if ( 'wporg' === section.params.action ) {
// Event listeners for remote queries with user-entered terms.
// Search terms. // Search terms.
debounced = _.debounce( section.checkTerm, 500 ); // Wait until there is no input for 500 milliseconds to initiate a search. debounced = _.debounce( section.checkTerm, 500 ); // Wait until there is no input for 500 milliseconds to initiate a search.
section.contentContainer.on( 'input', '.wp-filter-search', function() { section.contentContainer.on( 'input', '.wp-filter-search', function() {
@ -1740,8 +1742,9 @@
section.filtersChecked(); section.filtersChecked();
section.checkTerm( section ); section.checkTerm( section );
}); });
}
// Toggle feature filter sections. // Toggle feature filters.
section.contentContainer.on( 'click', '.feature-filter-toggle', function( e ) { section.contentContainer.on( 'click', '.feature-filter-toggle', function( e ) {
$( e.currentTarget ) $( e.currentTarget )
.toggleClass( 'open' ) .toggleClass( 'open' )
@ -1762,7 +1765,6 @@
section.contentContainer.find( '.themes' ).css( 'margin-top', 0 ); section.contentContainer.find( '.themes' ).css( 'margin-top', 0 );
} }
}); });
}
// Setup section cross-linking. // Setup section cross-linking.
section.contentContainer.on( 'click', '.no-themes-local .search-dotorg-themes', function() { section.contentContainer.on( 'click', '.no-themes-local .search-dotorg-themes', function() {
@ -1806,7 +1808,7 @@
// Try to load controls if none are loaded yet. // Try to load controls if none are loaded yet.
if ( 0 === section.loaded ) { if ( 0 === section.loaded ) {
section.loadControls(); section.loadThemes();
} }
// Collapse any sibling sections/panels // Collapse any sibling sections/panels
@ -1821,13 +1823,16 @@
section.contentContainer.find( '.wp-filter-search' ).val( searchTerm ); section.contentContainer.find( '.wp-filter-search' ).val( searchTerm );
// Directly initialize an empty remote search to avoid a race condition. // Directly initialize an empty remote search to avoid a race condition.
if ( '' === searchTerm && '' !== section.term && 'installed' !== section.params.action ) { if ( '' === searchTerm && '' !== section.term && 'local' !== section.params.filter_type ) {
section.term = ''; section.term = '';
section.initializeNewQuery( section.term, section.tags ); section.initializeNewQuery( section.term, section.tags );
} else { } else {
if ( 'remote' === section.params.filter_type ) {
section.checkTerm( section ); section.checkTerm( section );
} else if ( 'local' === section.params.filter_type ) {
section.filterSearch( searchTerm );
}
} }
section.filterSearch( section.contentContainer.find( '.wp-filter-search' ).get( 0 ) );
} }
otherSection.collapse( { duration: args.duration } ); otherSection.collapse( { duration: args.duration } );
} }
@ -1877,7 +1882,7 @@
* *
* @returns {void} * @returns {void}
*/ */
loadControls: function() { loadThemes: function() {
var section = this, params, page, request; var section = this, params, page, request;
if ( section.loading ) { if ( section.loading ) {
@ -1894,8 +1899,8 @@
'page': page 'page': page
}; };
// Add fields for wporg actions. // Add fields for remote filtering.
if ( 'wporg' === section.params.action ) { if ( 'remote' === section.params.filter_type ) {
params.search = section.term; params.search = section.term;
params.tags = section.tags; params.tags = section.tags;
} }
@ -1906,7 +1911,7 @@
section.container.find( '.no-themes' ).hide(); section.container.find( '.no-themes' ).hide();
request = wp.ajax.post( 'customize_load_themes', params ); request = wp.ajax.post( 'customize_load_themes', params );
request.done(function( data ) { request.done(function( data ) {
var themes = data.themes, newThemeControls; var themes = data.themes;
// Stop and try again if the term changed while loading. // Stop and try again if the term changed while loading.
if ( '' !== section.nextTerm || '' !== section.nextTags ) { if ( '' !== section.nextTerm || '' !== section.nextTags ) {
@ -1919,26 +1924,13 @@
section.nextTerm = ''; section.nextTerm = '';
section.nextTags = ''; section.nextTags = '';
section.loading = false; section.loading = false;
section.loadControls(); section.loadThemes();
return; return;
} }
if ( 0 !== themes.length ) { if ( 0 !== themes.length ) {
newThemeControls = [];
// Add controls for each theme. section.loadControls( themes, page );
_.each( themes, function( theme ) {
var themeControl = new api.controlConstructor.theme( section.params.action + '_theme_' + theme.id, {
type: 'theme',
section: section.params.id,
theme: theme,
priority: section.loaded + 1
} );
api.control.add( themeControl );
newThemeControls.push( themeControl );
section.loaded = section.loaded + 1;
});
if ( 1 === page ) { if ( 1 === page ) {
@ -1950,15 +1942,14 @@
img.src = src; img.src = src;
} }
}); });
if ( 'installed' !== section.params.action ) { if ( 'local' !== section.params.filter_type ) {
wp.a11y.speak( api.settings.l10n.themeSearchResults.replace( '%d', data.info.results ) ); wp.a11y.speak( api.settings.l10n.themeSearchResults.replace( '%d', data.info.results ) );
} }
} else {
Array.prototype.push.apply( section.screenshotQueue, newThemeControls ); // Add new themes to the screenshot queue.
} }
_.delay( section.renderScreenshots, 100 ); // Wait for the controls to become visible. _.delay( section.renderScreenshots, 100 ); // Wait for the controls to become visible.
if ( 'installed' === section.params.action || 100 > themes.length ) { // If we have less than the requested 100 themes, it's the end of the list. if ( 'local' === section.params.filter_type || 100 > themes.length ) { // If we have less than the requested 100 themes, it's the end of the list.
section.fullyLoaded = true; section.fullyLoaded = true;
} }
} else { } else {
@ -1969,7 +1960,7 @@
section.fullyLoaded = true; section.fullyLoaded = true;
} }
} }
if ( 'installed' === section.params.action ) { if ( 'local' === section.params.filter_type ) {
section.updateCount(); // Count of visible theme controls. section.updateCount(); // Count of visible theme controls.
} else { } else {
section.updateCount( data.info.results ); // Total number of results including pages not yet loaded. section.updateCount( data.info.results ); // Total number of results including pages not yet loaded.
@ -1994,6 +1985,37 @@
}); });
}, },
/**
* Loads controls into the section from data received from loadThemes().
*
* @since 4.9.0
* @param {Array} themes - Array of theme data to create controls with.
* @param {integer} page - Page of results being loaded.
* @returns {void}
*/
loadControls: function( themes, page ) {
var newThemeControls = [],
section = this;
// Add controls for each theme.
_.each( themes, function( theme ) {
var themeControl = new api.controlConstructor.theme( section.params.action + '_theme_' + theme.id, {
type: 'theme',
section: section.params.id,
theme: theme,
priority: section.loaded + 1
} );
api.control.add( themeControl );
newThemeControls.push( themeControl );
section.loaded = section.loaded + 1;
});
if ( 1 !== page ) {
Array.prototype.push.apply( section.screenshotQueue, newThemeControls ); // Add new themes to the screenshot queue.
}
},
/** /**
* Determines whether more themes should be loaded, and loads them. * Determines whether more themes should be loaded, and loads them.
* *
@ -2009,7 +2031,7 @@
threshold = container.prop( 'scrollHeight' ) - 3000; // Use a fixed distance to the bottom of loaded results to avoid unnecessarily loading results sooner when using a percentage of scroll distance. threshold = container.prop( 'scrollHeight' ) - 3000; // Use a fixed distance to the bottom of loaded results to avoid unnecessarily loading results sooner when using a percentage of scroll distance.
if ( bottom > threshold ) { if ( bottom > threshold ) {
section.loadControls(); section.loadThemes();
} }
} }
}, },
@ -2019,23 +2041,26 @@
* *
* @since 4.9.0 * @since 4.9.0
* *
* @param {Element} el - The search input element as a raw JS object. * @param {string} term - The raw search input value.
* @returns {void} * @returns {void}
*/ */
filterSearch: function( el ) { filterSearch: function( term ) {
var count = 0, var count = 0,
visible = false, visible = false,
section = this, section = this,
noFilter = ( undefined !== api.section( 'wporg_themes' ) && 'wporg' !== section.params.action ) ? '.no-themes-local' : '.no-themes', noFilter = ( api.section.has( 'wporg_themes' ) && 'remote' !== section.params.filter_type ) ? '.no-themes-local' : '.no-themes',
term = el.value.toLowerCase().trim().replace( '-', ' ' ), controls = section.controls(),
controls = section.controls(); terms;
if ( section.loading ) { if ( section.loading ) {
return; return;
} }
// Standardize search term format and split into an array of individual words.
terms = term.toLowerCase().trim().replace( /-/g, ' ' ).split( ' ' );
_.each( controls, function( control ) { _.each( controls, function( control ) {
visible = control.filter( term ); visible = control.filter( terms ); // Shows/hides and sorts control based on the applicability of the search term.
if ( visible ) { if ( visible ) {
count = count + 1; count = count + 1;
} }
@ -2049,6 +2074,7 @@
} }
section.renderScreenshots(); section.renderScreenshots();
api.reflowPaneContents();
// Update theme count. // Update theme count.
section.updateCount( count ); section.updateCount( count );
@ -2064,7 +2090,7 @@
*/ */
checkTerm: function( section ) { checkTerm: function( section ) {
var newTerm; var newTerm;
if ( 'wporg' === section.params.action ) { if ( 'remote' === section.params.filter_type ) {
newTerm = section.contentContainer.find( '.wp-filter-search' ).val(); newTerm = section.contentContainer.find( '.wp-filter-search' ).val();
if ( section.term !== newTerm ) { if ( section.term !== newTerm ) {
section.initializeNewQuery( newTerm, section.tags ); section.initializeNewQuery( newTerm, section.tags );
@ -2104,7 +2130,11 @@
if ( section.loading ) { if ( section.loading ) {
section.nextTags = tags; section.nextTags = tags;
} else { } else {
if ( 'remote' === section.params.filter_type ) {
section.initializeNewQuery( section.term, tags ); section.initializeNewQuery( section.term, tags );
} else if ( 'local' === section.params.filter_type ) {
section.filterSearch( tags.join( ' ' ) );
}
} }
} }
}, },
@ -2130,14 +2160,14 @@
section.fullyLoaded = false; section.fullyLoaded = false;
section.screenshotQueue = null; section.screenshotQueue = null;
// Run a new query, with loadControls handling paging, etc. // Run a new query, with loadThemes handling paging, etc.
if ( ! section.loading ) { if ( ! section.loading ) {
section.term = newTerm; section.term = newTerm;
section.tags = newTags; section.tags = newTags;
section.loadControls(); section.loadThemes();
} else { } else {
section.nextTerm = newTerm; // This will reload from loadControls() with the newest term once the current batch is loaded. section.nextTerm = newTerm; // This will reload from loadThemes() with the newest term once the current batch is loaded.
section.nextTags = newTags; // This will reload from loadControls() with the newest tags once the current batch is loaded. section.nextTags = newTags; // This will reload from loadThemes() with the newest tags once the current batch is loaded.
} }
if ( ! section.expanded() ) { if ( ! section.expanded() ) {
section.expand(); // Expand the section if it isn't expanded. section.expand(); // Expand the section if it isn't expanded.
@ -4875,20 +4905,50 @@
* Show or hide the theme based on the presence of the term in the title, description, tags, and author. * Show or hide the theme based on the presence of the term in the title, description, tags, and author.
* *
* @since 4.2.0 * @since 4.2.0
* @param {Array} terms - An array of terms to search for.
* @returns {boolean} Whether a theme control was activated or not. * @returns {boolean} Whether a theme control was activated or not.
*/ */
filter: function( term ) { filter: function( terms ) {
var control = this, var control = this,
matchCount = 0,
haystack = control.params.theme.name + ' ' + haystack = control.params.theme.name + ' ' +
control.params.theme.description + ' ' + control.params.theme.description + ' ' +
control.params.theme.tags + ' ' + control.params.theme.tags + ' ' +
control.params.theme.author; control.params.theme.author + ' ';
haystack = haystack.toLowerCase().replace( '-', ' ' ); haystack = haystack.toLowerCase().replace( '-', ' ' );
if ( -1 !== haystack.search( term ) ) {
// Back-compat for behavior in WordPress 4.2.0 to 4.8.X.
if ( ! _.isArray( terms ) ) {
terms = [ terms ];
}
// Always give exact name matches highest ranking.
if ( control.params.theme.name.toLowerCase() === terms.join( ' ' ) ) {
matchCount = 100;
} else {
// Search for and weight (by 10) complete term matches.
matchCount = matchCount + 10 * ( haystack.split( terms.join( ' ' ) ).length - 1 );
// Search for each term individually (as whole-word and partial match) and sum weighted match counts.
_.each( terms, function( term ) {
matchCount = matchCount + 2 * ( haystack.split( term + ' ' ).length - 1 ); // Whole-word, double-weighted.
matchCount = matchCount + haystack.split( term ).length - 1; // Partial word, to minimize empty intermediate searches while typing.
});
// Upper limit on match ranking.
if ( matchCount > 99 ) {
matchCount = 99;
}
}
if ( 0 !== matchCount ) {
control.activate(); control.activate();
control.params.priority = 101 - matchCount; // Sort results by match count.
return true; return true;
} else { } else {
control.deactivate(); control.deactivate(); // Hide control
control.params.priority = 101;
return false; return false;
} }
}, },

File diff suppressed because one or more lines are too long

View File

@ -4415,6 +4415,7 @@ final class WP_Customize_Manager {
$this->add_section( new WP_Customize_Themes_Section( $this, 'wporg_themes', array( $this->add_section( new WP_Customize_Themes_Section( $this, 'wporg_themes', array(
'title' => __( 'WordPress.org themes' ), 'title' => __( 'WordPress.org themes' ),
'action' => 'wporg', 'action' => 'wporg',
'filter_type' => 'remote',
'capability' => 'install_themes', 'capability' => 'install_themes',
'panel' => 'themes', 'panel' => 'themes',
'priority' => 5, 'priority' => 5,
@ -4947,23 +4948,48 @@ final class WP_Customize_Manager {
} }
$theme_action = sanitize_key( $_POST['theme_action'] ); $theme_action = sanitize_key( $_POST['theme_action'] );
$themes = array(); $themes = array();
$args = array();
// Define query filters based on user input.
if ( ! array_key_exists( 'search', $_POST ) ) {
$args['search'] = '';
} else {
$args['search'] = sanitize_text_field( wp_unslash( $_POST['search'] ) );
}
if ( ! array_key_exists( 'tags', $_POST ) ) {
$args['tag'] = '';
} else {
$args['tag'] = array_map( 'sanitize_text_field', wp_unslash( (array) $_POST['tags'] ) );
}
if ( ! array_key_exists( 'page', $_POST ) ) {
$args['page'] = 1;
} else {
$args['page'] = absint( $_POST['page'] );
}
require_once ABSPATH . 'wp-admin/includes/theme.php'; require_once ABSPATH . 'wp-admin/includes/theme.php';
if ( 'installed' === $theme_action ) { if ( 'installed' === $theme_action ) {
// Load all installed themes from wp_prepare_themes_for_js().
$themes = array( 'themes' => wp_prepare_themes_for_js() ); $themes = array( 'themes' => wp_prepare_themes_for_js() );
foreach ( $themes['themes'] as &$theme ) { foreach ( $themes['themes'] as &$theme ) {
$theme['type'] = 'installed'; $theme['type'] = 'installed';
$theme['active'] = ( isset( $_POST['customized_theme'] ) && $_POST['customized_theme'] === $theme['id'] ); $theme['active'] = ( isset( $_POST['customized_theme'] ) && $_POST['customized_theme'] === $theme['id'] );
} }
} elseif ( 'wporg' === $theme_action ) { } elseif ( 'wporg' === $theme_action ) {
// Load WordPress.org themes from the .org API and normalize data to match installed theme objects.
if ( ! current_user_can( 'install_themes' ) ) { if ( ! current_user_can( 'install_themes' ) ) {
wp_die( -1 ); wp_die( -1 );
} }
// Arguments for all queries. // Arguments for all queries.
$args = array( $wporg_args = array(
'per_page' => 100, 'per_page' => 100,
'page' => isset( $_POST['page'] ) ? absint( $_POST['page'] ) : 1,
'fields' => array( 'fields' => array(
'screenshot_url' => true, 'screenshot_url' => true,
'description' => true, 'description' => true,
@ -4979,18 +5005,7 @@ final class WP_Customize_Manager {
), ),
); );
// Define query filters based on user input. $args = array_merge( $wporg_args, $args );
if ( ! array_key_exists( 'search', $_POST ) ) {
$args['search'] = '';
} else {
$args['search'] = sanitize_text_field( wp_unslash( $_POST['search'] ) );
}
if ( ! array_key_exists( 'tags', $_POST ) ) {
$args['tag'] = '';
} else {
$args['tag'] = array_map( 'sanitize_text_field', wp_unslash( (array) $_POST['tags'] ) );
}
if ( '' === $args['search'] && '' === $args['tag'] ) { if ( '' === $args['search'] && '' === $args['tag'] ) {
$args['browse'] = 'new'; // Sort by latest themes by default. $args['browse'] = 'new'; // Sort by latest themes by default.
@ -5061,6 +5076,26 @@ final class WP_Customize_Manager {
unset( $theme->author ); unset( $theme->author );
} // End foreach(). } // End foreach().
} // End if(). } // End if().
/**
* Filters the theme data loaded in the customizer.
*
* This allows theme data to be loading from an external source,
* or modification of data loaded from `wp_prepare_themes_for_js()`
* or WordPress.org via `themes_api()`.
*
* @since 4.9.0
*
* @see wp_prepare_themes_for_js()
* @see themes_api()
* @see WP_Customize_Manager::__construct()
*
* @param array $themes Nested array of theme data.
* @param array $args List of arguments, such as page, search term, and tags to query for.
* @param WP_Customize_Manager $manager Instance of Customize manager.
*/
$themes = apply_filters( 'customize_load_themes', $themes, $args, $this );
wp_send_json_success( $themes ); wp_send_json_success( $themes );
} }

View File

@ -36,6 +36,17 @@ class WP_Customize_Themes_Section extends WP_Customize_Section {
*/ */
public $action = ''; public $action = '';
/**
* Theme section filter type.
*
* Determines whether filters are applied to loaded (local) themes or by initiating a new remote query (remote).
* When filtering is local, the initial themes query is not paginated by default.
*
* @since 4.9.0
* @var string
*/
public $filter_type = 'local';
/** /**
* Get section parameters for JS. * Get section parameters for JS.
* *
@ -45,6 +56,7 @@ class WP_Customize_Themes_Section extends WP_Customize_Section {
public function json() { public function json() {
$exported = parent::json(); $exported = parent::json();
$exported['action'] = $this->action; $exported['action'] = $this->action;
$exported['filter_type'] = $this->filter_type;
return $exported; return $exported;
} }

View File

@ -4,7 +4,7 @@
* *
* @global string $wp_version * @global string $wp_version
*/ */
$wp_version = '4.9-beta1-41806'; $wp_version = '4.9-beta1-41807';
/** /**
* Holds the WordPress DB revision, increments when changes are made to the WordPress DB schema. * Holds the WordPress DB revision, increments when changes are made to the WordPress DB schema.