[CLEANUP] Remove Debug Logs, Outdated TODOs, and Unused Props (#754)

src/components/CodeEditor/index.jsx

@@ -20,7 +20,7 @@ export default function CodeEditor({
       .writeText(props.value)
       .then(() => Toast.success(t("copied_to_clipboard")))
       .catch((e) => {
-        console.log(e);
+        console.error(e);
       });
   };
 

src/components/EditorHeader/ControlPanel.jsx

@@ -806,7 +806,7 @@ export default function ControlPanel({
         }
       })
       .catch((error) => {
-        console.log(error);
+        console.error(error);
         Toast.error(t("didnt_find_diagram"));
       });
   };

src/components/EditorHeader/Modal/Modal.jsx

@@ -156,7 +156,7 @@ export default function Modal({
         }
       })
       .catch((error) => {
-        console.log(error);
+        console.error(error);
         Toast.error(t("didnt_find_diagram"));
       });
   };

src/components/EditorSidePanel/TypesTab/TypeInfo.jsx

@@ -15,16 +15,15 @@ import TypeField from "./TypeField";
 import { useTranslation } from "react-i18next";
 import { nanoid } from "nanoid";
 
-export default function TypeInfo({ index, data }) {
+export default function TypeInfo({ data }) {
   const { layout } = useLayout();
   const { deleteType, updateType } = useTypes();
   const { tables, updateField } = useDiagram();
   const { setUndoStack, setRedoStack } = useUndoRedo();
   const [editField, setEditField] = useState({});
   const { t } = useTranslation();
 
-  // TODO: remove indexes, not a valid case after adding id to types
-  const typeId = data.id ?? index;
+  const typeId = data.id;
 
   return (
     <div id={`scroll_type_${typeId}`}>
@@ -34,7 +33,7 @@ export default function TypeInfo({ index, data }) {
             {data.name}
           </div>
         }
-        itemKey={`${index}`}
+        itemKey={`${data.id}`}
       >
         <div className="flex items-center mb-2.5">
           <div className="text-md font-semibold break-keep">{t("name")}: </div>
@@ -90,7 +89,7 @@ export default function TypeInfo({ index, data }) {
           />
         </div>
         {data.fields.map((f, j) => (
-          <TypeField key={j} data={f} fid={j} tid={index} />
+          <TypeField key={j} data={f} fid={j} tid={data.id} />
         ))}
         <Card
           bodyStyle={{ padding: "4px" }}

src/components/Workspace.jsx

@@ -227,7 +227,7 @@ export default function WorkSpace() {
           }
         })
         .catch((error) => {
-          console.log(error);
+          console.error(error);
         });
     };
 
@@ -288,7 +288,7 @@ export default function WorkSpace() {
           }
         })
         .catch((error) => {
-          console.log(error);
+          console.error(error);
         });
     };
 
@@ -346,7 +346,7 @@ export default function WorkSpace() {
           }
         })
         .catch((error) => {
-          console.log(error);
+          console.error(error);
           if (selectedDb === "") setShowSelectDbModal(true);
         });
     };
@@ -393,7 +393,7 @@ export default function WorkSpace() {
           );
         }
       } catch (e) {
-        console.log(e);
+        console.error(e);
         setSaveState(State.FAILED_TO_LOAD);
       }
     };

src/data/db.js

@@ -9,5 +9,5 @@ db.version(6).stores({
 });
 
 db.on("populate", (transaction) => {
-  transaction.templates.bulkAdd(templateSeeds).catch((e) => console.log(e));
+  transaction.templates.bulkAdd(templateSeeds).catch((e) => console.error(e));
 });

src/utils/exportAs/dbml.js

@@ -99,7 +99,6 @@ function columnComment(field) {
 }
 
 function processType(type) {
-  // TODO: remove after a while
   if (type.toUpperCase() === "TIMESTAMP WITH TIME ZONE") {
     return "timestamptz";
   }