diff options
author | Rafael George <george.rafael@gmail.com> | 2018-10-17 09:25:21 -0400 |
---|---|---|
committer | George Goldberg <george@gberg.me> | 2018-10-17 14:25:21 +0100 |
commit | 7d66ef0805895c4a56c0415c90de9d146ed25839 (patch) | |
tree | 953894c517f9522adbd5dcd2e956468d33014bad /app | |
parent | 7562f940bb7b0de80bcfb8f5c3fc1b751a1a6b16 (diff) | |
download | chat-7d66ef0805895c4a56c0415c90de9d146ed25839.tar.gz chat-7d66ef0805895c4a56c0415c90de9d146ed25839.tar.bz2 chat-7d66ef0805895c4a56c0415c90de9d146ed25839.zip |
[MM-12608] Migrate to idiomatic error handling in the file `app/scheme.go` in the mattermost-server repo (#9592)
* Add idiomatic error handling
* Remove unused else statement and fix compilation errors
* Add an inline error initialization
The error its just been use once for the return no need to have it in
separate lines.
* Remove uneeded code
- Remove extra line
- Remove initializing in two lines when not needed
Diffstat (limited to 'app')
-rw-r--r-- | app/scheme.go | 54 |
1 files changed, 26 insertions, 28 deletions
diff --git a/app/scheme.go b/app/scheme.go index 1d7ddf28a..0fa2647b2 100644 --- a/app/scheme.go +++ b/app/scheme.go @@ -7,7 +7,6 @@ import ( "net/http" "github.com/mattermost/mattermost-server/model" - "github.com/mattermost/mattermost-server/store" ) func (a *App) GetScheme(id string) (*model.Scheme, *model.AppError) { @@ -15,11 +14,11 @@ func (a *App) GetScheme(id string) (*model.Scheme, *model.AppError) { return nil, err } - if result := <-a.Srv.Store.Scheme().Get(id); result.Err != nil { + result := <-a.Srv.Store.Scheme().Get(id) + if result.Err != nil { return nil, result.Err - } else { - return result.Data.(*model.Scheme), nil } + return result.Data.(*model.Scheme), nil } func (a *App) GetSchemeByName(name string) (*model.Scheme, *model.AppError) { @@ -27,11 +26,11 @@ func (a *App) GetSchemeByName(name string) (*model.Scheme, *model.AppError) { return nil, err } - if result := <-a.Srv.Store.Scheme().GetByName(name); result.Err != nil { + result := <-a.Srv.Store.Scheme().GetByName(name) + if result.Err != nil { return nil, result.Err - } else { - return result.Data.(*model.Scheme), nil } + return result.Data.(*model.Scheme), nil } func (a *App) GetSchemesPage(scope string, page int, perPage int) ([]*model.Scheme, *model.AppError) { @@ -47,11 +46,11 @@ func (a *App) GetSchemes(scope string, offset int, limit int) ([]*model.Scheme, return nil, err } - if result := <-a.Srv.Store.Scheme().GetAllPage(scope, offset, limit); result.Err != nil { + result := <-a.Srv.Store.Scheme().GetAllPage(scope, offset, limit) + if result.Err != nil { return nil, result.Err - } else { - return result.Data.([]*model.Scheme), nil } + return result.Data.([]*model.Scheme), nil } func (a *App) CreateScheme(scheme *model.Scheme) (*model.Scheme, *model.AppError) { @@ -68,11 +67,11 @@ func (a *App) CreateScheme(scheme *model.Scheme) (*model.Scheme, *model.AppError scheme.UpdateAt = 0 scheme.DeleteAt = 0 - if result := <-a.Srv.Store.Scheme().Save(scheme); result.Err != nil { + result := <-a.Srv.Store.Scheme().Save(scheme) + if result.Err != nil { return nil, result.Err - } else { - return scheme, nil } + return scheme, nil } func (a *App) PatchScheme(scheme *model.Scheme, patch *model.SchemePatch) (*model.Scheme, *model.AppError) { @@ -94,11 +93,11 @@ func (a *App) UpdateScheme(scheme *model.Scheme) (*model.Scheme, *model.AppError return nil, err } - if result := <-a.Srv.Store.Scheme().Save(scheme); result.Err != nil { + result := <-a.Srv.Store.Scheme().Save(scheme) + if result.Err != nil { return nil, result.Err - } else { - return scheme, nil } + return scheme, nil } func (a *App) DeleteScheme(schemeId string) (*model.Scheme, *model.AppError) { @@ -106,11 +105,11 @@ func (a *App) DeleteScheme(schemeId string) (*model.Scheme, *model.AppError) { return nil, err } - if result := <-a.Srv.Store.Scheme().Delete(schemeId); result.Err != nil { + result := <-a.Srv.Store.Scheme().Delete(schemeId) + if result.Err != nil { return nil, result.Err - } else { - return result.Data.(*model.Scheme), nil } + return result.Data.(*model.Scheme), nil } func (a *App) GetTeamsForSchemePage(scheme *model.Scheme, page int, perPage int) ([]*model.Team, *model.AppError) { @@ -126,11 +125,11 @@ func (a *App) GetTeamsForScheme(scheme *model.Scheme, offset int, limit int) ([] return nil, err } - if result := <-a.Srv.Store.Team().GetTeamsByScheme(scheme.Id, offset, limit); result.Err != nil { + result := <-a.Srv.Store.Team().GetTeamsByScheme(scheme.Id, offset, limit) + if result.Err != nil { return nil, result.Err - } else { - return result.Data.([]*model.Team), nil } + return result.Data.([]*model.Team), nil } func (a *App) GetChannelsForSchemePage(scheme *model.Scheme, page int, perPage int) (model.ChannelList, *model.AppError) { @@ -145,12 +144,11 @@ func (a *App) GetChannelsForScheme(scheme *model.Scheme, offset int, limit int) if err := a.IsPhase2MigrationCompleted(); err != nil { return nil, err } - - if result := <-a.Srv.Store.Channel().GetChannelsByScheme(scheme.Id, offset, limit); result.Err != nil { + result := <-a.Srv.Store.Channel().GetChannelsByScheme(scheme.Id, offset, limit) + if result.Err != nil { return nil, result.Err - } else { - return result.Data.(model.ChannelList), nil } + return result.Data.(model.ChannelList), nil } func (a *App) IsPhase2MigrationCompleted() *model.AppError { @@ -170,8 +168,8 @@ func (a *App) IsPhase2MigrationCompleted() *model.AppError { func (a *App) SchemesIterator(batchSize int) func() []*model.Scheme { offset := 0 return func() []*model.Scheme { - var result store.StoreResult - if result = <-a.Srv.Store.Scheme().GetAllPage("", offset, batchSize); result.Err != nil { + result := <-a.Srv.Store.Scheme().GetAllPage("", offset, batchSize) + if result.Err != nil { return []*model.Scheme{} } offset += batchSize |