Opened 11 years ago
Last modified 8 years ago
#11179 new defect
[Patch] Milestones on versions page are not shown in 0.12/trunk & /versions not working
Reported by: | AlexK | Owned by: | |
---|---|---|---|
Priority: | highest | Component: | ExtendedVersionPlugin |
Severity: | major | Keywords: | |
Cc: | Trac Release: | 1.0 |
Description
When the patch I wrote was applied 2 years ago, the whole "show milestones underneath each version" feature was only applied to the 0.11 folder but not 0.12. Therefore this feature is no longer part of trunk!
Attachments (4)
Change History (13)
comment:1 Changed 11 years ago by
comment:2 follow-up: 5 Changed 11 years ago by
Priority: | high → highest |
---|---|
Summary: | Milestones on versions page are not shown in 0.12/trunk → [Patch] Milestones on versions page are not shown in 0.12/trunk & /versions not working |
Patch fixes milestones to be available in version view again, adapted for trac 1.0 CSS style. Also fixes an infinite recursion caused by incorrect regex in versions.py when trying to view /versions as it was incorrectly matched here (caused by previous commit that tried to fix new version page not matched).
comment:3 Changed 11 years ago by
Oops, had a bug due to incorrect indention in the first version of the patch.
Changed 11 years ago by
Attachment: | roadmap.3.diff added |
---|
Previous patch plus removed ul/li marker in roadmap view
comment:4 Changed 11 years ago by
Status: | new → assigned |
---|
I probably won't get to this for a few days, but it's in my near-term work plan.
comment:5 Changed 11 years ago by
Replying to AlexK:
Also fixes an infinite recursion caused by incorrect regex in versions.py when trying to view /versions as it was incorrectly matched here (caused by previous commit that tried to fix new version page not matched).
I haven't successfully reproduced the infinite recursion issue, however I confirm that the regex doesn't work as intended. I setup some tests and modified the regex. The previous commit that caused the issue was [12231].
As for the rest of the changes, they'll take me a while to sort through. If you were to break up this up into a series of patches representing concise changes, that would speed up the process.
Changed 8 years ago by
Attachment: | roadmap4.diff added |
---|
Adaptions for Trac 1.2 and re-integrated the progress bars on the version overview page
comment:8 Changed 8 years ago by
Status: | assigned → accepted |
---|
comment:9 Changed 8 years ago by
Owner: | Ryan J Ollos deleted |
---|---|
Status: | accepted → new |
@AlexK: Would you consider rebasing your changes on the trunk again? There were some major changes in r16610 to adapt to the Trac 1.0 database API. If you revise the patch I'll commit it.
Are you referring to [10468] / #7620?