CollaboratorsPermissionLogic.has_perm() - Code Metrics - Inspection of "Merge pull request #61 from AmbientLighter/feature..." - lambdalisue/django-permission - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 194382...d1a5fb )

by Lambda

created 2016-07-13 02:07 UTC

CollaboratorsPermissionLogic.has_perm() F

↳ Parent: CollaboratorsPermissionLogic

Complexity

Conditions

Size

Total Lines

Duplication

Lines	66
Ratio	100 %

Importance

Changes	2
Bugs	0	Features	0

Metric	Value
cc	16
c	2
b	0
f	0
dl	66
loc	66
rs	2.7057

How to fix Long Method Complexity

# coding=utf-8
"""
Permission logic module for collaborators based permission system
"""
from permission.conf import settings
from permission.logics.base import PermissionLogic
from permission.utils.field_lookup import field_lookup


class CollaboratorsPermissionLogic(PermissionLogic):

    """
    Permission logic class for collaborators based permission system
    """
    def __init__(self,
                 field_name=None,
                 any_permission=None,
                 change_permission=None,
                 delete_permission=None):
        """
        Constructor

        Parameters
        ----------
        field_name : string
            A field name of object which store the collaborators as django
            relational fields for django user model.
            You can specify the related object with '__' like django queryset
            filter.
            Default value will be taken from
            ``PERMISSION_DEFAULT_COLLABORATORS_PERMISSION_LOGIC_FIELD_NAME`` in
            settings.
        any_permission : boolean
            True for give any permission of the specified object to the
            collaborators.
            Default value will be taken from
            ``PERMISSION_DEFAULT_COLLABORATORS_PERMISSION_LOGIC_ANY_PERMISSION``
            in settings.
        change_permission : boolean
            True for give change permission of the specified object to the
            collaborators.
            It will be ignored if :attr:`any_permission` is True.
            Default value will be taken from
            ``PERMISSION_DEFAULT_COLLABORATORS_PERMISSION_LOGIC_CHANGE_PERMISSION``
            in settings.
        delete_permission : boolean
            True for give delete permission of the specified object to the
            collaborators.
            It will be ignored if :attr:`any_permission` is True.
            Default value will be taken from
            ``PERMISSION_DEFAULT_COLLABORATORS_PERMISSION_LOGIC_DELETE_PERMISSION``
            in settings.
        """
        self.field_name = field_name
        self.any_permission = any_permission
        self.change_permission = change_permission
        self.delete_permission = delete_permission

        if self.field_name is None:
            self.field_name = \
                settings.PERMISSION_DEFAULT_CPL_FIELD_NAME
        if self.any_permission is None:
            self.any_permission = \
                settings.PERMISSION_DEFAULT_CPL_ANY_PERMISSION
        if self.change_permission is None:
            self.change_permission = \
                settings.PERMISSION_DEFAULT_CPL_CHANGE_PERMISSION
        if self.delete_permission is None:
            self.delete_permission = \
                settings.PERMISSION_DEFAULT_CPL_DELETE_PERMISSION

    def has_perm(self, user_obj, perm, obj=None):
        """
        Check if user have permission (of object)

        If the user_obj is not authenticated, it return ``False``.

        If no object is specified, it return ``True`` when the corresponding
        permission was specified to ``True`` (changed from v0.7.0).
        This behavior is based on the django system.
        https://code.djangoproject.com/wiki/RowLevelPermissions


        If an object is specified, it will return ``True`` if the user is
        found in ``field_name`` of the object (e.g. ``obj.collaborators``).
        So once the object store the user as a collaborator in
        ``field_name`` attribute (default: ``collaborators``), the collaborator
        can change or delete the object (you can change this behavior to set
        ``any_permission``, ``change_permission`` or ``delete_permission``
        attributes of this instance).

        Parameters
        ----------
        user_obj : django user model instance
            A django user model instance which be checked
        perm : string
            `app_label.codename` formatted permission string
        obj : None or django model instance
            None or django model instance for object permission

        Returns
        -------
        boolean
            Whether the specified user have specified permission (of specified
            object).
        """
        if not user_obj.is_authenticated():
            return False
        # construct the permission full name
        change_permission = self.get_full_permission_string('change')
        delete_permission = self.get_full_permission_string('delete')
        if obj is None:
            # object permission without obj should return True
            # Ref: https://code.djangoproject.com/wiki/RowLevelPermissions
            if self.any_permission:
                return True
            if self.change_permission and perm == change_permission:
                return True
            if self.delete_permission and perm == delete_permission:
                return True
            return False
        elif user_obj.is_active:
            # get collaborator queryset
            collaborators = field_lookup(obj, self.field_name)
            if hasattr(collaborators, 'all'):
                collaborators = collaborators.all()
            if user_obj in collaborators:
                if self.any_permission:
                    # have any kind of permissions to the obj
                    return True
                if (self.change_permission and
                        perm == change_permission):
                    return True
                if (self.delete_permission and
                        perm == delete_permission):
                    return True
        return False


1		# coding=utf-8
2		"""
3		Permission logic module for collaborators based permission system
4		"""
5		from permission.conf import settings
6		from permission.logics.base import PermissionLogic
7		from permission.utils.field_lookup import field_lookup
8
9
10	View Code Duplication	class CollaboratorsPermissionLogic(PermissionLogic):
		0 ignored issues – show Duplication introduced 2016-07-13 02:08 UTC by Report Bug Copy Issue Report This code seems to be duplicated in your project. Loading history...
11		"""
12		Permission logic class for collaborators based permission system
13		"""
14		def __init__(self,
15		field_name=None,
16		any_permission=None,
17		change_permission=None,
18		delete_permission=None):
19		"""
20		Constructor
21
22		Parameters
23		----------
24		field_name : string
25		A field name of object which store the collaborators as django
26		relational fields for django user model.
27		You can specify the related object with '__' like django queryset
28		filter.
29		Default value will be taken from
30		``PERMISSION_DEFAULT_COLLABORATORS_PERMISSION_LOGIC_FIELD_NAME`` in
31		settings.
32		any_permission : boolean
33		True for give any permission of the specified object to the
34		collaborators.
35		Default value will be taken from
36		``PERMISSION_DEFAULT_COLLABORATORS_PERMISSION_LOGIC_ANY_PERMISSION``
37		in settings.
38		change_permission : boolean
39		True for give change permission of the specified object to the
40		collaborators.
41		It will be ignored if :attr:`any_permission` is True.
42		Default value will be taken from
43		``PERMISSION_DEFAULT_COLLABORATORS_PERMISSION_LOGIC_CHANGE_PERMISSION``
44		in settings.
45		delete_permission : boolean
46		True for give delete permission of the specified object to the
47		collaborators.
48		It will be ignored if :attr:`any_permission` is True.
49		Default value will be taken from
50		``PERMISSION_DEFAULT_COLLABORATORS_PERMISSION_LOGIC_DELETE_PERMISSION``
51		in settings.
52		"""
53		self.field_name = field_name
54		self.any_permission = any_permission
55		self.change_permission = change_permission
56		self.delete_permission = delete_permission
57
58		if self.field_name is None:
59		self.field_name = \
60		settings.PERMISSION_DEFAULT_CPL_FIELD_NAME
61		if self.any_permission is None:
62		self.any_permission = \
63		settings.PERMISSION_DEFAULT_CPL_ANY_PERMISSION
64		if self.change_permission is None:
65		self.change_permission = \
66		settings.PERMISSION_DEFAULT_CPL_CHANGE_PERMISSION
67		if self.delete_permission is None:
68		self.delete_permission = \
69		settings.PERMISSION_DEFAULT_CPL_DELETE_PERMISSION
70
71		def has_perm(self, user_obj, perm, obj=None):
72		"""
73		Check if user have permission (of object)
74
75		If the user_obj is not authenticated, it return ``False``.
76
77		If no object is specified, it return ``True`` when the corresponding
78		permission was specified to ``True`` (changed from v0.7.0).
79		This behavior is based on the django system.
80		https://code.djangoproject.com/wiki/RowLevelPermissions
81
82
83		If an object is specified, it will return ``True`` if the user is
84		found in ``field_name`` of the object (e.g. ``obj.collaborators``).
85		So once the object store the user as a collaborator in
86		``field_name`` attribute (default: ``collaborators``), the collaborator
87		can change or delete the object (you can change this behavior to set
88		``any_permission``, ``change_permission`` or ``delete_permission``
89		attributes of this instance).
90
91		Parameters
92		----------
93		user_obj : django user model instance
94		A django user model instance which be checked
95		perm : string
96		`app_label.codename` formatted permission string
97		obj : None or django model instance
98		None or django model instance for object permission
99
100		Returns
101		-------
102		boolean
103		Whether the specified user have specified permission (of specified
104		object).
105		"""
106		if not user_obj.is_authenticated():
107		return False
108		# construct the permission full name
109		change_permission = self.get_full_permission_string('change')
110		delete_permission = self.get_full_permission_string('delete')
111		if obj is None:
112		# object permission without obj should return True
113		# Ref: https://code.djangoproject.com/wiki/RowLevelPermissions
114		if self.any_permission:
115		return True
116		if self.change_permission and perm == change_permission:
117		return True
118		if self.delete_permission and perm == delete_permission:
119		return True
120		return False
121		elif user_obj.is_active:
122		# get collaborator queryset
123		collaborators = field_lookup(obj, self.field_name)
124		if hasattr(collaborators, 'all'):
125		collaborators = collaborators.all()
126		if user_obj in collaborators:
127		if self.any_permission:
128		# have any kind of permissions to the obj
129		return True
130		if (self.change_permission and
131		perm == change_permission):
132		return True
133		if (self.delete_permission and
134		perm == delete_permission):
135		return True
136		return False
137

lambdalisue / django-permission

GitHub Access Token became invalid

Push — master ( 194382...d1a5fb )

CollaboratorsPermissionLogic.has_perm() F

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Complexity

Duplication Side-by-Side

Filter issues like