outer() - Code Metrics - Inspection of "Merge branch 'issue/253' into develop" - fuzeman/Plex-Trakt-Scrobbler - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — develop ( af752c...572338 )

by Dean

created 2016-01-14 20:45 UTC

outer() A

↳ Parent: expose()

Complexity

Conditions

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	20

Metric	Value
cc	4
dl	0
loc	14
ccs	0
cts	7
cp	0
crap	20
rs	9.2

from plugin.api.core.exceptions import ApiError
from plugin.api.core.manager import ApiManager
# .scrutinizer.yml
before_commands:
    - sudo pip install abc # Python2
    - sudo pip3 install abc # Python3
from plugin.core.helpers import decorator

import logging

log = logging.getLogger(__name__)


class AuthenticationRequiredError(ApiError):
    code = 'authentication.required'
    message = 'API authentication required'


class ServiceMeta(type):
    def __init__(cls, name, bases, attributes):
        super(ServiceMeta, cls).__init__(name, bases, attributes)

        if '__metaclass__' in attributes:
            return

        # Register API service
        ApiManager.register(cls)


class Service(object):
    __metaclass__ = ServiceMeta
    __key__ = None

    manager = None

    def __init__(self, manager):
        self.manager = manager

    @property
    def context(self):
        return self.manager.context


@decorator.wraps
def expose(authenticated=True):
    def outer(func):
        def inner(self, *args, **kwargs):
            if authenticated and self.context.token is None:
                raise AuthenticationRequiredError

            return func(self, *args, **kwargs)

        # Attach meta to wrapper
        inner.__meta__ = {
            'authenticated': authenticated,
            'exposed': True
        }

        return inner

    return outer


Push — develop ( af752c...572338 )

outer() A

Complexity

Size

Duplication

Code Coverage

1. Missing Dependencies

2. Missing init.py files

1			from plugin.api.core.exceptions import ApiError
2			from plugin.api.core.manager import ApiManager
			0 ignored issues – show Bug introduced 2015-12-19 01:32 UTC by Report Bug Copy Issue Report The name `manager` does not seem to exist in module `plugin.api.core`. Loading history... Configuration introduced 2015-12-19 01:32 UTC by Report Bug Copy Issue Report Unable to import 'plugin.api.core.manager' (invalid syntax (<string>, line 77)) This can be caused by one of the following: 1. Missing Dependencies This error could indicate a configuration issue of Pylint. Make sure that your libraries are available by adding the necessary commands. # .scrutinizer.yml before_commands: - sudo pip install abc # Python2 - sudo pip3 install abc # Python3 Tip: We are currently not using virtualenv to run pylint, when installing your modules make sure to use the command for the correct version. 2. Missing __init__.py files This error could also result from missing `__init__.py` files in your module folders. Make sure that you place one file in each sub-folder. Loading history...
3			from plugin.core.helpers import decorator
4
5			import logging
6
7			log = logging.getLogger(__name__)
8
9
10			class AuthenticationRequiredError(ApiError):
11			code = 'authentication.required'
12			message = 'API authentication required'
13
14
15			class ServiceMeta(type):
16			def __init__(cls, name, bases, attributes):
17			super(ServiceMeta, cls).__init__(name, bases, attributes)
18
19			if '__metaclass__' in attributes:
20			return
21
22			# Register API service
23			ApiManager.register(cls)
24
25
26			class Service(object):
27			__metaclass__ = ServiceMeta
28			__key__ = None
29
30			manager = None
31
32			def __init__(self, manager):
33			self.manager = manager
34
35			@property
36			def context(self):
37			return self.manager.context
38
39
40			@decorator.wraps
41			def expose(authenticated=True):
42			def outer(func):
43			def inner(self, args, *kwargs):
44			if authenticated and self.context.token is None:
45			raise AuthenticationRequiredError
46
47			return func(self, args, *kwargs)
48
49			# Attach meta to wrapper
50			inner.__meta__ = {
51			'authenticated': authenticated,
52			'exposed': True
53			}
54
55			return inner
56
57			return outer
58

fuzeman / Plex-Trakt-Scrobbler

Push — develop ( af752c...572338 )

outer() A

Complexity

Size

Duplication

Code Coverage

1. Missing Dependencies

2. Missing __init__.py files

Duplication Side-by-Side

Filter issues like

2. Missing init.py files