Merge pull request #363 from wp-cli/add/try-make-php

README.md

@@ -227,6 +227,32 @@ wp i18n make-mo <source> [<destination>]
 
 
 
+### wp i18n make-php
+
+Create PHP files from PO files.
+
+~~~
+wp i18n make-php <source> [<destination>]
+~~~
+
+**OPTIONS**
+
+	<source>
+		Path to an existing PO file or a directory containing multiple PO files.
+
+	[<destination>]
+		Path to the destination directory for the resulting PHP files. Defaults to the source directory.
+
+**EXAMPLES**
+
+    # Create PHP files for all PO files in the current directory.
+    $ wp i18n make-php .
+
+    # Create a PHP file from a single PO file in a specific directory.
+    $ wp i18n make-php example-plugin-de_DE.po languages
+
+
+
 ### wp i18n update-po
 
 Update PO files from a POT file.

composer.json

@@ -42,6 +42,7 @@
             "i18n make-pot",
             "i18n make-json",
             "i18n make-mo",
+            "i18n make-php",
             "i18n update-po"
         ]
     },

features/makejson.feature

@@ -7,7 +7,7 @@ Feature: Split PO files into JSON files.
     When I try `wp i18n make-json foo`
     Then STDERR should contain:
       """
-      Error: Source file or directory does not exist!
+      Error: Source file or directory does not exist.
       """
     And the return code should be 1
 

features/makemo.feature

@@ -7,7 +7,7 @@ Feature: Generate MO files from PO files
     When I try `wp i18n make-mo foo`
     Then STDERR should contain:
       """
-      Error: Source file or directory does not exist!
+      Error: Source file or directory does not exist.
       """
     And the return code should be 1
   Scenario: Bail for destination being a file when source is a folder
@@ -18,7 +18,7 @@ Feature: Generate MO files from PO files
     When I try `wp i18n make-mo foo test.mo `
     Then STDERR should contain:
       """
-      Error: Destination file not supported when source is a directory!
+      Error: Destination file not supported when source is a directory.
       """
     And the return code should be 1
   Scenario: Uses source folder as destination by default

features/makephp.feature

@@ -0,0 +1,167 @@
+Feature: Generate PHP files from PO files
+
+  Background:
+    Given an empty directory
+
+  Scenario: Bail for invalid source directories
+    When I try `wp i18n make-php foo`
+    Then STDERR should contain:
+      """
+      Error: Source file or directory does not exist.
+      """
+    And the return code should be 1
+
+  Scenario: Uses source folder as destination by default
+    Given an empty foo-plugin directory
+    And a foo-plugin/foo-plugin-de_DE.po file:
+      """
+      # Copyright (C) 2018 Foo Plugin
+      # This file is distributed under the same license as the Foo Plugin package.
+      msgid ""
+      msgstr ""
+      "Project-Id-Version: Foo Plugin\n"
+      "Report-Msgid-Bugs-To: https://wordpress.org/support/plugin/foo-plugin\n"
+      "Last-Translator: FULL NAME <EMAIL@ADDRESS>\n"
+      "Language-Team: LANGUAGE <[email protected]>\n"
+      "Language: de_DE\n"
+      "MIME-Version: 1.0\n"
+      "Content-Type: text/plain; charset=UTF-8\n"
+      "Content-Transfer-Encoding: 8bit\n"
+      "POT-Creation-Date: 2018-05-02T22:06:24+00:00\n"
+      "PO-Revision-Date: 2018-05-02T22:06:24+00:00\n"
+      "X-Domain: foo-plugin\n"
+      "Plural-Forms: nplurals=2; plural=(n != 1);\n"
+
+      #: foo-plugin.js:15
+      msgid "Foo Plugin"
+      msgstr "Foo Plugin"
+      """
+
+    When I run `wp i18n make-php foo-plugin`
+    Then STDOUT should contain:
+      """
+      Success: Created 1 file.
+      """
+    And the return code should be 0
+    And the foo-plugin/foo-plugin-de_DE.l10n.php file should exist
+
+  Scenario: Allows setting custom destination directory
+    Given an empty foo-plugin directory
+    And a foo-plugin/foo-plugin-de_DE.po file:
+      """
+      # Copyright (C) 2018 Foo Plugin
+      # This file is distributed under the same license as the Foo Plugin package.
+      msgid ""
+      msgstr ""
+      "Project-Id-Version: Foo Plugin\n"
+      "Report-Msgid-Bugs-To: https://wordpress.org/support/plugin/foo-plugin\n"
+      "Last-Translator: FULL NAME <EMAIL@ADDRESS>\n"
+      "Language-Team: LANGUAGE <[email protected]>\n"
+      "Language: de_DE\n"
+      "MIME-Version: 1.0\n"
+      "Content-Type: text/plain; charset=UTF-8\n"
+      "Content-Transfer-Encoding: 8bit\n"
+      "POT-Creation-Date: 2018-05-02T22:06:24+00:00\n"
+      "PO-Revision-Date: 2018-05-02T22:06:24+00:00\n"
+      "X-Domain: foo-plugin\n"
+      "Plural-Forms: nplurals=2; plural=(n != 1);\n"
+
+      #: foo-plugin.js:15
+      msgid "Foo Plugin"
+      msgstr "Foo Plugin"
+      """
+
+    When I run `wp i18n make-php foo-plugin result`
+    Then STDOUT should contain:
+      """
+      Success: Created 1 file.
+      """
+    And the return code should be 0
+    And the result/foo-plugin-de_DE.l10n.php file should exist
+
+  Scenario: Does include headers
+    Given an empty foo-plugin directory
+    And a foo-plugin/foo-plugin-de_DE.po file:
+      """
+      # Copyright (C) 2018 Foo Plugin
+      # This file is distributed under the same license as the Foo Plugin package.
+      msgid ""
+      msgstr ""
+      "Project-Id-Version: Foo Plugin\n"
+      "Report-Msgid-Bugs-To: https://wordpress.org/support/plugin/foo-plugin\n"
+      "Last-Translator: FULL NAME <EMAIL@ADDRESS>\n"
+      "Language-Team: LANGUAGE <[email protected]>\n"
+      "Language: de_DE\n"
+      "MIME-Version: 1.0\n"
+      "Content-Type: text/plain; charset=UTF-8\n"
+      "Content-Transfer-Encoding: 8bit\n"
+      "POT-Creation-Date: 2018-05-02T22:06:24+00:00\n"
+      "PO-Revision-Date: 2018-05-02T22:06:24+00:00\n"
+      "X-Domain: foo-plugin\n"
+      "Plural-Forms: nplurals=2; plural=(n != 1);\n"
+
+      #: foo-plugin.js:15
+      msgid "Foo Plugin"
+      msgstr "Foo Plugin"
+      """
+
+    When I run `wp i18n make-php foo-plugin`
+    Then STDOUT should contain:
+      """
+      Success: Created 1 file.
+      """
+    And the return code should be 0
+    And STDERR should be empty
+    And the foo-plugin/foo-plugin-de_DE.l10n.php file should contain:
+      """
+      'language'=>'de_DE'
+      """
+    And the foo-plugin/foo-plugin-de_DE.l10n.php file should contain:
+      """
+      'domain'=>'foo-plugin'
+      """
+    And the foo-plugin/foo-plugin-de_DE.l10n.php file should contain:
+      """
+      'plural-forms'=>'nplurals=2; plural=(n != 1);'
+      """
+    And the foo-plugin/foo-plugin-de_DE.l10n.php file should contain:
+      """
+      'messages'=>[''=>['Foo Plugin'=>['Foo Plugin']]]
+      """
+
+  Scenario: Does include translations
+    Given an empty foo-plugin directory
+    And a foo-plugin/foo-plugin-de_DE.po file:
+      """
+      # Copyright (C) 2018 Foo Plugin
+      # This file is distributed under the same license as the Foo Plugin package.
+      msgid ""
+      msgstr ""
+      "Project-Id-Version: Foo Plugin\n"
+      "Report-Msgid-Bugs-To: https://wordpress.org/support/plugin/foo-plugin\n"
+      "Last-Translator: FULL NAME <EMAIL@ADDRESS>\n"
+      "Language-Team: LANGUAGE <[email protected]>\n"
+      "Language: de_DE\n"
+      "MIME-Version: 1.0\n"
+      "Content-Type: text/plain; charset=UTF-8\n"
+      "Content-Transfer-Encoding: 8bit\n"
+      "POT-Creation-Date: 2018-05-02T22:06:24+00:00\n"
+      "PO-Revision-Date: 2018-05-02T22:06:24+00:00\n"
+      "X-Domain: foo-plugin\n"
+      "Plural-Forms: nplurals=2; plural=(n != 1);\n"
+
+      #: foo-plugin.js:15
+      msgid "Foo Plugin"
+      msgstr "Bar Plugin"
+      """
+
+    When I run `wp i18n make-php foo-plugin`
+    Then STDOUT should contain:
+      """
+      Success: Created 1 file.
+      """
+    And the return code should be 0
+    And the foo-plugin/foo-plugin-de_DE.l10n.php file should contain:
+      """
+      'messages'=>[''=>['Foo Plugin'=>['Bar Plugin']]]
+      """