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

Timestamp:

02/18/2018 02:54:16 PM (7 years ago)

Author:

ocean90

Message:

Rosetta Roles: Pass the full user object to the translation_editor_* actions.

Also include the list of added/removed project IDs.

See #1613.

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

@@ -399 +399 @@
         ", $user->ID, $this->gp_locale->slug ) );
 
-        do_action( 'translation_editor_removed', $user->ID );
+        do_action( 'translation_editor_removed', $user );
 
         return true;
@@ -485 +485 @@
 
         if ( $update ) {
-            do_action( 'translation_editor_updated', $user->ID );
+            do_action( 'translation_editor_updated', $user, $projects_to_add, $projects_to_remove );
         } else {
-            do_action( 'translation_editor_added', $user->ID );
+            do_action( 'translation_editor_added', $user, $projects_to_add, $projects_to_remove );
         }
 
@@ -497 +497 @@
      * profiles.wordpress.org.
      *
-     * @param int $user_id User ID.
-     */
-    public function update_wporg_profile_badge( $user_id ) {
+     * @param \WP_User $user The user object of the translation editor.
+     */
+    public function update_wporg_profile_badge( $user ) {
         $action = 'translation_editor_added' === current_filter() ? 'add' : 'remove';
 
-        $this->notify_profiles_wporg_translation_editor_update( $user_id, $action );
+        $this->notify_profiles_wporg_translation_editor_update( $user->ID, $action );
     }