Merge pull request #34 from localheinz/fix/anonymous

Fix: Ignore anonymous classes

Author: weierophinney

src/ClassFileLocator.php

@@ -122,6 +122,17 @@ public function accept()
                     if ($i > 0 && is_array($tokens[$i - 1]) && $tokens[$i - 1][0] === T_DOUBLE_COLON) {
                         break;
                     }
+
+                    // ignore anonymous classes on PHP 7.1 and greater
+                    if ($i >= 2
+                        && \is_array($tokens[$i - 1])
+                        && T_WHITESPACE === $tokens[$i - 1][0]
+                        && \is_array($tokens[$i - 2])
+                        && T_NEW === $tokens[$i - 2][0]
+                    ) {
+                        break;
+                    }
+
                     // no break
                 case T_INTERFACE:
                     // Abstract class, class, interface or trait found

test/ClassFileLocatorTest.php

@@ -3,14 +3,15 @@
  * Zend Framework (http://framework.zend.com/)
  *
  * @link      http://github.com/zendframework/zf2 for the canonical source repository
- * @copyright Copyright (c) 2005-2015 Zend Technologies USA Inc. (http://www.zend.com)
+ * @copyright Copyright (c) 2005-2017 Zend Technologies USA Inc. (http://www.zend.com)
  * @license   http://framework.zend.com/license/new-bsd New BSD License
  */
 
 namespace ZendTest\File;
 
 use Zend\File\ClassFileLocator;
 use Zend\File\Exception;
+use Zend\File\PhpClassFile;
 
 /**
  * Test class for Zend\File\ClassFileLocator
@@ -147,4 +148,29 @@ public function testIterationShouldNotCountFQCNScalarResolutionConstantAsClass()
             $this->assertCount(1, $file->getClasses());
         }
     }
+
+    /**
+     * @requires PHP 7.1
+     */
+    public function testIgnoresAnonymousClasses()
+    {
+        $classFileLocator = new ClassFileLocator(__DIR__ . '/TestAsset/Anonymous');
+
+        $classFiles = \iterator_to_array($classFileLocator);
+
+        $this->assertCount(1, $classFiles);
+
+        $classNames = \array_reduce($classFiles, function (array $classNames, PhpClassFile $classFile) {
+            return \array_merge(
+                $classNames,
+                $classFile->getClasses()
+            );
+        }, []);
+
+        $expected = [
+            TestAsset\Anonymous\WithAnonymousClass::class,
+        ];
+
+        $this->assertEquals($expected, $classNames);
+    }
 }

test/TestAsset/Anonymous/WithAnonymousClass.php

@@ -0,0 +1,24 @@
+<?php
+
+/**
+ * Zend Framework (http://framework.zend.com/)
+ *
+ * @link      https://github.com/zendframework/zend-file for the canonical source repository
+ * @copyright Copyright (c) 2017 Zend Technologies USA Inc. (http://www.zend.com)
+ * @license   https://github.com/zendframework/zend-file/blob/master/LICENSE.md New BSD License
+ */
+
+namespace ZendTest\File\TestAsset\Anonymous;
+
+final class WithAnonymousClass
+{
+    private $anonymous;
+
+    public function __construct()
+    {
+        $this->anonymous = new class extends \stdClass {
+
+        };
+    }
+}
+

test/TestAsset/Anonymous/with-anonymous-class.php

@@ -0,0 +1,13 @@
+<?php
+
+/**
+ * Zend Framework (http://framework.zend.com/)
+ *
+ * @link      https://github.com/zendframework/zend-file for the canonical source repository
+ * @copyright Copyright (c) 2017 Zend Technologies USA Inc. (http://www.zend.com)
+ * @license   https://github.com/zendframework/zend-file/blob/master/LICENSE.md New BSD License
+ */
+
+$anonymous = new class extends \stdClass {
+
+};