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

[Documentation] Merging "Query Result Formats" with "Hydration Modes" #11403

Merged
merged 1 commit into from
Mar 28, 2024

Conversation

ThomasLandauer
Copy link
Contributor

Page: https://www.doctrine-project.org/projects/doctrine-orm/en/2.19/reference/dql-doctrine-query-language.html#query-result-formats

As announced in #11372 (comment), I merged the (mostly) identical sections.

  • I changed the consts from Query to AbstractQuery

  • I deleted this - mainly cause I didn't find a nice place for it:

    In parentheses are the constants of the Query class which you can use with the
    general-purpose method Query::execute(array $params = [], $hydrationMode = Query::HYDRATE_OBJECT).

    Should I bring it back?

@greg0ire
Copy link
Member

Please squash your commits.

Page: https://www.doctrine-project.org/projects/doctrine-orm/en/2.19/reference/dql-doctrine-query-language.html#query-result-formats

As announced in doctrine#11372 (comment), I merged the (mostly) identical sections.

* I changed the `const`s from `Query` to `AbstractQuery`
* I deleted this - mainly cause I didn't find a nice place for it:
    > In parentheses are the constants of the ``Query`` class which you can use with the
general-purpose method ``Query::execute(array $params = [], $hydrationMode = Query::HYDRATE_OBJECT)``.
@ThomasLandauer
Copy link
Contributor Author

Done :-)

@greg0ire greg0ire merged commit e9e60f2 into doctrine:2.19.x Mar 28, 2024
1 check passed
@greg0ire
Copy link
Member

Thanks @ThomasLandauer !

@ThomasLandauer ThomasLandauer deleted the patch-10 branch March 28, 2024 14:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants