Changeset 11937

Timestamp:

07/05/2022 04:59:28 PM (15 months ago)

Author:

iandunn

Message:

GP Profiles: Process bulk translation activity.

Only approvals are done at the moment, but reject/fuzzy will be added when we start to credit editor review actions.

See ​https://github.com/WordPress/five-for-the-future/issues/196

Location:

sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-gp-profiles

Files:

• profiles.php (modified) (3 diffs)
• tests/e2e.php (modified) (2 diffs)

sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-gp-profiles/profiles.php

@@ -10 +10 @@
 
 use WordPressdotorg\Profiles as Profiles_API;
-use GP, GP_Translation;
+use GP, GP_Translation, GP_Project, GP_Locale, GP_Translation_Set;
 
 defined( 'WPINC' ) || die();
@@ -23 +23 @@
 }
 
-add_action( 'gp_translation_created', __NAMESPACE__ . '\add_translation_activity' );
-add_action( 'gp_translation_saved', __NAMESPACE__ . '\add_translation_activity', 10, 2 );
+add_action( 'gp_translation_created', __NAMESPACE__ . '\add_single_translation_activity' );
+add_action( 'gp_translation_saved', __NAMESPACE__ . '\add_single_translation_activity', 10, 2 );
+add_action( 'gp_translation_set_bulk_action_post', __NAMESPACE__ . '\add_bulk_translation_activity', 10, 4 );
 
 /**
  * Add a activity when strings are suggested and approved.
  */
-function add_translation_activity( GP_Translation $new_translation, GP_Translation $previous_translation = null ) : void {
+function add_single_translation_activity( GP_Translation $new_translation, GP_Translation $previous_translation = null ) : void {
+    $bulk_request = gp_post( 'bulk', null );
+
+    // Bulk actions are handled by `add_bulk_translation_activity()`.
+    if ( $bulk_request ) {
+        return;
+    }
+
     $current_user_is_editor = GP::$permission->current_user_can(
         'approve',
@@ -58 +66 @@
     Profiles_API\api( $request_body );
 }
+
+/**
+ * Add activity when bulk actions are performed.
+ */
+function add_bulk_translation_activity( GP_Project $project, GP_Locale $locale, GP_Translation_Set $translation_set, array $bulk ) : void {
+    switch ( $bulk['action'] ) {
+        case 'approve':
+            $type = 'glotpress_translation_approved';
+            break;
+
+        default:
+            return;
+    }
+
+    $translation_ids = array();
+    foreach ( $bulk['row-ids'] as $row_id ) {
+        $parts             = explode( '-', $row_id );
+        $translation_ids[] = intval( $parts[1] ?? 0 );
+    }
+
+    $translations = GP::$translation->find_many(
+        sprintf( 'id IN ( %s )', implode( ',', $translation_ids ) )
+    );
+
+    $user_type_counts = array();
+    foreach ( $translations as $translation ) {
+        // `GP_Route_Translation::_bulk_approve()` tracks which `set_status()` calls succeed, but that information
+        // isn't passed to this callback. Check this just in case any of them failed.
+        if ( 'current' !== $translation->status ) {
+            continue;
+        }
+
+        $user_id = $translation->user_id;
+
+        if ( isset( $user_type_counts[ $user_id ][ $type ] ) ) {
+            $user_type_counts[ $user_id ][ $type ]++;
+        } else {
+            $user_type_counts[ $user_id ][ $type ] = 1;
+        }
+    }
+
+    $activities = array();
+    foreach ( $user_type_counts as $user_id => $types ) {
+        foreach ( $types as $type => $count ) {
+            $activities[] = array(
+                'component' => 'glotpress',
+                'type'      => $type,
+                'user_id'   => $user_id,
+                'bump'      => $count,
+            );
+        }
+    }
+
+    $request_body = array(
+        'action'     => 'wporg_handle_activity',
+        'component'  => 'glotpress',
+        'activities' => $activities,
+    );
+
+    Profiles_API\api( $request_body );
+}

sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-gp-profiles/tests/e2e.php

@@ -45 +45 @@
         'status'  => 'waiting',
     ) );
-    GlotPress_Profiles\add_translation_activity( $translation );
+    GlotPress_Profiles\add_single_translation_activity( $translation );
 
     $translation = new GP_Translation( array(
@@ -51 +51 @@
         'status'  => 'current',
     ) );
-    GlotPress_Profiles\add_translation_activity( $translation );
+    GlotPress_Profiles\add_single_translation_activity( $translation );
 
     echo "\nThe daily digest count should have been bumped on https://profiles.wordpress.org/$user->user_nicename/ \n";