Fixed #538 (review)

flightphp · Feb 4, 2024 · afba9c1 · afba9c1
1 parent db02086
commit afba9c1
Show file tree

Hide file tree

Showing 6 changed files with 13 additions and 7 deletions.
diff --git a/composer.json b/composer.json
@@ -28,8 +28,7 @@
     },
     "autoload": {
         "files": [
-            "flight/autoload.php",
-            "flight/Flight.php"
+            "flight/autoload.php"
         ]
     },
     "autoload-dev": {

diff --git a/flight/Flight.php b/flight/Flight.php
@@ -10,6 +10,8 @@
 use flight\template\View;
 use flight\net\Route;
 
+require_once __DIR__ . '/autoload.php';
+
 /**
  * The Flight class is a static representation of the framework.
  *

diff --git a/flight/autoload.php b/flight/autoload.php
@@ -4,6 +4,7 @@
 
 use flight\core\Loader;
 
+require_once __DIR__ . '/Flight.php';
 require_once __DIR__ . '/core/Loader.php';
 
 Loader::autoload(true, [dirname(__DIR__)]);
diff --git a/flight/core/Loader.php b/flight/core/Loader.php
@@ -190,12 +190,13 @@ public static function autoload(bool $enabled = true, $dirs = []): void
      */
     public static function loadClass(string $class): void
     {
-        $class_file = str_replace(['\\', '_'], '/', $class) . '.php';
+        $classFile = str_replace(['\\', '_'], '/', $class) . '.php';
 
         foreach (self::$dirs as $dir) {
-            $file = $dir . '/' . $class_file;
-            if (file_exists($file)) {
-                require $file;
+            $filePath = "$dir/$classFile";
+
+            if (file_exists($filePath)) {
+                require_once $filePath;
 
                 return;
             }

diff --git a/index.php b/index.php
@@ -1,6 +1,7 @@
 <?php
 
-require 'flight/Flight.php';
+// require 'flight/Flight.php';
+require 'flight/autoload.php';
 
 Flight::route('/', function () {
     echo 'hello world!';

diff --git a/phpcs.xml b/phpcs.xml
@@ -36,6 +36,8 @@
         <exclude name="Generic.CodeAnalysis.AssignmentInCondition.FoundInWhileCondition" />
         <exclude name="Generic.Commenting.DocComment.MissingShort" />
         <exclude name="Generic.Commenting.DocComment.SpacingBeforeTags" />
+        <exclude name="Generic.WhiteSpace.ArbitraryParenthesesSpacing.SpaceAfterOpen" />
+        <exclude name="Generic.WhiteSpace.ArbitraryParenthesesSpacing.SpaceBeforeClose" />
     </rule>
     <file>flight/</file>
     <file>tests/</file>