antonioribeiro / countries

src/package/Support/Collection.php

Spacing +3 added lines, -3 removed lines

@@ -99,7 +99,7 @@ 
      */
     private function arrayFinder(string $propertyName, $find, Closure $finderClosure)
     {
-        return $this->filter(function ($data) use ($find, $propertyName, $finderClosure) {
+        return $this->filter(function($data) use ($find, $propertyName, $finderClosure) {
             try {
                 $attributeValue = $data->{$propertyName};
             } catch (\Exception $e) {
@@ -121,7 +121,7 @@ 
      */
     private function _whereKey(string $arrayName, $value)
     {
-        $finderClosure = function ($value, $attributeValue) {
+        $finderClosure = function($value, $attributeValue) {
             return Arr::has($attributeValue, $value);
         };
 
@@ -137,7 +137,7 @@ 
      */
     private function _whereAttribute(string $arrayName, $value)
     {
-        $finderClosure = function ($value, $attributeValue) {
+        $finderClosure = function($value, $attributeValue) {
             return in_array($value, $attributeValue->toArray());
         };
 

src/package/Support/helpers.php

Spacing +3 added lines, -3 removed lines

@@ -2,7 +2,7 @@ 
 
 use PragmaRX\Countries\Package\Support\Collection;
 
-if (! function_exists('array_sort_by_keys_recursive')) {
+if (!function_exists('array_sort_by_keys_recursive')) {
     /**
      * Recursively sort array by keys.
      *
@@ -21,7 +21,7 @@ 
     }
 }
 
-if (! function_exists('countriesCollect')) {
+if (!function_exists('countriesCollect')) {
     /**
      * Check if array is multidimensional.
      *
@@ -34,7 +34,7 @@ 
     }
 }
 
-if (! function_exists('arrayable')) {
+if (!function_exists('arrayable')) {
     /**
      * Recursively change all array keys case.
      *

src/package/Services/Config.php

Doc Comments +2 added lines, -2 removed lines

@@ -16,7 +16,7 @@ 
     /**
      * Config constructor.
      *
-     * @param array|null $config
+     * @param Helper $config
      */
     public function __construct($config = null)
     {
@@ -24,7 +24,7 @@ 
     }
 
     /**
-     * @param $key
+     * @param string $key
      * @return \PragmaRX\Coollection\Package\Coollection
      */
     public function get($key)

src/package/Services/Countries.php

Doc Comments +1 added lines, -1 removed lines

@@ -235,7 +235,7 @@
     }
 
     /**
-     * @param $repository
+     * @param null|Repository $repository
      * @return Repository
      */
     protected function instantiateRepository($repository)

Unused Use Statements +2 added lines, -2 removed lines

@@ -2,10 +2,10 @@
 
 namespace PragmaRX\Countries\Package\Services;
 
-use PragmaRX\Countries\Update\Updater;
-use PragmaRX\Countries\Package\Support\Base;
 use PragmaRX\Coollection\Package\Coollection;
 use PragmaRX\Countries\Package\Data\Repository;
+use PragmaRX\Countries\Package\Support\Base;
+use PragmaRX\Countries\Update\Updater;
 
 class Countries extends Base
 {

Spacing +8 added lines, -8 removed lines

@@ -102,14 +102,14 @@ 
     {
         $instance = $this;
 
-        Coollection::macro('hydrate', function ($elements = null) use ($instance) {
+        Coollection::macro('hydrate', function($elements = null) use ($instance) {
             return $instance->hydrate($this, $elements);
         });
 
         foreach (Hydrator::HYDRATORS as $hydrator) {
             $hydrator = 'hydrate'.studly_case($hydrator);
 
-            Coollection::macro($hydrator, function () use ($hydrator, $instance) {
+            Coollection::macro($hydrator, function() use ($hydrator, $instance) {
                 return $instance->getRepository()->getHydrator()->{$hydrator}($this);
             });
         }
@@ -175,8 +175,8 @@ 
      */
     protected function instantiateCache(Cache $cache = null)
     {
-        if (is_null($this->cache) || ! is_null($cache)) {
-            $this->cache = ! is_null($cache)
+        if (is_null($this->cache) || !is_null($cache)) {
+            $this->cache = !is_null($cache)
                 ? $cache
                 : new Cache($this->config);
         }
@@ -192,8 +192,8 @@ 
      */
     protected function instantiateConfig($config = null)
     {
-        if (is_null($this->config) || ! is_null($config)) {
-            $this->config = ! is_null($config)
+        if (is_null($this->config) || !is_null($config)) {
+            $this->config = !is_null($config)
                 ? $config
                 : new Config($this->helper);
         }
@@ -224,8 +224,8 @@ 
      */
     protected function instantiateHydrator(Hydrator $hydrator = null)
     {
-        if (is_null($this->hydrator) || ! is_null($hydrator)) {
-            $this->hydrator = ! is_null($hydrator)
+        if (is_null($this->hydrator) || !is_null($hydrator)) {
+            $this->hydrator = !is_null($hydrator)
                 ? $hydrator
                 : new Hydrator($this->config);
         }

src/update/Rinvex.php

Spacing +7 added lines, -7 removed lines

@@ -61,7 +61,7 @@ 
             'latlng'       => 'geo',
         ];
 
-        coollect($mergeable)->each(function ($to, $key) use (&$natural) {
+        coollect($mergeable)->each(function($to, $key) use (&$natural) {
             if (isset($natural[$key])) {
                 $natural->overwrite([$to => [$key => $natural[$key]]]);
 
@@ -113,7 +113,7 @@ 
      */
     public function findRinvexState($country, $needle)
     {
-        $states = $this->findRinvex($country, 'divisions')->map(function ($state, $postal) {
+        $states = $this->findRinvex($country, 'divisions')->map(function($state, $postal) {
             $state['postal'] = $postal;
 
             $state['name'] = $this->helper->fixUtf8($state['name']);
@@ -125,12 +125,12 @@ 
             return $states;
         }
 
-        $state = $states->filter(function ($rinvexState) use ($needle) {
+        $state = $states->filter(function($rinvexState) use ($needle) {
             return $rinvexState->postal == $needle->postal ||
                 $rinvexState->name == $needle['name'] ||
                 utf8_encode($rinvexState->name) == $needle['name'] ||
                 $rinvexState->alt_names->contains($needle['name']) ||
-                $rinvexState->alt_names->contains(function ($name) use ($needle) {
+                $rinvexState->alt_names->contains(function($name) use ($needle) {
                     return $needle->alt_names->contains($name);
                 });
         })->first();
@@ -204,7 +204,7 @@ 
                 continue;
             }
 
-            if ($rinvexValue !== $naturalValue && ! $defaultToRinvex->contains($key)) {
+            if ($rinvexValue !== $naturalValue && !$defaultToRinvex->contains($key)) {
                 $result[$key.$suffix] = $rinvexValue; // Natural Earth Vector
             }
 
@@ -213,7 +213,7 @@ 
                 : $naturalValue; // Natural Earth Vector
         }
 
-        return coollect($result)->sortBy(function ($value, $key) {
+        return coollect($result)->sortBy(function($value, $key) {
             return $key;
         });
     }
@@ -226,7 +226,7 @@ 
      */
     public function mergeCountryStatesWithRinvex($states)
     {
-        return coollect($states)->map(function ($state) {
+        return coollect($states)->map(function($state) {
             return $this->mergeStateWithRinvex($state);
         });
     }

Unused Use Statements +1 added lines, -1 removed lines

@@ -3,8 +3,8 @@
 namespace PragmaRX\Countries\Update;
 
 use Exception;
-use PragmaRX\Countries\Package\Support\Base;
 use PragmaRX\Coollection\Package\Coollection;
+use PragmaRX\Countries\Package\Support\Base;
 
 class Rinvex extends Base
 {

Doc Comments +1 added lines, -1 removed lines

@@ -157,7 +157,7 @@
      *
      * @param Coollection $natural
      * @param Coollection $rinvex
-     * @param $translation
+     * @param null|Coollection $translation
      * @param string $suffix
      * @return mixed|\PragmaRX\Coollection\Package\Coollection
      */

src/update/Timezones.php

Indentation +8 added lines, -8 removed lines

@@ -101,9 +101,9 @@ 
         })
         ->mapWithKeys(function ($item, $cca2) {
             $fields = [
-               ['cca2', 'cca2'],
-               ['name.common', 'name'],
-               ['name.official', 'name'],
+                ['cca2', 'cca2'],
+                ['name.common', 'name'],
+                ['name.official', 'name'],
             ];
 
             list($country) = $this->updater->findByFields($this->updater->getCountries(), $item, $fields, 'cca2');
@@ -113,11 +113,11 @@ 
             }
 
             return [
-               $country->cca3 => [
-                   'cca2' => $country->cca2,
-                   'cca3' => $country->cca3,
-                   'name' => $item['name'],
-               ],
+                $country->cca3 => [
+                    'cca2' => $country->cca2,
+                    'cca3' => $country->cca3,
+                    'name' => $item['name'],
+                ],
             ];
         })->map(function ($country) use ($zones, $abbreviations) {
             $country['timezones'] = $zones->where('country_code', $country['cca2'])->mapWithKeys(function ($zone) use ($abbreviations, $country) {

Unused Use Statements +1 added lines, -1 removed lines

@@ -2,10 +2,10 @@
 
 namespace PragmaRX\Countries\Update;
 
-use PragmaRX\Countries\Package\Support\Base;
 use PragmaRX\Coollection\Package\Coollection;
 use PragmaRX\Countries\Package\Services\Cache;
 use PragmaRX\Countries\Package\Services\Config;
+use PragmaRX\Countries\Package\Support\Base;
 
 class Timezones extends Base
 {

Spacing +15 added lines, -15 removed lines

@@ -50,7 +50,7 @@ 
         $countries = $this->cache->remember(
             'updateTimezone.countries',
             160,
-            function () {
+            function() {
                 return $this->helper->loadCsv($this->helper->dataDir('third-party/timezonedb/country.csv'));
             }
         );
@@ -60,8 +60,8 @@ 
         $zones = $this->cache->remember(
             'updateTimezone.zones',
             160,
-            function () {
-                return $this->helper->loadCsv($this->helper->dataDir('third-party/timezonedb/zone.csv'))->mapWithKeys(function ($value) {
+            function() {
+                return $this->helper->loadCsv($this->helper->dataDir('third-party/timezonedb/zone.csv'))->mapWithKeys(function($value) {
                     return [
                         $value[0] => [
                             'zone_id'      => $value[0],
@@ -78,8 +78,8 @@ 
         $timezones = $this->cache->remember(
             'updateTimezone.timezones',
             160,
-            function () {
-                return $this->helper->loadCsv($this->helper->dataDir('third-party/timezonedb/timezone.csv'))->map(function ($timezone) {
+            function() {
+                return $this->helper->loadCsv($this->helper->dataDir('third-party/timezonedb/timezone.csv'))->map(function($timezone) {
                     return [
                         'zone_id' => $timezone[0],
                         'abbreviation' => $timezone[1],
@@ -96,9 +96,9 @@ 
         $abbreviations = $this->cache->remember(
             'updateTimezone.abbreviations',
             160,
-            function () use ($timezones) {
-                return $timezones->groupBy('zone_id')->map(function (Coollection $timezones) {
-                    return $timezones->map(function ($timezone) {
+            function() use ($timezones) {
+                return $timezones->groupBy('zone_id')->map(function(Coollection $timezones) {
+                    return $timezones->map(function($timezone) {
                         return $timezone['abbreviation'];
                     })->unique()->sort()->values();
                 });
@@ -107,13 +107,13 @@ 
 
         $this->helper->progress('Updating countries timezones...');
 
-        $countries = $countries->mapWithKeys(function ($item) {
+        $countries = $countries->mapWithKeys(function($item) {
             return [$item[0] => [
                 'cca2' => $item[0],
                 'name' => $item[1],
             ]];
         })
-        ->mapWithKeys(function ($item, $cca2) {
+        ->mapWithKeys(function($item, $cca2) {
             $fields = [
                ['cca2', 'cca2'],
                ['name.common', 'name'],
@@ -133,8 +133,8 @@ 
                    'name' => $item['name'],
                ],
             ];
-        })->map(function ($country) use ($zones, $abbreviations) {
-            $country['timezones'] = $zones->where('country_code', $country['cca2'])->mapWithKeys(function ($zone) use ($abbreviations, $country) {
+        })->map(function($country) use ($zones, $abbreviations) {
+            $country['timezones'] = $zones->where('country_code', $country['cca2'])->mapWithKeys(function($zone) use ($abbreviations, $country) {
                 $zone['abbreviations'] = $abbreviations[$zone['zone_id']];
 
                 $zone['cca3'] = isset($country['cca3']) ? $country['cca3'] : null;
@@ -153,14 +153,14 @@ 
 
         $this->helper->message('Generating timezone files...');
 
-        $getCountryCodeClosure = function () {
+        $getCountryCodeClosure = function() {
         };
 
-        $normalizeCountryClosure = function ($country) {
+        $normalizeCountryClosure = function($country) {
             return [$country['timezones']];
         };
 
-        $dummyClosure = function ($country) {
+        $dummyClosure = function($country) {
             return $country;
         };
 

src/update/Cities.php

Spacing +3 added lines, -3 removed lines

@@ -43,7 +43,7 @@ 
 
         $this->helper->message('Processing cities...');
 
-        $normalizerClosure = function ($item) {
+        $normalizerClosure = function($item) {
             $item = $this->updater->addDataSource($item, 'natural');
 
             $item = $this->updater->addRecordType($item, 'city');
@@ -51,11 +51,11 @@ 
             return $this->updater->normalizeStateOrCityData($item);
         };
 
-        $codeGeneratorClosure = function ($item) {
+        $codeGeneratorClosure = function($item) {
             return $this->helper->caseForKey($item['nameascii']);
         };
 
-        $mergerClosure = function ($item) {
+        $mergerClosure = function($item) {
             return $item;
         };
 

src/update/Currencies.php

Spacing +4 added lines, -4 removed lines

@@ -47,13 +47,13 @@ 
             throw new Exception("No currencies found in {$directory}");
         }
 
-        $currencies = $currencies->mapWithKeys(function ($currency) {
+        $currencies = $currencies->mapWithKeys(function($currency) {
             return $currency;
         });
 
         $this->helper->message('Processing currencies...');
 
-        $normalizerClosure = function ($item) {
+        $normalizerClosure = function($item) {
             $item = $this->updater->addDataSource($item, 'world-currencies');
 
             $item = $this->updater->addRecordType($item, 'currency');
@@ -61,10 +61,10 @@ 
             return [$item];
         };
 
-        $getCodeClosure = function () {
+        $getCodeClosure = function() {
         };
 
-        $generateTaxData = function ($tax) {
+        $generateTaxData = function($tax) {
             return $tax;
         };
 

src/update/Taxes.php

Spacing +4 added lines, -4 removed lines

@@ -38,7 +38,7 @@ 
 
         $taxes = $this->helper->loadJsonFiles($this->helper->dataDir('third-party/commerceguys/taxes/types'));
 
-        $taxes = $taxes->mapWithKeys(function ($vat, $key) {
+        $taxes = $taxes->mapWithKeys(function($vat, $key) {
             $parts = coollect(explode('_', $key));
             $cca2 = $parts->first();
             $type = $parts->last();
@@ -69,15 +69,15 @@ 
 
         $this->helper->message('Processing taxes...');
 
-        $normalizerClosure = function ($item) {
+        $normalizerClosure = function($item) {
             return $item;
         };
 
-        $getCodeClosure = function ($item) {
+        $getCodeClosure = function($item) {
             return $item['tax_type'];
         };
 
-        $generateTaxData = function ($tax) {
+        $generateTaxData = function($tax) {
             return $this->normalizeTax($tax);
         };