From 7364a20d5df4e10320eba107986678acb4396b55 Mon Sep 17 00:00:00 2001
From: glax <johanna@bernloehr.eu>
Date: Sun, 20 Oct 2024 21:20:18 +0200
Subject: [PATCH] Add Progressbar to jobs Add Cancel-Buttons to running jobs
 Auto-update Queue

---
 Website/modules/Footer.tsx                    |  8 ++--
 Website/modules/QueuePopUp.tsx                | 47 +++++++++++--------
 Website/modules/interfaces/IManga.tsx         | 32 +++++++++++++
 Website/modules/interfaces/IProgressToken.tsx | 13 ++++-
 Website/styles/queuePopUp.css                 | 30 ++++++++++--
 package-lock.json                             | 11 +++++
 package.json                                  |  1 +
 7 files changed, 114 insertions(+), 28 deletions(-)

diff --git a/Website/modules/Footer.tsx b/Website/modules/Footer.tsx
index da9299a..b230400 100644
--- a/Website/modules/Footer.tsx
+++ b/Website/modules/Footer.tsx
@@ -10,7 +10,7 @@ export default function Footer({connectedToBackend, apiUri} : {connectedToBacken
     const [AllJobsCount, setAllJobsCount] = React.useState(0);
     const [RunningJobsCount, setRunningJobsCount] = React.useState(0);
     const [StandbyJobsCount, setStandbyJobsCount] = React.useState(0);
-    const [countUpdateInterval, setcountUpdateInterval] = React.useState<number>();
+    const [countUpdateInterval, setCountUpdateInterval] = React.useState<number>();
 
     function UpdateBackendState(){
         Job.GetMonitoringJobs(apiUri).then((jobs) => setMonitoringJobsCount(jobs.length));
@@ -22,19 +22,19 @@ export default function Footer({connectedToBackend, apiUri} : {connectedToBacken
     useEffect(() => {
         if(connectedToBackend){
             UpdateBackendState();
-            setcountUpdateInterval(setInterval(() => {
+            setCountUpdateInterval(setInterval(() => {
                 UpdateBackendState();
             }, 2000));
         }else{
             clearInterval(countUpdateInterval);
-            setcountUpdateInterval(undefined);
+            setCountUpdateInterval(undefined);
         }
     }, [connectedToBackend]);
 
     return (
         <footer>
             <div className="statusBadge"><Icon path={mdiEyeCheck} size={1}/> <span>{MonitoringJobsCount}</span></div>
-            <QueuePopUp apiUri={apiUri}>
+            <QueuePopUp connectedToBackend={connectedToBackend} apiUri={apiUri}>
                 <div className="statusBadge hoverHand"><Icon path={mdiTrayFull} size={1}/> <span>{StandbyJobsCount}</span></div>
                 <div className="statusBadge hoverHand"><Icon path={mdiRun} size={1}/> <span>{RunningJobsCount}</span></div>
             </QueuePopUp>
diff --git a/Website/modules/QueuePopUp.tsx b/Website/modules/QueuePopUp.tsx
index 5c6ac4e..480d349 100644
--- a/Website/modules/QueuePopUp.tsx
+++ b/Website/modules/QueuePopUp.tsx
@@ -1,20 +1,39 @@
 import React, {useEffect, useState} from 'react';
-import IJob, {JobTypeFromNumber} from "./interfaces/IJob";
+import IJob from "./interfaces/IJob";
 import '../styles/queuePopUp.css';
 import '../styles/popup.css';
 import {Job} from "./Job";
-import IManga from "./interfaces/IManga";
+import IManga, {QueueItem} from "./interfaces/IManga";
 import {Manga} from "./Manga";
 
-export default function QueuePopUp({children, apiUri} : {children: JSX.Element[], apiUri: string}) {
+export default function QueuePopUp({connectedToBackend, children, apiUri} : {connectedToBackend: boolean, children: JSX.Element[], apiUri: string}) {
 
     const [StandbyJobs, setStandbyJobs] = React.useState<IJob[]>([]);
     const [StandbyJobsManga, setStandbyJobsManga] = React.useState<IManga[]>([]);
     const [RunningJobs, setRunningJobs] = React.useState<IJob[]>([]);
     const [RunningJobsManga, setRunningJobsManga] = React.useState<IManga[]>([]);
     const [showQueuePopup, setShowQueuePopup] = useState<boolean>(false);
+    const [queueListInterval, setQueueListInterval] = React.useState<number>();
 
     useEffect(() => {
+        if(!showQueuePopup)
+            return;
+        UpdateMonitoringJobsList();
+    }, [showQueuePopup]);
+
+    useEffect(() => {
+        if(connectedToBackend){
+            UpdateMonitoringJobsList();
+            setQueueListInterval(setInterval(() => {
+                UpdateMonitoringJobsList();
+            }, 2000));
+        }else{
+            clearInterval(queueListInterval);
+            setQueueListInterval(undefined);
+        }
+    }, [connectedToBackend]);
+
+    function UpdateMonitoringJobsList(){
         Job.GetStandbyJobs(apiUri)
             .then((jobs) => {
                 if(jobs.length > 0)
@@ -37,7 +56,7 @@ export default function QueuePopUp({children, apiUri} : {children: JSX.Element[]
                 //console.debug("Removing Metadata Jobs");
                 setRunningJobs(jobs.filter(job => job.jobType <= 1));
             });
-    }, []);
+    }
 
     useEffect(() => {
         if(StandbyJobs.length < 1)
@@ -65,7 +84,7 @@ export default function QueuePopUp({children, apiUri} : {children: JSX.Element[]
             {showQueuePopup
                 ? <div className="popup" id="QueuePopUp">
                     <div className="popupHeader">
-                        <h1>Queue Status {showQueuePopup ? "true" : "false"}</h1>
+                        <h1>Queue Status</h1>
                         <img alt="Close Search" className="close" src="../media/close-x.svg"
                              onClick={() => setShowQueuePopup(false)}/>
                     </div>
@@ -76,12 +95,8 @@ export default function QueuePopUp({children, apiUri} : {children: JSX.Element[]
                                 {RunningJobs.map((job: IJob) => {
                                     const manga = RunningJobsManga.find(manga => manga.internalId == job.mangaInternalId || manga.internalId == job.chapter?.parentManga.internalId);
                                     if (manga === undefined || manga === null)
-                                        return <div key={"QueueJob-" + job.id}>Error. Could not find matching manga
-                                            for {job.id}</div>
-                                    return <div className="QueueJob" key={"QueueJob-" + job.id}>
-                                        <img src={Manga.GetMangaCoverUrl(apiUri, manga.internalId)} alt="Manga Cover" />
-                                        <p>{JobTypeFromNumber(job.jobType)}</p>
-                                    </div>;
+                                        return <div key={"QueueJob-" + job.id}>Error. Could not find matching manga for {job.id}</div>
+                                    return QueueItem(apiUri, manga, job, UpdateMonitoringJobsList);
                                 })}
                             </div>
                         </div>
@@ -91,14 +106,8 @@ export default function QueuePopUp({children, apiUri} : {children: JSX.Element[]
                                 {StandbyJobs.map((job: IJob) => {
                                     const manga = StandbyJobsManga.find(manga => manga.internalId == job.mangaInternalId || manga.internalId == job.chapter?.parentManga.internalId);
                                     if (manga === undefined || manga === null)
-                                        return <div key={"QueueJob-" + job.id}>Error. Could not find matching manga
-                                            for {job.id}</div>
-                                    return <div className="QueueJob" key={"QueueJob-" + job.id}>
-                                        <img src={Manga.GetMangaCoverUrl(apiUri, manga.internalId)} alt="Manga Cover" />
-                                        <p className="QueueJob-Name">{manga.sortName}</p>
-                                        <p className="QueueJob-JobType">{JobTypeFromNumber(job.jobType)}</p>
-                                        <p className="QueueJob-additional">{job.jobType == 0 ? `Vol.${job.chapter?.volumeNumber} Ch.${job.chapter?.chapterNumber}` : ""}</p>
-                                    </div>;
+                                        return <div key={"QueueJob-" + job.id}>Error. Could not find matching manga for {job.id}</div>
+                                    return QueueItem(apiUri, manga, job, UpdateMonitoringJobsList);
                                 })}
                             </div>
                         </div>
diff --git a/Website/modules/interfaces/IManga.tsx b/Website/modules/interfaces/IManga.tsx
index 7d3ac10..6b46467 100644
--- a/Website/modules/interfaces/IManga.tsx
+++ b/Website/modules/interfaces/IManga.tsx
@@ -5,6 +5,9 @@ import React, {ReactElement} from "react";
 import Icon from '@mdi/react';
 import { mdiTagTextOutline, mdiAccountEdit } from '@mdi/js';
 import MarkdownPreview from '@uiw/react-markdown-preview';
+import IJob, {JobTypeFromNumber} from "./IJob";
+import {Job} from "../Job";
+import ProgressBar from "@ramonak/react-progress-bar";
 
 export default interface IManga{
     "sortName": string,
@@ -68,4 +71,33 @@ export function SearchResult(apiUri: string, manga: IManga, createJob: (internal
             }}>Monitor
             </button>
         </div>);
+}
+
+function ProgressbarStr(job: IJob): string {
+    return job.progressToken.timeRemaining.substring(0,job.progressToken.timeRemaining.indexOf(".")).concat(" ", ToPercentString(job.progressToken.progress));
+}
+
+function ToPercentString(n: number): string {
+    return n.toString().substring(2,4).concat("%");
+}
+
+export function QueueItem(apiUri: string, manga: IManga, job: IJob, triggerUpdate: () => void){
+    return (
+        <div className="QueueJob" key={"QueueJob-" + job.id}>
+            <img src={Manga.GetMangaCoverUrl(apiUri, manga.internalId)} alt="Manga Cover"/>
+            <p className="QueueJob-Name">{manga.sortName}</p>
+            <p className="QueueJob-JobType">{JobTypeFromNumber(job.jobType)}</p>
+            <p className="QueueJob-additional">{job.jobType == 0 ? `Vol.${job.chapter?.volumeNumber} Ch.${job.chapter?.chapterNumber}` : ""}</p>
+            {job.progressToken.state === 0
+            ? <ProgressBar labelColor={"#000"} height={"10px"} labelAlignment={"outside"} className="QueueJob-Progressbar" completed={job.progressToken.progress} maxCompleted={1} customLabel={ProgressbarStr(job)}/>
+            : <div className="QueueJob-Progressbar"></div>}
+            <div className="QueueJob-actions">
+                <button className="QueueJob-Cancel" onClick={() => Job.CancelJob(apiUri, job.id).then(triggerUpdate)}>Cancel</button>
+                {job.parentJobId != null
+                    ? <button className="QueueJob-Cancel" onClick={() => Job.CancelJob(apiUri, job.parentJobId!).then(triggerUpdate)}>Cancel all related</button>
+                    : <></>
+                }
+            </div>
+        </div>
+    );
 }
\ No newline at end of file
diff --git a/Website/modules/interfaces/IProgressToken.tsx b/Website/modules/interfaces/IProgressToken.tsx
index a639248..f33a507 100644
--- a/Website/modules/interfaces/IProgressToken.tsx
+++ b/Website/modules/interfaces/IProgressToken.tsx
@@ -5,6 +5,17 @@ export default interface IProgressToken{
     progress: number;
     lastUpdate: Date;
     executionStarted: Date;
-    timeRemaining: Date;
+    timeRemaining: string;
     state: number;
+}
+
+export function GetProgressStateFromNumber(n: number): string {
+    switch (n){
+        case 0: return "Running";
+        case 1: return "Complete";
+        case 2: return "Standby";
+        case 3: return "Cancelled";
+        case 4: return "Waiting";
+    }
+    return "";
 }
\ No newline at end of file
diff --git a/Website/styles/queuePopUp.css b/Website/styles/queuePopUp.css
index 59713d8..f2abf91 100644
--- a/Website/styles/queuePopUp.css
+++ b/Website/styles/queuePopUp.css
@@ -12,6 +12,7 @@
 #QueuePopUp #QueuePopUpBody h1 {
     padding: 0;
     margin: 0 0 5px 0;
+    color: var(--primary-color);
 }
 
 #QueuePopUp #QueuePopUpBody > *:first-child {
@@ -30,12 +31,18 @@
     height: 200px;
     display: grid;
     grid-template-columns: 150px auto;
-    grid-template-rows: 30% 30% auto;
+    grid-template-rows: 25% 20% auto 15% 12%;
     column-gap: 10px;
     grid-template-areas:
     "cover name"
     "cover jobType"
-    "cover additional"
+    "cover additionalInfo"
+    "cover progress"
+    "cover actions"
+}
+
+.QueueJob p {
+    margin: 2px 0;
 }
 
 .QueueJob img{
@@ -54,6 +61,21 @@
     grid-area: jobType;
 }
 
-.QueueJob .QueueJob-additional {
-    grid-area: additional;
+.QueueJob .QueueJob-additionalInfo {
+    grid-area: additionalInfo;
+}
+
+.QueueJob .QueueJob-actions {
+    grid-area: actions;
+    display: flex;
+    justify-content: space-evenly;
+}
+
+.QueueJob .QueueJob-Cancel {
+    grid-area: actions;
+    width: 150px;
+}
+
+.QueueJob .QueueJob-Progressbar {
+    grid-area: progress;
 }
\ No newline at end of file
diff --git a/package-lock.json b/package-lock.json
index 6b5d46c..6d78c38 100644
--- a/package-lock.json
+++ b/package-lock.json
@@ -5,6 +5,7 @@
   "packages": {
     "": {
       "dependencies": {
+        "@ramonak/react-progress-bar": "^5.3.0",
         "@uiw/react-markdown-preview": "^5.1.3"
       },
       "devDependencies": {
@@ -438,6 +439,16 @@
         "prop-types": "^15.7.2"
       }
     },
+    "node_modules/@ramonak/react-progress-bar": {
+      "version": "5.3.0",
+      "resolved": "https://registry.npmjs.org/@ramonak/react-progress-bar/-/react-progress-bar-5.3.0.tgz",
+      "integrity": "sha512-PjpOcSBAVSQNyx2cvYyBCI14Tg2eFM0psC9m2ic33PYBIdOzO9/DieWndq9BUQTSjIIarhSpa/lqJ33W/mFJMw==",
+      "license": "MIT",
+      "peerDependencies": {
+        "react": "^16.0.0 || ^17 || ^18",
+        "react-dom": "^16.0.0 || ^17 || ^18"
+      }
+    },
     "node_modules/@rollup/rollup-android-arm-eabi": {
       "version": "4.24.0",
       "resolved": "https://registry.npmjs.org/@rollup/rollup-android-arm-eabi/-/rollup-android-arm-eabi-4.24.0.tgz",
diff --git a/package.json b/package.json
index 2f553a0..aea735a 100644
--- a/package.json
+++ b/package.json
@@ -15,6 +15,7 @@
     "preview": "vite preview"
   },
   "dependencies": {
+    "@ramonak/react-progress-bar": "^5.3.0",
     "@uiw/react-markdown-preview": "^5.1.3"
   }
 }