Cleaning up logs

Author: deusprogrammer

src/main/main.ts

@@ -169,9 +169,6 @@ const importZip = async (filePath: string, game: string) => {
         '/'
     )}${subtitleDirectoryStack.join('/')}`;
 
-    console.log(`${sourceVideoDirectory} => ${targetVideoDirectory}`);
-    console.log(`${sourceSubtitlesDirectory} => ${targetSubtitlesDirectory}`);
-
     Object.values(entries).forEach((entry: any) => console.log(entry.name));
 
     // Try with standard directory names
@@ -503,7 +500,7 @@ const installExtensions = async () => {
             extensions.map((name) => installer[name]),
             forceDownload
         )
-        .catch(console.log);
+        .catch(console.error);
 };
 
 const createWindow = async () => {
@@ -585,13 +582,12 @@ app.whenReady()
             if (mainWindow === null) createWindow();
         });
     })
-    .catch(console.log);
+    .catch(console.error);
 
 // Bridged functionality
 
 ipcMain.handle('fileExists', (event, filePath) => {
     const exists = fs.existsSync(filePath);
-    console.log(`${filePath} exists?  ${exists}`);
     return exists;
 });
 
@@ -615,8 +611,6 @@ ipcMain.handle('clipExists', (event, { title, clipNumber, game }) => {
         fs.existsSync(videoFilePath) ||
         fs.existsSync(videoFilePath + '.disabled');
 
-    console.log(`${videoFilePath} exists? ${exists}`);
-
     return exists;
 });
 
@@ -764,7 +758,7 @@ ipcMain.handle('getVideos', (event, game) => {
 });
 
 ipcMain.handle('getVideo', (event, { id, game }) => {
-    console.log('Opening: ' + id + ' from game ' + game);
+    console.log('OPENING: ' + id + ' from game ' + game);
 
     let directory = null;
     if (game === 'rifftrax') {

src/renderer/api/ConfigAPI.js

@@ -0,0 +1,12 @@
+const getConfig = async () => {
+    return await window.api.send('getConfig');
+};
+
+const storeConfig = async (config) => {
+    return await window.api.send('updateConfig', config);
+};
+
+export default {
+    getConfig,
+    storeConfig,
+};

src/renderer/components/SubtitleList.jsx

@@ -42,8 +42,6 @@ export default ({
         setCollections(collections);
     };
 
-    console.log('CURRENT SUB: ' + currentSub);
-
     let currentSubObject = subs[currentSub];
     return (
         <div className="subtitle-window">

src/renderer/components/TimeLine.jsx

@@ -137,7 +137,6 @@ export default ({
                 step={1}
                 max={videoLengthMs}
                 onChange={(e) => {
-                    console.log('OFFSET: ' + offset);
                     onSliderPositionChange(parseFloat(e.target.value) + offset);
                 }}
             />

src/renderer/components/WhatTheDubPlayer.jsx

@@ -25,17 +25,14 @@ export default (props) => {
     });
 
     useEffect(() => {
-        console.log('Video position changed');
         videoElement.current.currentTime = props.videoPosition;
         isTalking = false;
     }, [props.videoPosition]);
 
     useEffect(() => {
         if (props.isPlaying) {
-            console.log('Play');
             videoElement.current.play();
         } else {
-            console.log('Pause');
             videoElement.current.pause();
         }
     }, [props.isPlaying]);
@@ -88,7 +85,6 @@ export default (props) => {
         });
 
         if (index !== currentIndex) {
-            console.log('INDEX CHANGED: ' + index);
             if (isTalking) {
                 video.pause();
                 return;

src/renderer/routes/VideoList.jsx

@@ -2,11 +2,13 @@ import React, { useState, useEffect } from 'react';
 import { Link, useParams } from 'react-router-dom';
 import { toast } from 'react-toastify';
 import BatchAPI from 'renderer/api/BatchAPI';
+import ConfigAPI from 'renderer/api/ConfigAPI';
 
 let VideoList = () => {
     const { game } = useParams();
     const [videos, setVideos] = useState([]);
     const [batchCount, setBatchCount] = useState(0);
+    const [config, setConfig] = useState({});
 
     useEffect(() => {
         loadVideos();
@@ -15,8 +17,10 @@ let VideoList = () => {
     const loadVideos = async () => {
         const videos = await window.api.send('getVideos', game);
         const hasBatch = await BatchAPI.hasBatch();
+        const config = await ConfigAPI.getConfig();
         setVideos(videos);
         setBatchCount(hasBatch);
+        setConfig(config);
     };
 
     const deleteFile = async (id, game, isActive) => {
@@ -32,13 +36,17 @@ let VideoList = () => {
             <Link to={`/create/${game}`}>
                 <button>New Clip</button>
             </Link>
-            <Link to={`/batch/${game}`}>
-                <button>New Batch</button>
-            </Link>
-            {batchCount > 0 ? (
-                <Link to={`/create/${game}?batch=true`}>
-                    <button>Continue Batch ({batchCount})</button>
-                </Link>
+            {config.editor === 'advanced' ? (
+                <>
+                    <Link to={`/batch/${game}`}>
+                        <button>New Batch</button>
+                    </Link>
+                    {batchCount > 0 ? (
+                        <Link to={`/create/${game}?batch=true`}>
+                            <button>Continue Batch ({batchCount})</button>
+                        </Link>
+                    ) : null}
+                </>
             ) : null}
             <h3>Clips</h3>
             {videos.filter((video) => video._id.startsWith('_')).length > 0 ? (

src/renderer/util/VideoTools.js

@@ -13,8 +13,6 @@ export let convertTimestampToSeconds = (timestamp) => {
     let regex = /(\d\d):(\d\d):(\d\d),(\d\d\d)/;
     let match = regex.exec(timestamp);
 
-    console.log('TIMESTAMP: ' + timestamp);
-
     let h = parseInt(match[1]);
     let m = parseInt(match[2]);
     let s = parseInt(match[3]);
@@ -73,7 +71,6 @@ export let convertSrtToSubtitles = (srtBase64) => {
     let srt = atob(srtBase64);
     let n = 0;
     srt.split('\n').forEach((line) => {
-        console.log('LINE: ' + line);
         switch (n++) {
             case 0:
                 break;
@@ -85,7 +82,6 @@ export let convertSrtToSubtitles = (srtBase64) => {
 
                 let startTime = match[1];
                 let endTime = match[2];
-                console.log(startTime + ' => ' + endTime);
                 subtitle.startTime =
                     convertTimestampToSeconds(startTime) * 1000;
                 subtitle.endTime = convertTimestampToSeconds(endTime) * 1000;
@@ -132,8 +128,6 @@ export let convertSubtitlesToWebVtt = (subtitles, substitution, offset = 0) => {
             })
             .join('\n\n');
 
-    console.log(webvtt);
-
     return webvtt;
 };
 
@@ -152,7 +146,6 @@ export let addVideo = async (
     type,
     isBatch
 ) => {
-    console.log('SUBS: ' + convertSubtitlesToSrt(subtitles, type));
     if (isBatch) {
         return await window.api.send('processBatchClip', {
             subtitles: convertSubtitlesToSrt(subtitles, type),