Skip to content

Update inheritance-mapping.rst#8704

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

Update inheritance-mapping.rst#8704
greg0ire merged 2 commits into
doctrine: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.

Comment thread docs/en/reference/inheritance-mapping.rst Outdated
Comment thread docs/en/reference/inheritance-mapping.rst Outdated
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