The roadmap module doesn't read [milestone-groups]

[tjorvi@…]

The roadmap module doesn't read [milestone-groups] section in trac.ini.

There's an incorrect line in _get_ticket_groups() in roadmap.py that reads:

if 'milestone-groups' in self.config:

But it must be:

if 'milestone-groups' in self.config.sections():

[Noah Kantrowitz]

The two are the same​, unless I am missing something. Are you actually seeing an error of some kind?

[Christian Boos]

I think this uncovers a bug: parser.has_section doesn't take into account inherited config files, so I think __contains__ should be implemented in term of ... in sections().

[Christian Boos]

Should be fixed by r6079.

[tjorvi@…]

Yes it now works correctly on my setup. You're right about the [inherited] bit, I had the miltstone-groups in an inherited ini file.