Add event to allow inspecting and changing multipart responses

sabre-io · Aug 26, 2018 · 190e42c · 190e42c
1 parent de8a4c9
commit 190e42c
Show file tree

Hide file tree

Showing 2 changed files with 38 additions and 0 deletions.
diff --git a/lib/DAV/Server.php b/lib/DAV/Server.php
@@ -1656,6 +1656,7 @@ function getResourceTypeForNode(INode $node) {
      * @return string
      */
     function generateMultiStatus($fileProperties, $strip404s = false) {
+        $this->emit('beforeMultiStatus', [&$fileProperties]);
 
         $w = $this->xml->getWriter();
         $w->openMemory();

diff --git a/tests/Sabre/DAV/ServerEventsTest.php b/tests/Sabre/DAV/ServerEventsTest.php
@@ -12,6 +12,8 @@ class ServerEventsTest extends AbstractServer {
 
     private $exception;
 
+    private $fileProperties;
+
     function testAfterBind() {
 
         $this->server->on('afterBind', [$this, 'afterBindHandler']);
@@ -123,4 +125,39 @@ function testMethod() {
 
     }
 
+    function multiStatusHandler(&$fileProperties) {
+        $this->fileProperties = $fileProperties;
+        $fileProperties = array_slice($fileProperties, 0, 1);
+    }
+
+    function testBeforeMultiStatus() {
+
+        $this->server->on('beforeMultiStatus', [$this, 'multiStatusHandler']);
+
+        $response = $this->server->generateMultiStatus([
+            [
+                'href' => 'foo',
+                '200' => [
+                    'd:getcontentlength' => 10
+                ],
+            ],
+            [
+                'href' => 'bar',
+                '200' => [
+                    'd:getcontentlength' => 11
+                ],
+            ]
+        ]);
+
+        $body = preg_replace("/xmlns(:[A-Za-z0-9_])?=(\"|\')DAV:(\"|\')/", "xmlns\\1=\"urn:DAV\"", $response);
+        $xml = simplexml_load_string($body);
+        $xml->registerXPathNamespace('d', 'urn:DAV');
+
+        $data = $xml->xpath('/d:multistatus/d:response/d:propstat/d:prop/d:getcontentlength');
+        $this->assertCount(1, $data);
+
+        $this->assertCount(2, $this->fileProperties);
+
+    }
+
 }