Merge pull request #37 from diogogomeswww/feature/ignore-abstract-cla…

…sses Feature - Ignore abstract classes
beyondcode · Sep 19, 2018 · e9b835e · e9b835e
2 parents 620914e + eb53562
commit e9b835e
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 1 deletion.
diff --git a/src/ModelFinder.php b/src/ModelFinder.php
@@ -11,6 +11,7 @@
 use PhpParser\NodeTraverser;
 use PhpParser\NodeVisitor\NameResolver;
 use PhpParser\ParserFactory;
+use ReflectionClass;
 
 class ModelFinder
 {
@@ -36,7 +37,9 @@ public function getModelsInDirectory(string $directory): Collection
         })->map(function ($path) {
             return $this->getFullyQualifiedClassNameFromFile($path);
         })->filter(function (string $className) {
-            return !empty($className) && is_subclass_of($className, EloquentModel::class);
+            return !empty($className)
+                && is_subclass_of($className, EloquentModel::class)
+                && ! (new ReflectionClass($className))->isAbstract();
         })->diff($ignoreModels);
     }
 

diff --git a/tests/Models/SomeAbstractClass.php b/tests/Models/SomeAbstractClass.php
@@ -0,0 +1,10 @@
+<?php
+
+namespace BeyondCode\ErdGenerator\Tests\Models;
+
+use Illuminate\Database\Eloquent\Model;
+
+abstract class SomeAbstractClass extends Model
+{
+
+}