chrismichaels84 / data-manager

src/Decoders/JsonDecoder.php

Doc Comments +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
 
     /**
      * Returns MimeType
-     * @return string
+     * @return string[]
      */
     public function getMimeType()
     {

src/Decoders/YamlDecoder.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/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)
     {

Unused Use Statements -1 removed lines

@@ -7,7 +7,6 @@
 use Michaels\Manager\Exceptions\NestingUnderNonArrayException;
 use Michaels\Manager\Helpers;
 use Michaels\Manager\Messages\NoItemFoundMessage;
-use Traversable;
 
 /**
  * Manages complex, nested data

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/CollectionTrait.php

Doc Comments +1 added lines, -1 removed lines

@@ -142,7 +142,7 @@
      *
      * @param string $method name
      * @param array $arguments to be passed along
-     * @param object $instance of the Collection
+     * @param CollectionTrait $instance of the Collection
      * @param string $flag corresponding to the properties above
      * @param string $subject Alias of data in Manager
      * @return mixed

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

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 +10 added lines, -10 removed lines

@@ -56,7 +56,7 @@ 
      */
     public function fetch($alias, $fallback = '_michaels_no_fallback')
     {
-        $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
@@ -69,7 +69,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;
             }
         }
@@ -90,11 +90,11 @@ 
      */
     public function di($alias, $factory, array $declared = null)
     {
-        $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);
         }
 
         return $this;
@@ -107,7 +107,7 @@ 
      */
     public function share($alias)
     {
-        $this->add($this->nameOfIocManifest . "._singletons.$alias", true);
+        $this->add($this->nameOfIocManifest."._singletons.$alias", true);
         return $this;
     }
 
@@ -119,7 +119,7 @@ 
      */
     public function setup($alias, $pipeline)
     {
-        $this->add($this->nameOfIocManifest . "._pipelines.$alias", $pipeline);
+        $this->add($this->nameOfIocManifest."._pipelines.$alias", $pipeline);
         return $this;
     }
 
@@ -154,7 +154,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) {
@@ -166,13 +166,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;
@@ -203,7 +203,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/Contracts/ManagesIocInterface.php

Doc Comments +3 added lines, -2 removed lines

@@ -11,7 +11,7 @@ 
     /**
      * Initializes IoC Container
      * @param array $components
-     * @return mixed
+     * @return \Michaels\Manager\Traits\ManagesIocTrait
      */
     public function initDi(array $components = []);
 
@@ -32,7 +32,7 @@ 
     /**
      * Turns a dependency into a singleton.
      * @param $alias
-     * @return mixed
+     * @return \Michaels\Manager\Traits\ManagesIocTrait
      */
     public function share($alias);
 
@@ -40,6 +40,7 @@ 
      * Add a pipeline to to the que for a specific item
      * @param $alias
      * @param $pipeline
+     * @return \Michaels\Manager\Traits\ManagesIocTrait
      */
     public function setup($alias, $pipeline);