locomotivemtl / charcoal-translator

src/Charcoal/Translator/Translator.php

Spacing +2 added lines, -2 removed lines

@@ -29,7 +29,7 @@ 
      *
      * @var string[]
      */
-    private $domains = [ 'messages' ];
+    private $domains = ['messages'];
 
     /**
      * @param array $data Constructor data.
@@ -266,7 +266,7 @@ 
         if (is_array($val)) {
             return !!array_filter(
                 $val,
-                function ($v, $k) {
+                function($v, $k) {
                     if (is_string($k) && strlen($k) > 0) {
                         if (is_string($v) && strlen($v) > 0) {
                             return true;

Doc Comments +1 added lines, -1 removed lines

@@ -130,7 +130,7 @@
      *
      * @uses   SymfonyTranslator::trans()
      * @uses   Translator::translation()
-     * @param  mixed       $val        The string or translation-object to retrieve.
+     * @param  string       $val        The string or translation-object to retrieve.
      * @param  array       $parameters An array of parameters for the message.
      * @param  string|null $domain     The domain for the message or NULL to use the default.
      * @param  string|null $locale     The locale or NULL to use the default.

Unused Use Statements -2 removed lines

@@ -2,8 +2,6 @@
 
 namespace Charcoal\Translator;
 
-use RuntimeException;
-
 // From 'symfony/translation'
 use Symfony\Component\Translation\MessageSelector;
 use Symfony\Component\Translation\Translator as SymfonyTranslator;

src/Charcoal/Translator/Script/TranslationParserScript.php

Doc Comments +1 added lines, -1 removed lines

@@ -213,7 +213,7 @@
 
     /**
      * Available locales (languages)
-     * @return array Locales.
+     * @return string[] Locales.
      */
     protected function locales()
     {

Spacing +3 added lines, -3 removed lines

@@ -359,7 +359,7 @@ 
                 $t = count($array[$index]);
                 $locales = $this->locales();
 
-                for (; $i<$t; $i++) {
+                for (; $i < $t; $i++) {
                     $this->climate()->inline('.');
                     $orig = $array[$index][$i];
                     foreach ($locales as $lang) {
@@ -388,7 +388,7 @@ 
         $max = $this->maxRecursiveLevel();
         $i = 1;
         $files = glob($pattern, $flags);
-        foreach (glob(dirname($pattern).'/*', (GLOB_ONLYDIR|GLOB_NOSORT)) as $dir) {
+        foreach (glob(dirname($pattern).'/*', (GLOB_ONLYDIR | GLOB_NOSORT)) as $dir) {
             $files = array_merge($files, $this->globRecursive($dir.'/'.basename($pattern), $flags));
             $i++;
             // if ($i >= $max) {
@@ -473,7 +473,7 @@ 
             }
 
             foreach ($trans as $key => $translation) {
-                $data = [ $key, $translation ];
+                $data = [$key, $translation];
                 fputcsv($file, $data, $separator, $enclosure);
             }
             fclose($file);

src/Charcoal/Translator/Middleware/LanguageMiddleware.php

Doc Comments +1 added lines, -1 removed lines

@@ -231,7 +231,7 @@
     }
 
     /**
-     * @return mixed
+     * @return string
      */
     private function getLanguageFromBrowser()
     {

Unused Use Statements -2 removed lines

@@ -7,10 +7,8 @@
 use Psr\Http\Message\ResponseInterface;
 
 // From Pimple
-use Pimple\Container;
 
 // From `charcoal-translator`
-use Charcoal\Translator\LocalesManager;
 use Charcoal\Translator\TranslatorAwareTrait;
 
 /**

src/Charcoal/Translator/ServiceProvider/TranslatorServiceProvider.php

Spacing +6 added lines, -6 removed lines

@@ -131,7 +131,7 @@ 
          * @param  Container $container Pimple DI container.
          * @return LocalesManager
          */
-        $container['locales/manager'] = function (Container $container) {
+        $container['locales/manager'] = function(Container $container) {
             return new LocalesManager([
                 'locales'             => $container['locales/languages'],
                 'default_language'    => $container['locales/default-language'],
@@ -150,7 +150,7 @@ 
          * @param  Container $container Pimple DI container.
          * @return TranslatorConfig
          */
-        $container['translator/config'] = function (Container $container) {
+        $container['translator/config'] = function(Container $container) {
             $config = isset($container['config']) ? $container['config'] : [];
             $translatorConfig = isset($config['translator']) ? $config['translator'] : null;
             return new TranslatorConfig($translatorConfig);
@@ -160,7 +160,7 @@ 
          * @param  Container $container Pimple DI container.
          * @return array
          */
-        $container['translator/translations'] = function (Container $container) {
+        $container['translator/translations'] = function(Container $container) {
             $translatorConfig = $container['translator/config'];
             return $translatorConfig['translations'];
         };
@@ -168,7 +168,7 @@ 
         /**
          * @return MessageSelector
          */
-        $container['translator/message-selector'] = function () {
+        $container['translator/message-selector'] = function() {
             return new MessageSelector();
         };
 
@@ -177,7 +177,7 @@ 
          * @param  Container $container Pimple DI container.
          * @return Translator
          */
-        $container['translator'] = function (Container $container) {
+        $container['translator'] = function(Container $container) {
             $translatorConfig = $container['translator/config'];
             $translator = new Translator([
                 'manager'           => $container['locales/manager'],
@@ -314,7 +314,7 @@ 
          * @param Container $container
          * @return LanguageMiddleware
          */
-        $container['middlewares/charcoal/translator/middleware/language'] = function (Container $container) {
+        $container['middlewares/charcoal/translator/middleware/language'] = function(Container $container) {
             $middlewareConfig = $container['config']['middlewares']['charcoal/translator/middleware/language'];
             $middlewareConfig = array_replace(
                 [