refactor for searching user (#1038)

* refactor for searching user

* fix like bug

* better format for builder cond

models/user.go

@@ -23,6 +23,7 @@ import (
 	"unicode/utf8"
 
 	"github.com/Unknwon/com"
+	"github.com/go-xorm/builder"
 	"github.com/go-xorm/xorm"
 	"github.com/nfnt/resize"
 	"golang.org/x/crypto/pbkdf2"
@@ -1235,27 +1236,28 @@ func SearchUserByName(opts *SearchUserOptions) (users []*User, _ int64, _ error)
 		opts.Page = 1
 	}
 
-	searchQuery := "%" + opts.Keyword + "%"
 	users = make([]*User, 0, opts.PageSize)
+
 	// Append conditions
-	sess := x.
+	cond := builder.And(
-		Where("LOWER(lower_name) LIKE ?", searchQuery).
+		builder.Eq{"type": opts.Type},
-		Or("LOWER(full_name) LIKE ?", searchQuery).
+		builder.Or(
-		And("type = ?", opts.Type)
+			builder.Like{"lower_name", opts.Keyword},
+			builder.Like{"LOWER(full_name)", opts.Keyword},
+		),
+	)
 
-	var countSess xorm.Session
+	count, err := x.Where(cond).Count(new(User))
-	countSess = *sess
-	count, err := countSess.Count(new(User))
 	if err != nil {
 		return nil, 0, fmt.Errorf("Count: %v", err)
 	}
 
+	sess := x.Where(cond).
+		Limit(opts.PageSize, (opts.Page-1)*opts.PageSize)
 	if len(opts.OrderBy) > 0 {
 		sess.OrderBy(opts.OrderBy)
 	}
-	return users, count, sess.
+	return users, count, sess.Find(&users)
-		Limit(opts.PageSize, (opts.Page-1)*opts.PageSize).
-		Find(&users)
 }
 
 // ___________    .__  .__