refactor: code cleanup

Author: thorsten

phpmyfaq/admin/assets/src/api/configuration.ts

@@ -0,0 +1,34 @@
+/**
+ * Fetch data for configuration
+ *
+ * This Source Code Form is subject to the terms of the Mozilla Public License,
+ * v. 2.0. If a copy of the MPL was not distributed with this file, You can
+ * obtain one at https://mozilla.org/MPL/2.0/.
+ *
+ * @package   phpMyFAQ
+ * @author    Thorsten Rinne <[email protected]>
+ * @copyright 2025 phpMyFAQ Team
+ * @license   http://www.mozilla.org/MPL/2.0/ Mozilla Public License Version 2.0
+ * @link      https://www.phpmyfaq.de
+ * @since     2025-01-21
+ */
+
+import { Response } from '../interfaces';
+
+export const saveConfiguration = async (data: FormData): Promise<void> => {
+  try {
+    const response = (await fetch('api/configuration', {
+      method: 'POST',
+      body: data,
+    })) as unknown as Response;
+
+    if (response.success) {
+      return await response.json();
+    } else {
+      throw new Error('Network response was not ok.');
+    }
+  } catch (error) {
+    console.error('Error updating configuration: ', error);
+    throw error;
+  }
+};

phpmyfaq/admin/assets/src/api/index.ts

@@ -1,5 +1,6 @@
 export * from './attachment';
 export * from './category';
+export * from './configuration';
 export * from './faqs';
 export * from './forms';
 export * from './glossary';

phpmyfaq/admin/assets/src/configuration/configuration.ts

@@ -15,11 +15,12 @@
 
 import { Tab } from 'bootstrap';
 import { pushErrorNotification, pushNotification } from '../../../../assets/src/utils';
+import { saveConfiguration } from '../api';
+import { Response } from '../interfaces';
 
 export const handleConfiguration = async (): Promise<void> => {
   const configTabList: HTMLElement[] = [].slice.call(document.querySelectorAll('#configuration-list a'));
   const result = document.getElementById('pmf-configuration-result') as HTMLElement;
-  const language = document.getElementById('pmf-language') as HTMLInputElement;
   if (configTabList.length) {
     let tabLoaded = false;
     configTabList.forEach((element) => {
@@ -80,22 +81,12 @@ export const handleSaveConfiguration = async (): Promise<void> => {
       const form = document.getElementById('configuration-list') as HTMLFormElement;
       const formData = new FormData(form);
 
-      const response = await fetch('./api/configuration', {
-        method: 'POST',
-        body: formData,
-      });
-
-      if (!response.ok) {
-        console.error('Request failed!');
-        return;
-      }
-
-      const json = await response.json();
+      const response = (await saveConfiguration(formData)) as unknown as Response;
 
-      if (json.success) {
-        pushNotification(json.success);
+      if (typeof response.success === 'string') {
+        pushNotification(response.success);
       } else {
-        pushErrorNotification(json.error);
+        pushErrorNotification(response.error as string);
       }
     });
   }

phpmyfaq/admin/assets/src/interfaces/response.ts

@@ -1,4 +1,5 @@
 export interface Response {
+  json(): void | PromiseLike<void>;
   success: boolean;
   message?: string;
   error?: string;