Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[11.x] Introduce TFetchMode generic to support for different PDO fetch modes #53477

Conversation

kayw-geek
Copy link
Contributor

@kayw-geek kayw-geek commented Nov 12, 2024

PR Description:
This PR introduces the TFetchMode generic to improve PHPDoc annotations in the Query Builder, ensuring better clarity and support for various PDO fetch modes.

Reason:
The current default fetch mode in Connection is PDO::FETCH_OBJ. When a custom Connection is used, the Query Builder's PHPDoc does not reflect the correct fetch mode, leading to potential issues in type hinting and IDE support.

@kayw-geek kayw-geek changed the title Introduce TFetchMode generic to support for different PDO fetch modes [11.x] Introduce TFetchMode generic to support for different PDO fetch modes Nov 12, 2024
@kayw-geek kayw-geek force-pushed the bugfix/fix-incorrect-generic-param-passing-for-build branch from 8489f31 to 5b34eac Compare November 12, 2024 09:21
@taylorotwell
Copy link
Member

No plans to mess with this atm.

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