Changeset 8902

Timestamp:

05/31/2019 07:51:49 AM (7 years ago)

Author:

dd32

Message:

Plugin Directory: Sanitize some search terms to allow better matching and more cache hits for searches.

See #4469.

File:

• sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/class-plugin-directory.php (modified) (1 diff)

sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/class-plugin-directory.php

@@ -868 +868 @@
                         $wp_query->query_vars['post_status'][] = 'disabled';
                         $wp_query->query_vars['post_status']   = array_unique( $wp_query->query_vars['post_status'] );
+                }
+
+                // Sanitize / cleanup the search query a little bit.
+                if ( $wp_query->is_search() ) {
+                        $s = $wp_query->get( 's' );
+                        $s = urldecode( $s );
+
+                        // If a URL-like request comes in, reduce to a slug
+                        if ( preg_match( '!^http.+/plugins/([^/]+)(/|$)!i', $s, $m ) ) {
+                                $s = $m[1];
+                        }
+
+                        // Jetpack Search has a limit, limit to 200char. This is intentionally using ASCII length + Multibyte substr.
+                        if ( strlen( $s ) > 200 ) {
+                                $s = mb_substr( $s, 0, 200 );
+                        }
+
+                        // Trim off special characters, only allowing wordy characters at the end of searches.s
+                        $s = preg_replace( '!(\W+)$!i', '', $s );
+                        // ..and whitespace
+                        $s = trim( $s );
+
+                        $wp_query->set( 's', $s );
                 }