Several phpdoc fixes (#74)

src/Cascade.php

@@ -10,16 +10,16 @@
  */
 namespace Cascade;
 
+use Monolog\Handler\HandlerInterface;
 use Monolog\Logger;
 use Monolog\Registry;
 
-use Cascade\Config;
 use Cascade\Config\ConfigLoader;
 
 /**
  * Module class that manages Monolog Logger object
- * @see Monolog\Logger
- * @see Monolog\Registry
+ * @see Logger
+ * @see Registry
  *
  * @author Raphael Antonmattei <[email protected]>
  */
@@ -34,12 +34,12 @@ class Cascade
 
     /**
      * Create a new Logger object and push it to the registry
-     * @see Monolog\Logger::__construct
+     * @see Logger::__construct
      *
      * @throws \InvalidArgumentException if no name is given
      *
      * @param string $name The logging channel
-     * @param Monolog\Handler\HandlerInterface[] $handlers Optional stack of handlers, the first
+     * @param HandlerInterface[] $handlers Optional stack of handlers, the first
      * one in the array is called first, etc.
      * @param callable[] $processors Optional array of processors
      *
@@ -90,7 +90,7 @@ public static function logger($name)
     /**
      * Return the config options
      *
-     * @return array Array with configuration options
+     * @return Config Array with configuration options
      */
     public static function getConfig()
     {

src/Config.php

@@ -10,7 +10,7 @@
  */
 namespace Cascade;
 
-use Monolog\Registry;
+use Monolog;
 
 use Cascade\Config\ConfigLoader;
 use Cascade\Config\Loader\ClassLoader\FormatterLoader;
@@ -99,7 +99,7 @@ public function configure()
         }
 
         if ($this->options['disable_existing_loggers']) {
-            Registry::clear();
+            Monolog\Registry::clear();
         }
 
         if (isset($this->options['formatters'])) {

src/Config/Loader/ClassLoader/FormatterLoader.php

@@ -10,7 +10,7 @@
  */
 namespace Cascade\Config\Loader\ClassLoader;
 
-use Monolog\Formatter\LineFormatter;
+use Monolog;
 
 use Cascade\Config\Loader\ClassLoader;
 
@@ -63,7 +63,7 @@ public static function initExtraOptionsHandlers()
     {
         self::$extraOptionHandlers = array(
             'Monolog\Formatter\LineFormatter' => array(
-                'includeStacktraces' => function (LineFormatter $instance, $include) {
+                'includeStacktraces' => function (Monolog\Formatter\LineFormatter $instance, $include) {
                     $instance->includeStacktraces($include);
                 }
             )

src/Config/Loader/ClassLoader/HandlerLoader.php

@@ -34,7 +34,7 @@ class HandlerLoader extends ClassLoader
     /**
      * Constructor
      * @see ClassLoader::__construct
-     * @see Monolog\Handler classes for handler options
+     * @see \Monolog\Handler classes for handler options
      *
      * @param array $handlerOptions Handler options
      * @param FormatterInterface[] $formatters Array of formatter to pick from

src/Config/Loader/ClassLoader/LoggerLoader.php

@@ -11,7 +11,8 @@
 namespace Cascade\Config\Loader\ClassLoader;
 
 use Cascade\Cascade;
-use Cascade\Config\Loader\ClassLoader;
+
+use Monolog;
 
 /**
  * Logger Loader. Instantiate a Logger and set passed in handlers and processors if any

src/Config/Loader/ClassLoader/ProcessorLoader.php

@@ -12,6 +12,8 @@
 
 use Cascade\Config\Loader\ClassLoader;
 
+use Monolog;
+
 /**
  * Processor Loader. Loads the Processor options, validate them and instantiates
  * a Processor object (implementing Monolog\Processor\ProcessorInterface) with all

src/Config/Loader/ClassLoader/Resolver/ExtraOptionsResolver.php

@@ -98,7 +98,7 @@ public static function generateParamsHashKey($params)
     /**
      * Configure options for the provided OptionResolver to match extra params requirements
      *
-     * @param  OptionsResolver $optionsResolver OptionResolver to configure
+     * @param  OptionsResolver $resolver OptionResolver to configure
      * @param  ClassLoader|null $classLoader Optional class loader if you want to use custom
      * handlers for some of the extra options
      */

tests/CascadeTest.php

@@ -14,7 +14,6 @@
 use Monolog\Registry;
 
 use Cascade\Cascade;
-use Cascade\Tests\Fixtures;
 
 /**
  * Class CascadeTest
@@ -49,11 +48,11 @@ public function testRegistry()
     }
 
     /**
-     * @expectedException InvalidArgumentException
+     * @expectedException \InvalidArgumentException
      */
     public function testRegistryWithInvalidName()
     {
-        $logger = Cascade::getLogger(null);
+        Cascade::getLogger(null);
     }
 
     public function testFileConfig()

tests/Config/Loader/ClassLoader/FormatterLoaderTest.php

@@ -43,8 +43,8 @@ public function tearDown()
      *
      * @param  string $class Class name the handler applies to
      * @param  string $optionName Option name
-     *
      * @return \Closure Closure
+     * @throws \Exception
      */
     private function getHandler($class, $optionName)
     {

tests/Config/Loader/ClassLoader/HandlerLoaderTest.php

@@ -54,7 +54,7 @@ public function testHandlerLoaderWithNoOptions()
     }
 
     /**
-     * @expectedException InvalidArgumentException
+     * @expectedException \InvalidArgumentException
      */
     public function testHandlerLoaderWithInvalidFormatter()
     {
@@ -67,7 +67,7 @@ public function testHandlerLoaderWithInvalidFormatter()
     }
 
     /**
-     * @expectedException InvalidArgumentException
+     * @expectedException \InvalidArgumentException
      */
     public function testHandlerLoaderWithInvalidProcessor()
     {
@@ -84,7 +84,7 @@ public function testHandlerLoaderWithInvalidProcessor()
     }
 
     /**
-     * @expectedException InvalidArgumentException
+     * @expectedException \InvalidArgumentException
      */
     public function testHandlerLoaderWithInvalidHandler()
     {
@@ -102,7 +102,7 @@ public function testHandlerLoaderWithInvalidHandler()
     }
 
     /**
-     * @expectedException InvalidArgumentException
+     * @expectedException \InvalidArgumentException
      */
     public function testHandlerLoaderWithInvalidHandlers()
     {
@@ -128,8 +128,8 @@ public function testHandlerLoaderWithInvalidHandlers()
      *
      * @param  string $class Class name the handler applies to
      * @param  string $optionName Option name
-     *
      * @return \Closure Closure
+     * @throws \Exception
      */
     private function getHandler($class, $optionName)
     {

tests/Config/Loader/ClassLoader/LoggerLoaderTest.php

@@ -57,7 +57,7 @@ public function testResolveHandlers()
     }
 
     /**
-     * @expectedException InvalidArgumentException
+     * @expectedException \InvalidArgumentException
      */
     public function testResolveHandlersWithMismatch()
     {
@@ -96,7 +96,7 @@ public function testResolveProcessors()
     }
 
     /**
-     * @expectedException InvalidArgumentException
+     * @expectedException \InvalidArgumentException
      */
     public function testResolveProcessorsWithMismatch()
     {

tests/Config/Loader/ClassLoader/Resolver/ConstructorResolverTest.php

@@ -13,6 +13,8 @@
 use Cascade\Util;
 use Cascade\Config\Loader\ClassLoader\Resolver\ConstructorResolver;
 
+use Symfony;
+
 /**
  * Class ConstructorResolverTest
  *
@@ -55,7 +57,7 @@ public function tearDown()
     /**
      * Return the contructor args of the reflected class
      *
-     * @return ReflectionParameter[] array of params
+     * @return \ReflectionParameter[] array of params
      */
     protected function getConstructorArgs()
     {

tests/Config/Loader/ClassLoader/Resolver/ExtraOptionsResolverTest.php

@@ -12,6 +12,8 @@
 
 use Cascade\Config\Loader\ClassLoader\Resolver\ExtraOptionsResolver;
 
+use Symfony;
+
 /**
  * Class ExtraOptionsResolverTest
  *

tests/Config/Loader/FileLoader/FileLoaderAbstractTest.php

@@ -95,7 +95,7 @@ public function extensionsDataProvider()
      * Test validating the extension
      *
      * @param boolean $expected Expected boolean value
-     * @param string filepath Filepath to validate
+     * @param string $filepath Filepath to validate
      * @dataProvider extensionsDataProvider
      */
     public function testValidateExtension($expected, $filepath)
@@ -152,7 +152,7 @@ public function testGetSectionOf(array $array, $section, array $expected)
     /**
      * Test loading an invalid file
      *
-     * @expectedException RuntimeException
+     * @expectedException \RuntimeException
      */
     public function testloadFileFromInvalidFile()
     {

tests/Fixtures/SampleClass.php

@@ -65,6 +65,7 @@ class SampleClass
      * @param mixed $mandatory Some mandatory param
      * @param string $optionalA Some optional param
      * @param string $optionalB Some other optional param
+     * @param string $optional_snake Some optional snake param
      */
     public function __construct(
         $mandatory,