diff options
author | George Goldberg <george@gberg.me> | 2017-10-02 14:15:51 +0100 |
---|---|---|
committer | Harrison Healey <harrisonmhealey@gmail.com> | 2017-10-02 09:15:51 -0400 |
commit | 078467ee96bdc74cc385810a24eeae2a2318be46 (patch) | |
tree | 0341c62a9374e166522688d309be4b7cf9a73a4c /store/sqlstore/job_store.go | |
parent | 76bd1bb212177824379c485c553e54530a854009 (diff) | |
download | chat-078467ee96bdc74cc385810a24eeae2a2318be46.tar.gz chat-078467ee96bdc74cc385810a24eeae2a2318be46.tar.bz2 chat-078467ee96bdc74cc385810a24eeae2a2318be46.zip |
PLT-7759: Make GetNewestJobByStatusAndType not fail when no jobs match. (#7540)
Diffstat (limited to 'store/sqlstore/job_store.go')
-rw-r--r-- | store/sqlstore/job_store.go | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/store/sqlstore/job_store.go b/store/sqlstore/job_store.go index 0ae5a6a07..8cd921217 100644 --- a/store/sqlstore/job_store.go +++ b/store/sqlstore/job_store.go @@ -344,8 +344,8 @@ func (jss SqlJobStore) GetNewestJobByStatusAndType(status string, jobType string Type = :Type ORDER BY CreateAt DESC - LIMIT 1`, map[string]interface{}{"Status": status, "Type": jobType}); err != nil { - result.Err = model.NewAppError("SqlJobStore.GetAllByStatus", "store.sql_job.get_newest_job_by_status_and_type.app_error", nil, "Status="+status+", "+err.Error(), http.StatusInternalServerError) + LIMIT 1`, map[string]interface{}{"Status": status, "Type": jobType}); err != nil && err != sql.ErrNoRows { + result.Err = model.NewAppError("SqlJobStore.GetNewestJobByStatusAndType", "store.sql_job.get_newest_job_by_status_and_type.app_error", nil, "Status="+status+", "+err.Error(), http.StatusInternalServerError) } else { result.Data = job } |