adjust: move ParseURL from models into config, and rename it into ParseConfigURL

Author: SysIntruder

Diff for: helpers/config.go

@@ -1,11 +1,14 @@
 package helpers
 
 import (
+	"fmt"
+	"net/url"
 	"os"
 	"path/filepath"
 
 	"github.com/pelletier/go-toml/v2"
 
+	"github.com/jorgerojas26/lazysql/drivers"
 	"github.com/jorgerojas26/lazysql/models"
 )
 
@@ -26,14 +29,43 @@ func LoadConfig() (Config, error) {
 		return config, err
 	}
 
-	for idx, cfg := range config.Connections {
-		cfg.ParseURL()
-		config.Connections[idx].URL = cfg.URL
+	for idx, conn := range config.Connections {
+		config.Connections[idx].URL = ParseConfigURL(&conn)
 	}
 
 	return config, nil
 }
 
+// ParseConfigURL will manually parse config url if url empty
+//
+// it main purpose is for handling username & password with special characters
+//
+// only sqlserver for now
+func ParseConfigURL(conn *models.Connection) string {
+	if conn.URL != "" {
+		return conn.URL
+	}
+
+	// only sqlserver for now
+	if conn.Provider != drivers.DriverMSSQL {
+		return conn.URL
+	}
+
+	user := url.QueryEscape(conn.Username)
+	pass := url.QueryEscape(conn.Password)
+
+	return fmt.Sprintf(
+		"%s://%s:%s@%s:%s?database=%s%s",
+		conn.Provider,
+		user,
+		pass,
+		conn.Hostname,
+		conn.Port,
+		conn.DBName,
+		conn.URLParams,
+	)
+}
+
 func LoadConnections() (connections []models.Connection, err error) {
 	config, err := LoadConfig()
 	if err != nil {

Diff for: models/models.go

@@ -1,9 +1,6 @@
 package models
 
 import (
-	"fmt"
-	"net/url"
-
 	"github.com/rivo/tview"
 )
 
@@ -25,38 +22,6 @@ type Connection struct {
 	Commands []*Command
 }
 
-// ParseURL will manually parse url if url empty
-//
-// for handling username & password with special characters
-//
-// only sqlserver for now
-//
-// need to refactor if wanted to reuse driver list in drivers/constants.go
-func (c *Connection) ParseURL() {
-	if c.URL != "" {
-		return
-	}
-
-	// only sqlserver for now
-	if c.Provider != "sqlserver" {
-		return
-	}
-
-	user := url.QueryEscape(c.Username)
-	pass := url.QueryEscape(c.Password)
-
-	c.URL = fmt.Sprintf(
-		"%s://%s:%s@%s:%s?database=%s%s",
-		c.Provider,
-		user,
-		pass,
-		c.Hostname,
-		c.Port,
-		c.DBName,
-		c.URLParams,
-	)
-}
-
 type Command struct {
 	Command     string
 	WaitForPort string