diff options
author | =Corey Hulen <corey@hulen.com> | 2015-08-20 14:48:04 -0700 |
---|---|---|
committer | =Corey Hulen <corey@hulen.com> | 2015-08-20 14:48:04 -0700 |
commit | 42822df5b3451114e71aff946d755f6dd3cfd1c9 (patch) | |
tree | b590d63af8b20cb3320f30c98f9d14b660c2eb9e | |
parent | 1b4135ab0da3c1635fae23f8f64247eb78866819 (diff) | |
download | chat-42822df5b3451114e71aff946d755f6dd3cfd1c9.tar.gz chat-42822df5b3451114e71aff946d755f6dd3cfd1c9.tar.bz2 chat-42822df5b3451114e71aff946d755f6dd3cfd1c9.zip |
Fixing race condition
-rw-r--r-- | store/sql_channel_store.go | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/store/sql_channel_store.go b/store/sql_channel_store.go index 25cd46088..a7577e645 100644 --- a/store/sql_channel_store.go +++ b/store/sql_channel_store.go @@ -4,6 +4,7 @@ package store import ( + "fmt" "github.com/mattermost/platform/model" "github.com/mattermost/platform/utils" ) @@ -155,8 +156,8 @@ func (s SqlChannelStore) extraUpdated(channel *model.Channel) StoreChannel { if count, err := s.GetMaster().Update(channel); err != nil { result.Err = model.NewAppError("SqlChannelStore.extraUpdated", "Problem updating members last updated time", "id="+channel.Id+", "+err.Error()) - } else if count == 0 { - result.Err = model.NewAppError("SqlChannelStore.extraUpdated", "Problem updating members last updated time", "id="+channel.Id) + } else if count != 1 { + result.Err = model.NewAppError("SqlChannelStore.extraUpdated", "Problem updating members last updated time", fmt.Sprintf("id=%v, count=%v", channel.Id, count)) } storeChannel <- result |