Changeset 3952 for sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-gp-routes/inc/routes/class-locale.php

Timestamp:

09/04/2016 07:57:05 PM (9 years ago)

Author:

ocean90

Message:

Translate, Rosetta Roles: Refactor the plugin to use an autoloader and add an initial list table to manage translators.

See #1770.

File:

• sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-gp-routes/inc/routes/class-locale.php (modified) (7 diffs)

sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-gp-routes/inc/routes/class-locale.php

@@ -8 +8 @@
 use GP_Route;
 use stdClass;
+use WordPressdotorg\GlotPress\Rosetta_Roles\Plugin as Rosetta_Roles;
 
 /**
@@ -17 +18 @@
 
     /**
+     * Adapter for the rosetta roles plugin.
+     *
+     * @var null|Rosetta_Roles
+     */
+    private $roles_adapter = null;
+
+    public function __construct() {
+        parent::__construct();
+
+        if ( method_exists( Rosetta_Roles::class, 'get_instance' ) ) {
+            $this->roles_adapter = Rosetta_Roles::get_instance();
+        }
+    }
+
+    /**
      * Prints projects/translation sets of a top level project.
      *
@@ -45 +61 @@
         if (
             ! is_user_logged_in() ||
-            ! function_exists( 'wporg_gp_rosetta_roles' ) || // Rosetta Roles plugin is not enabled
+            ! $this->roles_adapter || // Rosetta Roles plugin is not enabled
             ! (
-                wporg_gp_rosetta_roles()->is_global_administrator( $user_id ) || // Not a global admin
-                wporg_gp_rosetta_roles()->is_approver_for_locale( $user_id, $locale_slug ) // Doesn't have project-level access either
+                $this->roles_adapter->is_global_administrator( $user_id ) || // Not a global admin
+                $this->roles_adapter->is_approver_for_locale( $user_id, $locale_slug ) // Doesn't have project-level access either
             )
             // Add check to see if there are any waiting translations for this locale?
@@ -565 +581 @@
         // Special Waiting Project Tab
         // This removes the parent_project_id restriction and replaces it with all-translation-editer-projects
-        if ( 'waiting' == $project->slug && is_user_logged_in() && function_exists( 'wporg_gp_rosetta_roles' ) ) {
+        if ( 'waiting' == $project->slug && is_user_logged_in() && $this->roles_adapter ) {
 
             if ( ! $filter ) {
@@ -574 +590 @@
 
             // Global Admin or Locale-specific admin
-            $can_approve_for_all = wporg_gp_rosetta_roles()->is_global_administrator( $user_id );
+            $can_approve_for_all = $this->roles_adapter->is_global_administrator( $user_id );
 
             // Check to see if they have any special approval permissions
             $allowed_projects = array();
-            if ( ! $can_approve_for_all && wporg_gp_rosetta_roles()->is_approver_for_locale( $user_id, $locale ) ) {
-                $allowed_projects = wporg_gp_rosetta_roles()->get_project_id_access_list( $user_id, $locale, true );
+            if ( ! $can_approve_for_all && $this->roles_adapter->is_approver_for_locale( $user_id, $locale ) ) {
+                $allowed_projects = $this->roles_adapter->get_project_id_access_list( $user_id, $locale, true );
 
                 // Check to see if they can approve for all projects in this locale.
@@ -588 +604 @@
             }
 
-            $parent_project_sql = '';
             if ( $can_approve_for_all ) {
                 // The current user can approve for all projects, so just grab all with any waiting strings.
@@ -607 +622 @@
             // Limit to only showing base-level projects
             $parent_project_sql .= " AND tp.parent_project_id IN( (SELECT id FROM {$wpdb->gp_projects} WHERE parent_project_id IS NULL AND active = 1) )";
-
         }