golint

adapter/beego/beego.go

@@ -98,7 +98,7 @@ func (bee *Beego) Content(contextInterface interface{}, c types.GetPanel) {
 		return
 	}
 
-	user, ok := auth.GetCurUserById(int64(userID.(float64)))
+	user, ok := auth.GetCurUserByID(int64(userID.(float64)))
 
 	if !ok {
 		ctx.Redirect(http.StatusFound, globalConfig.Url("/login"))
@@ -140,7 +140,7 @@ func (bee *Beego) Content(contextInterface interface{}, c types.GetPanel) {
 
 	buf := new(bytes.Buffer)
 	err = tmpl.ExecuteTemplate(buf, tmplName, types.NewPage(user,
-		*(menu.GetGlobalMenu(user).SetActiveClass(globalConfig.UrlRemovePrefix(ctx.Request.URL.String()))),
+		*(menu.GetGlobalMenu(user).SetActiveClass(globalConfig.URLRemovePrefix(ctx.Request.URL.String()))),
 		panel, globalConfig, template.GetComponentAssetListsHTML()))
 	if err != nil {
 		logger.Error("Beego Content", err)

adapter/buffalo/buffalo.go

@@ -144,7 +144,7 @@ func (bu *Buffalo) Content(contextInterface interface{}, c types.GetPanel) {
 		return
 	}
 
-	user, ok := auth.GetCurUserById(int64(userID.(float64)))
+	user, ok := auth.GetCurUserByID(int64(userID.(float64)))
 
 	if !ok {
 		_ = ctx.Redirect(http.StatusFound, globalConfig.Url("/login"))
@@ -185,7 +185,7 @@ func (bu *Buffalo) Content(contextInterface interface{}, c types.GetPanel) {
 
 	buf := new(bytes.Buffer)
 	err = tmpl.ExecuteTemplate(buf, tmplName, types.NewPage(user,
-		*(menu.GetGlobalMenu(user).SetActiveClass(globalConfig.UrlRemovePrefix(ctx.Request().URL.String()))),
+		*(menu.GetGlobalMenu(user).SetActiveClass(globalConfig.URLRemovePrefix(ctx.Request().URL.String()))),
 		panel, globalConfig, template.GetComponentAssetListsHTML()))
 	if err != nil {
 		logger.Error("Buffalo Content", err)

adapter/chi/chi.go

@@ -152,7 +152,7 @@ func (bu *Chi) Content(contextInterface interface{}, c types.GetPanel) {
 		return
 	}
 
-	user, ok := auth.GetCurUserById(int64(userID.(float64)))
+	user, ok := auth.GetCurUserByID(int64(userID.(float64)))
 
 	if !ok {
 		http.Redirect(ctx.Response, ctx.Request, config.Url("/login"), http.StatusFound)
@@ -191,7 +191,7 @@ func (bu *Chi) Content(contextInterface interface{}, c types.GetPanel) {
 
 	buf := new(bytes.Buffer)
 	err = tmpl.ExecuteTemplate(buf, tmplName, types.NewPage(user,
-		*(menu.GetGlobalMenu(user).SetActiveClass(config.UrlRemovePrefix(ctx.Request.URL.String()))),
+		*(menu.GetGlobalMenu(user).SetActiveClass(config.URLRemovePrefix(ctx.Request.URL.String()))),
 		panel, config, template.GetComponentAssetListsHTML()))
 	if err != nil {
 		logger.Error("Chi Content", err)

adapter/echo/echo.go

@@ -99,7 +99,7 @@ func (e *Echo) Content(contextInterface interface{}, c types.GetPanel) {
 		return
 	}
 
-	user, ok := auth.GetCurUserById(int64(userID.(float64)))
+	user, ok := auth.GetCurUserByID(int64(userID.(float64)))
 
 	if !ok {
 		_ = ctx.Redirect(http.StatusFound, globalConfig.Url("/login"))
@@ -138,7 +138,7 @@ func (e *Echo) Content(contextInterface interface{}, c types.GetPanel) {
 
 	buf := new(bytes.Buffer)
 	err = tmpl.ExecuteTemplate(buf, tmplName, types.NewPage(user,
-		*(menu.GetGlobalMenu(user).SetActiveClass(globalConfig.UrlRemovePrefix(ctx.Request().URL.String()))),
+		*(menu.GetGlobalMenu(user).SetActiveClass(globalConfig.URLRemovePrefix(ctx.Request().URL.String()))),
 		panel, globalConfig, template.GetComponentAssetListsHTML()))
 	if err != nil {
 		logger.Error("Echo Content", err)

adapter/fasthttp/fasthttp.go

@@ -164,7 +164,7 @@ func (fast *Fasthttp) Content(contextInterface interface{}, c types.GetPanel) {
 		return
 	}
 
-	user, ok := auth.GetCurUserById(int64(userID.(float64)))
+	user, ok := auth.GetCurUserByID(int64(userID.(float64)))
 
 	if !ok {
 		ctx.Redirect(globalConfig.Url("/login"), http.StatusFound)
@@ -206,7 +206,7 @@ func (fast *Fasthttp) Content(contextInterface interface{}, c types.GetPanel) {
 
 	buf := new(bytes.Buffer)
 	err = tmpl.ExecuteTemplate(buf, tmplName,
-		types.NewPage(user, *(menu.GetGlobalMenu(user).SetActiveClass(globalConfig.UrlRemovePrefix(ctx.Request.URI().String()))),
+		types.NewPage(user, *(menu.GetGlobalMenu(user).SetActiveClass(globalConfig.URLRemovePrefix(ctx.Request.URI().String()))),
 			panel, globalConfig, template.GetComponentAssetListsHTML()))
 	if err != nil {
 		logger.Error("Fasthttp Content", err)

adapter/gin/gin.go

@@ -102,7 +102,7 @@ func (gins *Gin) Content(contextInterface interface{}, c types.GetPanel) {
 		return
 	}
 
-	user, ok := auth.GetCurUserById(int64(userID.(float64)))
+	user, ok := auth.GetCurUserByID(int64(userID.(float64)))
 
 	if !ok {
 		ctx.Redirect(http.StatusFound, globalConfig.Url("/login"))
@@ -142,7 +142,7 @@ func (gins *Gin) Content(contextInterface interface{}, c types.GetPanel) {
 
 	buf := new(bytes.Buffer)
 	err = tmpl.ExecuteTemplate(buf, tmplName, types.NewPage(user,
-		*(menu.GetGlobalMenu(user).SetActiveClass(globalConfig.UrlRemovePrefix(ctx.Request.URL.String()))),
+		*(menu.GetGlobalMenu(user).SetActiveClass(globalConfig.URLRemovePrefix(ctx.Request.URL.String()))),
 		panel, globalConfig, template.GetComponentAssetListsHTML()))
 	if err != nil {
 		logger.Error("Gin Content", err)

adapter/gorilla/gorilla.go

@@ -126,7 +126,7 @@ func (g *Gorilla) Content(contextInterface interface{}, c types.GetPanel) {
 		return
 	}
 
-	user, ok := auth.GetCurUserById(int64(userID.(float64)))
+	user, ok := auth.GetCurUserByID(int64(userID.(float64)))
 
 	if !ok {
 		http.Redirect(ctx.Response, ctx.Request, globalConfig.Url("/login"), http.StatusFound)
@@ -165,7 +165,7 @@ func (g *Gorilla) Content(contextInterface interface{}, c types.GetPanel) {
 
 	buf := new(bytes.Buffer)
 	err = tmpl.ExecuteTemplate(buf, tmplName, types.NewPage(user,
-		*(menu.GetGlobalMenu(user).SetActiveClass(globalConfig.UrlRemovePrefix(ctx.Request.URL.String()))),
+		*(menu.GetGlobalMenu(user).SetActiveClass(globalConfig.URLRemovePrefix(ctx.Request.URL.String()))),
 		panel, globalConfig, template.GetComponentAssetListsHTML()))
 	if err != nil {
 		logger.Error("Gorilla Content", err)

adapter/iris/iris.go

@@ -107,7 +107,7 @@ func (is *Iris) Content(contextInterface interface{}, c types.GetPanel) {
 		return
 	}
 
-	user, ok := auth.GetCurUserById(int64(userID.(float64)))
+	user, ok := auth.GetCurUserByID(int64(userID.(float64)))
 
 	if !ok {
 		ctx.Redirect(globalConfig.Url("/login"), http.StatusFound)
@@ -149,7 +149,7 @@ func (is *Iris) Content(contextInterface interface{}, c types.GetPanel) {
 
 	buf := new(bytes.Buffer)
 	err = tmpl.ExecuteTemplate(buf, tmplName,
-		types.NewPage(user, *(menu.GetGlobalMenu(user).SetActiveClass(globalConfig.UrlRemovePrefix(ctx.Request().URL.String()))),
+		types.NewPage(user, *(menu.GetGlobalMenu(user).SetActiveClass(globalConfig.URLRemovePrefix(ctx.Request().URL.String()))),
 			panel, globalConfig, template.GetComponentAssetListsHTML()))
 	if err != nil {
 		logger.Error("Iris Content", err)

context/context.go

@@ -29,7 +29,7 @@ type Context struct {
 	handlers  Handlers
 }
 
-// Path is used in the matching of request and response. URL stores the
+// Path is used in the matching of request and response. Url stores the
 // raw register url. RegUrl contains the wildcard which on behalf of
 // the route params.
 type Path struct {

examples/datamodel/mysql_types.go

@@ -57,7 +57,7 @@ func GetAllTypesTable() table.Table {
 	info.AddField("Type_40", "type_40", db.Multipoint)
 	info.AddField("Type_42", "type_42", db.Geometrycollection)
 	info.AddField("Type_50", "type_50", db.Double)
-	info.AddField("Type_51", "type_51", db.Json)
+	info.AddField("Type_51", "type_51", db.JSON)
 
 	info.SetTable("all_types").SetTitle("All_types").SetDescription("All_types")
 
@@ -107,7 +107,7 @@ func GetAllTypesTable() table.Table {
 	formList.AddField("Type_40", "type_40", db.Multipoint, form.Text)
 	formList.AddField("Type_42", "type_42", db.Geometrycollection, form.Text)
 	formList.AddField("Type_50", "type_50", db.Double, form.Text)
-	formList.AddField("Type_51", "type_51", db.Json, form.Text)
+	formList.AddField("Type_51", "type_51", db.JSON, form.Text)
 
 	formList.SetTable("all_types").SetTitle("All_types").SetDescription("All_types")
 

modules/auth/auth.go

@@ -38,7 +38,7 @@ func Check(password string, username string) (user models.UserModel, ok bool) {
 
 func comparePassword(comPwd, pwdHash string) bool {
 	err := bcrypt.CompareHashAndPassword([]byte(pwdHash), []byte(comPwd))
-	return err != nil
+	return err == nil
 }
 
 // EncodePassword encode the password.
@@ -52,7 +52,7 @@ func EncodePassword(pwd []byte) string {
 
 // SetCookie set the cookie.
 func SetCookie(ctx *context.Context, user models.UserModel) bool {
-	InitSession(ctx).Set("user_id", user.Id)
+	InitSession(ctx).Add("user_id", user.Id)
 	return true
 }
 
@@ -73,6 +73,7 @@ var (
 	CsrfTokenLock sync.Mutex
 )
 
+// AddToken add the token to the CSRFToken.
 func (csrf *CSRFToken) AddToken() string {
 	CsrfTokenLock.Lock()
 	defer CsrfTokenLock.Unlock()
@@ -81,6 +82,8 @@ func (csrf *CSRFToken) AddToken() string {
 	return tokenStr
 }
 
+// CheckToken check the given token with tokens in the CSRFToken, if exist
+// return true.
 func (csrf *CSRFToken) CheckToken(toCheckToken string) bool {
 	for i := 0; i < len(*csrf); i++ {
 		if (*csrf)[i] == toCheckToken {

modules/auth/middleware.go

@@ -61,11 +61,13 @@ func SetPrefix(prefix string) *Invoker {
 	return i
 }
 
+// SetAuthFailCallback set the authFailCallback of Invoker.
 func (invoker *Invoker) SetAuthFailCallback(callback MiddlewareCallback) *Invoker {
 	invoker.authFailCallback = callback
 	return invoker
 }
 
+// SetPermissionDenyCallback set the permissionDenyCallback of Invoker.
 func (invoker *Invoker) SetPermissionDenyCallback(callback MiddlewareCallback) *Invoker {
 	invoker.permissionDenyCallback = callback
 	return invoker
@@ -74,6 +76,7 @@ func (invoker *Invoker) SetPermissionDenyCallback(callback MiddlewareCallback) *
 // MiddlewareCallback is type of callback function.
 type MiddlewareCallback func(ctx *context.Context)
 
+// Middleware get the auth middleware from Invoker.
 func (invoker *Invoker) Middleware() context.Handler {
 	return func(ctx *context.Context) {
 		user, authOk, permissionOk := Filter(ctx)
@@ -112,7 +115,7 @@ func Filter(ctx *context.Context) (models.UserModel, bool, bool) {
 		return user, false, false
 	}
 
-	user, ok = GetCurUserById(int64(id))
+	user, ok = GetCurUserByID(int64(id))
 
 	if !ok {
 		return user, false, false
@@ -121,8 +124,8 @@ func Filter(ctx *context.Context) (models.UserModel, bool, bool) {
 	return user, true, CheckPermissions(user, ctx.Path(), ctx.Method())
 }
 
-// GetCurUserById return the user model of given user id.
-func GetCurUserById(id int64) (user models.UserModel, ok bool) {
+// GetCurUserByID return the user model of given user id.
+func GetCurUserByID(id int64) (user models.UserModel, ok bool) {
 
 	user = models.User().Find(id)
 

modules/auth/session.go

@@ -53,8 +53,8 @@ func (ses *Session) Get(key string) interface{} {
 	return ses.Values[key]
 }
 
-// Get set the session value of key.
-func (ses *Session) Set(key string, value interface{}) {
+// Add add the session value of key.
+func (ses *Session) Add(key string, value interface{}) {
 	ses.Values[key] = value
 	ses.Driver.Update(ses.Sid, ses.Values)
 	cookie := http.Cookie{
@@ -121,11 +121,11 @@ func (driver *DBDriver) Load(sid string) map[string]interface{} {
 
 	if sesModel == nil {
 		return map[string]interface{}{}
-	} else {
-		var values map[string]interface{}
-		_ = json.Unmarshal([]byte(sesModel["values"].(string)), &values)
-		return values
 	}
+
+	var values map[string]interface{}
+	_ = json.Unmarshal([]byte(sesModel["values"].(string)), &values)
+	return values
 }
 
 func deleteOverdueSession() {
@@ -151,7 +151,7 @@ func deleteOverdueSession() {
 		cmd = `delete from goadmin_session where strftime('%s', created_at) < strftime('%s', 'now') - ` + duration
 	}
 
-	logger.LogSql(cmd, nil)
+	logger.LogSQL(cmd, nil)
 
 	_, _ = db.Query(cmd)
 }