Merge pull request #37 from EtienneBruines/master

Updated require_once statement to valid php syntax
DarkGhostHunter · May 28, 2020 · 8b7fd69 · 8b7fd69
2 parents f467db7 + 679862c
commit 8b7fd69
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/src/PreloaderCompiler.php b/src/PreloaderCompiler.php
@@ -66,7 +66,7 @@ public function compile() : string
             '@output'       => $this->scriptRealPath(),
             '@generated_at' => date('Y-m-d H:i:s e'),
             '@autoload'     => isset($this->autoloader)
-                ? 'require_once ' . realpath($this->autoloader) : null,
+                ? 'require_once \'' . realpath($this->autoloader) . '\';': null,
             '@list'         => $this->parseList(),
             '@mechanism'    => $this->useRequire
                 ? 'require_once $file' : 'opcache_compile_file($file)',

diff --git a/tests/PreloaderTest.php b/tests/PreloaderTest.php
@@ -373,9 +373,11 @@ public function test_uses_require_instead_of_compile()
 
         $contents = file_get_contents($this->preloaderPath);
 
-        $this->assertStringContainsString('require_once ' . realpath(implode(DIRECTORY_SEPARATOR, [
+        $path = realpath(implode(DIRECTORY_SEPARATOR, [
             $this->workdir, 'autoload.php'
-        ])), $contents);
+        ]));
+        $expectedRequire = "require_once '$path';";
+        $this->assertStringContainsString($expectedRequire, $contents);
         $this->assertStringContainsString('require_once $file', $contents);
     }