appview/labels: reduce the opset after validation #605

merged
opened by oppi.li targeting master from push-plpuqtyrprou

validation also performs a transformation upon labels: it resolves handles to DIDs, it is important that this transform be present when determining redundant ops.

Signed-off-by: oppiliappan me@oppi.li

Changed files
+3 -3
appview
labels
+3 -3
appview/labels/labels.go
··· 162 162 } 163 163 } 164 164 165 - // reduce the opset 166 - labelOps = models.ReduceLabelOps(labelOps) 167 - 168 165 for i := range labelOps { 169 166 def := actx.Defs[labelOps[i].OperandKey] 170 167 if err := l.validator.ValidateLabelOp(def, repo, &labelOps[i]); err != nil { ··· 173 170 } 174 171 } 175 172 173 + // reduce the opset 174 + labelOps = models.ReduceLabelOps(labelOps) 175 + 176 176 // next, apply all ops introduced in this request and filter out ones that are no-ops 177 177 validLabelOps := labelOps[:0] 178 178 for _, op := range labelOps {