Revert "rfac: changed logic for checking valid paths"

This reverts commit 8934f5d.

conf/constants.go

@@ -58,14 +58,3 @@ const (
 	StatusMemoryLimitExceeded = "MLE"
 	StatusPartial             = "PTL"
 )
-
-var ValidPaths = []string{"username", "fullname", "handle.codechef", "handle.codeforces", "handle.hackerearth", "handle.hackerrank", "handle.spoj"}
-
-func IsPathValid(s string) bool {
-	for _, path := range ValidPaths {
-		if path == s {
-			return true
-		}
-	}
-	return false
-}

controllers/search.go

@@ -6,7 +6,6 @@ import (
 	"strconv"
 
 	"github.com/getsentry/sentry-go"
-	. "github.com/mdg-iitr/Codephile/conf"
 	. "github.com/mdg-iitr/Codephile/errors"
 	"github.com/mdg-iitr/Codephile/models"
 )
@@ -37,9 +36,16 @@ func (u *UserController) Search() {
 	}
 	path := u.GetString("path")
 	//Checking for possible fields, default is "username"
-	if !IsPathValid(path) {
+	var possible_path []string
+	var good_path bool = false
+	possible_path = []string{"username", "fullname", "handle.codechef", "handle.codeforces", "handle.hackerearth", "handle.hackerrank", "handle.spoj"}
+	for _, itr := range possible_path {
+		if itr == path {
+			good_path = true
+		}
+	}
+	if !good_path {
 		path = "username"
-		return
 	}
 	results, err := models.SearchUser(query, c, path)
 	if err != nil {

routers/router.go

@@ -9,14 +9,13 @@
 package routers
 
 import (
-	"net/http"
-	"os"
-	"path"
-
 	"github.com/astaxie/beego"
 	"github.com/astaxie/beego/context"
 	"github.com/mdg-iitr/Codephile/controllers"
 	"github.com/mdg-iitr/Codephile/middleware"
+	"net/http"
+	"os"
+	"path"
 )
 
 func init() {