From 5ba980d33aa5a3e6b77cd8e08fb8025b26bb8ff5 Mon Sep 17 00:00:00 2001
From: Xinos <userxinos@gmail.com>
Date: Thu, 17 Aug 2023 15:17:02 +0700
Subject: [PATCH] Revert "Refactoring"

This reverts commit b0bc29227e0915d397a986aac1ec6c543188b76e.
---
 src/components/widgets/openmediavault/updates.jsx    | 12 +++++-------
 src/pages/api/widgets/openmediavault.js              |  1 +
 .../methods/downloader_get_downloadlist.jsx          |  6 +++---
 .../openmediavault/methods/services_get_status.jsx   |  6 +++---
 .../openmediavault/methods/smart_get_list.jsx        |  6 +++---
 src/widgets/openmediavault/methods/zfs_get_stats.jsx |  4 ++--
 6 files changed, 17 insertions(+), 18 deletions(-)

diff --git a/src/components/widgets/openmediavault/updates.jsx b/src/components/widgets/openmediavault/updates.jsx
index 93a64298..c50751ea 100644
--- a/src/components/widgets/openmediavault/updates.jsx
+++ b/src/components/widgets/openmediavault/updates.jsx
@@ -1,16 +1,14 @@
 import useSWR from "swr";
-import { FaDownload } from "react-icons/fa";
+import { FaCheck } from "react-icons/fa";
 import { useTranslation } from "next-i18next";
 
 import Resource from "../widget/resource";
 import Error from "../widget/error";
 
-const params = new URLSearchParams({ method: "apt.enumerateUpgraded" }).toString();
-const useApi = () => useSWR(`/api/widgets/openmediavault?${params}`);
-
 export default function Uptime({ expanded }) {
-  const { t } = useTranslation();
-  const { data, error } = useApi();
+  const { t, i18n } = useTranslation();
+  const params = { lang: i18n.language, method: "apt.enumerateUpgraded" };
+  const { data, error } = useSWR(`/api/widgets/openmediavault?${new URLSearchParams(params).toString()}`);
 
   if (error || data?.error) {
     return <Error />;
@@ -22,7 +20,7 @@ export default function Uptime({ expanded }) {
 
   return (
     <Resource
-      icon={FaDownload}
+      icon={FaCheck}
       value={t("openmediavault.updatesAvailable")}
       expandedValue={t("openmediavault.packageCount", { value: data.response.length })}
       expanded={expanded}
diff --git a/src/pages/api/widgets/openmediavault.js b/src/pages/api/widgets/openmediavault.js
index 64b5e20e..dcf49f2b 100644
--- a/src/pages/api/widgets/openmediavault.js
+++ b/src/pages/api/widgets/openmediavault.js
@@ -4,6 +4,7 @@ import { getPrivateWidgetOptions } from "utils/config/widget-helpers";
 
 export default async function handler(req, res) {
   const { index, method } = req.query;
+
   const [{ options }] = await getPrivateWidgetOptions("openmediavault", index);
   const widget = {
     type: "openmediavault",
diff --git a/src/widgets/openmediavault/methods/downloader_get_downloadlist.jsx b/src/widgets/openmediavault/methods/downloader_get_downloadlist.jsx
index e48eaf7e..f19b2f47 100644
--- a/src/widgets/openmediavault/methods/downloader_get_downloadlist.jsx
+++ b/src/widgets/openmediavault/methods/downloader_get_downloadlist.jsx
@@ -20,13 +20,13 @@ const items = [
 export default function Component({ service }) {
   const { data, error } = useWidgetAPI(service.widget);
 
-  if (error || data?.error) {
-    return <Container service={service} error={error || data?.error} />;
+  if (error) {
+    return <Container service={service} error={error} />;
   }
 
   const itemsWithData = items.map((item) => ({
     ...item,
-    number: data?.response?.data ? item.getNumber(data.response.data) : null,
+    number: data?.response?.data ? item.getNumber(data?.response?.data) : null,
   }));
 
   return (
diff --git a/src/widgets/openmediavault/methods/services_get_status.jsx b/src/widgets/openmediavault/methods/services_get_status.jsx
index 12887a79..b808f60e 100644
--- a/src/widgets/openmediavault/methods/services_get_status.jsx
+++ b/src/widgets/openmediavault/methods/services_get_status.jsx
@@ -25,13 +25,13 @@ const items = [
 export default function Component({ service }) {
   const { data, error } = useWidgetAPI(service.widget);
 
-  if (error || data?.error) {
-    return <Container service={service} error={error || data?.error} />;
+  if (error) {
+    return <Container service={service} error={error} />;
   }
 
   const itemsWithData = items.map((item) => ({
     ...item,
-    number: data?.response?.data ? item.getNumber(data.response.data) : null,
+    number: data?.response?.data ? item.getNumber(data?.response?.data) : null,
   }));
 
   return (
diff --git a/src/widgets/openmediavault/methods/smart_get_list.jsx b/src/widgets/openmediavault/methods/smart_get_list.jsx
index 48919e07..1a105a36 100644
--- a/src/widgets/openmediavault/methods/smart_get_list.jsx
+++ b/src/widgets/openmediavault/methods/smart_get_list.jsx
@@ -25,13 +25,13 @@ const items = [
 export default function Component({ service }) {
   const { data, error } = useWidgetAPI(service.widget);
 
-  if (error || data?.error) {
-    return <Container service={service} error={error || data?.error} />;
+  if (error) {
+    return <Container service={service} error={error} />;
   }
 
   const itemsWithData = items.map((item) => ({
     ...item,
-    number: data?.response?.output ? item.getNumber(JSON.parse(data.response.output).data) : null,
+    number: data?.response?.output ? item.getNumber(JSON.parse(data?.response?.output || "{}")?.data) : null,
   }));
 
   return (
diff --git a/src/widgets/openmediavault/methods/zfs_get_stats.jsx b/src/widgets/openmediavault/methods/zfs_get_stats.jsx
index 7a5b6d3f..ec8b7a25 100644
--- a/src/widgets/openmediavault/methods/zfs_get_stats.jsx
+++ b/src/widgets/openmediavault/methods/zfs_get_stats.jsx
@@ -38,10 +38,10 @@ export default function Component({ service }) {
     }
   }, [data]);
 
-  if (error || data?.error) {
+  if (error) {
     return (
       <Container>
-        <Error error={error || data?.error} />
+        <Error error={error} />
       </Container>
     );
   }