UpdateWSession.to_dict() - Code Metrics - Inspection of "Updated peewee_migrate" - fuzeman/Plex-Trakt-Scrobbler - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — develop ( b3bfc7...6fb855 )

by Dean

created 2016-02-09 02:11 UTC

UpdateWSession.to_dict() C

↳ Parent: UpdateWSession

Complexity

Conditions

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	56

Metric	Value
cc	7
dl	0
loc	79
ccs	0
cts	31
cp	0
crap	56
rs	5.3799

How to fix Long Method

from plugin.core.helpers.variable import to_integer, merge, resolve
from plugin.core.session_status import SessionStatus
from plugin.managers.core.base import Manager
# .scrutinizer.yml
before_commands:
    - sudo pip install abc # Python2
    - sudo pip3 install abc # Python3
from plugin.managers.client import ClientManager
from plugin.managers.core.exceptions import FilteredException
from plugin.managers.session.base import GetSession, UpdateSession
# .scrutinizer.yml
before_commands:
    - sudo pip install abc # Python2
    - sudo pip3 install abc # Python3
from plugin.managers.user import UserManager
from plugin.models import Session

from datetime import datetime
from plex import Plex
import apsw
# .scrutinizer.yml
before_commands:
    - sudo pip install abc # Python2
    - sudo pip3 install abc # Python3
import logging
import peewee


log = logging.getLogger(__name__)


class GetWSession(GetSession):

    def __call__(self, info):
        session_key = to_integer(info.get('sessionKey'))

        return super(GetWSession, self).__call__(
            Session.session_key == self.build_session_key(session_key)

        )

    def or_create(self, info, fetch=False):
        session_key = to_integer(info.get('sessionKey'))

        try:
            # Create new session
            obj = self.manager.create(

                rating_key=to_integer(info.get('ratingKey')),
                session_key=self.build_session_key(session_key),


                state='create'
            )

            # Update newly created object
            self.manager.update(obj, info, fetch)


            # Update active sessions
            SessionStatus.on_created(obj)

            return obj
        except (apsw.ConstraintError, peewee.IntegrityError):
            # Return existing object
            return self(info)


class UpdateWSession(UpdateSession):

    def __call__(self, obj, info, fetch=False):
        data = self.to_dict(obj, info, fetch)

        success = super(UpdateWSession, self).__call__(
            obj, data
        )

        SessionStatus.on_updated(obj)
        return success

    def to_dict(self, obj, info, fetch=False):
        fetch = resolve(fetch, obj, info)

        view_offset = to_integer(info.get('viewOffset'))

        result = {
            'rating_key': to_integer(info.get('ratingKey')),
            'view_offset': view_offset,

            'updated_at': datetime.utcnow()
        }

        if not fetch:
            # Return simple update
            return merge(result, {
                'progress': self.get_progress(obj.duration, view_offset)

            })

        # Retrieve session key
        session_key = to_integer(info.get('sessionKey'))

        if not session_key:
            log.warn('Missing session key, unable to fetch session details')
            return result

        # Retrieve session details
        log.debug('Fetching details for session #%s', session_key)

        p_item = Plex['status'].sessions().get(session_key)

        if not p_item:
            log.warn('Unable to find session with key %r', session_key)
            return result

        # Retrieve metadata and guid
        p_metadata, guid = self.get_metadata(p_item.rating_key)


        if not p_metadata:
            log.warn('Unable to retrieve metadata for rating_key %r', p_item.rating_key)
            return result

        if not guid:
            return merge(result, {
                'duration': p_metadata.duration,
                'progress': self.get_progress(p_metadata.duration, view_offset)

            })

        try:
            # Create/Retrieve `Client` for session
            result['client'] = ClientManager.get.or_create(

                p_item.session.player,

                fetch=True,
                match=True,
                filtered_exception=True
            )

            # Create/Retrieve `User` for session
            result['user'] = UserManager.get.or_create(

                p_item.session.user,

                fetch=True,
                match=True,
                filtered_exception=True
            )

            # Pick account from `client` or `user` objects
            result['account'] = self.get_account(result)

        except FilteredException:
            log.debug('Activity has been filtered')

            result['client'] = None
            result['user'] = None

            result['account'] = None

        return merge(result, {
            'duration': p_metadata.duration,
            'progress': self.get_progress(p_metadata.duration, view_offset)

        })


class WSessionManager(Manager):

    get = GetWSession
    update = UpdateWSession

    model = Session


1			from plugin.core.helpers.variable import to_integer, merge, resolve
2			from plugin.core.session_status import SessionStatus
3			from plugin.managers.core.base import Manager
			0 ignored issues – show Bug introduced 2015-12-19 01:32 UTC by Report Bug Copy Issue Report The name `base` does not seem to exist in module `plugin.managers.core`. Loading history... Configuration introduced 2015-12-19 01:32 UTC by Report Bug Copy Issue Report Unable to import 'plugin.managers.core.base' (invalid syntax (<string>, line 39)) 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...
4			from plugin.managers.client import ClientManager
5			from plugin.managers.core.exceptions import FilteredException
6			from plugin.managers.session.base import GetSession, UpdateSession
			0 ignored issues – show Bug introduced 2015-12-19 01:32 UTC by Report Bug Copy Issue Report The name `base` does not seem to exist in module `plugin.managers.session`. Loading history... Configuration introduced 2015-12-19 01:32 UTC by Report Bug Copy Issue Report Unable to import 'plugin.managers.session.base' (invalid syntax (<string>, line 63)) 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...
7			from plugin.managers.user import UserManager
8			from plugin.models import Session
9
10			from datetime import datetime
11			from plex import Plex
12			import apsw
			0 ignored issues – show Configuration introduced 2015-12-19 01:32 UTC by Report Bug Copy Issue Report The import `apsw` could not be resolved. 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...
13			import logging
14			import peewee
15
16
17			log = logging.getLogger(__name__)
18
19
20			class GetWSession(GetSession):
			0 ignored issues – show Coding Style introduced 2015-12-19 01:32 UTC by Report Bug Copy Issue Report This class has no `__init__` method. Loading history...
21			def __call__(self, info):
22			session_key = to_integer(info.get('sessionKey'))
23
24			return super(GetWSession, self).__call__(
25			Session.session_key == self.build_session_key(session_key)
			0 ignored issues – show Bug introduced 2015-12-19 01:32 UTC by Report Bug Copy Issue Report The Instance of `GetWSession` does not seem to have a member named `build_session_key`. This check looks for calls to members that are non-existent. These calls will fail. The member could have been renamed or removed. Loading history...
26			)
27
28			def or_create(self, info, fetch=False):
29			session_key = to_integer(info.get('sessionKey'))
30
31			try:
32			# Create new session
33			obj = self.manager.create(
			0 ignored issues – show Bug introduced 2015-12-19 01:32 UTC by Report Bug Copy Issue Report The Instance of `GetWSession` does not seem to have a member named `manager`. This check looks for calls to members that are non-existent. These calls will fail. The member could have been renamed or removed. Loading history...
34			rating_key=to_integer(info.get('ratingKey')),
35			session_key=self.build_session_key(session_key),
			0 ignored issues – show Bug introduced 2015-12-19 01:32 UTC by Report Bug Copy Issue Report The Instance of `GetWSession` does not seem to have a member named `build_session_key`. This check looks for calls to members that are non-existent. These calls will fail. The member could have been renamed or removed. Loading history...
36
37			state='create'
38			)
39
40			# Update newly created object
41			self.manager.update(obj, info, fetch)
			0 ignored issues – show Bug introduced 2015-12-19 01:32 UTC by Report Bug Copy Issue Report The Instance of `GetWSession` does not seem to have a member named `manager`. This check looks for calls to members that are non-existent. These calls will fail. The member could have been renamed or removed. Loading history...
42
43			# Update active sessions
44			SessionStatus.on_created(obj)
45
46			return obj
47			except (apsw.ConstraintError, peewee.IntegrityError):
48			# Return existing object
49			return self(info)
50
51
52			class UpdateWSession(UpdateSession):
			0 ignored issues – show Coding Style introduced 2015-12-19 01:32 UTC by Report Bug Copy Issue Report This class has no `__init__` method. Loading history...
53			def __call__(self, obj, info, fetch=False):
54			data = self.to_dict(obj, info, fetch)
55
56			success = super(UpdateWSession, self).__call__(
57			obj, data
58			)
59
60			SessionStatus.on_updated(obj)
61			return success
62
63			def to_dict(self, obj, info, fetch=False):
64			fetch = resolve(fetch, obj, info)
65
66			view_offset = to_integer(info.get('viewOffset'))
67
68			result = {
69			'rating_key': to_integer(info.get('ratingKey')),
70			'view_offset': view_offset,
71
72			'updated_at': datetime.utcnow()
73			}
74
75			if not fetch:
76			# Return simple update
77			return merge(result, {
78			'progress': self.get_progress(obj.duration, view_offset)
			0 ignored issues – show Bug introduced 2015-12-19 01:32 UTC by Report Bug Copy Issue Report The Instance of `UpdateWSession` does not seem to have a member named `get_progress`. This check looks for calls to members that are non-existent. These calls will fail. The member could have been renamed or removed. Loading history...
79			})
80
81			# Retrieve session key
82			session_key = to_integer(info.get('sessionKey'))
83
84			if not session_key:
85			log.warn('Missing session key, unable to fetch session details')
86			return result
87
88			# Retrieve session details
89			log.debug('Fetching details for session #%s', session_key)
90
91			p_item = Plex['status'].sessions().get(session_key)
92
93			if not p_item:
94			log.warn('Unable to find session with key %r', session_key)
95			return result
96
97			# Retrieve metadata and guid
98			p_metadata, guid = self.get_metadata(p_item.rating_key)
			0 ignored issues – show Bug introduced 2015-12-19 01:32 UTC by Report Bug Copy Issue Report The Instance of `UpdateWSession` does not seem to have a member named `get_metadata`. This check looks for calls to members that are non-existent. These calls will fail. The member could have been renamed or removed. Loading history...
99
100			if not p_metadata:
101			log.warn('Unable to retrieve metadata for rating_key %r', p_item.rating_key)
102			return result
103
104			if not guid:
105			return merge(result, {
106			'duration': p_metadata.duration,
107			'progress': self.get_progress(p_metadata.duration, view_offset)
			0 ignored issues – show Bug introduced 2015-12-19 01:32 UTC by Report Bug Copy Issue Report The Instance of `UpdateWSession` does not seem to have a member named `get_progress`. This check looks for calls to members that are non-existent. These calls will fail. The member could have been renamed or removed. Loading history...
108			})
109
110			try:
111			# Create/Retrieve `Client` for session
112			result['client'] = ClientManager.get.or_create(
			0 ignored issues – show Bug introduced 2015-12-19 01:32 UTC by Report Bug Copy Issue Report It seems like a value for argument `player` is missing in the unbound method call. Loading history...
113			p_item.session.player,
114
115			fetch=True,
116			match=True,
117			filtered_exception=True
118			)
119
120			# Create/Retrieve `User` for session
121			result['user'] = UserManager.get.or_create(
			0 ignored issues – show Bug introduced 2015-12-19 01:32 UTC by Report Bug Copy Issue Report It seems like a value for argument `user` is missing in the unbound method call. Loading history...
122			p_item.session.user,
123
124			fetch=True,
125			match=True,
126			filtered_exception=True
127			)
128
129			# Pick account from `client` or `user` objects
130			result['account'] = self.get_account(result)
			0 ignored issues – show Bug introduced 2015-12-19 01:32 UTC by Report Bug Copy Issue Report The Instance of `UpdateWSession` does not seem to have a member named `get_account`. This check looks for calls to members that are non-existent. These calls will fail. The member could have been renamed or removed. Loading history...
131			except FilteredException:
132			log.debug('Activity has been filtered')
133
134			result['client'] = None
135			result['user'] = None
136
137			result['account'] = None
138
139			return merge(result, {
140			'duration': p_metadata.duration,
141			'progress': self.get_progress(p_metadata.duration, view_offset)
			0 ignored issues – show Bug introduced 2015-12-19 01:32 UTC by Report Bug Copy Issue Report The Instance of `UpdateWSession` does not seem to have a member named `get_progress`. This check looks for calls to members that are non-existent. These calls will fail. The member could have been renamed or removed. Loading history...
142			})
143
144
145			class WSessionManager(Manager):
			0 ignored issues – show Coding Style introduced 2015-12-19 01:32 UTC by Report Bug Copy Issue Report This class has no `__init__` method. Loading history...
146			get = GetWSession
147			update = UpdateWSession
148
149			model = Session
150

Push — develop ( b3bfc7...6fb855 )

UpdateWSession.to_dict() C

Complexity

Size

Duplication

Code Coverage

How to fix Long Method

Long Method

1. Missing Dependencies

2. Missing init.py files

1. Missing Dependencies

2. Missing init.py files

1. Missing Dependencies

2. Missing init.py files

fuzeman / Plex-Trakt-Scrobbler

Push — develop ( b3bfc7...6fb855 )

UpdateWSession.to_dict() C

Complexity

Size

Duplication

Code Coverage

How to fix Long Method

Long Method

1. Missing Dependencies

2. Missing __init__.py files

1. Missing Dependencies

2. Missing __init__.py files

1. Missing Dependencies

2. Missing __init__.py files

Duplication Side-by-Side

Filter issues like

2. Missing init.py files

2. Missing init.py files

2. Missing init.py files