8000 [Dotenv] Check required dotenvs by pizzaminded · Pull Request #32729 · symfony/symfony · GitHub
[go: up one dir, main page]
Skip to content

[Dotenv] Check required dotenvs #32729

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 8 commits into from
Closed

[Dotenv] Check required dotenvs #32729

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
a9e75c0
[Dotenv] added new method which checks if there are all envs defined …
pizzaminded Jul 24, 2019
c8560cc
[Dotenv] added space in exception message, added void return type
pizzaminded Jul 24, 2019
9dbe86a
[Dotenv] added unit tests for checkRequired() method
pizzaminded Jul 24, 2019
3cdf29d
[Dotenv] updated changelog file
pizzaminded Jul 24, 2019
e199b3c
[Dotenv] applied coding standards patch
pizzaminded Jul 24, 2019
6706afa
[Dotenv] make sure that _ENV does not have variables that are populat…
pizzaminded Jul 24, 2019
e661de0
[Dotenv] fixed CS
pizzaminded Jul 24, 2019
de9f772
[Dotenv] added constructor arguments in tests
pizzaminded Jul 24, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
[Dotenv] added new method which checks if there are all envs defined … 
…in $_ENV superglobal
  • Loading branch information
@pizzaminded
pizzaminded committed Jul 24, 2019
commit a9e75c0c2e819f15a0d3f99f73a2ff08f699b1c4
23 changes: 23 additions & 0 deletions src/Symfony/Component/Dotenv/Dotenv.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -216,6 +216,29 @@ public function parse(string $data, string $path = '.env'): array
}
}

/**
* Checks if all required variables are populated.
*
* @param string[] $variables Array with variable names that should be set in $_ENV
*
* @throws \RuntimeException when some of declared variables are not defined
*/
public function checkRequired(array $variables)
{
$missingKeys = [];

foreach ($variables as $variable) {
if(!isset($_ENV[$variable])) {
$missingKeys[] = $variable;
}
}

if(count($missingKeys) > 0) {
throw new \RuntimeException(sprintf('Following environment variables are missing: %s', implode(',', $missingKeys)));
}

}

private function lexVarname()
{
// var name + optional export
Expand Down
0