Changeset 8902

Timestamp:

05/31/2019 07:51:49 AM (6 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 );
         }