Merge pull request #10 from msztorc/empty-value

#9 - empty value hotfix

Author: msztorc

src/Env.php

@@ -152,7 +152,7 @@ private function _preg_quote_except(string $str, string $exclude, ?string $delim
      */
     private function _prepareValue(string $value): string
     {
-        if (false !== strpos($value, ' ') || in_array($value[0], ['=', '$'])) {
+        if (false !== strpos($value, ' ') || (strlen($value) && in_array($value[0], ['=', '$']))) {
             $value = '"' . $value . '"';
         }
 

tests/EnvArtisanTest.php

@@ -151,4 +151,17 @@ public function testUrls(): void
             ->expectsOutput($app_url)
             ->assertExitCode(0);
     }
+
+    public function testEmptyValue(): void
+    {
+        $app_name = '';
+
+        $this->artisan('env:set', ['key' => 'APP_NAME' .'=' . ''])
+            ->expectsOutput("Environment variable with key 'APP_NAME' has been set to '{$app_name}'")
+            ->assertExitCode(0);
+
+        $this->artisan('env:get', ['key' => 'APP_NAME'])
+            ->expectsOutput($app_name)
+            ->assertExitCode(0);
+    }
 }

tests/EnvClassTest.php

@@ -208,4 +208,13 @@ public function testUrls(): void
         $ver_value = $env->getValue('APP_URL');
         $this->assertEquals($app_url, $ver_value);
     }
+
+    public function testEmptyValue(): void
+    {
+        $env = new Env();
+        $env->setValue('APP_NAME', '');
+        $ver_value = $env->getValue('APP_NAME');
+        $this->assertEmpty($ver_value);
+        $this->assertEquals('', $ver_value);
+    }
 }