From cbe1f5892dbea30a267a7a00d436be7b587e4596 Mon Sep 17 00:00:00 2001 From: Tobias Lindberg Date: Wed, 19 May 2021 19:54:43 +0200 Subject: [PATCH] changing if false statements and commenting out unused functions --- src/v1_TeslaMateAPICarsCommand.go | 2 +- src/v1_TeslaMateAPICarsLogging.go | 2 +- src/webserver.go | 11 +++++++++-- 3 files changed, 11 insertions(+), 4 deletions(-) diff --git a/src/v1_TeslaMateAPICarsCommand.go b/src/v1_TeslaMateAPICarsCommand.go index 4193500..e179d32 100644 --- a/src/v1_TeslaMateAPICarsCommand.go +++ b/src/v1_TeslaMateAPICarsCommand.go @@ -20,7 +20,7 @@ func TeslaMateAPICarsCommandV1(c *gin.Context) { var err error // check if commands are enabled.. if not we need to abort - if getEnvAsBool("ENABLE_COMMANDS", false) == false { + if !getEnvAsBool("ENABLE_COMMANDS", false) { log.Println("[warning] TeslaMateAPICarsCommandV1 ENABLE_COMMANDS is not true.. returning 403 forbidden.") c.JSON(http.StatusForbidden, gin.H{"error": "You are not allowed to access commands"}) return diff --git a/src/v1_TeslaMateAPICarsLogging.go b/src/v1_TeslaMateAPICarsLogging.go index 27b7e9a..4de9f3f 100644 --- a/src/v1_TeslaMateAPICarsLogging.go +++ b/src/v1_TeslaMateAPICarsLogging.go @@ -19,7 +19,7 @@ func TeslaMateAPICarsLoggingV1(c *gin.Context) { var err error // check if commands are enabled.. if not we need to abort - if getEnvAsBool("ENABLE_COMMANDS", false) == false { + if !getEnvAsBool("ENABLE_COMMANDS", false) { log.Println("[warning] TeslaMateAPICarsLoggingV1 ENABLE_COMMANDS is not true.. returning 403 forbidden.") c.JSON(http.StatusForbidden, gin.H{"error": "You are not allowed to access logging commands"}) return diff --git a/src/webserver.go b/src/webserver.go index b27a269..6bfb462 100644 --- a/src/webserver.go +++ b/src/webserver.go @@ -6,7 +6,6 @@ import ( "log" "net/http" "os" - "reflect" "strconv" "time" @@ -34,7 +33,7 @@ func main() { log.SetFlags(log.Ldate | log.Lmicroseconds) // setting application to ReleaseMode if DEBUG_MODE is false - if getEnvAsBool("DEBUG_MODE", false) == false { + if !getEnvAsBool("DEBUG_MODE", false) { // setting GIN_MODE to ReleaseMode gin.SetMode(gin.ReleaseMode) log.Printf("[info] TeslaMateApi running in release mode.") @@ -201,6 +200,7 @@ func getTimeInTimeZone(datestring string) string { return ReturnDate } +/* func isNil(i interface{}) bool { if i == nil { return true @@ -211,6 +211,7 @@ func isNil(i interface{}) bool { } return false } +*/ // getEnv func - read an environment or return a default value func getEnv(key string, defaultVal string) string { @@ -229,6 +230,7 @@ func getEnvAsBool(name string, defaultVal bool) bool { return defaultVal } +/* // getEnvAsFloat func - read an environment variable into a float64 or return default value func getEnvAsFloat(name string, defaultVal float64) float64 { valStr := getEnv(name, "") @@ -237,6 +239,7 @@ func getEnvAsFloat(name string, defaultVal float64) float64 { } return defaultVal } +*/ // getEnvAsInt func - read an environment variable into integer or return a default value func getEnvAsInt(name string, defaultVal int) int { @@ -288,6 +291,7 @@ func kilometersToMilesNilSupport(km NullFloat64) NullFloat64 { return (km) } +/* // milesToKilometers func func milesToKilometers(mi float64) float64 { return (mi * 1.609344) @@ -298,6 +302,7 @@ func milesToKilometersNilSupport(mi NullFloat64) NullFloat64 { mi.Float64 = (mi.Float64 * 1.609344) return (mi) } +*/ // celsiusToFahrenheit func func celsiusToFahrenheit(c float64) float64 { @@ -310,6 +315,7 @@ func celsiusToFahrenheitNilSupport(c NullFloat64) NullFloat64 { return (c) } +/* // fahrenheitToCelsius func func fahrenheitToCelsius(f float64) float64 { return ((f - 32) * 5 / 9) @@ -320,6 +326,7 @@ func fahrenheitToCelsiusNilSupport(f NullFloat64) NullFloat64 { f.Float64 = ((f.Float64 - 32) * 5 / 9) return (f) } +*/ // checkArrayContainsString func - check if string is inside stringarray func checkArrayContainsString(s []string, e string) bool {