antonioribeiro / countries

src/package/Data/Repository.php

Doc Comments +3 added lines, -3 removed lines

@@ -63,7 +63,7 @@ 
      * @param CacheContract $cache
      * @param Hydrator $hydrator
      * @param Helper $helper
-     * @param object $config
+     * @param \PragmaRX\Countries\Package\Services\Config $config
      */
     public function __construct(CacheContract $cache, Hydrator $hydrator, Helper $helper, $config)
     {
@@ -179,7 +179,7 @@ 
      * Load currency json file.
      *
      * @param $code
-     * @return string
+     * @return \PragmaRX\Coollection\Package\Coollection
      */
     public function loadCurrenciesForCountry($code)
     {
@@ -311,7 +311,7 @@ 
      * Find a country timezone.
      *
      * @param $countryCode
-     * @return null
+     * @return \PragmaRX\Coollection\Package\Coollection
      */
     public function findTimezones($countryCode)
     {

Unused Use Statements -1 removed lines

@@ -4,7 +4,6 @@
 
 use PragmaRX\Countries\Package\Services\Cache;
 use PragmaRX\Countries\Package\Services\Helper;
-use PragmaRX\Countries\Package\Contracts\Config;
 use PragmaRX\Countries\Package\Services\Hydrator;
 use Psr\SimpleCache\CacheInterface as CacheContract;
 

src/package/Services/Cache.php

Doc Comments +3 added lines, -3 removed lines

@@ -71,7 +71,7 @@ 
     /**
      * Check if cache is enabled.
      *
-     * @return bool
+     * @return \PragmaRX\Coollection\Package\Coollection
      */
     protected function enabled()
     {
@@ -132,7 +132,7 @@ 
      * Delete an item from the cache by its unique key.
      *
      * @param string $key
-     * @return bool
+     * @return boolean|null
      */
     public function delete($key)
     {
@@ -197,7 +197,7 @@ 
      * Get an item from the cache, or store the default value.
      *
      * @param  string $key
-     * @param  \DateTimeInterface|\DateInterval|float|int $minutes
+     * @param  integer $minutes
      * @param Closure $callback
      * @return mixed
      */

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

@@ -3,8 +3,8 @@
 namespace PragmaRX\Countries\Package\Services;
 
 use Closure;
-use Psr\SimpleCache\CacheInterface;
 use PragmaRX\Countries\Package\Services\Cache\Managers\Nette as NetteManager;
+use Psr\SimpleCache\CacheInterface;
 
 class Cache implements CacheInterface
 {

Spacing +1 added lines, -1 removed lines

@@ -203,7 +203,7 @@
      */
     public function remember($key, $minutes, Closure $callback)
     {
-        if (! is_null($value = $this->manager->get($key))) {
+        if (!is_null($value = $this->manager->get($key))) {
             return $value;
         }
 

src/package/Services/Cache/Managers/Nette.php

Doc Comments +2 added lines, -2 removed lines

@@ -49,7 +49,7 @@ 
     /**
      * Check if cache is enabled.
      *
-     * @return bool
+     * @return \PragmaRX\Coollection\Package\Coollection
      */
     protected function enabled()
     {
@@ -135,7 +135,7 @@ 
      * Delete an item from the cache by its unique key.
      *
      * @param string $key
-     * @return bool
+     * @return boolean|null
      */
     public function delete($key)
     {

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

@@ -3,11 +3,10 @@
 namespace PragmaRX\Countries\Package\Services\Cache\Managers;
 
 use Closure;
-use Exception;
-use Psr\SimpleCache\CacheInterface;
 use Nette\Caching\Cache as NetteCache;
 use Nette\Caching\Storages\FileStorage;
 use PragmaRX\Countries\Package\Services\Config;
+use Psr\SimpleCache\CacheInterface;
 
 class Nette implements CacheInterface
 {

src/package/Services/Helper.php

Doc Comments +3 added lines, -3 removed lines

@@ -14,7 +14,7 @@ 
     /**
      * Rinvex constructor.
      *
-     * @param object $config
+     * @param Config $config
      */
     public function __construct($config)
     {
@@ -64,7 +64,7 @@ 
     /**
      * Load json files from dir.
      *
-     * @param $dir
+     * @param string $dir
      * @return \PragmaRX\Coollection\Package\Coollection
      */
     public function loadJsonFiles($dir)
@@ -105,7 +105,7 @@ 
     }
 
     /**
-     * @param $contents
+     * @param string $contents
      * @return string
      */
     public function sanitizeFile($contents)

Unused Use Statements -2 removed lines

@@ -2,8 +2,6 @@
 
 namespace PragmaRX\Countries\Package\Services;
 
-use PragmaRX\Countries\Package\Contracts\Config;
-
 class Helper
 {
     /**

src/package/Services/Hydrator.php

Doc Comments +7 added lines, -7 removed lines

@@ -46,7 +46,7 @@ 
     /**
      * Hydrator constructor.
      *
-     * @param object $config
+     * @param Config $config
      */
     public function __construct($config)
     {
@@ -57,7 +57,7 @@ 
      * Can hydrate?
      *
      * @param $element
-     * @param $enabled
+     * @param boolean $enabled
      * @param $countryCode
      * @return bool
      */
@@ -93,7 +93,7 @@ 
      *
      * @param $countries
      * @param $elements
-     * @return mixed
+     * @return \PragmaRX\Countries\Package\Support\Collection
      */
     private function hydrateCountries($countries, $elements = null)
     {
@@ -109,7 +109,7 @@ 
      *
      * @param $country
      * @param $elements
-     * @return mixed
+     * @return \PragmaRX\Countries\Package\Support\Collection
      */
     private function hydrateCountry($country, $elements)
     {
@@ -285,7 +285,7 @@ 
      * Hydrate flag.
      *
      * @param $country
-     * @return mixed
+     * @return Coollection
      */
     public function hydrateFlag($country)
     {
@@ -396,7 +396,7 @@ 
      * Get country by country code.
      *
      * @param $countryCode
-     * @return mixed
+     * @return \PragmaRX\Countries\Package\Support\Collection
      */
     public function getCountry($countryCode)
     {
@@ -455,7 +455,7 @@ 
     /**
      * Repository setter.
      *
-     * @param $repository
+     * @param \PragmaRX\Countries\Package\Data\Repository $repository
      */
     public function setRepository($repository)
     {

Spacing +11 added lines, -11 removed lines

@@ -64,7 +64,7 @@ 
     protected function canHydrate($element, $enabled, $countryCode)
     {
         return ($enabled || $this->config->get('hydrate.elements.'.$element)) &&
-                ! isset($this->repository->countries[$countryCode]['hydrated'][$element]);
+                !isset($this->repository->countries[$countryCode]['hydrated'][$element]);
     }
 
     /**
@@ -72,7 +72,7 @@ 
      */
     protected function createHydrated($countryCode)
     {
-        if (! isset($this->repository->countries[$countryCode]['hydrated'])) {
+        if (!isset($this->repository->countries[$countryCode]['hydrated'])) {
             $this->repository->countries[$countryCode]['hydrated'] = [];
         }
 
@@ -81,7 +81,7 @@ 
 
     protected function fixCurrencies($country)
     {
-        if (! isset($country['currencies']) && isset($country['currency'])) {
+        if (!isset($country['currencies']) && isset($country['currency'])) {
             $country['currencies'] = $country['currency'];
         }
 
@@ -98,7 +98,7 @@ 
     private function hydrateCountries($countries, $elements = null)
     {
         return countriesCollect(
-            $countries->map(function ($country) use ($elements) {
+            $countries->map(function($country) use ($elements) {
                 return $this->hydrateCountry($country, $elements);
             })
         );
@@ -192,7 +192,7 @@ 
      */
     protected function needsHydration($countryCode, $element, $enabled = false)
     {
-        if (! $this->canHydrate($element, $enabled, $countryCode)) {
+        if (!$this->canHydrate($element, $enabled, $countryCode)) {
             return false;
         }
 
@@ -305,7 +305,7 @@ 
     public function hydrateBorders($country)
     {
         $country['borders'] = isset($country['borders'])
-            ? $country['borders'] = countriesCollect($country['borders'])->map(function ($border) {
+            ? $country['borders'] = countriesCollect($country['borders'])->map(function($border) {
                 return $this->repository->call('where', ['cca3', $border])->first();
             })
             : countriesCollect();
@@ -334,7 +334,7 @@ 
     {
         $country = $this->hydrateTimezones($country);
 
-        $country['timezones'] = $country->timezones->map(function ($timezone) {
+        $country['timezones'] = $country->timezones->map(function($timezone) {
             return $timezone->overwrite(['times' => $this->repository->findTimezoneTime($timezone['zone_id'])]);
         });
 
@@ -354,7 +354,7 @@ 
         $country = $this->fixCurrencies($country);
 
         if (isset($country['currencies'])) {
-            $currencies = countriesCollect($country['currencies'])->mapWithKeys(function ($code) {
+            $currencies = countriesCollect($country['currencies'])->mapWithKeys(function($code) {
                 if ($this->isCurrenciesArray($code)) {
                     return [
                         $code['ISO4217Code'] => $code,
@@ -411,7 +411,7 @@ 
      */
     public function addCountry($countryCode, $country)
     {
-        if (! isset($this->repository->countries[$countryCode])) {
+        if (!isset($this->repository->countries[$countryCode])) {
             $this->repository->countries[$countryCode] = $country;
         }
     }
@@ -438,14 +438,14 @@ 
      */
     protected function checkHydrationElements($elements)
     {
-        $elements = countriesCollect($elements)->mapWithKeys(function ($value, $key) {
+        $elements = countriesCollect($elements)->mapWithKeys(function($value, $key) {
             if (is_numeric($key)) {
                 $key = $value;
                 $value = true;
             }
 
             return [$key => $value];
-        })->filter(function ($element) {
+        })->filter(function($element) {
             return $element;
         });
 

src/package/Services/Countries.php

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);
         }