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;

src/Charcoal/Translator/Middleware/LanguageMiddleware.php

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;
 
 /**

Spacing +1 added lines, -1 removed lines

@@ -115,7 +115,7 @@
             'browser_language' => null,
 
             'use_path'         => true,
-            'excluded_path'    => [ '^/admin\b' ],
+            'excluded_path'    => ['^/admin\b'],
             'path_regexp'      => '^/([a-z]{2})\b',
 
             'use_params'       => false,

Doc Comments +2 added lines, -2 removed lines

@@ -171,7 +171,7 @@ 
 
     /**
      * @param  RequestInterface $request The PSR-7 HTTP request.
-     * @return null|string
+     * @return string
      */
     private function getLanguage(RequestInterface $request)
     {
@@ -282,7 +282,7 @@ 
     }
 
     /**
-     * @return mixed
+     * @return string
      */
     private function getLanguageFromBrowser()
     {

src/Charcoal/Translator/Script/TranslationParserScript.php

Doc Comments +2 added lines, -2 removed lines

@@ -220,7 +220,7 @@ 
 
     /**
      * Available locales (languages)
-     * @return array Locales.
+     * @return string[] Locales.
      */
     protected function locales()
     {
@@ -445,7 +445,7 @@ 
 
     /**
      * @param array $translations The translations to save in CSV.
-     * @return TranslateScript Chainable
+     * @return TranslationParserScript Chainable
      */
     public function toCSV(array $translations)
     {

Spacing +3 added lines, -3 removed lines

@@ -364,7 +364,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) {
@@ -394,7 +394,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) {
@@ -478,7 +478,7 @@ 
             }
 
             foreach ($trans as $key => $translation) {
-                $data = [ $key, $translation ];
+                $data = [$key, $translation];
                 fputcsv($file, $data, $separator, $enclosure);
             }
             fclose($file);

src/Charcoal/Translator/ServiceProvider/TranslatorServiceProvider.php

Spacing +8 added lines, -8 removed lines

@@ -105,11 +105,11 @@ 
              * as the default language.
              */
             $localesConfig    = $container['locales/config'];
-            $availableLocales = array_filter($localesConfig['languages'], function ($locale) {
+            $availableLocales = array_filter($localesConfig['languages'], function($locale) {
                 return !(isset($locale['active']) && !$locale['active']);
             });
 
-            $acceptedLanguages  = explode(',', $_SERVER['HTTP_ACCEPT_LANGUAGE']);
+            $acceptedLanguages = explode(',', $_SERVER['HTTP_ACCEPT_LANGUAGE']);
             foreach ($acceptedLanguages as $acceptedLang) {
                 $lang = explode(';', $acceptedLang);
                 $lang = trim($lang[0]);
@@ -162,7 +162,7 @@ 
          * @param  Container $container Pimple DI container.
          * @return LocalesManager
          */
-        $container['locales/manager'] = function (Container $container) {
+        $container['locales/manager'] = function(Container $container) {
             $localesConfig = $container['locales/config'];
             return new LocalesManager([
                 'locales'          => $localesConfig['languages'],
@@ -183,7 +183,7 @@ 
          * @param  Container $container Pimple DI container.
          * @return TranslatorConfig
          */
-        $container['translator/config'] = function (Container $container) {
+        $container['translator/config'] = function(Container $container) {
             $appConfig        = isset($container['config']) ? $container['config'] : [];
             $translatorConfig = isset($appConfig['translator']) ? $appConfig['translator'] : null;
             return new TranslatorConfig($translatorConfig);
@@ -195,7 +195,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'];
         };
@@ -205,7 +205,7 @@ 
          *
          * @return MessageSelector
          */
-        $container['translator/message-selector'] = function () {
+        $container['translator/message-selector'] = function() {
             return new MessageSelector();
         };
 
@@ -216,7 +216,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'],
@@ -353,7 +353,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(
                 [