Ignore 0 values for assignment search params

edshadi · edshadi · commit a90ace6588cc · 2020-02-27T12:08:15.000+01:00
diff --git a/pkg/datastore/assignment.go b/pkg/datastore/assignment.go
@@ -45,15 +45,15 @@ func (as *AssignmentStore) GetAssignments(p assignment.Params) (assignment.Assig
 	paramsQuery := []string{}
 	args := []interface{}{}
 
-	if p.WorkerID != "" {
+	if p.WorkerID != "" && p.WorkerID != "0" {
 		args = append(args, p.WorkerID)
 		paramsQuery = append(paramsQuery, "worker_id=?")
 	}
-	if p.JobID != "" {
+	if p.JobID != "" && p.JobID != "0" {
 		args = append(args, p.JobID)
 		paramsQuery = append(paramsQuery, "job_id=?")
 	}
-	if p.TaskID != "" {
+	if p.TaskID != "" && p.TaskID != "0" {
 		args = append(args, p.TaskID)
 		paramsQuery = append(paramsQuery, "task_id=?")
 	}

Original file line number	Diff line number	Diff line change
`@@ -45,15 +45,15 @@ func (as *AssignmentStore) GetAssignments(p assignment.Params) (assignment.Assig`
`45`	`45`	`paramsQuery := []string{}`
`46`	`46`	`args := []interface{}{}`
`47`	`47`
`48`		`- if p.WorkerID != "" {`
	`48`	`+ if p.WorkerID != "" && p.WorkerID != "0" {`
`49`	`49`	`args = append(args, p.WorkerID)`
`50`	`50`	`paramsQuery = append(paramsQuery, "worker_id=?")`
`51`	`51`	`}`
`52`		`- if p.JobID != "" {`
	`52`	`+ if p.JobID != "" && p.JobID != "0" {`
`53`	`53`	`args = append(args, p.JobID)`
`54`	`54`	`paramsQuery = append(paramsQuery, "job_id=?")`
`55`	`55`	`}`
`56`		`- if p.TaskID != "" {`
	`56`	`+ if p.TaskID != "" && p.TaskID != "0" {`
`57`	`57`	`args = append(args, p.TaskID)`
`58`	`58`	`paramsQuery = append(paramsQuery, "task_id=?")`
`59`	`59`	`}`