Fix negate on FieldMatcherPolicy

passbook/core/models.py

@@ -284,8 +284,7 @@ class FieldMatcherPolicy(Policy):
         if self.match_action == FieldMatcherPolicy.MATCH_REGEXP:
             pattern = re.compile(self.value)
             passes = bool(pattern.match(user_field_value))
-        if self.negate:
-            passes = not passes
+
         LOGGER.debug("User got '%r'", passes)
         return passes
 

passbook/core/policies.py

@@ -25,6 +25,7 @@ def _policy_engine_task(user_pk, policy_pk, **kwargs):
     # Invert result if policy.negate is set
     if policy_obj.negate:
         policy_result = not policy_result
+    LOGGER.debug("Policy %r#%s got %s", policy_obj.name, policy_obj.pk.hex, policy_result)
     return policy_obj.action, policy_result, message
 
 class PolicyEngine: