fix: small fixes

`hasFieldValue` should return undefined instead of empty string to reset default folder.
SheffeyG · Oct 7, 2024 · 287f9e7 · 287f9e7
1 parent 3f3f7d5
commit 287f9e7
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 7 deletions.
diff --git a/src/utils/options.ts b/src/utils/options.ts
@@ -40,12 +40,12 @@ export class Options {
     return key in this.#options && this.#options[key] === value;
   }
 
-  getFieldValue(key: string): string {
+  getFieldValue(key: string): string | undefined {
     if (key in this.#options) {
       // Unescape spaces to get real value
       return this.#options[key].replace(/^"(.*)"$/, "$1").replace(/\\ /g, " ");
     }
-    return "";
+    return undefined;
   }
 
   setFieldValue(key: string, value: string): string {

diff --git a/src/utils/patch.tsx b/src/utils/patch.tsx
@@ -48,8 +48,8 @@ const contextMenuPatch = (LibraryContextMenu: any) => {
     if (!patches.inner) {
       patches.inner = afterPatch(component.type.prototype, "shouldComponentUpdate", ([nextProps]: any, shouldUpdate: any) => {
         try {
-          const sgdbIdx = nextProps.children.findIndex((x: any) => x?.key === "cheat-deck");
-          if (sgdbIdx != -1) nextProps.children.splice(sgdbIdx, 1);
+          const idx = nextProps.children.findIndex((x: any) => x?.key === "cheat-deck");
+          if (idx != -1) nextProps.children.splice(idx, 1);
         }
         // eslint-disable-next-line @typescript-eslint/no-unused-vars
         catch (error) {

diff --git a/src/views/Custom/index.tsx b/src/views/Custom/index.tsx
@@ -106,9 +106,7 @@ const Custom: FC<{ appid: number }> = ({ appid }) => {
         cusOptList.map((opt: CustomOption) => (
           // eslint-disable-next-line react/jsx-key
           <Focusable className="CD_EntryContainer">
-            <Focusable
-              className="CD_ToggleContainer"
-            >
+            <Focusable className="CD_ToggleContainer">
               <ToggleField
                 bottomSeparator="none"
                 label={<span className="CD_Label">{opt.label}</span>}