tracim.lib.utils.request.TracimRequest.current_workspace() - Code Metrics - Inspection of "new authorization mecanism with decorators" - tracim/tracim_backend - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 68bc61...042191 )

unknown

created 2018-04-12 10:19 UTC

TracimRequest.current_workspace() A

↳ Parent: tracim.lib.utils.request

Complexity

Conditions

Size

Total Lines	5
Code Lines	4

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
eloc	4
dl	0
loc	5
rs	9.4285
c	0
b	0
f	0
cc	2
nop	2

from contextlib import contextmanager
class SomeClass:
    def some_method(self):
        """Do x and return foo."""

from pyramid.decorator import reify

from pyramid.request import Request

from tracim.models import User
from tracim.models.data import Workspace
from tracim.lib.utils.auth import get_safe_user
from tracim.lib.utils.auth import get_workspace


class TracimRequest(Request):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
    def __init__(
            self,
            environ,
            charset=None,
            unicode_errors=None,
            decode_param_names=None,
            **kw
    ):
        super().__init__(
            environ,
            charset,
            unicode_errors,
            decode_param_names,
            **kw
        )
        self._current_workspace = None  # type: Workspace
        self._current_user = None  # type: User

    @property
    def current_workspace(self) -> Workspace:
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        if self._current_workspace is None:
            self.current_workspace = get_workspace(self.current_user, self)
        return self._current_workspace

    @current_workspace.setter
    def current_workspace(self, workspace: Workspace) -> None:
        assert self._current_workspace is None
        self._current_workspace = workspace

    @property
    def current_user(self) -> User:
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        if self._current_user is None:
            self.current_user = get_safe_user(self)
        return self._current_user

    @current_user.setter
    def current_user(self, user: User) -> None:
        assert self._current_user is None
        self._current_user = user


1			from contextlib import contextmanager
			0 ignored issues – show Coding Style introduced 2018-04-12 10:22 UTC by Report Bug Copy Issue Report This module should have a docstring. The coding style of this project requires that you add a docstring to this code element. Below, you find an example for methods: class SomeClass: def some_method(self): """Do x and return foo.""" If you would like to know more about docstrings, we recommend to read PEP-257: Docstring Conventions. Loading history... Unused Code introduced 2018-04-12 10:22 UTC by Report Bug Copy Issue Report Unused contextmanager imported from contextlib Loading history...
2
3			from pyramid.decorator import reify
			0 ignored issues – show Unused Code introduced 2018-04-12 10:22 UTC by Report Bug Copy Issue Report Unused reify imported from pyramid.decorator Loading history...
4			from pyramid.request import Request
5
6			from tracim.models import User
7			from tracim.models.data import Workspace
8			from tracim.lib.utils.auth import get_safe_user
9			from tracim.lib.utils.auth import get_workspace
10
11
12			class TracimRequest(Request):
			0 ignored issues – show Coding Style introduced 2018-04-12 10:22 UTC by Report Bug Copy Issue Report This class should have a docstring. The coding style of this project requires that you add a docstring to this code element. Below, you find an example for methods: class SomeClass: def some_method(self): """Do x and return foo.""" If you would like to know more about docstrings, we recommend to read PEP-257: Docstring Conventions. Loading history... best-practice introduced 2018-04-12 10:22 UTC by Report Bug Copy Issue Report Too many ancestors (10/7) Loading history... Unused Code introduced 2018-04-12 10:22 UTC by Report Bug Copy Issue Report The variable `__class__` seems to be unused. Loading history...
13			def __init__(
14			self,
15			environ,
16			charset=None,
17			unicode_errors=None,
18			decode_param_names=None,
19			**kw
20			):
21			super().__init__(
22			environ,
23			charset,
24			unicode_errors,
25			decode_param_names,
26			**kw
27			)
28			self._current_workspace = None # type: Workspace
29			self._current_user = None # type: User
30
31			@property
32			def current_workspace(self) -> Workspace:
			0 ignored issues – show Coding Style introduced 2018-04-12 10:22 UTC by Report Bug Copy Issue Report This method should have a docstring. The coding style of this project requires that you add a docstring to this code element. Below, you find an example for methods: class SomeClass: def some_method(self): """Do x and return foo.""" If you would like to know more about docstrings, we recommend to read PEP-257: Docstring Conventions. Loading history...
33			if self._current_workspace is None:
34			self.current_workspace = get_workspace(self.current_user, self)
35			return self._current_workspace
36
37			@current_workspace.setter
38			def current_workspace(self, workspace: Workspace) -> None:
39			assert self._current_workspace is None
40			self._current_workspace = workspace
41
42			@property
43			def current_user(self) -> User:
			0 ignored issues – show Coding Style introduced 2018-04-12 10:22 UTC by Report Bug Copy Issue Report This method should have a docstring. The coding style of this project requires that you add a docstring to this code element. Below, you find an example for methods: class SomeClass: def some_method(self): """Do x and return foo.""" If you would like to know more about docstrings, we recommend to read PEP-257: Docstring Conventions. Loading history...
44			if self._current_user is None:
45			self.current_user = get_safe_user(self)
46			return self._current_user
47
48			@current_user.setter
49			def current_user(self, user: User) -> None:
50			assert self._current_user is None
51			self._current_user = user
52

tracim / tracim_backend

Push — master ( 68bc61...042191 )

TracimRequest.current_workspace() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like