From 66e8262530fb7906a3f8f5933fd6dbd6152bf898 Mon Sep 17 00:00:00 2001 From: G Date: Tue, 1 Aug 2017 01:47:31 -0400 Subject: [PATCH] Replace calls to xorm UseBool with Where (#2237) --- models/login_source.go | 2 +- models/oauth2.go | 11 +++-------- 2 files changed, 4 insertions(+), 9 deletions(-) diff --git a/models/login_source.go b/models/login_source.go index 2e70228c9..e4dc77714 100644 --- a/models/login_source.go +++ b/models/login_source.go @@ -677,7 +677,7 @@ func UserSignIn(username, password string) (*User, error) { } sources := make([]*LoginSource, 0, 5) - if err = x.UseBool().Find(&sources, &LoginSource{IsActived: true}); err != nil { + if err = x.Where("is_actived = ?", true).Find(&sources); err != nil { return nil, err } diff --git a/models/oauth2.go b/models/oauth2.go index 158f52b96..a1917540d 100644 --- a/models/oauth2.go +++ b/models/oauth2.go @@ -56,7 +56,7 @@ var OAuth2DefaultCustomURLMappings = map[string]*oauth2.CustomURLMapping{ // GetActiveOAuth2ProviderLoginSources returns all actived LoginOAuth2 sources func GetActiveOAuth2ProviderLoginSources() ([]*LoginSource, error) { sources := make([]*LoginSource, 0, 1) - if err := x.UseBool().Find(&sources, &LoginSource{IsActived: true, Type: LoginOAuth2}); err != nil { + if err := x.Where("is_actived = ? and type = ?", true, LoginOAuth2).Find(&sources); err != nil { return nil, err } return sources, nil @@ -64,13 +64,8 @@ func GetActiveOAuth2ProviderLoginSources() ([]*LoginSource, error) { // GetActiveOAuth2LoginSourceByName returns a OAuth2 LoginSource based on the given name func GetActiveOAuth2LoginSourceByName(name string) (*LoginSource, error) { - loginSource := &LoginSource{ - Name: name, - Type: LoginOAuth2, - IsActived: true, - } - - has, err := x.UseBool().Get(loginSource) + loginSource := new(LoginSource) + has, err := x.Where("name = ? and type = ? and is_actived = ?", name, LoginOAuth2, true).Get(loginSource) if !has || err != nil { return nil, err }