Deprecate the `\Doctrine\ORM\Query\Parser::setCustomOutputTreeWalker(…

…)` method

We use this method only from within one of our own test cases, and I don't see how it would be useful to anybody else outside – it has to be called on the `Parser` instance which exists internally in the `Query` only.

Deprecating and removing it in 3.x allows for a slight simplification in the `Parser` there, since we do no longer need the field (it can be a local variable).

src/Query/Parser.php

@@ -200,6 +200,13 @@ public function __construct(Query $query)
      */
     public function setCustomOutputTreeWalker($className)
     {
+        Deprecation::trigger(
+            'doctrine/orm',
+            'https://github.com/doctrine/orm/pull/11641',
+            '%s is deprecated, set the output walker class in a query hint instead',
+            __METHOD__
+        );
+
         $this->customOutputWalker = $className;
     }
 

tests/Tests/ORM/Query/LanguageRecognitionTest.php

@@ -51,7 +51,7 @@ public function parseDql(string $dql, array $hints = []): ParserResult
         $parser = new Query\Parser($query);
 
         // We do NOT test SQL output here. That only unnecessarily slows down the tests!
-        $parser->setCustomOutputTreeWalker(NullSqlWalker::class);
+        $query->setHint(Query::HINT_CUSTOM_OUTPUT_WALKER, NullSqlWalker::class);
 
         return $parser->parse();
     }