Regression: Collator::getSortKey Reflection (collator $coll, string $str) — Zend string $string → string|false; named string rejected (ext/intl/collator)

[PurHur]

Category

bug · php-src-strict · Reflection + named args · ext/intl/collator

Problem

Collator::getSortKey is registered with the procedural parameter list ($coll, $str) on the instance method. Zend’s OOP stub is getSortKey(string $string): string|false. Runtime one-arg calls work, but Reflection and named args diverge.

Probed 2026-08-07 on master @4c2f13ea09 under PHP_COMPILER_PROFILE=8.4.

Repro	Zend 8.4	VM PROFILE=8.4 (2026-08-07)
Reflection params	string $string	collator $coll, string $str
return type	string|false	none
$c->getSortKey('abc')	string	string (ok)
$c->getSortKey(string: 'abc')	string	TypeError / internal coerce fail (named str only matches Reflection)

php-src reference

• php/php-src ext/intl/collator/collator.stub.php — public function getSortKey(string $string): string|false

PHP implementation target

• ext/intl/ Collator method registration / Reflection metadata — align OOP arginfo with Zend stub; keep procedural collator_get_sort_key separate
• PHP-in-PHP only; no new runtime/*.c logic

Repro

./script/docker-exec.sh -- bash -lc 'PHP_COMPILER_PROFILE=8.4 php bin/vm.php -r '"'"'
$r = new ReflectionMethod(Collator::class, "getSortKey");
foreach ($r->getParameters() as $p) {
  echo $p->getName(), " ", ($p->hasType() ? (string)$p->getType() : "none"), "\n";
}
echo "ret=", $r->hasReturnType() ? (string)$r->getReturnType() : "none", "\n";
$c = new Collator("en_US");
var_export($c->getSortKey(string: "abc"));
'"'"''

Done when

• Reflection reports getSortKey(string $string): string|false
• Named arg string: works; phantom $coll param gone from instance method
• Compliance / repro guard; php-src-strict

[PurHur]

claim: PHP Compiler Ext — I/O & Crypto — starting this run

[PurHur]

blocker: this RunForge Docker image has no host php-intl (extension_loaded(intl)=false), so Collator is not advertised and the issue repro cannot run. Handing off — picking a different owned extension issue that is verifiable here.

[PurHur]

claim: PHP Compiler Ext — I/O & Crypto — continuing this run (prior claim earlier today; no PR yet)

[PurHur]

Merged via #31544

Verification (Docker):

./script/docker-exec.sh -- bash -lc 'php test/repro/issue_28785_collator_getsortkey_named.php'
# arity=1 req=1; string string; ret=string|false; named_string=string; Unknown named parameter $str; pos=string
./script/phpunit.sh test/unit/CollatorGetSortKeyArginfoTest.php  # OK 8 assertions