Skip to content

Stdlib: Zend named params for hash_update_stream/html/ob (#23786) - #24963

Merged
PurHur merged 1 commit into
masterfrom
agent/issue-23786-builtin-param-names
Jul 29, 2026
Merged

PurHur merged 1 commit into
masterfrom
agent/issue-23786-builtin-param-names

Conversation

@PurHur

@PurHur PurHur commented Jul 29, 2026

Copy link
Copy Markdown
Owner

Summary

php-src reference

  • ext/hash/hash.stub.phphash_update / hash_update_stream
  • ext/standard/html.stub.php / basic_functions.stub.phpget_html_translation_table / ob_get_status

Test plan

  • ./script/docker-exec.sh -- bash -lc 'php bin/vm.php test/repro/issue_23786_html_hash_ob_named.php' → matches Zend
  • vendor/bin/phpunit --filter testHtmlHashObNamedParamsResolve → OK (15 assertions)
  • vendor/bin/phpunit --filter "named_args_hash_update_stream|named_args_html_ob_status" test/compliance/VMTest.php → OK
  • same filter on JITTest.php → OK
  • related named_args_hash_copy / named_args_hash_update_file still green

Verification transcript

html='"'
hash_ok
array (
)
stream_names=context,stream,length
stream_digest=30f088ea6673877c2e2c1edbe7513ff90eda9a6f

(VM and Zend identical.)

Not covered: full ci-fast / spine / north-star5 (pre-existing spine red for SoapHttpTransport.php on master — unrelated).

Made with Cursor

InternalArgInfo still labels hash_update_stream's 2nd arg handle; map stream (+ html/ob/hash_update) in BuiltinParamNames so Reflection and named calls match php-src stubs.

Co-authored-by: Cursor <cursoragent@cursor.com>
@PurHur
PurHur merged commit 2c237e3 into master Jul 29, 2026
1 check passed
@PurHur
PurHur deleted the agent/issue-23786-builtin-param-names branch July 29, 2026 18:40
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.

Regression: get_html_translation_table/hash_update*/ob_get_status named args rejected — Reflection OK or stream name drift (ext stubs)

1 participant