8000 [Bridge/PhpUnit] Prefer $_SERVER['argv'] over $argv · Pull Request #25304 · symfony/symfony · GitHub
[go: up one dir, main page]
Skip to content

[Bridge/PhpUnit] Prefer $_SERVER['argv'] over $argv #25304

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 5 commits into from
Closed

[Bridge/PhpUnit] Prefer $_SERVER['argv'] over $argv #25304

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
9f8c89b
Prefer $_SERVER['argv'] over $argv.
ricardodevries Dec 4, 2017
67cc253
Put back some code that was removed for debug purposes.
ricardodevries Dec 4, 2017
3c28b42
Put back the correct variable names.
ricardodevries Dec 4, 2017
8fbf912
Reverting changes and init the variables at the beginning.
ricardodevries Dec 4, 2017
6140de9
Add global variables.
ricardodevries Dec 4, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
8000
Diff view
Diff view
Prev Previous commit
Add global variables.
  • Loading branch information
@ricardodevries
ricardodevries committed Dec 4, 2017
commit 6140de9a9d32de5ed48e38c39f919e8f2c615f61
1 change: 1 addition & 0 deletions src/Symfony/Bridge/PhpUnit/bin/simple-phpunit
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -119,6 +119,7 @@ EOPHP

}

global $argv, $argc;
$argv = isset($_SERVER['argv']) ? $_SERVER['argv'] : array();
Copy link
Member
@nicolas-grekas nicolas-grekas Dec 4, 2017 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

onle last thing: before this line, add: global $argv, $argc;

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch. Fixed in 6140de9.

$argc = isset($_SERVER['argc']) ? $_SERVER['argc'] : 0;
$components = array();
Expand Down
0