enlitepro / enlite-monolog

src/EnliteMonolog/Service/MonologServiceAbstractFactory.php

Doc Comments +4 added lines, -1 removed lines

@@ -60,6 +60,9 @@ 
         return $this->createLogger($container, $requestedName);
     }
 
+    /**
+     * @param ContainerInterface $container
+     */
     private function createLogger($container, $requestedName)
     {
         $config = $this->getConfig($container);
@@ -70,7 +73,7 @@ 
 
     /**
      * @param ServiceLocatorInterface|ContainerInterface $container
-     * @return array
+     * @return ContainerInterface
      */
     public function getConfig($container)
     {

src/EnliteMonolog/Service/MonologServiceFactory.php

Unused Use Statements -1 removed lines

@@ -9,7 +9,6 @@
 use Closure;
 use Exception;
 use Interop\Container\ContainerInterface;
-use Monolog\Formatter\LineFormatter;
 use Monolog\Handler\HandlerInterface;
 use Monolog\Logger;
 use Monolog\Formatter\FormatterInterface;

Indentation +34 added lines, -34 removed lines

@@ -132,16 +132,16 @@ 
                 /** @var HandlerInterface $instance */
                 $instance = $reflection->newInstanceArgs($parameters);
             } else {
-	            $class = $handler['name'];
+                $class = $handler['name'];
 
                 /** @var HandlerInterface $instance */
-	            $instance = new $class();
+                $instance = new $class();
             }
 
-	        if (isset($handler['formatter'])) {
-		        $formatter = $this->createFormatter($container, $handler['formatter']);
-		        $instance->setFormatter($formatter);
-	        }
+            if (isset($handler['formatter'])) {
+                $formatter = $this->createFormatter($container, $handler['formatter']);
+                $instance->setFormatter($formatter);
+            }
 
             return $instance;
         }
@@ -155,34 +155,34 @@ 
      * @throws \Interop\Container\Exception\ContainerException
      * @throws RuntimeException
      */
-	public function createFormatter($container, $formatter)
-	{
-		if (is_string($formatter) && $container->has($formatter)) {
-			return $container->get($formatter);
-		} else {
-			if (!isset($formatter['name'])) {
-				throw new RuntimeException('Cannot create logger formatter');
-			}
-
-			if (!class_exists($formatter['name'])) {
-				throw new RuntimeException('Cannot create logger formatter (' . $formatter['name'] . ')');
-			}
-
-			if (isset($formatter['args'])) {
-				if (!is_array($formatter['args'])) {
-					throw new RuntimeException('Arguments of formatter(' . $formatter['name'] . ') must be array');
-				}
-
-				$reflection = new \ReflectionClass($formatter['name']);
-
-				return call_user_func_array(array($reflection, 'newInstance'), $formatter['args']);
-			}
-
-			$class = $formatter['name'];
-
-			return new $class();
-		}
-	}
+    public function createFormatter($container, $formatter)
+    {
+        if (is_string($formatter) && $container->has($formatter)) {
+            return $container->get($formatter);
+        } else {
+            if (!isset($formatter['name'])) {
+                throw new RuntimeException('Cannot create logger formatter');
+            }
+
+            if (!class_exists($formatter['name'])) {
+                throw new RuntimeException('Cannot create logger formatter (' . $formatter['name'] . ')');
+            }
+
+            if (isset($formatter['args'])) {
+                if (!is_array($formatter['args'])) {
+                    throw new RuntimeException('Arguments of formatter(' . $formatter['name'] . ') must be array');
+                }
+
+                $reflection = new \ReflectionClass($formatter['name']);
+
+                return call_user_func_array(array($reflection, 'newInstance'), $formatter['args']);
+            }
+
+            $class = $formatter['name'];
+
+            return new $class();
+        }
+    }
 
     /**
      * @param ServiceLocatorInterface|ContainerInterface $container

Spacing +1 added lines, -1 removed lines

@@ -117,7 +117,7 @@
                     throw new RuntimeException(sprintf('Handler(%s) requires at least %d params. Only %d passed.', $handler['name'], $requiredArgsCount, count($handlerOptions)));
                 }
 
-                foreach($reflection->getConstructor()->getParameters() as $parameter) {
+                foreach ($reflection->getConstructor()->getParameters() as $parameter) {
                     if (!$parameter->isOptional() && !isset($handlerOptions[$parameter->getName()])) {
                         $argumentValue = array_shift($handlerOptions);
                     } elseif (isset($handlerOptions[$parameter->getName()])) {