Merge pull request #46 from symfony-cmf/patch-return-new-document

Return the new document after PATCH has finished

Author: dbu

src/Controller/ResourceController.php

@@ -133,13 +133,17 @@ public function patchResourceAction($repositoryName, $path, Request $request)
                     $targetPath = $action['target'];
                     $repository->move($path, $targetPath);
 
+                    $resource = $repository->get($targetPath);
+
                     break;
                 default:
-                    return $this->badRequestResponse(sprintf('Only operation "%s" is not supported, supported operations: move.', $action['operation']));
+                    return $this->badRequestResponse(sprintf('Operation "%s" is not supported, supported operations: move.', $action['operation']));
             }
         }
 
-        return $this->createResponse('', Response::HTTP_NO_CONTENT);
+        $this->resourceHandler->setMaxDepth(0);
+
+        return $this->createResponse($resource, Response::HTTP_OK);
     }
 
     /**

tests/Features/resource_api_phpcr.feature

@@ -49,7 +49,7 @@ Feature: PHPCR resource repository
             """
             [{"operation": "move", "target": "/foo-bar"}]
             """
-        Then the response code should be 204
+        Then the response code should be 200
         And there is an "Article" document at "/cmf/articles/foo-bar"
             | title | Article 1          |
             | body  | This is my article |
@@ -65,7 +65,7 @@ Feature: PHPCR resource repository
             """
             [{"operation": "move", "target": "/bar/foo"}]
             """
-        Then the response code should be 204
+        Then the response code should be 200
         And there is an "Article" document at "/cmf/articles/bar/foo"
             | title | Article 1          |
             | body  | This is my article |

tests/Features/resource_api_phpcr_odm.feature

@@ -108,7 +108,7 @@ Feature: PHPCR-ODM resource repository
             """
             [{"operation": "move", "target": "/foo-bar"}]
             """
-        Then the response code should be 204
+        Then the response code should be 200
         And there is an "Article" document at "/cmf/articles/foo-bar":
             | title | Article 1          |
             | body  | This is my article |
@@ -124,7 +124,21 @@ Feature: PHPCR-ODM resource repository
             """
             [{"operation": "move", "target": "/bar/foo"}]
             """
-        Then the response code should be 204
+        Then the response code should be 200
+        And the response should contain json:
+            """
+            {
+                "repository_alias": "phpcrodm_repo",
+                "repository_type": "doctrine/phpcr-odm",
+                "payload_alias": "article",
+                "payload_type": "Symfony\\Cmf\\Bundle\\ResourceRestBundle\\Tests\\Resources\\TestBundle\\Document\\Article",
+                "path": "\/bar\/foo",
+                "node_name": "foo",
+                "label": "foo",
+                "repository_path": "\/bar\/foo",
+                "children": []
+            }
+            """
         And there is an "Article" document at "/cmf/articles/bar/foo":
             | title | Article 1          |
             | body  | This is my article |