Merge pull request #121 from fiskhandlarn/patch-2

Use ReflectionClass to determine Controller inheritance
soberwp · Jul 17, 2019 · 523d3fd · 523d3fd
2 parents 4bb384f + fa020a5
commit 523d3fd
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 12 deletions.
diff --git a/src/Loader.php b/src/Loader.php
@@ -100,7 +100,7 @@ protected function setClassesToRun()
             }
 
             // Exclude non-Controller classes
-            if (!Utils::doesFileContain($filename, 'extends Controller')) {
+            if (!$reflection->isSubclassOf('Sober\Controller\Controller')) {
                 continue;
             }
 

diff --git a/src/Utils.php b/src/Utils.php
@@ -15,17 +15,6 @@ public static function isFilePhp($filename)
         return (in_array(pathinfo($filename, PATHINFO_EXTENSION), ['php']));
     }
 
-    /**
-     * Does File Contain
-     *
-     * Determine if the file contains a string
-     * @return boolean
-     */
-    public static function doesFileContain($filename, $str)
-    {
-        return strpos(file_get_contents($filename), $str) !== false;
-    }
-
     /**
      * Is Array Indexed
      *