Changeset 10358 for sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-gp-rosetta-roles/inc/class-plugin.php

Timestamp:

10/11/2020 04:01:43 PM (4 years ago)

Author:

ocean90

Message:

Translate: Introduce the concept of read-only translation sets.

See #5471.

File:

• sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-gp-rosetta-roles/inc/class-plugin.php (modified) (4 diffs)

sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-gp-rosetta-roles/inc/class-plugin.php

@@ -37 +37 @@
      */
     const LOCALE_MANAGER_ROLE = 'locale_manager';
+
+    /**
+     * Paths of translation sets the user cannot edit.
+     */
+    const READ_ONLY_TRANSLATION_SETS = [ 'sr/latin' ];
 
     /**
@@ -114 +119 @@
         }
 
-        // Allow logged in users to submit translations.
-        if ( 'edit' == $args['action'] && 'translation-set' === $args['object_type'] ) {
-            return is_user_logged_in();
-        }
-
         // Get locale and current project ID.
         $locale_and_project_id = (object) $this->get_locale_and_project_id( $args['object_type'], $args['object_id'], $args );
         if ( ! $locale_and_project_id ) {
             return false;
+        }
+
+        $set_path = $locale_and_project_id->locale . '/' . $locale_and_project_id->set_slug;
+        if ( \in_array( $set_path, self::READ_ONLY_TRANSLATION_SETS, true ) ) {
+            return false;
+        }
+
+        // Allow logged in users to submit translations.
+        if ( 'edit' == $args['action'] && 'translation-set' === $args['object_type'] ) {
+            return true;
         }
 
@@ -512 +522 @@
                 }
 
-                return array( 'locale' => $set->locale, 'project_id' => (int) $set->project_id );
+                return array( 'locale' => $set->locale, 'set_slug' => $set->slug, 'project_id' => (int) $set->project_id );
 
             case 'translation-set' :
@@ -522 +532 @@
                 }
 
-                return array( 'locale' => $set->locale, 'project_id' => (int) $set->project_id );
+                return array( 'locale' => $set->locale, 'set_slug' => $set->slug, 'project_id' => (int) $set->project_id );
 
             case 'project|locale|set-slug' :
-                list( $project_id, $locale ) = explode( '|', $object_id );
-                return array( 'locale' => $locale, 'project_id' => (int) $project_id );
+                list( $project_id, $locale, $set_slug ) = explode( '|', $object_id );
+                return array( 'locale' => $locale, 'set_slug' => $set_slug, 'project_id' => (int) $project_id );
         }