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

Timestamp:

02/22/2016 08:50:25 PM (9 years ago)

Author:

ocean90

Message:

Rosetta, Roles: Improve performance of the edit page.

• When searching filter the sub-projects collection directly and re-render the list once. Previously a CSS class was added/removed and thus every item was re-rendered. H/t adamsilverstein.
• Only render the first 100 sub-projects of a project. The others can be revealed through a search.
• Add an Ajax handler for the projects data and store the JSON data in browser's local storage.

See #1590.

File:

• sites/trunk/global.wordpress.org/public_html/wp-content/mu-plugins/roles/rosetta-roles.php (modified) (3 diffs)

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

@@ -77 +77 @@
         add_action( 'translation_editor_added', array( $this, 'update_wporg_profile_badge' ) );
         add_action( 'translation_editor_removed', array( $this, 'update_wporg_profile_badge' ) );
+
+        add_action( 'wp_ajax_rosetta-get-projects', array( $this, 'ajax_rosetta_get_projects' ) );
     }
 
@@ -619 +621 @@
      */
     private function render_edit_translation_editor( $user_id ) {
+        global $wpdb;
+
+        $project_access_list = $this->get_users_projects( $user_id );
+
+        $last_updated = $wpdb->get_var( 'SELECT meta_value FROM translate_meta WHERE object_type = "gp_option" AND meta_key = "wporg_projects_last_updated"' );
+
+        wp_localize_script( 'rosetta-roles', '_rosettaProjectsSettings', array(
+            'l10n' => array(
+                'searchPlaceholder' => esc_attr__( 'Search...', 'rosetta' ),
+            ),
+            'lastUpdated' => $last_updated,
+            'accessList' => $project_access_list,
+        ) );
+
+        $feedback_message = $this->get_feedback_message();
+
+        require __DIR__ . '/views/edit-translation-editor.php';
+    }
+
+    /**
+     * Ajax handler for retrieving projects.
+     */
+    public function ajax_rosetta_get_projects() {
         $projects = $this->get_translate_projects();
         $project_tree = $this->get_project_tree( $projects, 0, 1 );
@@ -632 +657 @@
         $project_tree = array_values( $project_tree );
 
-        $project_access_list = $this->get_users_projects( $user_id );
-
-        wp_localize_script( 'rosetta-roles', '_rosettaProjectsSettings', array(
-            'l10n' => array(
-                'searchPlaceholder' => esc_attr__( 'Search...', 'rosetta' ),
-            ),
-            'data' => $project_tree,
-            'accessList' => $project_access_list,
-        ) );
-
-        $feedback_message = $this->get_feedback_message();
-
-        require __DIR__ . '/views/edit-translation-editor.php';
+        ob_start( 'ob_gzhandler' ); // Compress JSON.
+        wp_send_json_success( $project_tree );
     }