feat: xml export supports child keys

close #7

README.md

@@ -52,7 +52,7 @@ Route::apiResource('documents', DocumentController::class)
     ->name('documents');
 ```
 
-The `ExportCsv` and `ExportXlsx` middlewares allows you to specify an array key which will be used to retrieve the data. "Dot" notation is supported.
+You can specify an array key which will be used to retrieve the data. "Dot" notation is supported.
 
 ```php
 Route::apiResource('documents', DocumentController::class)
@@ -63,7 +63,9 @@ Route::apiResource('documents', DocumentController::class)
         ExportXlsx::with([
             'key' => 'data',
         ]),
-        ExportXml::class
+        ExportXml::class::with([
+            'key' => 'data',
+        ]),
     ])
     ->name('documents');
 ```

src/Macros/JsonResponse/ToCsv.php

@@ -11,7 +11,9 @@ public function __invoke()
     {
         return function (string $filename, string $key = null): StreamedResponse {
             $data = $this->getData(true);
-            $data = ! is_null($key) ? Arr::get($data, $key) : $data;
+            if (! is_null($key)) {
+                $data = Arr::get($data, $key);
+            }
             $collection = Arr::isAssoc($data) ? collect([$data]) : collect($data);
 
             return $collection->toCsv($filename);

src/Macros/JsonResponse/ToXlsx.php

@@ -11,7 +11,9 @@ public function __invoke()
     {
         return function (string $filename, string $key = null): StreamedResponse {
             $data = $this->getData(true);
-            $data = ! is_null($key) ? Arr::get($data, $key) : $data;
+            if (! is_null($key)) {
+                $data = Arr::get($data, $key);
+            }
             $collection = Arr::isAssoc($data) ? collect([$data]) : collect($data);
 
             return $collection->toXlsx($filename);

src/Macros/JsonResponse/ToXml.php

@@ -11,8 +11,11 @@ class ToXml
 {
     public function __invoke()
     {
-        return function (string $filename): Response {
+        return function (string $filename, string $key = null): Response {
             $data = $this->getData(true);
+            if (! is_null($key)) {
+                $data = Arr::get($data, $key);
+            }
             $data = Arr::isAssoc($data) ? $data : ['data' => $data];
             $xml = ArrayToXml::convert($data);
 

src/Middleware/ExportXml.php

@@ -7,24 +7,27 @@
 use Closure;
 use Illuminate\Http\JsonResponse;
 use Illuminate\Http\Request;
+use TiMacDonald\Middleware\HasParameters;
 
 class ExportXml
 {
+    use HasParameters;
+
     /**
      * @SuppressWarnings("unused")
      */
     public function __construct(protected FilenameGeneratorContract $filenameGenerator)
     {
     }
 
-    public function handle(Request $request, Closure $next)
+    public function handle(Request $request, Closure $next, string $key = null)
     {
         $response = $next($request);
 
         if ($request->wants(MimeType::Xml()) && $response instanceof JsonResponse) {
             $filename = $this->filenameGenerator->get($request, $response);
 
-            return $response->toXml($filename);
+            return $response->toXml($filename, $key);
         }
 
         return $response;

tests/Feature/XmlTest.php

@@ -16,7 +16,7 @@ public function testItGetsXmlFromWrappedResponse()
 
         $this->assertEquals("attachment; filename=\"documents.xml\"; filename*=utf-8''documents.xml", $response->headers->get('Content-Disposition'));
         $this->assertEquals('<?xml version="1.0"?>
-<root><data><id>1</id><name>Navn</name><data><foo>bar</foo></data><meta/></data><data><id>2</id><name>Noe annet</name><data><foo>bar</foo><bar>foo</bar></data></data></root>
+<root><data><id>1</id><name>Navn</name><data><foo>bar</foo></data><meta/></data><data><id>2</id><name>Noe annet</name><data><foo>bar</foo><bar>foo</bar></data></data><meta><page>1</page></meta></root>
 ', $response->getContent());
     }
 }

tests/TestCase.php

@@ -124,6 +124,9 @@ protected function getWrappedResponse(): JsonResponse
                     ],
                 ],
             ],
+            'meta' => [
+                'page' => 1,
+            ],
         ]);
     }
 

tests/Unit/JsonResponseToXmlTest.php

@@ -27,6 +27,19 @@ public function testItGeneratesXmlFromWrappedResponse()
         Collection::macro('toXml', app(ToXml::class)());
         JsonResponse::macro('JsonResponseToXml', app(ToXml::class)());
 
+        $response = $this->getWrappedResponse()->toXml('filename.xml');
+
+        $this->assertEquals("attachment; filename=\"filename.xml\"; filename*=utf-8''filename.xml", $response->headers->get('Content-Disposition'));
+        $this->assertEquals('<?xml version="1.0"?>
+<root><data><id>1</id><name>Navn</name><data><foo>bar</foo></data><meta/></data><data><id>2</id><name>Noe annet</name><data><foo>bar</foo><bar>foo</bar></data></data><meta><page>1</page></meta></root>
+', $response->getContent());
+    }
+
+    public function testItGeneratesXmlFromWrappedResponseUsingKey()
+    {
+        Collection::macro('toXml', app(ToXml::class)());
+        JsonResponse::macro('JsonResponseToXml', app(ToXml::class)());
+
         $response = $this->getWrappedResponse()->toXml('filename.xml', 'data');
 
         $this->assertEquals("attachment; filename=\"filename.xml\"; filename*=utf-8''filename.xml", $response->headers->get('Content-Disposition'));