Extract/update only for one bundle (#166)

Author: mvrhov

Command/BundleTrait.php

@@ -0,0 +1,38 @@
+<?php
+
+/*
+ * This file is part of the PHP Translation package.
+ *
+ * (c) PHP Translation team <tobias.nyholm@gmail.com>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace Translation\Bundle\Command;
+
+use Symfony\Component\Console\Input\InputInterface;
+use Translation\Bundle\Model\Configuration;
+
+trait BundleTrait
+{
+    private function configureBundleDirs(InputInterface $input, Configuration $config)
+    {
+        if ($bundleName = $input->getOption('bundle')) {
+            if ('@' === $bundleName[0]) {
+                if (false === $pos = strpos($bundleName, '/')) {
+                    $bundleName = substr($bundleName, 1);
+                } else {
+                    $bundleName = substr($bundleName, 1, $pos - 2);
+                }
+            }
+
+            $bundle = $this->getApplication()
+                ->getKernel()
+                ->getBundle($bundleName)
+            ;
+
+            $config->reconfigureBundleDirs($bundle->getPath(), $bundle->getPath().'/Resources/translations');
+        }
+    }
+}

Command/DeleteObsoleteCommand.php

@@ -15,6 +15,7 @@
 use Symfony\Component\Console\Helper\ProgressBar;
 use Symfony\Component\Console\Input\InputArgument;
 use Symfony\Component\Console\Input\InputInterface;
+use Symfony\Component\Console\Input\InputOption;
 use Symfony\Component\Console\Output\OutputInterface;
 use Symfony\Component\Console\Question\ConfirmationQuestion;
 
@@ -23,13 +24,17 @@
  */
 class DeleteObsoleteCommand extends ContainerAwareCommand
 {
+    use BundleTrait;
+
     protected function configure()
     {
         $this
             ->setName('translation:delete-obsolete')
             ->setDescription('Delete all translations marked as obsolete.')
             ->addArgument('configuration', InputArgument::OPTIONAL, 'The configuration to use', 'default')
-            ->addArgument('locale', InputArgument::OPTIONAL, 'The locale ot use. If omitted, we use all configured locales.', null);
+            ->addArgument('locale', InputArgument::OPTIONAL, 'The locale ot use. If omitted, we use all configured locales.', null)
+            ->addOption('bundle', 'b', InputOption::VALUE_REQUIRED, 'The bundle you want remove translations from.')
+        ;
     }
 
     protected function execute(InputInterface $input, OutputInterface $output)
@@ -43,6 +48,7 @@ protected function execute(InputInterface $input, OutputInterface $output)
 
         $catalogueManager = $container->get('php_translation.catalogue_manager');
         $config = $container->get('php_translation.configuration_manager')->getConfiguration($configName);
+        $this->configureBundleDirs($input, $config);
         $catalogueManager->load($container->get('php_translation.catalogue_fetcher')->getCatalogues($config, $locales));
 
         $storage = $container->get('php_translation.storage.'.$configName);

Command/DownloadCommand.php

@@ -25,13 +25,16 @@
  */
 class DownloadCommand extends ContainerAwareCommand
 {
+    use BundleTrait;
+
     protected function configure()
     {
         $this
             ->setName('translation:download')
             ->setDescription('Replace local messages with messages from remote')
             ->addArgument('configuration', InputArgument::OPTIONAL, 'The configuration to use', 'default')
             ->addOption('cache', null, InputOption::VALUE_NONE, 'Clear the cache if the translations have changed')
+            ->addOption('bundle', 'b', InputOption::VALUE_REQUIRED, 'The bundle you want update translations from.')
         ;
     }
 
@@ -44,6 +47,7 @@ protected function execute(InputInterface $input, OutputInterface $output)
         $storage = $container->get('php_translation.storage.'.$configName);
         /** @var Configuration $configuration */
         $configuration = $this->getContainer()->get('php_translation.configuration.'.$configName);
+        $this->configureBundleDirs($input, $configuration);
 
         if ($input->getOption('cache')) {
             $translationsDirectory = $configuration->getOutputDir();

Command/ExtractCommand.php

@@ -26,14 +26,18 @@
  */
 class ExtractCommand extends ContainerAwareCommand
 {
+    use BundleTrait;
+
     protected function configure()
     {
         $this
             ->setName('translation:extract')
             ->setDescription('Extract translations from source code.')
             ->addArgument('configuration', InputArgument::OPTIONAL, 'The configuration to use', 'default')
             ->addArgument('locale', InputArgument::OPTIONAL, 'The locale ot use. If omitted, we use all configured locales.', false)
-            ->addOption('hide-errors', null, InputOption::VALUE_NONE, 'If we should print error or not');
+            ->addOption('hide-errors', null, InputOption::VALUE_NONE, 'If we should print error or not')
+            ->addOption('bundle', 'b', InputOption::VALUE_REQUIRED, 'The bundle you want extract translations from.')
+        ;
     }
 
     protected function execute(InputInterface $input, OutputInterface $output)
@@ -51,6 +55,7 @@ protected function execute(InputInterface $input, OutputInterface $output)
         $catalogues = $container->get('php_translation.catalogue_fetcher')
             ->getCatalogues($config, $locales);
 
+        $this->configureBundleDirs($input, $config);
         $finder = $this->getConfiguredFinder($config);
 
         $result = $importer->extractToCatalogues($finder, $catalogues, [

Command/StatusCommand.php

@@ -23,14 +23,18 @@
  */
 class StatusCommand extends ContainerAwareCommand
 {
+    use BundleTrait;
+
     protected function configure()
     {
         $this
             ->setName('translation:status')
             ->setDescription('Show status about your translations.')
             ->addArgument('configuration', InputArgument::OPTIONAL, 'The configuration to use', 'default')
             ->addArgument('locale', InputArgument::OPTIONAL, 'The locale ot use. If omitted, we use all configured locales.', false)
-            ->addOption('json', null, InputOption::VALUE_NONE, 'If we should output in Json format');
+            ->addOption('json', null, InputOption::VALUE_NONE, 'If we should output in Json format')
+            ->addOption('bundle', 'b', InputOption::VALUE_REQUIRED, 'The translations for bundle you want to check.')
+        ;
     }
 
     protected function execute(InputInterface $input, OutputInterface $output)
@@ -39,6 +43,7 @@ protected function execute(InputInterface $input, OutputInterface $output)
         $counter = $container->get('php_translation.catalogue_counter');
         $config = $container->get('php_translation.configuration_manager')
             ->getConfiguration($input->getArgument('configuration'));
+        $this->configureBundleDirs($input, $config);
 
         $locales = [];
         if ($inputLocale = $input->getArgument('locale')) {

Model/Configuration.php

@@ -217,4 +217,17 @@ public function getXliffVersion()
     {
         return $this->xliffVersion;
     }
+
+    /**
+     * Reconfigures the directories so we can use one configuration for extracting
+     * the messages only from one bundle.
+     *
+     * @param string $bundleDir
+     * @param string $outputDir
+     */
+    public function reconfigureBundleDirs($bundleDir, $outputDir)
+    {
+        $this->dirs = is_array($bundleDir) ? $bundleDir : [$bundleDir];
+        $this->outputDir = $outputDir;
+    }
 }