Changeset 2037

Timestamp:

11/04/2015 04:59:09 PM (9 years ago)

Author:

ocean90

Message:

Rosetta Roles: Include project slugs in search fields.

Also bump checked projects to the top.

Fixes #1356.

Location:

sites/trunk/global.wordpress.org/public_html/wp-content/mu-plugins/roles

Files:

• js/rosetta-roles.js (modified) (2 diffs)
• rosetta-roles.php (modified) (2 diffs)

sites/trunk/global.wordpress.org/public_html/wp-content/mu-plugins/roles/js/rosetta-roles.js

@@ -131 +131 @@
         // @uses RegExp
         search: function( term ) {
-            var match, name;
+            var match;
 
             // Escape the term string for RegExp meta characters
@@ -143 +143 @@
             // Find results
             this.each( function( project ) {
-                name = project.get( 'name' );
-                project.set( 'isVisible', match.test( name ) );
-            });
-        },
+                var haystack = _.union( project.get( 'name' ), project.get( 'slug' ) );
+                project.set( 'isVisible', match.test( haystack ) );
+            });
+        },
+
+        comparator: function( project ) {
+            return - project.get( 'checked' );
+        }
     });
 

sites/trunk/global.wordpress.org/public_html/wp-content/mu-plugins/roles/rosetta-roles.php

@@ -221 +221 @@
      */
     public function enqueue_scripts() {
-        wp_enqueue_script( 'rosetta-roles', plugins_url( '/js/rosetta-roles.js', __FILE__ ), array( 'jquery', 'wp-backbone' ), '2', true );
+        wp_enqueue_script( 'rosetta-roles', plugins_url( '/js/rosetta-roles.js', __FILE__ ), array( 'jquery', 'wp-backbone' ), '3', true );
     }
 
@@ -656 +656 @@
 
         $_projects = $wpdb->get_results( "
-            SELECT id, name, parent_project_id
+            SELECT id, name, parent_project_id, slug
             FROM translate_projects
             WHERE id NOT IN( " . implode( ',', $ignore_project_ids ) . " )