Skip to content

Deprecate returning non-string values from a user output handler #18932

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 15 commits into from
Jul 7, 2025
Merged

Deprecate returning non-string values from a user output handler #18932

Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
8d702ca
Deprecate returning non-string values from a user output handler
DanielEScherzer Jun 24, 2025
d213952
Only disable the problematic output handler
DanielEScherzer Jun 24, 2025
25c1183
Tests
DanielEScherzer Jun 24, 2025
cb1026a
upgrading
DanielEScherzer Jun 24, 2025
52f5e47
Handlers might be freed from OOM
DanielEScherzer Jun 26, 2025
5781571
Location not always given
DanielEScherzer Jun 26, 2025
470b7e9
context is also unavailable
DanielEScherzer Jun 26, 2025
7710240
No tabs
DanielEScherzer Jun 26, 2025
541028c
Memory limits
DanielEScherzer Jun 28, 2025
87756d7
Update tests for rebase
DanielEScherzer Jul 3, 2025
47c9836
Better variable name
DanielEScherzer Jul 3, 2025
198a391
Test returning objects
DanielEScherzer Jul 3, 2025
40aa105
More test fixes
DanielEScherzer Jul 3, 2025
1cb076f
Update exception_handler.phpt
DanielEScherzer Jul 3, 2025
0882664
Variable name + tests
DanielEScherzer Jul 6, 2025
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
Update exception_handler.phpt
  • Loading branch information
@DanielEScherzer
DanielEScherzer authored Jul 3, 2025
commit 1cb076fd70f3c5606c9995b3e70803ea2332be9b
4 changes: 2 additions & 2 deletions tests/output/ob_start_callback_bad_return/exception_handler.phpt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -127,7 +127,7 @@ return_zero: <<<Inside of return_zero
Testing: return_non_stringable
ErrorException: ob_end_flush(): Returning a non-string result from user output handler return_non_stringable is deprecated in %s:%d
Stack trace:
#0 [internal function]: {closure:%s:%d}(8192, 'ob_end_flush():...', '/usr/src/php/te...', 69)
#0 [internal function]: {closure:%s:%d}(8192, 'ob_end_flush():...', %s, 69)
#1 %s(%d): ob_end_flush()
#2 {main}

Expand All @@ -138,7 +138,7 @@ return_non_stringable: <<<Inside of return_non_stringable
Testing: return_stringable
ErrorException: ob_end_flush(): Returning a non-string result from user output handler return_stringable is deprecated in %s:%d
Stack trace:
#0 [internal function]: {closure:%s:%d}(8192, 'ob_end_flush():...', '/usr/src/php/te...', 69)
#0 [internal function]: {closure:%s:%d}(8192, 'ob_end_flush():...', %s, 69)
#1 %s(%d): ob_end_flush()
#2 {main}

Expand Down