From: Michael Shamoon <4887959+shamoon@users.noreply.github.com>
Date: Fri, 8 Jul 2022 20:55:27 +0000 (-0700)
Subject: simplify filter rule comparison
X-Git-Tag: v1.8.0-beta.rc1~14^2
X-Git-Url: http://git.ipfire.org/gitweb/gitweb.cgi?a=commitdiff_plain;h=refs%2Fpull%2F1133%2Fhead;p=thirdparty%2Fpaperless-ngx.git

simplify filter rule comparison
---

diff --git a/src-ui/src/app/data/filter-rule.ts b/src-ui/src/app/data/filter-rule.ts
index 4460d39ab1..b0b9cb415e 100644
--- a/src-ui/src/app/data/filter-rule.ts
+++ b/src-ui/src/app/data/filter-rule.ts
@@ -29,28 +29,19 @@ export function filterRulesDiffer(
   filterRulesA: FilterRule[],
   filterRulesB: FilterRule[]
 ): boolean {
-  let modified = false
+  let differ = false
   if (filterRulesA.length != filterRulesB.length) {
-    modified = true
+    differ = true
   } else {
-    modified = filterRulesA.some((rule) => {
+    differ = filterRulesA.some((rule) => {
       return (
         filterRulesB.find(
           (fri) => fri.rule_type == rule.rule_type && fri.value == rule.value
         ) == undefined
       )
     })
-
-    if (!modified) {
-      // only check other direction if we havent already determined is modified
-      modified = filterRulesB.some((rule) => {
-        filterRulesA.find(
-          (fr) => fr.rule_type == rule.rule_type && fr.value == rule.value
-        ) == undefined
-      })
-    }
   }
-  return modified
+  return differ
 }
 
 export interface FilterRule {