Sign upLogin

Terralego/django-terra-utils

View on GitHub
Star

Showing 21 of 25 total issues

Function _tree_flattend has a Cognitive Complexity of 24 (exceeds 5 allowed). Consider refactoring.
Open

    def _tree_flattend(self, flatten_tree, tree, parent='', sep='_'):
        for node, child in tree.items():
            parent_node = node if not parent else sep.join((parent, node))
            flatten_child = self.flatten_datatree(child,
                                                  parent=parent_node)
Severity: Minor
Found in terra_utils/helpers/__init__.py - About 3 hrs to fix

Cyclomatic complexity is too high in method get_ordering. (10)
Open

    def get_ordering(self, request, queryset, view):
        ordering = super().get_ordering(request, queryset, view)
 
        if not ordering:
            ordering = []
Severity: Minor
Found in terra_utils/filters.py by radon

Cyclomatic complexity is too high in class JSONFieldOrderingFilter. (10)
Open

class JSONFieldOrderingFilter(OrderingFilter):
 
    def get_ordering(self, request, queryset, view):
        ordering = super().get_ordering(request, queryset, view)
 
 
Severity: Minor
Found in terra_utils/filters.py by radon

Function build_header has a Cognitive Complexity of 18 (exceeds 5 allowed). Consider refactoring.
Open

    def build_header(self, data, parent_key=None):
        header = []
 
        if isinstance(data, dict):
            for colname, colvalue in data.items():
Severity: Minor
Found in terra_utils/helpers/__init__.py - About 2 hrs to fix

Cyclomatic complexity is too high in method build_header. (9)
Open

    def build_header(self, data, parent_key=None):
        header = []
 
        if isinstance(data, dict):
            for colname, colvalue in data.items():
Severity: Minor
Found in terra_utils/helpers/__init__.py by radon

Cyclomatic complexity is too high in method _tree_flattend. (9)
Open

    def _tree_flattend(self, flatten_tree, tree, parent='', sep='_'):
        for node, child in tree.items():
            parent_node = node if not parent else sep.join((parent, node))
            flatten_child = self.flatten_datatree(child,
                                                  parent=parent_node)
Severity: Minor
Found in terra_utils/helpers/__init__.py by radon

Cyclomatic complexity is too high in method filter_queryset. (8)
Open

    def filter_queryset(self, request, queryset, view):
        search_field = getattr(view, 'date_search_field', None)
        date_from = self.parse_date(request.GET.get('date_from', None))
        date_to = self.parse_date(request.GET.get('date_to', None))
 
 
Severity: Minor
Found in terra_utils/filters.py by radon

Cyclomatic complexity is too high in method handle. (7)
Open

    def handle(self, *args, **options):
        self.test_mode = True if options.get('test_data') else False
 
        if options.get('list', False):
            self.stdout.write('Applications with populate modules:')
Severity: Minor
Found in terra_utils/management/commands/populatedata.py by radon

Cyclomatic complexity is too high in function get_media_response. (7)
Open

def get_media_response(request, data, permissions=None, headers=None):
    # For compatibility purpose
    content, url = None, None
    if isinstance(data, (File, )):
        content, url = data, data.url
Severity: Minor
Found in terra_utils/helpers/responses.py by radon

Function get_ordering has a Cognitive Complexity of 12 (exceeds 5 allowed). Consider refactoring.
Open

    def get_ordering(self, request, queryset, view):
        ordering = super().get_ordering(request, queryset, view)
 
        if not ordering:
            ordering = []
Severity: Minor
Found in terra_utils/filters.py - About 1 hr to fix

Function get_media_response has a Cognitive Complexity of 10 (exceeds 5 allowed). Consider refactoring.
Open

def get_media_response(request, data, permissions=None, headers=None):
    # For compatibility purpose
    content, url = None, None
    if isinstance(data, (File, )):
        content, url = data, data.url
Severity: Minor
Found in terra_utils/helpers/responses.py - About 1 hr to fix

Avoid deeply nested control flow statements.
Open

                        if sub_child.keys() <= sub_tree.keys():
                            subtree_copy = deepcopy(sub_tree)
                            subtree_copy.update(sub_child)
                            flatten_tree.append(subtree_copy)
                            break
Severity: Major
Found in terra_utils/helpers/__init__.py - About 45 mins to fix

    Function VALUE_TO_CONST has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.
    Open

        def VALUE_TO_CONST(self):
            """
            Dict {"value": "const"}
            """
            if self._VALUE_TO_CONST is None:
    Severity: Minor
    Found in terra_utils/helpers/__init__.py - About 25 mins to fix

    Function REVERTED_CHOICES_DICT has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.
    Open

        def REVERTED_CHOICES_DICT(self):
            """
            Dict {"display_value": "value"}
            """
            # FIXME: rename in a more friendly name, like STRING_TO_VALUE?
    Severity: Minor
    Found in terra_utils/helpers/__init__.py - About 25 mins to fix

    Function handle has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.
    Open

        def handle(self, *args, **options):
            self.test_mode = True if options.get('test_data') else False
     
            if options.get('list', False):
                self.stdout.write('Applications with populate modules:')
    Severity: Minor
    Found in terra_utils/management/commands/populatedata.py - About 25 mins to fix

    Function CHOICES_DICT has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.
    Open

        def CHOICES_DICT(self):
            if self._CHOICES_DICT is None:
                self._CHOICES_DICT = {}
                for c, k, v in self.RAW_CHOICES:
                    if c in self._CONSTS:
    Severity: Minor
    Found in terra_utils/helpers/__init__.py - About 25 mins to fix

    Merge this if statement with the enclosing one.
    Open

            if not set(permissions).intersection(
    Severity: Major
    Found in terra_utils/helpers/responses.py by sonar-python

    Take the required action to fix the issue indicated by this "FIXME" comment.
    Open

            # FIXME: rename in a more friendly name, like STRING_TO_VALUE?
    Severity: Major
    Found in terra_utils/helpers/__init__.py by sonar-python

    Refactor this function to reduce its Cognitive Complexity from 18 to the 15 allowed.
    Open

        def build_header(self, data, parent_key=None):
    Severity: Critical
    Found in terra_utils/helpers/__init__.py by sonar-python

    Use of RawSQL potential SQL attack vector.
    Open

                        OrderBy(RawSQL('lower({})'.format(tpl), json_nested),
    Severity: Minor
    Found in terra_utils/filters.py by bandit
    Severity
    Category
    Status
    Source
    Language