Skip to content

Stdlib: pgsql legacy PHP_FALIAS aliases (#22219) - #22259

Merged
PurHur merged 1 commit into
masterfrom
agent/stdlib-22219-pgsql-legacy-aliases
Jul 22, 2026
Merged

PurHur merged 1 commit into
masterfrom
agent/stdlib-22219-pgsql-legacy-aliases

Conversation

@PurHur

@PurHur PurHur commented Jul 22, 2026

Copy link
Copy Markdown
Owner

Summary

  • Register 25 Zend-era ext/pgsql legacy aliases (pg_exec, pg_numrows, pg_cmdtuples, pg_loopen, …, pg_result→pg_fetch_result) as second registrations of the existing Internal handlers (PHP_FALIAS / stub @alias).
  • Optional $name ctor on alias targets; no duplicated execute bodies; no new runtime/*.c.
  • Compliance pgsql_legacy_aliases.phpt + test/repro/issue_22219_pgsql_legacy_aliases.php.

php-src: ext/pgsql/pgsql.stub.php, ext/pgsql/pgsql.c
PHP impl: ext/pgsql/Module.php + ctor pass-through on modern handlers

Closes #22219

Test plan

  • php bin/vm.php test/repro/issue_22219_pgsql_legacy_aliases.php → all Y / exit 0
  • php bin/compile.php -l test/repro/issue_22219_pgsql_legacy_aliases.php → exit 0
  • vendor/bin/phpunit --filter pgsql_legacy_aliases → OK (2 tests, VM+JIT)
  • php script/bootstrap-inventory.php --check → OK 6392/6392
  • Note: full ./phpc build standalone AOT currently red on master even for examples/000-HelloWorld (LLVM module verify); lint path is green for this repro.

Pre-claim gates

  • inventory OK 6392/6392
  • release-readiness --json → user_release_ready: yes

Made with Cursor

Wire Zend-era aliases (pg_exec, pg_numrows, pg_loopen, …) to the existing
modern handlers so function_exists and calls match php-src stub aliases.

Co-authored-by: Cursor <cursoragent@cursor.com>
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: pgsql legacy aliases (pg_exec/pg_numrows/…) missing after modern names (ext/pgsql/pgsql.c)

1 participant