Function __init__ has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

    def __init__(self, req, options):
        environ = {}
        environ.update(apache.build_cgi_env(req))

        environ['trac.web.frontend'] = 'mod_python'

• Read up
  Read up

Function get_change has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

    def get_change(self, cnum=None, cdate=None):
        """Return a ticket change by its number or date.
        """
        if cdate is None:
            row = self._find_change(cnum)

• Read up
  Read up

Function __init__ has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

    def __init__(self, env, tkt_id=None, version=None):
        self.env = env
        self.fields = TicketSystem(self.env).get_ticket_fields()
        self.editable_fields = \
            {f['name'] for f in self.fields

• Read up
  Read up

Function run has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

    def run(self):
        """Begin processing data from the socket."""
        self._keepGoing = True
        while self._keepGoing:
            try:

• Read up
  Read up

Avoid deeply nested control flow statements.

Open

                        if old_db_val is None:
                            field = self.fields.by_name(name)
                            default = self._custom_field_default(field)
                            if self.values.get(name) == default:
                                continue

Function arg_list_to_args has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

def arg_list_to_args(arg_list):
    """Convert a list of `(name, value)` tuples into into a `_RequestArgs`."""
    args = _RequestArgs()
    for name, value in arg_list:
        if name in args:

• Read up
  Read up

Function load has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

    def load(self, filename):
        """Load account information from apache style htdigest files,
        only users from the specified realm are used
        """
        # FIXME use a logger

• Read up
  Read up

Function get_available_actions has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

    def get_available_actions(self, req, ticket):
        """Returns a sorted list of available actions"""
        # The list should not have duplicates.
        actions = {}
        for controller in self.action_controllers:

• Read up
  Read up

Avoid deeply nested control flow statements.

Open

                        for name in sel:
                            milestone = model.Milestone(self.env, name)
                            milestone.move_tickets(None, req.authname,
                                                   "Milestone deleted")
                            milestone.delete()

Avoid deeply nested control flow statements.

Open

                        if ends:
                            vals = [start + '..' + end
                                    for (start, end) in zip(vals, ends)]
                    if k in remove_constraints:

Function milestone_stats_data has 6 arguments (exceeds 4 allowed). Consider refactoring.

Open

def milestone_stats_data(env, req, stat, name, grouped_by='component',

Avoid deeply nested control flow statements.

Open

                        for name in sel:
                            self._enum_cls(self.env, name).delete()
                            if name == default:
                                self.config.set('ticket',
                                                'default_%s' % self._type, '')

Function __init__ has 6 arguments (exceeds 4 allowed). Consider refactoring.

Open

    def __init__(self, targets, time, author, comment, new_values, action):

Avoid deeply nested control flow statements.

Open

                        for enum in self._enum_cls.select(self.env):
                            new_value = order[enum.value]
                            if new_value != enum.value:
                                enum.value = new_value
                                enum.update()

Avoid deeply nested control flow statements.

Open

                        if newv:
                            change_data[field] = {'oldvalue': old,
                                                  'newvalue': new}

Function execute_paginated_report has 6 arguments (exceeds 4 allowed). Consider refactoring.

Open

    def execute_paginated_report(self, req, id, sql, args, limit=0, offset=0):

Avoid deeply nested control flow statements.

Open

                            if a == b:
                                ids.append(str(a))
                            else:
                                id_clauses.append('t.id BETWEEN %s AND %s')
                                args.append(a)

Avoid deeply nested control flow statements.

Open

                        if prev_t == t:
                            ticket = ev[3][0]
                            batch_ev[3][0].append(ticket.id)
                        else:
                            yield batch_ev

Avoid deeply nested control flow statements.

Open

                        for name in sel:
                            model.Version(self.env, name).delete()
                            if name == default:
                                self.config.set('ticket',
                                                'default_version', '')

Function _send_csv has 6 arguments (exceeds 4 allowed). Consider refactoring.

Open

    def _send_csv(self, req, cols, rows, sep=',', mimetype='text/plain',