Skip to content

Support mysqli_result->fetch_assoc() and mysqli_result->fetch_all() - #815

Merged
staabm merged 7 commits into
mainfrom
assoc
Oct 31, 2026
Merged

staabm merged 7 commits into
mainfrom
assoc

Conversation

@staabm

@staabm staabm commented Oct 30, 2026

Copy link
Copy Markdown
Owner

No description provided.

@staabm staabm changed the title Support mysqli_result->fetch_assoc Support mysqli_result->fetch_assoc() Oct 30, 2026
Comment thread tests/default/data/mysqli.php Resolved
@staabm

staabm commented Oct 31, 2026

Copy link
Copy Markdown
Owner Author

@copilot implement the necessary MysqliReflection, MysqliResultObjectType, MysqliResultDynamicReturnTypeExtension changes to make the tests pass.

Copilot AI commented Oct 31, 2026

Copy link
Copy Markdown
Contributor

@copilot implement the necessary MysqliReflection, MysqliResultObjectType, MysqliResultDynamicReturnTypeExtension changes to make the tests pass.

Implemented union-aware fetch_assoc() row inference and immutable row-typed results in commit 2dce25f. Syntax and diff checks pass; PHPUnit could not run because vendor/bin/phpunit is absent.

@staabm staabm changed the title Support mysqli_result->fetch_assoc() Support mysqli_result->fetch_assoc() and mysqli_result->fetch_all() Oct 31, 2026
@staabm
staabm marked this pull request as ready for review October 31, 2026 07:17
@staabm
staabm merged commit b7c7525 into main Oct 31, 2026
30 checks passed
@staabm
staabm deleted the assoc branch October 31, 2026 07:18
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.

2 participants