Precompile module tags for translation dynamically

This ensures that any new tags added later will be included in the
translation set.

Author: George Stephanis

class.jetpack.php

@@ -1328,17 +1328,9 @@ public static function get_file_data( $file, $headers ) {
 	}
 
 	public static function translate_module_tag( $untranslated_tag ) {
+		// Tags are aggregated by tools/build-module-headings-translations.php
+		// and output in modules/module-headings.php
 		return _x( $untranslated_tag, 'Module Tag', 'jetpack' );
-
-		// Calls here are to populate translation files.
-		_x( 'Photos and Videos',   'Module Tag', 'jetpack' );
-		_x( 'Social',              'Module Tag', 'jetpack' );
-		_x( 'WordPress.com Stats', 'Module Tag', 'jetpack' );
-		_x( 'Writing',             'Module Tag', 'jetpack' );
-		_x( 'Appearance',          'Module Tag', 'jetpack' );
-		_x( 'Developers',          'Module Tag', 'jetpack' );
-		_x( 'Mobile',              'Module Tag', 'jetpack' );
-		_x( 'Other',               'Module Tag', 'jetpack' );
 	}
 
 	/**

modules/module-headings.php

@@ -143,3 +143,65 @@
 // modules/widgets.php
 _x( 'Extra Sidebar Widgets', 'Module Name', 'jetpack' );
 _x( 'Add images, Twitter streams, your site’s RSS links, and more to your sidebar.', 'Module Description', 'jetpack' );
+
+// Modules with `Other` tag:
+//  - modules/contact-form.php
+//  - modules/notes.php
+//  - modules/site-icon.php
+_x( 'Other', 'Module Tag', 'jetpack' );
+
+// Modules with `Writing` tag:
+//  - modules/after-the-deadline.php
+//  - modules/custom-content-types.php
+//  - modules/enhanced-distribution.php
+//  - modules/json-api.php
+//  - modules/latex.php
+//  - modules/markdown.php
+//  - modules/post-by-email.php
+//  - modules/shortcodes.php
+_x( 'Writing', 'Module Tag', 'jetpack' );
+
+// Modules with `Photos and Videos` tag:
+//  - modules/carousel.php
+//  - modules/photon.php
+//  - modules/shortcodes.php
+//  - modules/tiled-gallery.php
+//  - modules/videopress.php
+_x( 'Photos and Videos', 'Module Tag', 'jetpack' );
+
+// Modules with `Social` tag:
+//  - modules/comments.php
+//  - modules/gravatar-hovercards.php
+//  - modules/likes.php
+//  - modules/publicize.php
+//  - modules/sharedaddy.php
+//  - modules/shortcodes.php
+//  - modules/shortlinks.php
+//  - modules/subscriptions.php
+//  - modules/widgets.php
+_x( 'Social', 'Module Tag', 'jetpack' );
+
+// Modules with `Appearance` tag:
+//  - modules/custom-css.php
+//  - modules/gravatar-hovercards.php
+//  - modules/infinite-scroll.php
+//  - modules/minileven.php
+//  - modules/photon.php
+//  - modules/shortcodes.php
+//  - modules/widget-visibility.php
+//  - modules/widgets.php
+_x( 'Appearance', 'Module Tag', 'jetpack' );
+
+// Modules with `Developers` tag:
+//  - modules/json-api.php
+//  - modules/omnisearch.php
+//  - modules/sso.php
+_x( 'Developers', 'Module Tag', 'jetpack' );
+
+// Modules with `Mobile` tag:
+//  - modules/minileven.php
+_x( 'Mobile', 'Module Tag', 'jetpack' );
+
+// Modules with `WordPress.com Stats` tag:
+//  - modules/stats.php
+_x( 'WordPress.com Stats', 'Module Tag', 'jetpack' );

tools/build-module-headings-translations.php

@@ -12,7 +12,10 @@
 ";
 
 $jp_dir = dirname( dirname( __FILE__ ) ) . '/';
-$files = glob( "{$jp_dir}modules/*.php" );
+$files  = glob( "{$jp_dir}modules/*.php" );
+$tags   = array(
+	'Other' => array(),
+);
 foreach ( $files as $file ) {
 	$absolute_path  = $file;
 	$relative_path  = str_replace( $jp_dir, '', $file );
@@ -28,12 +31,20 @@
 	$all_headers = array(
 		'name'        => 'Module Name',
 		'description' => 'Module Description',
+		'tags'        => 'Module Tags',
 	);
 
 	foreach ( $all_headers as $field => $regex ) {
 		if ( preg_match( '/^[ \t\/*#@]*' . preg_quote( $regex, '/' ) . ':(.*)$/mi', $file_data, $match ) && $match[1] ) {
 			$string = trim( preg_replace( "/\s*(?:\*\/|\?>).*/", '', $match[1] ) );
-			$_file_contents .= "_x( '{$string}', '{$regex}', 'jetpack' );\r\n";
+			if ( 'Module Tags' === $regex ) {
+				$module_tags = array_map( 'trim', explode( ',', $string ) );
+				foreach ( $module_tags as $tag ) {
+					$tags[ $tag ][] = $relative_path;
+				}
+			} else {
+				$_file_contents .= "_x( '{$string}', '{$regex}', 'jetpack' );\r\n";
+			}
 		}
 	}
 
@@ -44,4 +55,12 @@
 
 }
 
+foreach ( $tags as $tag => $files ) {
+	$file_contents .= "\r\n// Modules with `{$tag}` tag:\r\n";
+	foreach ( $files as $file ) {
+		$file_contents .= "//  - {$file}\r\n";
+	}
+	$file_contents .= "_x( '{$tag}', 'Module Tag', 'jetpack' );\r\n";
+}
+
 file_put_contents( "{$jp_dir}modules/module-headings.php", $file_contents );