chrismichaels84 / data-manager

src/FileLoader.php

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

@@ -1,11 +1,11 @@
 <?php
 namespace Michaels\Manager;
 
-use Michaels\Manager\Contracts\DecoderInterface;
+use Exception;
 use Michaels\Manager\Bags\FileBag;
-use Michaels\Manager\Exceptions\UnsupportedFilesException;
+use Michaels\Manager\Contracts\DecoderInterface;
 use Michaels\Manager\Exceptions\BadFileDataException;
-use Exception;
+use Michaels\Manager\Exceptions\UnsupportedFilesException;
 
 /**
  * Loads configuration files and converts them to php arrays using Decoders

Doc Comments +2 added lines, -2 removed lines

@@ -101,7 +101,7 @@ 
      * Process file bag to load into the data manager.
      * A file bag is an array of SplFileInfo objects.
      *
-     * @param array|FileBag $fileBag
+     * @param FileBag $fileBag
      * @return array
      * @throws Exception
      */
@@ -172,7 +172,7 @@ 
      * Default decoder class factory method.
      * Checks to make sure we have a default decoder available and if so, adds it as a decoder to the file loader.
      *
-     * @param $mimeType
+     * @param string $mimeType
      */
     protected function checkAndAddDefaultDecoder($mimeType)
     {

Spacing +5 added lines, -5 removed lines

@@ -128,7 +128,7 @@ 
         if (!empty($this->unsupportedFiles)) {
             $badFiles = implode(", ", $this->unsupportedFiles);
             throw new UnsupportedFilesException(
-                'The file(s) ' . $badFiles . ' are not supported by the available decoders.'
+                'The file(s) '.$badFiles.' are not supported by the available decoders.'
             );
         }
 
@@ -176,11 +176,11 @@ 
      */
     protected function checkAndAddDefaultDecoder($mimeType)
     {
-        $decoderClass = ucfirst($mimeType) . "Decoder";
+        $decoderClass = ucfirst($mimeType)."Decoder";
 
-        if (file_exists(__DIR__ . '/Decoders/' . $decoderClass . '.php') && !$this->isSupportedMimeType($mimeType)) {
+        if (file_exists(__DIR__.'/Decoders/'.$decoderClass.'.php') && !$this->isSupportedMimeType($mimeType)) {
             $nameSpace = "Michaels\\Manager\\Decoders\\";
-            $fullQualifiedClassName = $nameSpace . $decoderClass;
+            $fullQualifiedClassName = $nameSpace.$decoderClass;
             $decoder = new $fullQualifiedClassName();
             $this->addDecoder($decoder);
         }
@@ -263,7 +263,7 @@ 
 
         } else {
             // We are namespacing using the file's name
-            $filename = rtrim($file->getBasename(), '.' . $file->getExtension());
+            $filename = rtrim($file->getBasename(), '.'.$file->getExtension());
             $namespace = $this->sanitizeNamespace($filename);
             return [$namespace, $file];
         }

src/Traits/ManagesItemsTrait.php

Doc Comments +1 added lines, -1 removed lines

@@ -435,7 +435,7 @@
 
     /**
      * Cycle through the nests to see if an item is protected
-     * @param $item
+     * @param string $item
      */
     protected function checkIfProtected($item)
     {

src/Helpers.php

Spacing +1 added lines, -1 removed lines

@@ -25,7 +25,7 @@
             return iterator_to_array($items);
         }
 
-        return (array)$items;
+        return (array) $items;
     }
 
     /**

src/Decoders/CustomXmlDecoder.php

Doc Comments +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
      * For instance, if you want to decode Yaml files with the extensions ".yml" and ".yaml",
      * then you want to set the return array to ['yaml', 'yml'].
      *
-     * @return array
+     * @return string[]
      */
     public function getMimeType()
     {

src/Traits/ManagesIocTrait.php

Doc Comments +1 added lines, -1 removed lines

@@ -103,7 +103,7 @@
     /**
      * Turns a dependency into a singleton.
      * @param $alias
-     * @return mixed
+     * @return ManagesIocTrait
      */
     public function share($alias)
     {

Spacing +12 added lines, -12 removed lines

@@ -57,13 +57,13 @@ 
     public function fetch($alias, $fallback = '_michaels_no_fallback')
     {
         // If this is a link, just go back to the master
-        $link = $this->getIfExists($this->nameOfIocManifest . ".$alias");
+        $link = $this->getIfExists($this->nameOfIocManifest.".$alias");
         if (is_string($link) && strpos($link, '_michaels_link_') !== false) {
             return $this->fetch(str_replace('_michaels_link_', '', $link));
         }
 
         // Otherwise, continue
-        $shared = $this->getIfExists($this->nameOfIocManifest . "._singletons.$alias");
+        $shared = $this->getIfExists($this->nameOfIocManifest."._singletons.$alias");
 
         if ($shared instanceof NoItemFoundMessage) {
             // This is not a shared item. We want a new one each time
@@ -76,7 +76,7 @@ 
             // This is shared, but we must produce and cache it
             } else {
                 $object = $this->produceDependency($alias, $fallback);
-                $this->set($this->nameOfIocManifest . "._singletons.$alias", $object);
+                $this->set($this->nameOfIocManifest."._singletons.$alias", $object);
                 return $object;
             }
         }
@@ -104,17 +104,17 @@ 
             unset($links[0]);
         }
 
-        $this->set($this->nameOfIocManifest . ".$alias", $factory);
+        $this->set($this->nameOfIocManifest.".$alias", $factory);
 
         // Setup any declared dependencies
         if ($declared) {
-            $this->set($this->nameOfIocManifest . "._declarations.$alias", $declared);
+            $this->set($this->nameOfIocManifest."._declarations.$alias", $declared);
         }
 
         // Add Links
         if (!empty($links)) {
             foreach ($links as $link) {
-                $this->set($this->nameOfIocManifest . ".$link", "_michaels_link_$alias");
+                $this->set($this->nameOfIocManifest.".$link", "_michaels_link_$alias");
             }
         }
 
@@ -128,7 +128,7 @@ 
      */
     public function share($alias)
     {
-        $this->add($this->nameOfIocManifest . "._singletons.$alias", true);
+        $this->add($this->nameOfIocManifest."._singletons.$alias", true);
         return $this;
     }
 
@@ -140,7 +140,7 @@ 
      */
     public function setup($alias, $pipeline)
     {
-        $this->add($this->nameOfIocManifest . "._pipelines.$alias", $pipeline);
+        $this->add($this->nameOfIocManifest."._pipelines.$alias", $pipeline);
         return $this;
     }
 
@@ -175,7 +175,7 @@ 
     protected function produceDependency($alias, $fallback = '_michaels_no_fallback')
     {
         /* Get the registered factory (string, closure, object, container, NoItemFoundMessage) */
-        $factory = $this->getIfExists($this->nameOfIocManifest . ".$alias");
+        $factory = $this->getIfExists($this->nameOfIocManifest.".$alias");
 
         /* Manage not founds and fallback */
         if ($factory instanceof NoItemFoundMessage) {
@@ -189,13 +189,13 @@ 
         }
 
         /* Get any declared dependencies */
-        $declared = $this->getIfExists($this->nameOfIocManifest . "._declarations.$alias");
+        $declared = $this->getIfExists($this->nameOfIocManifest."._declarations.$alias");
         $dependencies = [];
 
         // Now setup those dependencies into an array
         if (!$declared instanceof NoItemFoundMessage) {
             $dependencies = array_map(function(&$value) use ($alias) {
-                if (is_string($value) && $this->exists($this->nameOfIocManifest . ".$alias")) {
+                if (is_string($value) && $this->exists($this->nameOfIocManifest.".$alias")) {
                     return $this->fetch($value);
                 }
                 return $value;
@@ -226,7 +226,7 @@ 
         }
 
         /* Run the object through the pipeline, if desired */
-        $pipeline = $this->getIfExists($this->nameOfIocManifest . "._pipelines.$alias");
+        $pipeline = $this->getIfExists($this->nameOfIocManifest."._pipelines.$alias");
 
         if (!$pipeline instanceof NoItemFoundMessage) {
             /** @var \Closure $pipeline */

src/Traits/ChainsNestedItemsTrait.php

Spacing +4 added lines, -4 removed lines

@@ -40,7 +40,7 @@ 
     {
         $prefix = $this->buildPrefix();
 
-        return $this->get($prefix . $name);
+        return $this->get($prefix.$name);
     }
 
     /**
@@ -69,7 +69,7 @@ 
     {
         $prefix = $this->buildPrefix();
 
-        return $this->add($prefix . $key, $value);
+        return $this->add($prefix.$key, $value);
     }
 
     /**
@@ -85,7 +85,7 @@ 
             $prefix = $this->currentLevel;
             $this->currentLevel = false;
 
-            return $prefix . ".";
+            return $prefix.".";
         }
     }
 
@@ -97,7 +97,7 @@ 
     protected function addToChain($name)
     {
         $dot = ($this->currentLevel === false) ? '' : '.';
-        $this->currentLevel .= $dot . $name;
+        $this->currentLevel .= $dot.$name;
 
         return $this;
     }

src/Traits/CollectionTrait.php

Spacing +1 added lines, -1 removed lines

@@ -77,7 +77,7 @@
             } else {
                 throw new \BadMethodCallException(
                     "Call to undefined method. `$method` does not exist in "
-                    . get_called_class() . " and it is not part of the Collection API"
+                    . get_called_class()." and it is not part of the Collection API"
                 );
             }
         }