Changeset 11768 for sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-gp-plugin-directory/inc/sync/class-translation-sync.php

Timestamp:

04/12/2022 06:52:51 PM (4 years ago)

Author:

ocean90

Message:

Translate: Update Slack notifications for #meta-language-packs.

• Include project type in notifications for translation syncs.
• Exclude lines for sets without any translations when building new language packs.

File:

• sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-gp-plugin-directory/inc/sync/class-translation-sync.php (modified) (3 diffs)

sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-gp-plugin-directory/inc/sync/class-translation-sync.php

@@ -49 +49 @@
 
                 // Sync translations only if the stable project was updated.
-                if ( false === strpos( $project->path, '/stable' ) ) {
+                if ( str_ends_with( $project->path, '/dev' ) || str_ends_with( $project->path, '/dev-readme' ) ) {
                         return;
                 }
@@ -85 +85 @@
                 }
 
-                $sub_project = basename( $args['gp_project'] );
-                $sub_project_counterpart = $this->project_mapping[ $sub_project ];
-
                 $timestamp = time();
                 $message   = '';
-                $updates   = 0;
 
                 foreach ( $translation_sets as $translation_set ) {
@@ -98 +94 @@
 
                         // Sync translations in a separate process.
-                        $cmd = WPORGTRANSLATE_WPCLI . ' wporg-translate sync-plugin-translations ' . escapeshellarg( $args['gp_project'] ) . ' ' . escapeshellarg( $translation_set->locale ) . ' --set=' . escapeshellarg( $translation_set->slug ) . ' 2>&1';
-                        $output = '';
+                        $cmd        = WPORGTRANSLATE_WPCLI . ' wporg-translate sync-plugin-translations ' . escapeshellarg( $args['gp_project'] ) . ' ' . escapeshellarg( $translation_set->locale ) . ' --set=' . escapeshellarg( $translation_set->slug ) . ' 2>&1';
+                        $output     = [];
                         $return_var = 0;
                         exec( $cmd, $output, $return_var );
                         if ( $return_var ) {
-                                $message .= "\tFailure: " . implode( "\n\t", $output ) . "\n";
+                                $message .= 'Failure: ' . implode( "\n", $output ) . "\n";
                         } else {
-                                $message .= "\t" . implode( "\n\t", $output ) . "\n";
-                        }
-                        $updates += 1;
-                }
-
-                if ( ! $updates ) {
-                        $message .= "\tNo translations are available to sync.\n";
-                }
-
-                $message .= 'Translation sync was successfully processed.';
+                                $message .= implode( "\n", $output ) . "\n";
+                        }
+                }
+
+                $message = trim( $message );
+                if ( ! $message ) {
+                        $message = 'No translations are available to sync.';
+                }
+
+                $type = str_ends_with( $project->path, '/dev-readme' ) ? 'readme' : 'code';
 
                 $attachment = [
-                        'title'      => "Translation Sync for {$args['plugin']}",
+                        'title'      => "Translation sync for {$type} of {$args['plugin']}",
                         'title_link' => "https://translate.wordpress.org/projects/wp-plugins/{$args['plugin']}",
                         'text'       => $message,
-                        'fallback'   => "Translations for {$args['plugin']} were synced.",
+                        'fallback'   => "Translations for {$type} of {$args['plugin']} were synced.",
                         'color'      => '#00a0d2',
                         'mrkdwn_in'  => [ 'text' ],