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-2052 Doctrine 2 - ORM <p>Custom tree walkers have freedom in modifying the AST but when you try to add a new query component (i.e. new join in walkSelectStatement() ) to the AST then the SqlWalker throws an exception because it does not has the new component in its _queryComponents array. I see two possible ways to resolve this:<br/> 1. Modify the Parser class in order to allow tree walkers to modify queryComponents and pass changed queryComponents to the SqlWalker<br/> 2. Improve SqlWalker so it can extract and prepare needed information about queryComponent based on AST when it does not have them.</p> DDC-2052

Custom tree walkers are not allowed to add new components to the query

Improvement Major Reopened Unresolved Benjamin Eberlei Łukasz Cybula dql Tue, 2 Oct 2012 13:30:45 +0000 Tue, 14 May 2013 18:18:10 +0000 2.3 2.4 DQL 0 3 <p>Ok this is much more complicated to allow then i thought. The problem is that the QueryComponents are passed by value, as an array, not by reference. That prevents changing them because this change wouldn't be visible in the output walker.</p> <p>I can add a method to allow this in the OutputWalker for now, but generally this requires a bigger refactoring on the Query Components.</p> <p>Added setQueryComponent() in SQL Walker to allow modification in output walker.</p> <p>I'm afraid that this doesn't solve the initial problem at all. I'll try to describe it in more details to show what I mean. Suppose we have two doctrine extensions each of which contain its own tree walker. Each of these tree walkers need to modify AST and add new component to it (joined with some component already existing in the query). The first problem is that each tree walker has its own queryComponents array which is not passed between them, although they not necessary need to use queryComponents - they could use only AST. The second, bigger problem is that the Parser class does not know anything about modifications of queryComponents in tree walkers and cannot pass modified version to the OutputWalker. The goal of submitting this issue was to allow adding new components to the query in tree walkers which is not achievable by your fix. I think it may be the first step in the right direction. Maybe TreeWalkerAdapter should have public method getQueryComponents() which would be used by the Parser to pass modified queryComponents between different tree walkers and finally to the OutputWalker ? This would not break backward compatibility and solve this issue. What do you think about it?</p> <p>I've tried to implement the solution mentioned in previous comment but it's also not so clean and easy as I thought. Each tree walker (including TreeWalkerChain) would have to implement getQueryComponents() and setQueryComponent($alias, array $component) methods. The same with SqlWalker, so the TreeWalker interface should have these methods, which would break BC in some way (walkers that do not inherit from SqlWalker or TreeWalkerAdapter will fail to compile). So maybe my first solution (PR #464) is not so bad for now? In the future queryComponents could be replaced by a special object or could be passed by a reference to allow modifications.</p> <p>Marked as improvement as its not a bug.</p> <p>A solution might probably implement an object holding all the QueryComponent, implementing ArrayAccess. So that way the state can be shared.</p> <p>Just hit this while developing an ast walker... Will look into it too since I need it more than soon.</p> <p>As a VERY UGLY workaround, I used a static variable and a custom sql walker in combination with my AST walker.</p> <div class="code panel" style="border-width: 1px;"><div class="codeContent panelContent"> <pre class="code-java"> namespace Comcom\Versioning\ORM\Query; use Doctrine\ORM\Query\SqlWalker; class WorkaroundSqlWalker <span class="code-keyword">extends</span> SqlWalker { <span class="code-keyword">public</span> function __construct($query, $parserResult, array $queryComponents) { parent::__construct($query, $parserResult, $queryComponents); foreach (VersionWalker::$additionalAliases as $alias =&gt; $value) { $<span class="code-keyword">this</span>-&gt;setQueryComponent($alias, $value); } } } </pre> </div></div>