Avoid deeply nested control flow statements.

Open

                        if user:
                            vals[i] = val.replace('$USER', user)
                        elif val.endswith('$USER'):
                            del clause[field]
                            break

Avoid deeply nested control flow statements.

Open

                        if field == 'description':
                            new_descr = wrap(new, self.COLS, ' ', ' ', '\n',
                                             self.ambiwidth)
                            old_descr = wrap(old, self.COLS, '> ', '> ', '\n',
                                             self.ambiwidth)

Avoid deeply nested control flow statements.

Open

                        if cnum == 'description':
                            title = _("Description for #%(id)s: %(summary)s",
                                      id=resource.id, summary=summary)
                        else:
                            title = _("Comment %(cnum)s for #%(id)s: "

Avoid deeply nested control flow statements.

Open

                            if not word:
                                continue
                        clauses.append("COALESCE(%s,'') %s%s" % (col, cneg,

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

Open

    def _query_link(self, req, name, value, text=None, class_=None, query=None):

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

Open

    def _render_property_diff(self, req, ticket, field, old, new,

Avoid deeply nested control flow statements.

Open

                        with self.env.db_query as db:
                            for name, value in db(
                                    "SELECT name, %s FROM enum WHERE type=%%s"
                                    % db.cast('value', 'int'),
                                    (sort_col,)):

Avoid deeply nested control flow statements.

Open

                        if href is not None:
                            result['href'] = href.ticket(val)
                    elif name in self.time_fields:

Avoid deeply nested control flow statements.

Open

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

Avoid deeply nested control flow statements.

Open

                        if 0 <= idx < len(vals):
                            del vals[idx]
                            if not vals:
                                continue
                        else:

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

Open

    def expand_macro(self, formatter, name, content, args=None):
        args = args or {}
        reponame = args.get('repository') or ''
        rev = args.get('revision')
        repos = RepositoryManager(self.env).get_repository(reponame)

• Read up
  Read up

Avoid deeply nested control flow statements.

Open

                    if not data:
                        break
                    m.update(data)

Avoid deeply nested control flow statements.

Open

                        if k not in values:
                            values[k] = v['old']
                    skip = True

Avoid deeply nested control flow statements.

Open

                        if ev:
                            yield ev

            # Attachments
            if 'ticket_details' in filters:

Avoid deeply nested control flow statements.

Open

                        for k, v in change['fields'].items():
                            values[k] = v['new']
                    if 'description' in change['fields']:

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

Open

    def next_rev(self, rev, path='', find_initial_rev=False):
        """Return revision immediately following `rev`, eventually below
        given `path` or globally.
        """
        rev = self.normalize_rev(rev)

• Read up
  Read up

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

Open

    def milestone_changed(self, milestone, old_values):
        if not milestone.is_completed or 'completed' not in old_values \
                or old_values['completed'] is not None:
            return
        m = re.match(self.pattern, milestone.name)

• Read up
  Read up

Avoid deeply nested control flow statements.

Open

                        if prev_ev:
                            yield prev_ev
                        prev_ev = ev

Avoid deeply nested control flow statements.

Open

                        if older[0] == path:
                            # still on the path: 'newer' was an edit
                            yield newer[0], newer[1], Changeset.EDIT
                        else:
                            # the path changed: 'newer' was a copy

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

Open

    def __init__(self, git_dir, log, git_bin='git', git_fs_encoding=None,