Merge branch 'new-field-mixin' of github.com:gianpieropa/drf-access-p…

…olicy into new-field-mixin

.gitignore

@@ -7,4 +7,5 @@ drf_access_policy.egg-info
 site/
 venv.sh
 .venv
-.coverage
+.coverage
+

rest_access_policy/field_access_mixin.py

@@ -1,7 +1,4 @@
-from typing import List
-
 from rest_framework.request import Request
-
 from .access_policy import AccessPolicy
 
 
@@ -11,6 +8,7 @@ def __init__(self, *args, **kwargs):
         super().__init__(*args, **kwargs)
         self._apply_fields_access()
 
+
     @property
     def access_policy(self) -> AccessPolicy:
         meta = getattr(self, "Meta", None)
@@ -23,7 +21,7 @@ def access_policy(self) -> AccessPolicy:
         if not access_policy:
             raise Exception("Must set access_policy inside Meta for FieldAccessMixin")
 
-        if getattr(access_policy,"scope_fields",None) is None:
+        if getattr(access_policy, "scope_fields", None) is None:
             raise Exception("Must define scope_fields method on access_policy")
 
         return access_policy
@@ -41,7 +39,6 @@ def _apply_fields_access(self):
         fields = self.access_policy.scope_fields(self.request, self.fields, instance=self.instance)
         if fields is None:
             raise Exception("scope_fields method must return fields variable")
-
         self.fields = self.access_policy.scope_fields(self.request, self.fields, instance=self.instance)
 
-
+

test_project/testapp/access_policies.py

@@ -23,6 +23,7 @@ def scope_fields(cls, request, fields: dict, instance=None):
             fields["status"].read_only = True 
         return fields
 
+
 class LogsAccessPolicy(AccessPolicy):
     statements = [
         {"principal": "group:admin", "action": "*", "effect": "allow"},