From a3eb3a7f505a5898c4f98e1f425b553aa8031d4a Mon Sep 17 00:00:00 2001
From: ddwag1 <ddwag1@users.noreply.github.com>
Date: Wed, 1 Jan 2020 03:59:39 -0800
Subject: [PATCH 1/5] change "playlist url" to "video url" on /videos page

---
 static/videos.html | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/static/videos.html b/static/videos.html
index 4ef0a53..a3635d6 100644
--- a/static/videos.html
+++ b/static/videos.html
@@ -41,7 +41,7 @@
     </div>
     
     <div class="form-group mt-4">
-      <label for="playlist-url">Playlist URL</label>
+      <label for="playlist-url">Video URL</label>
       <input type="text" class="form-control" id="video-url" placeholder="ex. https://www.youtube.com/watch?v=wOMwO5T3yT4">
     </div>
     <div class="form-group form-inline">
@@ -99,4 +99,4 @@ <h2 class="mt-2">Download History: </h2>
   <script src="/static/js/helpers.js"></script>
   <script src="/static/js/version.js"></script>
 </body>
-</html>
\ No newline at end of file
+</html>

From 44cd70528870419185a28d2cf0b1e8733c2b2b61 Mon Sep 17 00:00:00 2001
From: Mike Kolesnik <mike.kolesnik@gmail.com>
Date: Mon, 6 Jan 2020 14:32:53 +0200
Subject: [PATCH 2/5] Moved updateCheckingInterval to helpers.js

---
 static/index.html      |  1 +
 static/js/helpers.js   | 26 ++++++++++++++++++++++++++
 static/js/index.js     | 27 ---------------------------
 static/js/playlists.js | 28 ----------------------------
 static/playlists.html  |  1 +
 5 files changed, 28 insertions(+), 55 deletions(-)

diff --git a/static/index.html b/static/index.html
index dfc3be9..3d6d8b5 100644
--- a/static/index.html
+++ b/static/index.html
@@ -91,6 +91,7 @@
     <div class="form-group form-inline">
       <label for="checking-interval">Check All Channels Every: </label>
       <input type="number" class="form-control ml-1" id="checking-interval">
+      <input type="hidden" id="list-type" value="Channel">
       <div class="form-group ml-2">
         <select class="form-control" id="time">
           <option value="minutes">minutes</option>
diff --git a/static/js/helpers.js b/static/js/helpers.js
index abae669..3c19261 100644
--- a/static/js/helpers.js
+++ b/static/js/helpers.js
@@ -72,3 +72,29 @@ function setCheckingInterval() {
         return checkingIntervalInput * 1440
     }
 }
+
+function updateCheckingInterval() {
+    startSpinner("update-checking-interval-spinner");
+    let checkingInterval = setCheckingInterval();
+    let type = document.getElementById("list-type").value;
+
+    let interval = {
+      checkingInterval: checkingInterval.toString(),
+      type
+    };
+
+    const options = {
+      method: "POST",
+      body: JSON.stringify(interval),
+      headers: new Headers({
+        "Content-Type": "application/json"
+      })
+    };
+
+    fetch("/api/update-checking-interval", options)
+        .then(res => res.json())
+        .then(res => {
+          handleResponse(res);
+          stopSpinner("update-checking-interval-spinner")
+        });
+}
diff --git a/static/js/index.js b/static/js/index.js
index ed857c7..e723e6e 100644
--- a/static/js/index.js
+++ b/static/js/index.js
@@ -1,32 +1,5 @@
 let channels = [];
 
-function updateCheckingInterval() {
-  startSpinner("update-checking-interval-spinner");
-  let checkingInterval = setCheckingInterval();
-
-  let type = "Channel";
-
-  let interval = {
-    checkingInterval: checkingInterval.toString(),
-    type
-  };
-
-  const options = {
-    method: "POST",
-    body: JSON.stringify(interval),
-    headers: new Headers({
-      "Content-Type": "application/json"
-    })
-  };
-
-  fetch("/api/update-checking-interval", options)
-    .then(res => res.json())
-    .then(res => {
-      handleResponse(res);
-      stopSpinner("update-checking-interval-spinner")
-    });
-}
-
 function addChannel() {
   startSpinner("add-channel-spinner");
   let downloadEntire = document.querySelector('#download-entire-channel').checked;
diff --git a/static/js/playlists.js b/static/js/playlists.js
index 3eb64ee..1b31113 100644
--- a/static/js/playlists.js
+++ b/static/js/playlists.js
@@ -36,34 +36,6 @@ function addPlaylist() {
     });
 }
 
-function updateCheckingInterval() {
-  startSpinner("update-checking-interval-spinner");
-  let checkingInterval = setCheckingInterval();
-
-
-  let type = "Playlist";
-
-  let interval = {
-    checkingInterval: checkingInterval.toString(),
-    type
-  };
-
-  const options = {
-    method: "POST",
-    body: JSON.stringify(interval),
-    headers: new Headers({
-      "Content-Type": "application/json"
-    })
-  };
-
-  fetch("/api/update-checking-interval", options)
-      .then(res => res.json())
-      .then(res => {
-        handleResponse(res)
-        stopSpinner("update-checking-interval-spinner")
-      });
-}
-
 function checkPlaylist(id) {
   startSpinner(id+"-spinner");
   let URL = id;
diff --git a/static/playlists.html b/static/playlists.html
index 23378b0..d2c62f1 100644
--- a/static/playlists.html
+++ b/static/playlists.html
@@ -91,6 +91,7 @@
     <div class="form-group form-inline">
       <label for="checking-interval">Check All Playlists Every: </label>
       <input type="number" class="form-control ml-1" id="checking-interval">
+      <input type="hidden" id="list-type" value="Playlist">
       <div class="form-group ml-2">
         <select class="form-control" id="time">
           <option value="minutes">minutes</option>

From 02060293d82d4ae9462e0ec5bb766d95b7ea5168 Mon Sep 17 00:00:00 2001
From: Mike Kolesnik <mike.kolesnik@gmail.com>
Date: Mon, 6 Jan 2020 16:27:58 +0200
Subject: [PATCH 3/5] Store checking interval time in server

Make calculations of the time multiplier on server not on the client,
this will allow to revtrieve the checking interval later
---
 consts.go                 |  6 ++++++
 database_handlers.go      | 17 +++++++++++++++--
 database_handlers_test.go |  5 ++---
 handlers.go               |  6 +++---
 static/js/helpers.js      | 17 +++--------------
 structs.go                |  6 ++++--
 6 files changed, 33 insertions(+), 24 deletions(-)

diff --git a/consts.go b/consts.go
index b830554..1aec014 100644
--- a/consts.go
+++ b/consts.go
@@ -9,3 +9,9 @@ var (
 	errRes Response
 	okRes  Response
 )
+
+var checkingIntervalMultipliers = map[string]int{
+	"minutes": 1,
+	"hours":   60,
+	"days":    1440,
+}
diff --git a/database_handlers.go b/database_handlers.go
index 5255e3c..267d9d0 100644
--- a/database_handlers.go
+++ b/database_handlers.go
@@ -69,7 +69,7 @@ func (db *Database) load() error {
 	return nil
 }
 
-func UpdateCheckingInterval(file, interval string) error {
+func UpdateCheckingInterval(file, time, intervalStr string) error {
 	log.Info("update checking interval")
 	cf, ok := confs[file]
 	if !ok {
@@ -78,8 +78,17 @@ func UpdateCheckingInterval(file, interval string) error {
 	if len(cf.contents) == 0 {
 		return fmt.Errorf("UpdateCheckingInterval: empty config list")
 	}
+	interval, err := strconv.Atoi(intervalStr)
+	if err != nil {
+		return fmt.Errorf("UpdateCheckingInterval: non-numeric interval %w", err)
+	}
+	_, ok = checkingIntervalMultipliers[time]
+	if !ok {
+		return fmt.Errorf("UpdateCheckingInterval: bad multiplier")
+	}
 	cf.Lock()
 	cf.contents[0].CheckingInterval = interval
+	cf.contents[0].CheckingIntervalTime = time
 	cf.write()
 	cf.Unlock()
 	return nil
@@ -196,5 +205,9 @@ func GetCheckingInterval(file string) (int, error) {
 	if len(cf.contents) == 0 {
 		return 0, fmt.Errorf("GetCheckingInterval: empty target list")
 	}
-	return strconv.Atoi(cf.contents[0].CheckingInterval)
+	target := cf.contents[0]
+	if target.CheckingInterval == 0 {
+		return 0, nil
+	}
+	return target.CheckingInterval * checkingIntervalMultipliers[target.CheckingIntervalTime], nil
 }
diff --git a/database_handlers_test.go b/database_handlers_test.go
index 14c73d3..4c7add4 100644
--- a/database_handlers_test.go
+++ b/database_handlers_test.go
@@ -16,7 +16,6 @@ var targets = []DownloadTarget{
 		PreferredExtensionForVideo: "",
 		DownloadHistory:            nil,
 		LastChecked:                "",
-		CheckingInterval:           "",
 		Type:                       "Channel",
 	},
 	{
@@ -28,7 +27,6 @@ var targets = []DownloadTarget{
 		PreferredExtensionForVideo: "mp4",
 		DownloadHistory:            nil,
 		LastChecked:                "",
-		CheckingInterval:           "",
 		Type:                       "Channel",
 	},
 	{
@@ -40,7 +38,8 @@ var targets = []DownloadTarget{
 		PreferredExtensionForVideo: "mp4",
 		DownloadHistory:            nil,
 		LastChecked:                "",
-		CheckingInterval:           "5",
+		CheckingInterval:           5,
+		CheckingIntervalTime:       "minutes",
 		Type:                       "Playlist",
 	},
 }
diff --git a/handlers.go b/handlers.go
index 47e8043..4798434 100644
--- a/handlers.go
+++ b/handlers.go
@@ -73,9 +73,9 @@ func HandleAddTarget(w http.ResponseWriter, r *http.Request) {
 		}
 
 		if targetData.DownloadMode == "Audio Only" {
-			target = DownloadTarget{URL: targetData.URL, DownloadMode: targetData.DownloadMode, Name: targetMetadata.Playlist, PreferredExtensionForAudio: targetData.FileExtension, DownloadHistory: []string{}, LastChecked: time.Now().Format("01-02-2006 15:04:05"), CheckingInterval: "", Type: targetData.Type, DownloadPath: targetData.DownloadPath}
+			target = DownloadTarget{URL: targetData.URL, DownloadMode: targetData.DownloadMode, Name: targetMetadata.Playlist, PreferredExtensionForAudio: targetData.FileExtension, DownloadHistory: []string{}, LastChecked: time.Now().Format("01-02-2006 15:04:05"), Type: targetData.Type, DownloadPath: targetData.DownloadPath}
 		} else if targetData.DownloadMode == "Video And Audio" {
-			target = DownloadTarget{URL: targetData.URL, DownloadMode: targetData.DownloadMode, Name: targetMetadata.Playlist, PreferredExtensionForVideo: targetData.FileExtension, DownloadHistory: []string{}, LastChecked: time.Now().Format("01-02-2006 15:04:05"), CheckingInterval: "", Type: targetData.Type, DownloadPath: targetData.DownloadPath}
+			target = DownloadTarget{URL: targetData.URL, DownloadMode: targetData.DownloadMode, Name: targetMetadata.Playlist, PreferredExtensionForVideo: targetData.FileExtension, DownloadHistory: []string{}, LastChecked: time.Now().Format("01-02-2006 15:04:05"), Type: targetData.Type, DownloadPath: targetData.DownloadPath}
 		}
 
 		err = target.Download(targetData.DownloadQuality, targetData.FileExtension, targetData.DownloadEntire)
@@ -221,7 +221,7 @@ func HandleUpdateCheckingInterval(w http.ResponseWriter, r *http.Request) {
 		ReturnResponse(w, errRes)
 		return
 	}
-	err = UpdateCheckingInterval(interval.Type, interval.CheckingInterval)
+	err = UpdateCheckingInterval(interval.Type, interval.Time, interval.CheckingInterval)
 	if err != nil {
 		errRes = Response{Type: "Error", Key: "ERROR_UPDATING_CHECKING_INTERVAL", Message: "There was an updating the checking interval: " + err.Error()}
 		ReturnResponse(w, errRes)
diff --git a/static/js/helpers.js b/static/js/helpers.js
index 3c19261..7368f1d 100644
--- a/static/js/helpers.js
+++ b/static/js/helpers.js
@@ -60,26 +60,15 @@ function displayDownloadHistory(channelName, downloadHistory) {
     })
 }
 
-function setCheckingInterval() {
-    let checkingIntervalInput = document.getElementById("checking-interval").value;
-    let time = document.getElementById("time").value;
-
-    if (time === "minutes") {
-        return checkingIntervalInput
-    } else if (time === "hours") {
-        return  checkingIntervalInput * 60
-    } else if (time === "days") {
-        return checkingIntervalInput * 1440
-    }
-}
-
 function updateCheckingInterval() {
     startSpinner("update-checking-interval-spinner");
-    let checkingInterval = setCheckingInterval();
+    let checkingInterval = document.getElementById("checking-interval").value;
+    let time = document.getElementById("time").value;
     let type = document.getElementById("list-type").value;
 
     let interval = {
       checkingInterval: checkingInterval.toString(),
+      time: time,
       type
     };
 
diff --git a/structs.go b/structs.go
index cfce8ab..148683e 100644
--- a/structs.go
+++ b/structs.go
@@ -140,7 +140,8 @@ type DownloadTarget struct {
 	PreferredExtensionForVideo string
 	DownloadHistory            []string
 	LastChecked                string
-	CheckingInterval           string
+	CheckingInterval           int
+	CheckingIntervalTime       string
 	Type                       string
 	DownloadPath               string
 }
@@ -156,6 +157,7 @@ type DeleteTargetPayload struct {
 
 type CheckingIntervalPayload struct {
 	CheckingInterval string
+	Time             string
 	Type             string
 }
 
@@ -206,4 +208,4 @@ type Database struct {
 	file     string
 	contents []DownloadTarget
 	lookup   map[string]*DownloadTarget
-}
\ No newline at end of file
+}

From a329749b8ab87cfae16b4b8dfe899ff2bb937b43 Mon Sep 17 00:00:00 2001
From: Mike Kolesnik <mike.kolesnik@gmail.com>
Date: Mon, 6 Jan 2020 15:42:24 +0200
Subject: [PATCH 4/5] Return checking interval

---
 database_handlers.go  | 21 +++++++++++++++------
 handlers.go           | 25 +++++++++++++++++++++++++
 main.go               |  1 +
 static/index.html     |  2 +-
 static/js/helpers.js  | 23 +++++++++++++++++++++++
 static/playlists.html |  2 +-
 structs.go            |  4 ++++
 7 files changed, 70 insertions(+), 8 deletions(-)

diff --git a/database_handlers.go b/database_handlers.go
index 267d9d0..74fe75b 100644
--- a/database_handlers.go
+++ b/database_handlers.go
@@ -196,18 +196,27 @@ func (t DownloadTarget) GetFromDatabase() (out DownloadTarget, err error) {
 }
 
 func GetCheckingInterval(file string) (int, error) {
+	interval, time, err := GetCheckingIntervalConfig(file)
+	if err != nil {
+		return 0, err
+	}
+
+	if interval == 0 {
+		return 0, nil
+	}
+	return interval * checkingIntervalMultipliers[time], nil
+}
+
+func GetCheckingIntervalConfig(file string) (int, string, error) {
 	cf, ok := confs[file]
 	if !ok {
-		return 0, fmt.Errorf("GetFromDatabase: bad conf type")
+		return 0, "", fmt.Errorf("GetFromDatabase: bad conf type")
 	}
 	cf.RLock()
 	defer cf.RUnlock()
 	if len(cf.contents) == 0 {
-		return 0, fmt.Errorf("GetCheckingInterval: empty target list")
+		return 0, "", fmt.Errorf("GetCheckingInterval: empty target list")
 	}
 	target := cf.contents[0]
-	if target.CheckingInterval == 0 {
-		return 0, nil
-	}
-	return target.CheckingInterval * checkingIntervalMultipliers[target.CheckingIntervalTime], nil
+	return target.CheckingInterval, target.CheckingIntervalTime, nil
 }
diff --git a/handlers.go b/handlers.go
index 4798434..c60e85d 100644
--- a/handlers.go
+++ b/handlers.go
@@ -3,6 +3,7 @@ package main
 import (
 	"encoding/json"
 	"net/http"
+	"strconv"
 	"strings"
 	"time"
 
@@ -210,6 +211,30 @@ func HandleCheckAllTargets(w http.ResponseWriter, r *http.Request) {
 	return
 }
 
+func HandleGetCheckingInterval(w http.ResponseWriter, r *http.Request) {
+	log.Info("received a request to get the checking interval")
+	w.Header().Set("Content-Type", "application/json")
+	var getCheckingInterval GetCheckingIntervalPayload
+	err := json.NewDecoder(r.Body).Decode(&getCheckingInterval)
+	if err != nil {
+		ReturnResponse(w, Response{Type: "Error", Key: "ERROR_PARSING_DATA", Message: "There was an error parsing json: " + err.Error()})
+		return
+	}
+	interval, time, err := GetCheckingIntervalConfig(getCheckingInterval.Type)
+	if err != nil {
+		ReturnResponse(w, Response{Type: "Error", Key: "ERROR_GETTING_CHECKING_INTERVAL", Message: "There was an error while getting the checking interval: " + err.Error()})
+		return
+	}
+	result := CheckingIntervalPayload{
+		CheckingInterval: strconv.Itoa(interval),
+		Time:             time,
+		Type:             getCheckingInterval.Type,
+	}
+	json.NewEncoder(w).Encode(result)
+	log.Info("Returning checking-interval %v", result)
+	return
+}
+
 func HandleUpdateCheckingInterval(w http.ResponseWriter, r *http.Request) {
 	log.Info("received a request to update the checking interval")
 	w.Header().Set("Content-Type", "application/json")
diff --git a/main.go b/main.go
index 1ae9d9c..dbfdc76 100644
--- a/main.go
+++ b/main.go
@@ -103,6 +103,7 @@ func main() {
 	r.HandleFunc("/api/get", HandleGetTargets).Methods("POST")
 	r.HandleFunc("/api/check-all", HandleCheckAllTargets).Methods("POST")
 
+	r.HandleFunc("/api/get-checking-interval", HandleGetCheckingInterval).Methods("POST")
 	r.HandleFunc("/api/update-checking-interval", HandleUpdateCheckingInterval).Methods("POST")
 
 	r.HandleFunc("/api/get-videos", HandleGetVideos).Methods("GET")
diff --git a/static/index.html b/static/index.html
index 3d6d8b5..232162d 100644
--- a/static/index.html
+++ b/static/index.html
@@ -8,7 +8,7 @@
   <title>GoAutoYT - Dashboard</title>
   
 </head>
-<body onload="getChannels()">
+<body onload="getChannels(); getCheckingInterval();">
   <ul class="nav nav-pills justify-content-center mt-3">
     <li class="nav-item">
       <a class="nav-link active" href="#">Channels</a>
diff --git a/static/js/helpers.js b/static/js/helpers.js
index 7368f1d..23fbbee 100644
--- a/static/js/helpers.js
+++ b/static/js/helpers.js
@@ -60,6 +60,29 @@ function displayDownloadHistory(channelName, downloadHistory) {
     })
 }
 
+function getCheckingInterval() {
+    let type = document.getElementById("list-type").value;
+
+    let getCheckingInterval = {
+        type
+    };
+
+    const options = {
+        method: "POST",
+        body: JSON.stringify(getCheckingInterval),
+        headers: new Headers({
+            "Content-Type": "application/json"
+        })
+    };
+
+    fetch("/api/get-checking-interval", options)
+        .then(res => res.json())
+        .then(interval => {
+            document.getElementById("checking-interval").value = interval.CheckingInterval
+            document.getElementById("time").value = interval.Time
+        });
+}
+
 function updateCheckingInterval() {
     startSpinner("update-checking-interval-spinner");
     let checkingInterval = document.getElementById("checking-interval").value;
diff --git a/static/playlists.html b/static/playlists.html
index d2c62f1..65b2607 100644
--- a/static/playlists.html
+++ b/static/playlists.html
@@ -8,7 +8,7 @@
   <title>GoAutoYT - Dashboard</title>
   
 </head>
-<body onload="getPlaylists()">
+<body onload="getPlaylists(); getCheckingInterval();">
   <ul class="nav nav-pills justify-content-center mt-3">
     <li class="nav-item">
       <a class="nav-link" href="/">Channels</a>
diff --git a/structs.go b/structs.go
index 148683e..2b5a26b 100644
--- a/structs.go
+++ b/structs.go
@@ -155,6 +155,10 @@ type DeleteTargetPayload struct {
 	Type string
 }
 
+type GetCheckingIntervalPayload struct {
+	Type string
+}
+
 type CheckingIntervalPayload struct {
 	CheckingInterval string
 	Time             string

From 6bd30d5ba4d6f23b3c6ef40f3f7e0b7c051129c4 Mon Sep 17 00:00:00 2001
From: Mike Kolesnik <mike.kolesnik@gmail.com>
Date: Mon, 6 Jan 2020 17:42:01 +0200
Subject: [PATCH 5/5] fix logging

---
 handlers.go | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/handlers.go b/handlers.go
index c60e85d..7c8c721 100644
--- a/handlers.go
+++ b/handlers.go
@@ -231,7 +231,7 @@ func HandleGetCheckingInterval(w http.ResponseWriter, r *http.Request) {
 		Type:             getCheckingInterval.Type,
 	}
 	json.NewEncoder(w).Encode(result)
-	log.Info("Returning checking-interval %v", result)
+	log.Info("Returning checking-interval: ", result)
 	return
 }