catch config() error in a more deliberate way to prevent issues with future code changes.

This commit is contained in:
Tony Murray
2020-06-21 22:13:37 -05:00
parent 5fff03cff1
commit 3ab0a92de9

View File

@@ -2,7 +2,7 @@
/** /**
* EnvHelper.php * EnvHelper.php
* *
* -Description- * Helper for manipulation of the .env file
* *
* This program is free software: you can redistribute it and/or modify * This program is free software: you can redistribute it and/or modify
* it under the terms of the GNU General Public License as published by * it under the terms of the GNU General Public License as published by
@@ -123,16 +123,18 @@ class EnvHelper
'INSTALL' => !file_exists(base_path('config.php')) ? 'true' : false, // if both .env and config.php are missing, assume install is needed 'INSTALL' => !file_exists(base_path('config.php')) ? 'true' : false, // if both .env and config.php are missing, assume install is needed
], [], $env_file); ], [], $env_file);
try {
config(['app.key' => $key]); config(['app.key' => $key]);
} catch (BindingResolutionException $e) {
// called outside of Laravel, ignore config() failure
}
return $key; return $key;
} }
return false; return false;
} catch (ErrorException $e) { } catch (ErrorException $e) {
throw new FileWriteFailedException($env_file, 0, $e); throw new FileWriteFailedException($env_file, 0, $e);
} catch (BindingResolutionException $e) {
// called outside of Laravel, ignore config() failure
return $key;
} }
} }