Skip to content

Stdlib: exception previous-chain compliance guard (#5104) - #17712

Merged
PurHur merged 1 commit into
masterfrom
agent/issue-5104-exception-previous-chain
Jul 10, 2026
Merged

PurHur merged 1 commit into
masterfrom
agent/issue-5104-exception-previous-chain

Conversation

@PurHur

@PurHur PurHur commented Jul 10, 2026

Copy link
Copy Markdown
Owner

Summary

Verification

./script/docker-exec.sh -- bash -lc 'source script/php-env.sh
php bin/vm.php test/repro/exception_previous_chain.php
php vendor/bin/phpunit test/compliance/VMTest.php --filter exception_previous_chain'
previous_is_exception=true
previous_message=inner
inner_previous_null=true
chain_has_next=true
chain_has_inner=true
chain_has_outer=true
OK (1 test, 1 assertion)

Closes #5104

Made with Cursor

Throwable getPrevious() and three-arg ctor chain already work on master;
add repro script and PHPT so the closed #7159 surface stays guarded.

Co-authored-by: Cursor <cursoragent@cursor.com>
@PurHur
PurHur merged commit 61c78f8 into master Jul 10, 2026
@PurHur
PurHur deleted the agent/issue-5104-exception-previous-chain branch July 10, 2026 19:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Stdlib: Exception::getPrevious() / Throwable chain — missing VM method (zend_exceptions.c)

1 participant