Refly

Complexity

Total Complexity

6

Size/Duplication

Total Lines	69
Duplicated Lines	100 %

Test Coverage

Coverage

80%

Importance

Changes	4
Bugs	0	Features	1

3 Methods

Rating	Name	Duplication	Size	Complexity
B	from_ini()	27	27	2
A	default()	5	5	2
A	to_ini()	11	11	2

# coding: utf-8

import zope.interface

from schematics.models import Model
from schematics.types import BooleanType, IntType, FloatType

from .constants import NO_DEATH_LIMIT_FLAG
from .helpers import field_from_ini, field_to_ini
from .interfaces import INISerializable, DefaultProvider


@zope.interface.implementer(INISerializable)

This code seems to be duplicated in your project.

@zope.interface.implementer(DefaultProvider)
class Refly(Model):
    enabled = BooleanType(
        default=True,
        required=True,
    )
    death_penalty = IntType(
        min_value=0,
        default=0,
        required=True,
    )
    death_penalty_multiplier = FloatType(
        min_value=0.0,
        default=0.0,
        required=True,
    )
    death_limit = IntType(
        min_value=1,
        default=None,
        required=False,
    )

    @classmethod
    def from_ini(cls, ini):
        death_limit = field_from_ini(
            cls.death_limit, ini,
            'NET', 'maxAllowedKIA',
        )
        death_limit = (
            None
            if death_limit == NO_DEATH_LIMIT_FLAG
            else death_limit
        )

        return cls({
            'enabled': not field_from_ini(
                cls.enabled, ini,
                'NET', 'reflyDisabled',
                False,
            ),
            'death_penalty': field_from_ini(
                cls.death_penalty, ini,
                'NET', 'reflyKIADelay',
            ),
            'death_penalty_multiplier': field_from_ini(
                cls.death_penalty_multiplier, ini,
                'NET', 'reflyKIADelayMultiplier',
            ),
            'death_limit': death_limit,
        })

    def to_ini(self, ini):
        death_limit = (
            NO_DEATH_LIMIT_FLAG
            if self.death_limit is None
            else self.death_limit
        )

        field_to_ini(not self.enabled, ini, 'NET', 'reflyDisabled')
        field_to_ini(self.death_penalty, ini, 'NET', 'reflyKIADelay')
        field_to_ini(self.death_penalty_multiplier, ini, 'NET', 'reflyKIADelayMultiplier')
        field_to_ini(death_limit, ini, 'NET', 'maxAllowedKIA')

    @classmethod
    def default(cls):
        return cls({
            field_name: field.default
            for field_name, field in cls.fields.items()
        })