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

Timestamp:

09/16/2016 08:56:17 PM (7 years ago)

Author:

ocean90

Message:

Translate, Rosetta Roles: Add support for the new locale manager role.

See #1398.

File:

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

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

@@ -594 +594 @@
             // Check to see if they have any special approval permissions
             $allowed_projects = array();
-            if ( ! $can_approve_for_all && $this->roles_adapter->is_approver_for_locale( $user_id, $locale ) ) {
+            if ( ! $can_approve_for_all && $role = $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.
-                if ( in_array( 'all', $allowed_projects ) ) {
+                if ( Rosetta_Roles::LOCALE_MANAGER_ROLE === $role || in_array( 'all', $allowed_projects ) ) {
                     $can_approve_for_all = true;
                     $allowed_projects = array();