Roave#134 suppress exceptions in parsing Fqsen expressions for documented property types

Author: Ocramius

src/DetectChanges/BCBreak/PropertyBased/PropertyDocumentedTypeChanged.php

@@ -4,6 +4,7 @@
 
 namespace Roave\BackwardCompatibility\DetectChanges\BCBreak\PropertyBased;
 
+use InvalidArgumentException;
 use Roave\BackwardCompatibility\Change;
 use Roave\BackwardCompatibility\Changes;
 use Roave\BackwardCompatibility\Formatter\ReflectionPropertyName;
@@ -34,8 +35,13 @@ public function __invoke(ReflectionProperty $fromProperty, ReflectionProperty $t
             return Changes::empty();
         }
 
-        $fromTypes = array_unique($fromProperty->getDocBlockTypeStrings());
-        $toTypes   = array_unique($toProperty->getDocBlockTypeStrings());
+        try {
+            $fromTypes = array_unique($fromProperty->getDocBlockTypeStrings());
+            $toTypes   = array_unique($toProperty->getDocBlockTypeStrings());
+        } catch (InvalidArgumentException $failedToParseDocblock) {
+            // @TODO #134 improve docblock parsing upstream to remove this generic try-catch
+            return Changes::empty();
+        }
 
         sort($fromTypes);
         sort($toTypes);

test/unit/DetectChanges/BCBreak/PropertyBased/PropertyDocumentedTypeChangedTest.php

@@ -108,6 +108,11 @@ class TheClass {
      * @var int
      */
     private $propertyTypeBeingDuplicatedAreNotBcBreaks;
+
+    /**
+     * @var GenericType<T1, T2>
+     */ 
+    public $propertyWithComplexDocblockThatCannotBeParsed;
 }
 PHP
             ,
@@ -172,6 +177,11 @@ class TheClass {
      * @var int|int
      */
     private $propertyTypeBeingDuplicatedAreNotBcBreaks;
+
+    /**
+     * @var GenericType<T1, T2>
+     */ 
+    public $propertyWithComplexDocblockThatCannotBeParsed;
 }
 PHP
             ,
@@ -196,6 +206,7 @@ class TheClass {
             'privateDocblockToDifferentDocblock'                                => ['[BC] CHANGED: Type documentation for property TheClass#$privateDocblockToDifferentDocblock changed from int to float'],
             'duplicatePropertyTypesBeingDeduplicatedAreNotBcBreaks'             => [],
             'propertyTypeBeingDuplicatedAreNotBcBreaks'                         => [],
+            'propertyWithComplexDocblockThatCannotBeParsed'                     => [],
         ];
 
         return array_combine(