Merge pull request #525 from leancloud/you-can-also-ignore-hooks-with-master-keys

fix: master key is also allowed to ignore hooks.

Author: weakish

leancloud/object_.py

@@ -576,21 +576,25 @@ def set_acl(self, acl):
 
     def disable_before_hook(self):
         hook_key = client.get_app_info().get("hook_key")
-        if not hook_key:
-            raise ValueError("disable_before_hook need LeanCloud hook key")
-        self.ignore_hook("beforeSave")
-        self.ignore_hook("beforeUpdate")
-        self.ignore_hook("beforeDelete")
-        return self
+        master_key = client.get_app_info().get("master_key")
+        if hook_key or master_key:
+            self.ignore_hook("beforeSave")
+            self.ignore_hook("beforeUpdate")
+            self.ignore_hook("beforeDelete")
+            return self
+        else:
+            raise ValueError("disable_before_hook needs master key or hook key")
 
     def disable_after_hook(self):
         hook_key = client.get_app_info().get("hook_key")
-        if not hook_key:
-            raise ValueError("`disable_before_hook` need LeanCloud hook key")
-        self.ignore_hook("afterSave")
-        self.ignore_hook("afterUpdate")
-        self.ignore_hook("afterDelete")
-        return self
+        master_key = client.get_app_info().get("master_key")
+        if hook_key or master_key:
+            self.ignore_hook("afterSave")
+            self.ignore_hook("afterUpdate")
+            self.ignore_hook("afterDelete")
+            return self
+        else:
+            raise ValueError("disable_after_hook needs master key or hook key")
 
     def ignore_hook(self, hook_name):
         if hook_name not in {