Fix wrong syntax for replaceRoot stage

doctrine · Jul 9, 2019 · ed02702 · ed02702
1 parent a50624c
commit ed02702
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 5 deletions.
diff --git a/lib/Doctrine/ODM/MongoDB/Aggregation/Stage/ReplaceRoot.php b/lib/Doctrine/ODM/MongoDB/Aggregation/Stage/ReplaceRoot.php
@@ -41,7 +41,9 @@ public function __construct(Builder $builder, DocumentManager $documentManager,
     public function getExpression() : array
     {
         return [
-            '$replaceRoot' => $this->expression !== null ? $this->convertExpression($this->expression) : $this->expr->getExpression(),
+            '$replaceRoot' => [
+                'newRoot' => $this->expression !== null ? $this->convertExpression($this->expression) : $this->expr->getExpression(),
+            ],
         ];
     }
 

diff --git a/tests/Doctrine/ODM/MongoDB/Tests/Aggregation/Stage/ReplaceRootTest.php b/tests/Doctrine/ODM/MongoDB/Tests/Aggregation/Stage/ReplaceRootTest.php
@@ -26,7 +26,9 @@ public function testTypeConversion()
         $this->assertEquals(
             [
                 '$replaceRoot' => [
-                    'isToday' => ['$eq' => ['$createdAt', $mongoDate]],
+                    'newRoot' => [
+                        'isToday' => ['$eq' => ['$createdAt', $mongoDate]],
+                    ],
                 ],
             ],
             $stage->getExpression()
@@ -49,7 +51,9 @@ public function testTypeConversionWithDirectExpression()
         $this->assertEquals(
             [
                 '$replaceRoot' => [
-                    'isToday' => ['$eq' => ['$createdAt', $mongoDate]],
+                    'newRoot' => [
+                        'isToday' => ['$eq' => ['$createdAt', $mongoDate]],
+                    ],
                 ],
             ],
             $stage->getExpression()
@@ -68,7 +72,9 @@ public function testFieldNameConversion()
         $this->assertEquals(
             [
                 '$replaceRoot' => [
-                    'someField' => ['$concat' => ['$ip', 'foo']],
+                    'newRoot' => [
+                        'someField' => ['$concat' => ['$ip', 'foo']],
+                    ],
                 ],
             ],
             $stage->getExpression()
@@ -83,7 +89,9 @@ public function testFieldNameConversionWithDirectExpression()
             ->replaceRoot('$authorIp');
 
         $this->assertEquals(
-            ['$replaceRoot' => '$ip'],
+            [
+                '$replaceRoot' => ['newRoot' => '$ip']
+            ],
             $stage->getExpression()
         );
     }