From 9ec8b4d6346cce3d664911aae06ff0bb86dd6677 Mon Sep 17 00:00:00 2001 From: Wenkai Yin Date: Wed, 17 Jan 2018 17:07:10 +0800 Subject: [PATCH] Support multiple status in query string of replication job listing API --- src/ui/api/replication_job.go | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/src/ui/api/replication_job.go b/src/ui/api/replication_job.go index 2acda5604..f2446688f 100644 --- a/src/ui/api/replication_job.go +++ b/src/ui/api/replication_job.go @@ -82,7 +82,7 @@ func (ra *RepJobAPI) List() { } repository := ra.GetString("repository") - status := ra.GetString("status") + statuses := ra.GetStrings("status") var startTime *time.Time startTimeStr := ra.GetString("start_time") @@ -108,15 +108,11 @@ func (ra *RepJobAPI) List() { page, pageSize := ra.GetPaginationParams() - statuses := []string{} - if len(status) > 0 { - statuses = append(statuses, status) - } jobs, total, err := dao.FilterRepJobs(policyID, repository, statuses, startTime, endTime, pageSize, pageSize*(page-1)) if err != nil { - log.Errorf("failed to filter jobs according policy ID %d, repository %s, status %s, start time %v, end time %v: %v", - policyID, repository, status, startTime, endTime, err) + log.Errorf("failed to filter jobs according policy ID %d, repository %s, status %v, start time %v, end time %v: %v", + policyID, repository, statuses, startTime, endTime, err) ra.CustomAbort(http.StatusInternalServerError, "") }