From f4aff5ec2c33315b3d9e73303d4bf885fc62d6f4 Mon Sep 17 00:00:00 2001
From: Martastain <martas@imm.cz>
Date: Mon, 1 Apr 2024 10:14:39 +0200
Subject: [PATCH] fix: make eslint happy

---
 frontend/src/components/Button.jsx            |  1 +
 frontend/src/components/Dropdown.jsx          |  3 +-
 frontend/src/components/InputDatetime.jsx     |  3 +-
 frontend/src/components/InputPassword.jsx     |  1 +
 frontend/src/components/InputText.jsx         |  1 +
 frontend/src/components/Video.jsx             |  2 +-
 frontend/src/containers/Browser/Browser.jsx   |  2 +
 .../src/containers/Browser/Formatting.jsx     | 87 +++++++++++--------
 frontend/src/hooks.jsx                        |  1 +
 frontend/src/pages/JobsPage/JobsPage.jsx      |  3 +
 10 files changed, 64 insertions(+), 40 deletions(-)

diff --git a/frontend/src/components/Button.jsx b/frontend/src/components/Button.jsx
index 0d009e4..75b01a2 100644
--- a/frontend/src/components/Button.jsx
+++ b/frontend/src/components/Button.jsx
@@ -89,6 +89,7 @@ const Button = forwardRef(
   }
 )
 
+Button.displayName = 'Button'
 Button.defaultProps = {
   iconOnRight: false,
   //component: 'button',
diff --git a/frontend/src/components/Dropdown.jsx b/frontend/src/components/Dropdown.jsx
index 8691076..bc887b9 100644
--- a/frontend/src/components/Dropdown.jsx
+++ b/frontend/src/components/Dropdown.jsx
@@ -64,14 +64,13 @@ const DropdownOption = ({
   currentValue,
   separator,
   disabled,
-  active,
   ...props
 }) => (
   <span>
     {separator && <hr />}
     <Button
       {...props}
-      disabled={props.disabled || currentValue === props.value}
+      disabled={disabled || currentValue === props.value}
     />
   </span>
 )
diff --git a/frontend/src/components/InputDatetime.jsx b/frontend/src/components/InputDatetime.jsx
index a7cdb49..fabedab 100644
--- a/frontend/src/components/InputDatetime.jsx
+++ b/frontend/src/components/InputDatetime.jsx
@@ -8,12 +8,11 @@ import Dialog from './Dialog'
 import BaseInput from './BaseInput'
 import Button from './Button'
 
-//import "react-datepicker/dist/react-datepicker.css"
-
 import './datepicker.sass'
 
 const timeRegex = /^(\d{4})-(\d{2})-(\d{2}) (\d{2}):(\d{2}):(\d{2})$/
 const dateRegex = /^(\d{4})-(\d{2})-(\d{2})$/
+//eslint-disable-next-line
 const allowedDateCharsRegex = /^[\d-\:\ ]*$/
 
 const DateTimeWrapper = styled.div`
diff --git a/frontend/src/components/InputPassword.jsx b/frontend/src/components/InputPassword.jsx
index 756fa9a..d56cbe6 100644
--- a/frontend/src/components/InputPassword.jsx
+++ b/frontend/src/components/InputPassword.jsx
@@ -15,5 +15,6 @@ const InputPassword = forwardRef(
     )
   }
 )
+InputPassword.displayName = 'InputPassword'
 
 export default InputPassword
diff --git a/frontend/src/components/InputText.jsx b/frontend/src/components/InputText.jsx
index 0969c9d..aebf6af 100644
--- a/frontend/src/components/InputText.jsx
+++ b/frontend/src/components/InputText.jsx
@@ -13,5 +13,6 @@ const InputText = forwardRef(({ value, onChange, tooltip, ...props }, ref) => {
     />
   )
 })
+InputText.displayName = 'InputText'
 
 export default InputText
diff --git a/frontend/src/components/Video.jsx b/frontend/src/components/Video.jsx
index c0c92ae..de4ba79 100644
--- a/frontend/src/components/Video.jsx
+++ b/frontend/src/components/Video.jsx
@@ -68,7 +68,7 @@ const Video = ({ src, style, showMarks, marks = {}, setMarks = () => {} }) => {
   const [videoPosition, setVideoPosition] = useState(0)
   const [videoDuration, setVideoDuration] = useState(0)
   const [trackbarPosition, setTrackbarPosition] = useState(0)
-  const [maximize, setMaximize] = useState(false)
+  //const [maximize, setMaximize] = useState(false)
 
   const frameRate = 25
 
diff --git a/frontend/src/containers/Browser/Browser.jsx b/frontend/src/containers/Browser/Browser.jsx
index edb9b03..8bb5b14 100644
--- a/frontend/src/containers/Browser/Browser.jsx
+++ b/frontend/src/containers/Browser/Browser.jsx
@@ -144,7 +144,9 @@ const BrowserTable = () => {
   }
 
   useEffect(() => {
+    // eslint-disable-next-line no-undef
     const token = PubSub.subscribe('objects_changed', handlePubSub)
+    // eslint-disable-next-line no-undef
     return () => PubSub.unsubscribe(token)
   }, [])
 
diff --git a/frontend/src/containers/Browser/Formatting.jsx b/frontend/src/containers/Browser/Formatting.jsx
index 45f10ba..3a38031 100644
--- a/frontend/src/containers/Browser/Formatting.jsx
+++ b/frontend/src/containers/Browser/Formatting.jsx
@@ -47,12 +47,59 @@ const getColumnWidth = (key) => {
 
 // Field formatters
 
+
+const getDefaultFormatter = (key) =>  {
+  const metaType = nebula.metaType(key)
+  switch (metaType.type) {
+    case 'boolean':
+      // eslint-disable-next-line
+      return (rowData, key) => <td>{rowData[key] ? '✓' : ''}</td>
+
+    case 'datetime':
+      // eslint-disable-next-line
+      return (rowData, key) => (
+        <td>
+          <Timestamp timestamp={rowData[key]} mode={metaType.mode} />{' '}
+        </td>
+      )
+
+    case 'select':
+      // eslint-disable-next-line
+      return (rowData, key) => {
+        if (!metaType.cs) return <td>{rowData[key]}</td>
+
+        const option = nebula
+          .csOptions(metaType.cs)
+          .find((opt) => opt.value === rowData[key])
+
+        return <td>{option?.title}</td>
+      }
+
+    case 'list':
+      // eslint-disable-next-line
+      return (rowData, key) => {
+        if (!metaType.cs) return <td>{rowData[key].join(', ')}</td>
+        const options = nebula
+          .csOptions(metaType.cs)
+          .filter((opt) => rowData[key].includes(opt.value))
+        return <td>{options.map((opt) => opt.title).join(', ')}</td>
+      }
+
+    default:
+      // eslint-disable-next-line
+      return (rowData, key) => <td>{rowData[key]}</td>
+  } // switch metaType
+}
+
+
 const getFormatter = (key) => {
   if (['title', 'subtitle', 'description'].includes(key))
+    // eslint-disable-next-line
     return (rowData, key) => <td>{rowData[key]}</td>
 
   switch (key) {
     case 'qc/state':
+      // eslint-disable-next-line
       return (rowData, key) => (
         <td>
           <QCState className={`qc-state-${rowData[key]}`} />
@@ -60,6 +107,7 @@ const getFormatter = (key) => {
       )
 
     case 'id_folder':
+      // eslint-disable-next-line
       return (rowData, key) => {
         const folder = nebula.settings.folders.find(
           (f) => f.id === rowData[key]
@@ -68,6 +116,7 @@ const getFormatter = (key) => {
       }
 
     case 'duration':
+      // eslint-disable-next-line
       return (rowData, key) => {
         const fps = rowData['video/fps_f'] || 25
         const duration = rowData[key] || 0
@@ -76,51 +125,19 @@ const getFormatter = (key) => {
       }
 
     case 'created_by':
+      // eslint-disable-next-line
       return (rowData, key) => {
         return <td>{nebula.getUserName(rowData[key])}</td>
       }
 
     case 'updated_by':
+      // eslint-disable-next-line
       return (rowData, key) => {
         return <td>{nebula.getUserName(rowData[key])}</td>
       }
 
     default:
-      const metaType = nebula.metaType(key)
-      switch (metaType.type) {
-        case 'boolean':
-          return (rowData, key) => <td>{rowData[key] ? '✓' : ''}</td>
-
-        case 'datetime':
-          return (rowData, key) => (
-            <td>
-              <Timestamp timestamp={rowData[key]} mode={metaType.mode} />{' '}
-            </td>
-          )
-
-        case 'select':
-          return (rowData, key) => {
-            if (!metaType.cs) return <td>{rowData[key]}</td>
-
-            const option = nebula
-              .csOptions(metaType.cs)
-              .find((opt) => opt.value === rowData[key])
-
-            return <td>{option?.title}</td>
-          }
-
-        case 'list':
-          return (rowData, key) => {
-            if (!metaType.cs) return <td>{rowData[key].join(', ')}</td>
-            const options = nebula
-              .csOptions(metaType.cs)
-              .filter((opt) => rowData[key].includes(opt.value))
-            return <td>{options.map((opt) => opt.title).join(', ')}</td>
-          }
-
-        default:
-          return (rowData, key) => <td>{rowData[key]}</td>
-      } // switch metaType
+      return getDefaultFormatter(key)
   } // end switch key
 } // end getFormatter
 
diff --git a/frontend/src/hooks.jsx b/frontend/src/hooks.jsx
index d646df7..ba17c20 100644
--- a/frontend/src/hooks.jsx
+++ b/frontend/src/hooks.jsx
@@ -13,6 +13,7 @@ const useConfirm = () => {
   const [message, setMessage] = useState('')
 
   const confirm = (title, message) =>
+    // eslint-disable-next-line
     new Promise((resolve, reject) => {
       setTitle(title)
       setMessage(message)
diff --git a/frontend/src/pages/JobsPage/JobsPage.jsx b/frontend/src/pages/JobsPage/JobsPage.jsx
index 35812a9..5a04bed 100644
--- a/frontend/src/pages/JobsPage/JobsPage.jsx
+++ b/frontend/src/pages/JobsPage/JobsPage.jsx
@@ -55,6 +55,7 @@ const JobsPage = () => {
     nebula.request('jobs', { abort: id }).then(() => loadJobs())
   }
 
+  // eslint-disable-next-line
   const formatAction = (rowData, key) => {
     if ([0, 1, 5].includes(rowData['status']))
       return (
@@ -168,7 +169,9 @@ const JobsPage = () => {
   } // handlePubSub
 
   useEffect(() => {
+    // eslint-disable-next-line no-undef
     const token = PubSub.subscribe('job_progress', handlePubSub)
+    // eslint-disable-next-line no-undef
     return () => PubSub.unsubscribe(token)
   }, [])