tcllib.devices.Device.set_mode() - Code Metrics - Inspection of "Moved device-specific settings to own class. Only..." - mbirth/tcl_ota_check - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( ba99c1...808282 )

by Markus

created 2018-02-05 23:23 UTC

tcllib.devices.Device.set_mode() A

↳ Parent: tcllib.devices

Complexity

Conditions

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
eloc	2
nop	2
dl	0
loc	4
rs	10
c	0
b	0
f	0

# -*- coding: utf-8 -*-

"""Pseudo-devices for desktop/mobile requests"""

class Device():
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
    CLTP_STATES = {
        "MOBILE": 10,     # only show actual newer versions or HTTP 206
        "DESKTOP": 2010,  # always show latest version for FULL updates
    }
    MODE_STATES = {"OTA": 2, "FULL": 4}
    CHNL_STATES = {"3G": 1, "WIFI": 2}
    CKTP_STATES = {"AUTO": 1, "MANUAL": 2}
    CKOT_STATES = {"ALL": 1, "AOTA_ONLY": 2, "FOTA_ONLY": 3}

    def __init__(self, curef, fwver):
        self.curef = curef
        self.imei = ""
        self.osver = "7.1.1"
        self.fwver = fwver
        self.rtd = 0
        self.cltp = self.CLTP_STATES["DESKTOP"]
        self.mode = self.MODE_STATES["FULL"]
        self.type = "Firmware"
        self.chnl = self.CHNL_STATES["WIFI"]
        self.cktp = self.CKTP_STATES["MANUAL"]
        self.ckot = self.CKOT_STATES["ALL"]
        self.ua = "tcl"

    def is_rooted(self):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        return (self.rtd == 1)


    def set_rooted(self, new_state: bool):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        if new_state:
            self.rtd = 1
        else:
            self.rtd = 0

    def set_cltp(self, new_cltp: str):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        # (Numerical CLTPs can be set by direct assigns.)
        # Throws exception when invalid cltp given:
        self.cltp = self.CLTP_STATES[new_cltp]

    def set_mode(self, new_mode: str):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        # (Numerical MODEs can be set by direct assigns.)
        # Throws exception when invalid mode given:
        self.mode = self.MODE_STATES[new_mode]

    def set_chnl(self, new_chnl: str):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        # (Numerical CHNLs can be set by direct assigns.)
        # Throws exception when invalid mode given:
        self.chnl = self.CHNL_STATES[new_chnl]

    def set_ckot(self, new_ckot: str):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        # (Numerical CKOTs can be set by direct assigns.)
        # Throws exception when invalid mode given:
        self.ckot = self.CKOT_STATES[new_ckot]

class MobileDevice(Device):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
    def __init__(self, curef="PRD-63117-011", fwver="AAO472"):
        super().__init__(curef, fwver)
        self.imei = "3531510"
        self.set_cltp("MOBILE")
        self.set_mode("OTA")
        self.ua = "com.tcl.fota/5.1.0.2.0029.0, Android"

class DesktopDevice(Device):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
    def __init__(self, curef="PRD-63117-011", fwver="AAA000"):
        super().__init__(curef, fwver)
        self.imei = "543212345000000"
        self.set_cltp("DESKTOP")
        self.set_mode("FULL")


1			# -- coding: utf-8 --
2
3			"""Pseudo-devices for desktop/mobile requests"""
4
5			class Device():
			0 ignored issues – show Coding Style introduced 2018-02-05 23:24 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-02-05 23:24 UTC by Report Bug Copy Issue Report Too many instance attributes (12/7) Loading history... Unused Code introduced 2018-02-05 23:24 UTC by Report Bug Copy Issue Report The variable `__class__` seems to be unused. Loading history...
6			CLTP_STATES = {
7			"MOBILE": 10, # only show actual newer versions or HTTP 206
8			"DESKTOP": 2010, # always show latest version for FULL updates
9			}
10			MODE_STATES = {"OTA": 2, "FULL": 4}
11			CHNL_STATES = {"3G": 1, "WIFI": 2}
12			CKTP_STATES = {"AUTO": 1, "MANUAL": 2}
13			CKOT_STATES = {"ALL": 1, "AOTA_ONLY": 2, "FOTA_ONLY": 3}
14
15			def __init__(self, curef, fwver):
16			self.curef = curef
17			self.imei = ""
18			self.osver = "7.1.1"
19			self.fwver = fwver
20			self.rtd = 0
21			self.cltp = self.CLTP_STATES["DESKTOP"]
22			self.mode = self.MODE_STATES["FULL"]
23			self.type = "Firmware"
24			self.chnl = self.CHNL_STATES["WIFI"]
25			self.cktp = self.CKTP_STATES["MANUAL"]
26			self.ckot = self.CKOT_STATES["ALL"]
27			self.ua = "tcl"
28
29			def is_rooted(self):
			0 ignored issues – show Coding Style introduced 2018-02-05 23:24 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...
30			return (self.rtd == 1)
			0 ignored issues – show Unused Code Coding Style introduced 2018-02-05 23:24 UTC by Report Bug Copy Issue Report There is an unnecessary parenthesis after `return`. Loading history...
31
32			def set_rooted(self, new_state: bool):
			0 ignored issues – show Coding Style introduced 2018-02-05 23:24 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 new_state:
34			self.rtd = 1
35			else:
36			self.rtd = 0
37
38			def set_cltp(self, new_cltp: str):
			0 ignored issues – show Coding Style introduced 2018-02-05 23:24 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...
39			# (Numerical CLTPs can be set by direct assigns.)
40			# Throws exception when invalid cltp given:
41			self.cltp = self.CLTP_STATES[new_cltp]
42
43			def set_mode(self, new_mode: str):
			0 ignored issues – show Coding Style introduced 2018-02-05 23:24 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			# (Numerical MODEs can be set by direct assigns.)
45			# Throws exception when invalid mode given:
46			self.mode = self.MODE_STATES[new_mode]
47
48			def set_chnl(self, new_chnl: str):
			0 ignored issues – show Coding Style introduced 2018-02-05 23:24 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...
49			# (Numerical CHNLs can be set by direct assigns.)
50			# Throws exception when invalid mode given:
51			self.chnl = self.CHNL_STATES[new_chnl]
52
53			def set_ckot(self, new_ckot: str):
			0 ignored issues – show Coding Style introduced 2018-02-05 23:24 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...
54			# (Numerical CKOTs can be set by direct assigns.)
55			# Throws exception when invalid mode given:
56			self.ckot = self.CKOT_STATES[new_ckot]
57
58			class MobileDevice(Device):
			0 ignored issues – show Coding Style introduced 2018-02-05 23:24 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... Unused Code introduced 2018-02-05 23:24 UTC by Report Bug Copy Issue Report The variable `__class__` seems to be unused. Loading history...
59			def __init__(self, curef="PRD-63117-011", fwver="AAO472"):
60			super().__init__(curef, fwver)
61			self.imei = "3531510"
62			self.set_cltp("MOBILE")
63			self.set_mode("OTA")
64			self.ua = "com.tcl.fota/5.1.0.2.0029.0, Android"
65
66			class DesktopDevice(Device):
			0 ignored issues – show Coding Style introduced 2018-02-05 23:24 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... Unused Code introduced 2018-02-05 23:24 UTC by Report Bug Copy Issue Report The variable `__class__` seems to be unused. Loading history...
67			def __init__(self, curef="PRD-63117-011", fwver="AAA000"):
68			super().__init__(curef, fwver)
69			self.imei = "543212345000000"
70			self.set_cltp("DESKTOP")
71			self.set_mode("FULL")
72

mbirth / tcl_ota_check

Push — master ( ba99c1...808282 )

tcllib.devices.Device.set_mode() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like