backend/apps/user/api/views.py from cnap-cobre/synapse

backend/apps/user/api/views.py

Summary

Maintainability

55 mins

Test Coverage

Cyclomatic complexity is too high in method get_serializer_class. (6)
Open

    def get_serializer_class(self):
        """
        - Limit signup fields for anonymous
        - Expose all fields to privileged users
        - Limit signup fields for non-privileged users

Found in backend/apps/user/api/views.py by radon

Read up
Read up
Exclude checks
- Disable engine
- Disable check

Cyclomatic complexity is too high in method get_permissions. (6)
Open

    def get_permissions(self):
        """
        - Anyone can create a user and view available methods
        - Updates require permission or ownership
        - Retrieval requires authentication

Found in backend/apps/user/api/views.py by radon

Read up
Read up
Exclude checks
- Disable engine
- Disable check

Avoid too many `return` statements within this function.
Open

        return basic.UserSerializer

Found in backend/apps/user/api/views.py - About 30 mins to fix

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

    def get_serializer_class(self):
        """
        - Limit signup fields for anonymous
        - Expose all fields to privileged users
        - Limit signup fields for non-privileged users

Found in backend/apps/user/api/views.py - About 25 mins to fix

Read up
Read up

Construct	Effect on CC	Reasoning
if	+1	An if statement is a single decision.
elif	+1	The elif statement adds another decision.
else	+0	The else statement does not cause a new decision. The decision is at the if.
for	+1	There is a decision at the start of the loop.
while	+1	There is a decision at the while statement.
except	+1	Each except branch adds a new conditional path of execution.
finally	+0	The finally block is unconditionally executed.
with	+1	The with statement roughly corresponds to a try/except block (see PEP 343 for details).
assert	+1	The assert statement internally roughly equals a conditional statement.
Comprehension	+1	A list/set/dict comprehension of generator expression is equivalent to a for loop.
Boolean Operator	+1	Every boolean operator (and, or) adds a decision point.

Construct	Effect on CC	Reasoning
if	+1	An if statement is a single decision.
elif	+1	The elif statement adds another decision.
else	+0	The else statement does not cause a new decision. The decision is at the if.
for	+1	There is a decision at the start of the loop.
while	+1	There is a decision at the while statement.
except	+1	Each except branch adds a new conditional path of execution.
finally	+0	The finally block is unconditionally executed.
with	+1	The with statement roughly corresponds to a try/except block (see PEP 343 for details).
assert	+1	The assert statement internally roughly equals a conditional statement.
Comprehension	+1	A list/set/dict comprehension of generator expression is equivalent to a for loop.
Boolean Operator	+1	Every boolean operator (and, or) adds a decision point.

cnap-cobre/synapse

Summary

Maintainability

Test Coverage

Showing 4 of 4 total issues

Cyclomatic complexity is too high in method get_serializer_class. (6)
Open

Cyclomatic complexity is too high in method get_permissions. (6)
Open

Avoid too many `return` statements within this function.
Open

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

There are no issues that match your filters.

Category

Status

Cyclomatic Complexity

Cyclomatic Complexity

Cognitive Complexity

A method's cognitive complexity is based on a few simple rules:

Further reading

cnap-cobre/synapse

Summary

Maintainability

Test Coverage

Showing 4 of 4 total issues

Cyclomatic complexity is too high in method get_serializer_class. (6) Open

Cyclomatic complexity is too high in method get_permissions. (6) Open

Avoid too many return statements within this function. Open

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

There are no issues that match your filters.

Category

Status

Cyclomatic Complexity

Cyclomatic Complexity

Cognitive Complexity

A method's cognitive complexity is based on a few simple rules:

Further reading

Cyclomatic complexity is too high in method get_serializer_class. (6)
Open

Cyclomatic complexity is too high in method get_permissions. (6)
Open

Avoid too many `return` statements within this function.
Open

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