Opened 13 years ago
Last modified 11 years ago
#8989 new enhancement
Show nicer error message when there is a ticket with multiple parents
| Reported by: | Ryan J Ollos | Owned by: | Chris Nelson |
|---|---|---|---|
| Priority: | normal | Component: | TracJsGanttPlugin |
| Severity: | normal | Keywords: | |
| Cc: | Trac Release: | 0.11 |
Description
Here is the error I saw when I tried to include a ticket in a milestone displayed in the TracJsGanttChart and that ticket had multiple parents (via SubticketsPlugin).
It might not be immediately obvious why this error is occurring, and that you can't have any tickets with multiple parents, so I propose to display a more user friendly error for this condition.
I can provide a patch this weekend.
Attachments (1)
Change History (5)
Changed 13 years ago by
| Attachment: | TracJsGanttWithMultipleParents.png added |
|---|
comment:1 follow-up: 2 Changed 12 years ago by
comment:2 Changed 12 years ago by
Replying to ChrisNelson:
Hmmm, it's been four months. ;-)
Oh yeah .. I totally forgot about this. I'm expecting to send you a patch for both this ticket and #9442 within the next two weeks.
comment:3 Changed 11 years ago by
It's been an awful lot longer than four months now! I'm not a Python programmer, nor really a Trac expert, and so I've just wasted the best part of an hour while upgrading Trac in order to work out what was going on.
I see the comment that says that the failure is deliberate, and I think it is poor design decision. I don't know why code from jsgantt is being called when I create or edit a ticket, so I don't know what the right behavious is. But I do know that it is not "crash with a cryptic message".
comment:4 Changed 11 years ago by
I'm not using the plugin at the moment, so it's unlikely that I'll find time to provide a patch in the near future.
Replying to rjollos:
Hmmm, it's been four months. ;-)