Showing 12 of 23 total issues
Function post
has a Cognitive Complexity of 259 (exceeds 5 allowed). Consider refactoring. Open
Open
def post(self, request, format=None):
# Some docs:
# Slack
# Basic formatting: https://api.slack.com/docs/message-formatting
generic_error_msg = _('Something went wrong here... I will tell the developers and hopefully they will manage to fix this.')
- Read upRead up
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"
Further reading
File restviews.py
has 402 lines of code (exceeds 250 allowed). Consider refactoring. Open
Open
# -*- coding: utf-8 -*-
from __future__ import absolute_import, unicode_literals
# TODO
# v Output formatting of dictionaries
Avoid deeply nested control flow statements. Open
Open
if('tags' in parser.specification['paths'][path][method]):
if(parameters['object'].lower() in [tag.lower() for tag in parser.specification['paths'][path][method]['tags']]):
operation['path'] = path
operation['method'] = method
operations.append(operation)
Avoid deeply nested control flow statements. Open
Open
if(parameters['object'].lower() in path):
operation['path'] = path
operation['method'] = method
operations.append(operation)
break
Avoid deeply nested control flow statements. Open
Open
if('operationId' in parser.specification['paths'][path][method]):
operation = {
'type': 'operation',
'value': parser.specification['paths'][path][method]['operationId'],
}
Avoid deeply nested control flow statements. Open
Open
if(parameters['object'].lower() in operation):
operation['path'] = path
operation['method'] = method
operations.append(operation)
break
Avoid deeply nested control flow statements. Open
Open
if(operations):
# Define buttons for Slack
actions = []
for operation in operations:
Avoid deeply nested control flow statements. Open
Open
if(definitions):
# Define buttons for Slack
actions = []
for definition in definitions:
Avoid deeply nested control flow statements. Open
Open
if(paths):
# Define buttons for Slack
actions = []
for path in paths:
Avoid deeply nested control flow statements. Open
Open
if('parameters' in parser.specification['paths'][path][method]):
for parameter in parser.specification['paths'][path][method]['parameters']:
# Regex the object out
try:
match = re.match(r'#/definitions/(\w+)', parameter['schema']['$ref'])
Avoid deeply nested control flow statements. Open
Open
if('responses' in parser.specification['paths'][path][method]):
try:
for status_code in parser.specification['paths'][path][method]['responses']:
if('schema' in parser.specification['paths'][path][method]['responses'][status_code]):
# In case of a list of objects
Function get_api
has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring. Open
Open
def get_api(self, parameters, contexts):
if 'api' in parameters:
return parameters['api']
elif contexts:
for context in contexts:
- Read upRead up
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"