Ticket #3876: 3876.diff

wordpress.org/public_html/wp-content/plugins/wporg-gp-customizations/inc/cli/class-language-pack.php

@@ -374 +374 @@
         }
 
         /**
-         * Builds a PO file for translations.
+         * Builds a mapping of JS file names to translation entries.
          *
-         * @param GP_Project         $gp_project The GlotPress project.
-         * @param GP_Locale          $gp_locale  The GlotPress locale.
-         * @param GP_Translation_Set $set        The translation set.
-         * @param string             $dest       Destination file name.
-         * @return string|WP_Error Last updated date on success, WP_Error on failure.
+         * @param GP_Project          $gp_project The GlotPress project.
+         * @param GP_Locale           $gp_locale  The GlotPress locale.
+         * @param GP_Translation_Set  $set        The translation set.
+         * @param Translation_Entry[] $entries    The translation entries.
+         * @param string              $dest       Destination file name.
+         * @return array An array of translation files built, may be empty if no translations in JS files exist.
          */
-        private function build_po_file( $gp_project, $gp_locale, $set, $dest ) {
-                $entries = GP::$translation->for_export( $gp_project, $set, [ 'status' => 'current' ] );
-                if ( ! $entries ) {
-                        return new WP_Error( 'no_translations', 'No current translations available.' );
+        private function build_json_files(  $gp_project, $gp_locale, $set, $entries, $base_dest ) {
+                // Build a mapping of JS files to translation entries occuring in those files.
+                $mapping = array();
+                foreach ( $entries as $entry ) {
+                        /** @var Translation_Entry $entry */
+                        foreach ( $entry->references as $reference ) {
+                                // Split the references to get the filename. References are in relative_to_root_path/file.exit:linenum form.
+                                $parts = explode( ':', $reference );
+                                $file  = $parts[0];
+
+                                if ( substr( $file, -3 ) === '.js' ) {
+                                        $mapping[ $file ][] = $entry;
+                                }
+                        }
                 }
 
+                // Export translations for each JS file to a separate translation file.
+                $files  = array();
+                $format = gp_array_get( GP::$formats, 'json' );
+                foreach ( $mapping as $file => $file_entries ) {
+                        $json_content = $format->print_exported_file( $gp_project, $gp_locale, $set, $file_entries );
+
+                        $hash = md5( $file );
+                        $dest = "{$base_dest}-{$hash}.json";
+
+                        file_put_contents( $dest, $json_content );
+
+                        $files[] = $dest;
+                }
+
+                return $files;
+        }
+
+        /**
+         * Builds a PO file for translations.
+         *
+         * @param GP_Project          $gp_project The GlotPress project.
+         * @param GP_Locale           $gp_locale  The GlotPress locale.
+         * @param GP_Translation_Set  $set        The translation set.
+         * @param Translation_Entry[] $entries    The translation entries.
+         * @param string              $dest       Destination file name.
+         * @return string|WP_Error Last updated date on success, WP_Error on failure.
+         */
+        private function build_po_file( $gp_project, $gp_locale, $set, $entries, $dest ) {
                 $format     = gp_array_get( GP::$formats, 'po' );
                 $po_content = $format->print_exported_file( $gp_project, $gp_locale, $set, $entries );
 
@@ -523 +562 @@
                         $export_directory = "{$data->svn_checkout}/{$data->domain}/{$data->version}/{$wp_locale}";
                         $build_directory  = self::BUILD_DIR . "/{$data->type}s/{$data->domain}/{$data->version}";
                         $filename         = "{$data->domain}-{$wp_locale}";
+                        $json_file_base   = "{$export_directory}/{$filename}";
                         $po_file          = "{$export_directory}/{$filename}.po";
                         $mo_file          = "{$export_directory}/{$filename}.mo";
                         $zip_file         = "{$export_directory}/{$filename}.zip";
@@ -531 +571 @@
                         // Update/create directories.
                         $this->update_svn_directory( $export_directory );
 
+                        $entries = GP::$translation->for_export( $data->gp_project, $set, [ 'status' => 'current' ] );
+                        if ( ! $entries ) {
+                                WP_CLI::warning( "No current translations available for {$wp_locale}." );
+
+                                continue;
+                        }
+
+                        $json_files = $this->build_json_files( $data->gp_project, $gp_locale, $set, $entries, $json_file_base );
+
                         // Create PO file.
-                        $last_modified = $this->build_po_file( $data->gp_project, $gp_locale, $set, $po_file );
+                        $last_modified = $this->build_po_file( $data->gp_project, $gp_locale, $set, $entries, $po_file );
 
                         if ( is_wp_error( $last_modified ) ) {
                                 WP_CLI::warning( sprintf( "PO generation for {$wp_locale} failed: %s", $last_modified->get_error_message() ) );
@@ -562 +611 @@
 
                         // Create ZIP file.
                         $result = $this->execute_command( sprintf(
-                                'zip -9 -j %s %s %s 2>&1',
+                                'zip -9 -j %s %s %s %s 2>&1',
                                 escapeshellarg( $zip_file ),
                                 escapeshellarg( $po_file ),
-                                escapeshellarg( $mo_file )
+                                escapeshellarg( $mo_file ),
+                                implode( ' ', array_map( 'escapeshellarg', $json_files ) )
                         ) );
 
                         if ( is_wp_error( $result ) ) {