Regression: http_build_query() legacy 3-arg int encoding_type — LogicException, Zend BC accepts (ext/standard/http.c) #13740

Category

stdlib · php-src-strict

Problem

php-src keeps backward compatibility for the pre-PHP-8 three-argument form where argument #3 is an integer encoding_type (PHP_QUERY_RFC1738 / PHP_QUERY_RFC3986), not arg_separator.

This compiler rejects the legacy call with:

LogicException: http_build_query() argument #3 ($arg_separator) must be a string in this compiler build

Zend (without declare(strict_types=1) on the call site) still returns the RFC3986 query string.

php-src reference

• php/php-src ext/standard/http.c — php_http_build_query(), legacy encoding arg handling
• php/php-src ext/standard/basic_functions.stub.php — http_build_query() signature

Repro (verified 2026-06-29)

php test/repro/maintainer_gap_http_build_query_legacy_encoding.php
php bin/vm.php test/repro/maintainer_gap_http_build_query_legacy_encoding.php

Repro intentionally omits declare(strict_types=1) to match typical php-src BC scripts.

Scope (PHP-in-PHP first)

Done when (php-src-strict)

• Legacy 3-arg int encoding repro prints ok on VM
• 4-arg explicit form unchanged
• Invalid separator types still TypeError/LogicException per php-src
• JIT/AOT agree when in scope

Verification

php test/repro/maintainer_gap_http_build_query_legacy_encoding.php
php bin/vm.php test/repro/maintainer_gap_http_build_query_legacy_encoding.php

Links

• Stdlib: http_build_query() inline nested array literal — LogicException not encoded query (ext/standard/http.c) #11300 inline nested array · Stdlib: http_build_query() — enum case values must encode like Zend objects not backing scalars (ext/standard/http.c) #8733 enum values · Epic: Self-host critical path — compiler compiles itself (M3→M5) #1492