http://www.doctrine-project.org/jira This file is an XML representation of an issue en-us 5.2.7 850 21-02-2013 http://www.doctrine-project.org/jira/browse/DDC-16 Doctrine 2 - ORM <p>I have a classes B and C which inherit from superclass A. I would like<br/> to get a list of all A's but filter the list to ignore those in C<br/> which have a property d set to 2.</p> <p>select a from A where a.d == 2 fails because "d" is not a property of A.</p> DDC-16

DQL Ignores properties of subclasses

Improvement Minor Closed Won't Fix All Marco Pivetta Avi Block Tue, 15 Sep 2009 15:10:54 +0000 Sun, 24 Feb 2013 00:19:29 +0000 Sun, 24 Feb 2013 00:19:29 +0000 2.0-ALPHA2 DQL 6 6 <p>Thats indeed tricky. That syntax alone can, however, never work, because there might be several subclasses that have a field named "d", so Doctrine would not know which field you mean.</p> <p>We might need special syntax for such constructs but I'm not sure it is worth it. If anyone has an idea, just shoot.</p> <p>Alternatively, apart from using a native query, what about this DQL:</p> <div class="code panel" style="border-width: 1px;"><div class="codeContent panelContent"> <pre class="code-java">select a from A a where a.id not in (select c.id from C c where c.d = 2) </pre> </div></div> <p>I am sorry, but isn't this sort of query a violation of object oriented programming, Accessing a Superclass A and checking for a property that exists on a certain child is not possible in any strict OO language without specific casting into the type beforehand.</p> <p>Thats why I said "we would need special syntax for such constructs", i.e. casting syntax <img class="emoticon" src="http://www.doctrine-project.org/jira/images/icons/emoticons/smile.gif" height="20" width="20" align="absmiddle" alt="" border="0"/></p> <p>However, the alternative query I showed should work just fine for such purposes.</p> <p>+1 for this request. </p> <p>When displaying data (stored in a database) to end-users using generated tables, server-side sorting and filtering on properties that belong to subclasses is a common task, for instance to provide the data presented in a JQuery Datatable: <a href="http://www.datatables.net" class="external-link">http://www.datatables.net</a>. </p> <p>For this purpose, DQL is great to generate the required queries dynamically. However, since properties of subclasses can currently not be 'accessed', sorting and filtering on these properties becomes impossible, or at least cumbersome to implement. Currently I have lifted/copied specific properties to a general 'value' property of the superclass, which CAN be used for ordering and filtering. Of course, this work-around significantly decreases the power of inheritance strategy in the originally envisioned model.</p> <p>In Hibernate this seems to have been solved in HQL using downcasting, which seems a viable approach, but as Benjamin already mentioned not a best practice in a strict OO approach. Therefore a specific syntax might be required to hint on the to-be-expected casts:</p> <ul class="alternate" type="square"> <li><a href="http://www.mail-archive.com/hibernate-devel@lists.sourceforge.net/msg03975.html" class="external-link">http://www.mail-archive.com/hibernate-devel@lists.sourceforge.net/msg03975.html</a></li> <li><a href="https://hibernate.onjira.com/browse/HB-792" class="external-link">https://hibernate.onjira.com/browse/HB-792</a></li> <li><a href="https://hibernate.onjira.com/browse/HB-132" class="external-link">https://hibernate.onjira.com/browse/HB-132</a></li> </ul> <p>For additional motivation of this subject, check a recent post of mine at the user groups: <a href="http://groups.google.com/group/doctrine-user/msg/caebf8139e06e01a" class="external-link">http://groups.google.com/group/doctrine-user/msg/caebf8139e06e01a</a></p> <p>Honestly I gave up on all of this a long time ago, and now use a CQRS approach. I make views, or denormalized tables for every view in my application, and just use straight SQL.</p> <p>When I need to do creating or updating or whatever, I use doctrine.</p> <p>I would like this as well (though my use case involves joining to the subclass).</p> <p>Registration has one Opponent (superclass to Player and TeamSelection). I would like to count the number of registrations for a particular team within a specific tournament. For example:</p> <div class="code panel" style="border-width: 1px;"><div class="codeContent panelContent"> <pre class="code-java">SELECT COUNT(*) FROM Registration r INNER JOIN r.opponent CAST TeamSelection ts INNER JOIN ts.team t WHERE r.tournament = 1 AND t.name = <span class="code-quote">"Team #1"</span></pre> </div></div> <p>Thinking about this a bit more, it can actually improve query performance, as it would no longer be necessary to join the other subclasses. In other words, since you're casting the opponent to a TeamSelection, it's not necessary to join to the players table.</p> <p>A cast should probably also restrict the result using the discriminator column, for example (SQL):</p> <div class="code panel" style="border-width: 1px;"><div class="codeContent panelContent"> <pre class="code-java">INNER JOIN opponents o ON o.id = r.opponent_d AND o.type = <span class="code-quote">"TeamSelection"</span></pre> </div></div> <p>Possible solution:</p> <div class="code panel" style="border-width: 1px;"><div class="codeContent panelContent"> <pre class="code-java">SELECT p FROM CAST(Person AS User) p </pre> </div></div> <p>Or:</p> <div class="code panel" style="border-width: 1px;"><div class="codeContent panelContent"> <pre class="code-java">SELECT a, p FROM Article a JOIN CAST(a.person AS User) p </pre> </div></div> <p>If you're casting a Person (superclass) to a User (subclass), then shouldn't the alias be "u" and not "p"?</p> <p>i.e. SELECT a, u FROM Article a JOIN CAST(a.person AS User) u</p> <p>the alias is that, an alias, you can use whatever you want.</p> <p>Yes I know that, I'm just making sure that I understand the syntax.</p> <p>I just ran into a similar problem and am not sure the above cast solution would do.</p> <p>I'm trying to do:</p> <div class="code panel" style="border-width: 1px;"><div class="codeContent panelContent"> <pre class="code-java">SELECT s FROM Superclass s WHERE s INSTANCE OF SubclassA OR s INSTANCE OF SubclassB AND s.foo = 'bar'; </pre> </div></div> <p>So I'd need to use CAST in a WHERE condition, not in the FROM/JOIN clause, e.g:</p> <div class="code panel" style="border-width: 1px;"><div class="codeContent panelContent"> <pre class="code-java">SELECT s FROM Superclass s WHERE s INSTANCE OF SubclassA OR s INSTANCE OF SubclassB AND (CAST s AS SubclassB).foo = 'bar'; </pre> </div></div> <p>Is there any workaround? Would be perfect if I could just: </p> <div class="preformatted panel" style="border-width: 1px;"><div class="preformattedContent panelContent"> <pre> SELECT a, p FROM Article a JOIN CAST(a.person AS User) p </pre> </div></div> <p>(+1 for Guilherme Blanco possible solution)</p> <p>I was trying a workaround: </p> <p>Before executing the dql query, I change the mapping info of the association to the concrete type that I was going to use. Like this:</p> <div class="preformatted panel" style="border-width: 1px;"><div class="preformattedContent panelContent"> <pre> $cmf = $this-&gt;em-&gt;getMetadataFactory(); $class = $cmf-&gt;getMetadataFor("Article"); $class-&gt;associationMappings["person"]["targetEntity"]="User"; </pre> </div></div> <p>But now my problem is that I'm actually using an inverse side (in query where this is needed), so when I change the targetEntity, the sql join that is generated, is using the table of the concrete class (User in this case) and not the base class (Person)</p> <div class="preformatted panel" style="border-width: 1px;"><div class="preformattedContent panelContent"> <pre>class Person { ... /** * @OneToOne(targetEntity="Order",cascade={"persist"}) * @JoinColumn(name="order_id", referencedColumnName="orden_id") */ private $order; ... } class Order { ... /** * @OneToOne(targetEntity="Person", mappedBy="order") */ private $person; ... } </pre> </div></div> <p>The SQL generated is joining as if the "user" table has orden_id field <img class="emoticon" src="http://www.doctrine-project.org/jira/images/icons/emoticons/sad.gif" height="20" width="20" align="absmiddle" alt="" border="0"/> </p> <p>I am closing this one.</p> <p>The requirement of this issue is basically violating OO principles.</p> <p>If you really need to filter across multiple child-entities in your inheritance, then try something as following instead:</p> <div class="code panel" style="border-width: 1px;"><div class="codeContent panelContent"> <pre class="code-java"> SELECT r FROM Root r WHERE r.id IN ( SELECT c.id FROM Child c WHERE c.field = :value ) </pre> </div></div> <p>Up-casting is not really a viable solution anyway, since it would be even more weird to cast in DQL and then have hydration still retrieve the root entity type.</p> Duplicate DDC-1377