Adding proper use of url package in Http package and more fixes

imdb.go

@@ -20,15 +20,14 @@ import (
 	"encoding/json"
 	"errors"
 	"fmt"
-	//"log"
 	"net/http"
-	"strings"
+	"net/url"
 )
 
 //=======================================================================
 //							Const
 //=======================================================================
-const baseUrl string = "http://www.omdbapi.com/?"
+const baseUrl string = "http://www.omdbapi.com"
 const plot string = "full"
 const tomatoes string = "true"
 
@@ -165,20 +164,31 @@ func omdbApiRequest(s string, i string, t string, y string) (resp *http.Response
 	//t = Title Parameter, if this is != nil then its a getMovieByTitle
 	//y = Year Parameter, Optional data for s and t search
 	//var res http.Response
-	var url string
+	var Url *url.URL
-	if s != "" {
+	Url, err = url.Parse(baseUrl)
-		s = strings.Replace(s, " ", "%20", -1)
+
-		url = fmt.Sprintf(baseUrl+"s=%s&y=%s", s, y)
+	if err != nil {
-	} else if i != "" {
+		return nil, err
-		url = fmt.Sprintf(baseUrl+"i=%s&plot=%s&tomatoes=%s", i, plot, tomatoes)
+	}
-	} else if t != "" {
+	Url.Path += "/"
-		t = strings.Replace(t, " ", "%20", -1)
+	parameters := url.Values{}
-		url = fmt.Sprintf(baseUrl+"t=%s&plot=%s&tomatoes=%s&y=%s", t, plot, tomatoes, y)
+	if len(s) > 0 {
+		parameters.Add("s", s)
+		parameters.Add("y", y)
+	} else if len(i) > 0 {
+		parameters.Add("i", i)
+		parameters.Add("plot", plot)
+		parameters.Add("tomatoes", tomatoes)
+	} else if len(t) > 0 {
+		parameters.Add("t", t)
+		parameters.Add("plot", plot)
+		parameters.Add("tomatoes", tomatoes)
+		parameters.Add("y", y)
 	} else {
 		return nil, errors.New("Invalid Request")
 	}
-	//log.Print(url) //DEBUG
+	Url.RawQuery = parameters.Encode()
-	res, err := http.Get(url)
+	res, err := http.Get(Url.String())
 	err = checkErrorStatus(res.StatusCode)
 	if err != nil {
 		return nil, err