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

Update inheritance-mapping.rst #8704

Merged
merged 2 commits into from
May 27, 2021
Merged

Update inheritance-mapping.rst #8704

merged 2 commits into from
May 27, 2021

Conversation

juaiglesias
Copy link
Contributor

I updated Inheritance Mapping documentation based on what we discussed in #5961.
I added what I think is an important comment about how it is not possible to query directly the base class in a CTI schema without getting all the LEFT JOINs with the leaf entities tables.

SenseException
SenseException previously approved these changes May 26, 2021
@greg0ire greg0ire changed the base branch from 2.8.x to 2.9.x May 27, 2021 06:16
@greg0ire greg0ire dismissed SenseException’s stale review May 27, 2021 06:16

The base branch was changed.

@greg0ire greg0ire merged commit f2c3dda into doctrine:2.9.x May 27, 2021
beberlei added a commit that referenced this pull request May 31, 2021
* Mark 2.8.x as unmaintained, and 2.9.x as current

* Fix ClassMetadataInfo template inference

* Fix metadata cache compatibility layer

* Bump doctrine/cache patch dependency to fix build with lowest deps

* Add generics to parameters

* Add note about performance and inheritance mapping (#8704)

Co-authored-by: Claudio Zizza <[email protected]>

* Adapt flush($argument) in documentation as it's deprecated. (#8728)

* [GH-8723] Remove use of nullability to automatically detect nullable status (#8732)

* [GH-8723] Remove use of nullability to automatically detect nullable status.

* [GH-8723] Make Column::$nullable default to false again, fix tests.

* Add automatic type detection for Embedded. (#8724)

* Add automatic type detection for Embedded.

* Inline statement.

Co-authored-by: Benjamin Eberlei <[email protected]>

Co-authored-by: Grégoire Paris <[email protected]>
Co-authored-by: Vincent Langlet <[email protected]>
Co-authored-by: Andreas Braun <[email protected]>
Co-authored-by: Fran Moreno <[email protected]>
Co-authored-by: Juan Iglesias <[email protected]>
Co-authored-by: Claudio Zizza <[email protected]>
Co-authored-by: Yup <[email protected]>
Co-authored-by: Benjamin Eberlei <[email protected]>
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.

3 participants