From 2c18f6268b20ac93098dec9f5fb3ba6428729464 Mon Sep 17 00:00:00 2001 From: Michael Shamoon <4887959+nikonratm@users.noreply.github.com> Date: Sun, 13 Dec 2020 11:34:58 -0800 Subject: [PATCH] Comment cleanup --- src-ui/src/app/services/filter-editor-view.service.ts | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src-ui/src/app/services/filter-editor-view.service.ts b/src-ui/src/app/services/filter-editor-view.service.ts index 3a3b70e9a..ee7961f0f 100644 --- a/src-ui/src/app/services/filter-editor-view.service.ts +++ b/src-ui/src/app/services/filter-editor-view.service.ts @@ -106,11 +106,11 @@ export class FilterEditorViewService { let existingRules = filterRules.filter(rule => rule.type.id == filterRuleType.id) let existingItemRule = existingRules?.find(rule => rule.value == item.id) - if (existingRules && existingItemRule) { - filterRules.splice(filterRules.indexOf(existingItemRule), 1) // if exact rule exists just remove + if (existingRules && existingItemRule) { // if exact rule exists just remove + filterRules.splice(filterRules.indexOf(existingItemRule), 1) } else if (existingRules.length > 0 && filterRuleType.multi) { // e.g. tags can have multiple filterRules.push({type: filterRuleType, value: item.id}) - } else if (existingRules.length > 0) { // Correspondents & DocumentTypes only one + } else if (existingRules.length > 0) { // correspondents & documentTypes can only be one filterRules.find(rule => rule.type.id == filterRuleType.id).value = item.id } else { filterRules.push({type: filterRuleType, value: item.id})