Skip to content

Update inheritance-mapping.rst#8704

Merged
greg0ire merged 2 commits intodoctrine:2.9.xfrom
juaiglesias:2.8.x
May 27, 2021
Merged

Update inheritance-mapping.rst#8704
greg0ire merged 2 commits intodoctrine:2.9.xfrom
juaiglesias:2.8.x

Conversation

@juaiglesias
Copy link
Copy Markdown
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.

Co-authored-by: Claudio Zizza <859964+SenseException@users.noreply.github.com>
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 <859964+SenseException@users.noreply.github.com>

* 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 <kontakt@beberlei.de>

Co-authored-by: Grégoire Paris <postmaster@greg0ire.fr>
Co-authored-by: Vincent Langlet <VincentLanglet@users.noreply.github.com>
Co-authored-by: Andreas Braun <git@alcaeus.org>
Co-authored-by: Fran Moreno <franmomu@gmail.com>
Co-authored-by: Juan Iglesias <juan.manuel.iglesias93@gmail.com>
Co-authored-by: Claudio Zizza <859964+SenseException@users.noreply.github.com>
Co-authored-by: Yup <warxcell@gmail.com>
Co-authored-by: Benjamin Eberlei <kontakt@beberlei.de>
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