Hey John, I'm just curious about how are we going to deal with this...

I'm just curious about how are we going to deal with this...

When I orignally but the roadmay together with this functionality, I had planned to make circular hirearchies simply break the chain as soon as a circle was detected, but either I forgot to put that in, or my logic is failing. :P

Regardless, now that my final exams are over for the quarter, I'm going to try to get time to fix this and the other roadmap bugs, and get a patch into trunk and push Victor for a 1.1.2 release with a fixed roadmap.

+1 for me, especially if the change is trivial

whouldn't it be better to check for circle relations on creation of the relation?

It could be wise to check how other projects (bugzilla, trac, etc) are handling such case

other solution of the problem:
currently tickets are displayed multiple in the roadmap, if they have multiple relations.
If you stop showing them more often than once, you would not end in that loop. (and on a roadmap i don't want to see all those small things a houndred time, that depend to 10 other items. view ticket 0008853)

suggestion: you could add a + symbol to tickets (like on a browser), to show the childs of this ticket (using ajax)

Fix commited to 1.1.2 and 1.2.x trunk, r5106 and r5015 respectively.