From 20da15d9e933dc235283b86ff9e0556c845a45e7 Mon Sep 17 00:00:00 2001
From: Oleg Bulkin <o.bulkin@gmail.com>
Date: Wed, 15 Feb 2017 18:35:00 -0800
Subject: [PATCH] [FIX] auditlog: Duplicate logging * Update _patch_methods and
 _revert_methods in auditlog.rule model to properly track whether a method has
 already been patched for logging purposes. This prevents duplicate logs from
 being produced in various cases (e.g. when a logging rule is created as part
 of a module install)

---
 auditlog/models/rule.py | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/auditlog/models/rule.py b/auditlog/models/rule.py
index fe31a4aaa..4dcdb7877 100644
--- a/auditlog/models/rule.py
+++ b/auditlog/models/rule.py
@@ -131,28 +131,28 @@ class AuditlogRule(models.Model):
             if getattr(rule, 'log_create') \
                     and not hasattr(model_model, check_attr):
                 model_model._patch_method('create', rule._make_create())
-                setattr(model_model, check_attr, True)
+                setattr(type(model_model), check_attr, True)
                 updated = True
             #   -> read
             check_attr = 'auditlog_ruled_read'
             if getattr(rule, 'log_read') \
                     and not hasattr(model_model, check_attr):
                 model_model._patch_method('read', rule._make_read())
-                setattr(model_model, check_attr, True)
+                setattr(type(model_model), check_attr, True)
                 updated = True
             #   -> write
             check_attr = 'auditlog_ruled_write'
             if getattr(rule, 'log_write') \
                     and not hasattr(model_model, check_attr):
                 model_model._patch_method('write', rule._make_write())
-                setattr(model_model, check_attr, True)
+                setattr(type(model_model), check_attr, True)
                 updated = True
             #   -> unlink
             check_attr = 'auditlog_ruled_unlink'
             if getattr(rule, 'log_unlink') \
                     and not hasattr(model_model, check_attr):
                 model_model._patch_method('unlink', rule._make_unlink())
-                setattr(model_model, check_attr, True)
+                setattr(type(model_model), check_attr, True)
                 updated = True
         return updated
 
@@ -166,6 +166,7 @@ class AuditlogRule(models.Model):
                 if getattr(rule, 'log_%s' % method) and hasattr(
                         getattr(model_model, method), 'origin'):
                     model_model._revert_method(method)
+                    delattr(type(model_model), 'auditlog_ruled_%s' % method)
                     updated = True
         if updated:
             modules.registry.RegistryManager.signal_registry_change(