Merge branch 'release/1.2.0'

Author: mikaelcom

.php_cs

@@ -1,10 +1,18 @@
 <?php
 
-$finder = Symfony\CS\Finder\DefaultFinder::create()
+$finder = PhpCsFixer\Finder::create()
     ->exclude('vendor')
     ->exclude('tests')
     ->in(__DIR__);
 
-return Symfony\CS\Config\Config::create()
-    ->level(Symfony\CS\FixerInterface::PSR2_LEVEL)
-    ->finder($finder);
+return PhpCsFixer\Config::create()
+	->setUsingCache(false)
+    ->setRules(array(
+        '@PSR2' => true,
+        'binary_operator_spaces' => true,
+        'no_whitespace_in_blank_line' => true,
+        'ternary_operator_spaces' => true,
+        'cast_spaces' => true,
+        'trailing_comma_in_multiline_array' => true
+	))
+    ->setFinder($finder);

.travis.yml

@@ -7,16 +7,17 @@ php:
     - 5.4
     - 5.5
     - 5.6
-    - 7
+    - 7.0
+    - 7.1
 
 before_install:
     - composer self-update
 
 install:
-    - composer install --no-dev
+    - composer install
 
 script:
-    - phpunit --coverage-text --coverage-clover=coverage.clover
+    - ./vendor/phpunit/phpunit/phpunit --coverage-text --coverage-clover=coverage.clover
 
 after_script:
     - wget https://scrutinizer-ci.com/ocular.phar

CHANGELOG.md

@@ -1,5 +1,8 @@
 # CHANGELOG
 
+## 1.2.0
+- issue #7 - Improve Exception message
+
 ## 1.1.0
 - issue #5 - No autocomplete because of return type on a new line after @return in annotation
 

composer.json

@@ -1,30 +1,29 @@
 {
-    "name": "wsdltophp/phpgenerator",
-    "description": "Generate php source file",
-    "type": "library",
-    "keywords" : ["php","generator"],
-    "homepage" : "https://github.com/WsdlToPhp/PhpGenerator",
-    "license" : "MIT",
-    "authors" : [
-        {
-            "name": "Mikaël DELSOL",
-            "email": "[email protected]",
-            "role": "Owner"
-        }
-    ],
-    "support" : {
-        "email" : "[email protected]"
-    },
-    "require": {
+    "name" : "wsdltophp/phpgenerator",
+    "description" : "Generate php source file",
+    "require" : {
         "php" : ">=5.3.3"
     },
     "require-dev": {
-        "fabpot/php-cs-fixer": "~1.8"
+        "friendsofphp/php-cs-fixer": "~2.0",
+        "phpunit/phpunit": "~4.0"
     },
-    "autoload": {
-        "psr-4": {
-            "WsdlToPhp\\PhpGenerator\\": "src",
-            "WsdlToPhp\\PhpGenerator\\Tests\\": "tests"
+    "license" : "MIT",
+    "keywords" : [ "php", "generator" ],
+    "autoload" : {
+        "psr-4" : {
+            "WsdlToPhp\\PhpGenerator\\" : "src",
+            "WsdlToPhp\\PhpGenerator\\Tests\\" : "tests"
         }
-    }
+    },
+    "type" : "library",
+    "support" : {
+        "email" : "[email protected]"
+    },
+    "homepage" : "https://github.com/WsdlToPhp/PhpGenerator",
+    "authors" : [ {
+        "name" : "Mikaël DELSOL",
+        "email" : "[email protected]",
+        "role" : "Owner"
+    } ]
 }

src/Element/AbstractAssignedValueElement.php

@@ -31,7 +31,7 @@ public function __construct($name, $value = null, $access = parent::ACCESS_PUBLI
     public function setValue($value)
     {
         if ($this->getAcceptNonScalarValue() === false && !is_scalar($value) && $value !== null) {
-            throw new \InvalidArgumentException(sprintf('Value of type "%s" is not a valid scalar value', gettype($value)));
+            throw new \InvalidArgumentException(sprintf('Value of type "%s" is not a valid scalar value for %s object', gettype($value), $this->getCalledClass()));
         }
         $this->value = $value;
         return $this;

src/Element/AbstractElement.php

@@ -32,7 +32,7 @@ public function __construct($name)
     public function setName($name)
     {
         if (!self::nameIsValid($name)) {
-            throw new \InvalidArgumentException(sprintf('Name "%s" is invalid, please provide a valid name', $name));
+            throw new \InvalidArgumentException(sprintf('Name "%s" is invalid when instantiating %s object', $name, $this->getCalledClass()));
         }
         $this->name = $name;
         return $this;
@@ -196,7 +196,7 @@ protected function childIsValid($child)
                 $valid |= (gettype($child) === $authorizedType) || self::objectIsValid($child, $authorizedType);
             }
         }
-        return (bool)$valid;
+        return (bool) $valid;
     }
     /**
      * @return AbstractElement[]|mixed[]
@@ -324,9 +324,16 @@ public function getIndentationString($indentation = null)
     public function getIndentedString($string, $indentation = null)
     {
         $strings = explode(self::BREAK_LINE_CHAR, $string);
-        foreach ($strings as $i=>$s) {
+        foreach ($strings as $i => $s) {
             $strings[$i] = sprintf('%s%s', $this->getIndentationString($indentation), $s);
         }
         return implode(self::BREAK_LINE_CHAR, $strings);
     }
+    /**
+     * @return string
+     */
+    final public function getCalledClass()
+    {
+        return substr(get_called_class(), strrpos(get_called_class(), '\\') + 1);
+    }
 }

src/Element/PhpAnnotationBlock.php

@@ -64,7 +64,7 @@ protected static function annotationsAreValid(array $annotations)
         foreach ($annotations as $annotation) {
             $valid &= self::annotationIsValid($annotation);
         }
-        return (bool)$valid;
+        return (bool) $valid;
     }
     /**
      * @param string|array|PhpAnnotation $annotation

src/Element/PhpClass.php

@@ -131,7 +131,7 @@ public static function interfacesAreValid(array $interfaces = array())
         foreach ($interfaces as $interface) {
             $valid &= self::interfaceIsValid($interface);
         }
-        return (bool)$valid;
+        return (bool) $valid;
     }
     /**
      * @param string|PhpClass $interface

src/Element/PhpFunction.php

@@ -65,7 +65,7 @@ public static function parametersAreValid(array $parameters)
         foreach ($parameters as $parameter) {
             $valid &= self::parameterIsValid($parameter);
         }
-        return (bool)$valid;
+        return (bool) $valid;
     }
     /**
      * @param string|array|PhpFunctionParameter $parameter

tests/Element/PhpAnnotationTest.php

@@ -68,4 +68,13 @@ public function testHasContent()
 
         $this->assertTrue($annotation->hasContent());
     }
+
+    public function testExceptionMessageOnName()
+    {
+        try {
+            new PhpAnnotation(0, '');
+        } catch (\InvalidArgumentException $e) {
+            $this->assertSame('Name "0" is invalid when instantiating PhpAnnotation object', $e->getMessage());
+        }
+    }
 }