8000 Fixing incorrect error message when passing null to join/implode's array parameter by CViniciusSDias · Pull Request #12683 · php/php-src · GitHub
[go: up one dir, main page]
Skip to content

Fixing incorrect error message when passing null to join/implode's array parameter #12683

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

Merged
merged 14 commits into from
Dec 1, 2023
Merged

Fixing incorrect error message when passing null to join/implode's array parameter #12683

Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
469981b
Fixes #12682
CViniciusSDias Nov 15, 2023
af47cdf
Adding correct check and error message
CViniciusSDias Nov 15, 2023
b2494f1
Splitting paths as suggested by @iluuu1994
CViniciusSDias Nov 15, 2023
0581bf1
Fixing parameters parsing
CViniciusSDias Nov 16, 2023
d3524bd
Simplifying the parameter check conditions
CViniciusSDias Nov 16, 2023
737d829
Tests fixed and code approach reverted to the original one
CViniciusSDias Nov 16, 2023
63648cc
Removing duplicated test
CViniciusSDias Nov 16, 2023
48a080e
Rewriting the error message and moving test conditions
CViniciusSDias Nov 17, 2023
b56c486
Removing deprecation test
CViniciusSDias Nov 17, 2023
5ac3431
Fixing tests with the new error message
CViniciusSDias Nov 17, 2023
f5b351e
Using the new error message always with null
CViniciusSDias Nov 20, 2023
2193972
Adding closing tag to test and removing unreacheable statement
CViniciusSDias Nov 21, 2023
e86f567
Reverting checks to original structure, changing only the error messages
CViniciusSDias Nov 22, 2023
95b8db9
Making tests PSR-12 compliant
CViniciusSDias Nov 22, 2023
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
Prev Previous commit
Next Next commit
Removing deprecation test
  • Loading branch information
@CViniciusSDias
CViniciusSDias committed Nov 22, 2023
commit b56c486a1797f71ea598f0e74cdeae37e2262644
11 changes: 0 additions & 11 deletions ext/standard/tests/strings/implode_error.phpt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -21,18 +21,7 @@ try {
} catch (TypeError $e) {
echo $e->getMessage(), "\n";
}

/* NULL as glue */
try {
var_dump( implode(NULL, "abcd") );
} catch (TypeError $e) {
echo $e->getMessage(), "\n";
}
?>
--EXPECTF--
implode(): Argument #2 ($array) must be of type array, null given
implode(): Argument #2 ($array) must be of type array, null given
implode(): Argument #2 ($array) must be of type array, string given

Deprecated: implode(): Passing null to parameter #1 ($separator) of type array|string is deprecated in %s on line %d
implode(): Argument #2 ($array) must be of type array, string given
0