Ref #19398 - Use application/json instead of text/plain for JSON export

williamdes · williamdes · commit 833645f8d16f · 2024-11-21T14:01:40.000+01:00
Ref: #19398 Signed-off-by: William Desportes <williamdes@wdes.fr>
diff --git a/ChangeLog b/ChangeLog
@@ -12,6 +12,7 @@ phpMyAdmin - ChangeLog
 - issue #17009 Implement the security header `Permissions-Policy`
 - issue #17951 Allow rollbacking SET queries
 - issue #18529 Add copy to clipboard for export as text
+- issue #19398 Use `application/json` instead of `text/plain` for JSON export
 
 5.2.2 (not yet released)
 - issue #17851 Fix total count of rows in not accurate
diff --git a/src/Plugins/Export/ExportJson.php b/src/Plugins/Export/ExportJson.php
@@ -63,7 +63,7 @@ protected function setProperties(): ExportPluginProperties
         $exportPluginProperties = new ExportPluginProperties();
         $exportPluginProperties->setText('JSON');
         $exportPluginProperties->setExtension('json');
-        $exportPluginProperties->setMimeType('text/plain');
+        $exportPluginProperties->setMimeType('application/json');
         $exportPluginProperties->setOptionsText(__('Options'));
 
         // create the root group that will be the options field for
diff --git a/tests/unit/Plugins/Export/ExportJsonTest.php b/tests/unit/Plugins/Export/ExportJsonTest.php
@@ -78,7 +78,7 @@ public function testSetProperties(): void
         );
 
         self::assertSame(
-            'text/plain',
+            'application/json',
             $properties->getMimeType(),
         );
 
