#12853 closed defect (fixed)
Trac 1.1+ requires dropping macros.html
Reported by: | lkraav | Owned by: | Russ Tyndall |
---|---|---|---|
Priority: | normal | Component: | TimingAndEstimationPlugin |
Severity: | normal | Keywords: | |
Cc: | Trac Release: | 1.2 |
Description (last modified by )
/billing
crashes on current trunk
.
Because of timingandestimationplugin/branches/trac1.0/timingandestimationplugin/templates/billing.html#L8
Which is logical, considering r12889.
Just removing xi:include macros.html
seems to work, but being a new user, I'm not sure what side-effects this might have.
Attachments (0)
Change History (13)
comment:1 Changed 8 years ago by
Description: | modified (diff) |
---|
comment:2 Changed 8 years ago by
comment:3 Changed 8 years ago by
I have been quite busy this year as my company recently lost a valuable employee. I will try to get to this soon :)
comment:4 Changed 7 years ago by
Hello, is there any progress on this? I'm curious since distributions are moving to more modern trac and having billing.html not working is kind of pity. From my own experience the plugin has worked well on OpenBSD up to 6.0. Now, on 6.1 which distributes trac-1.2 it does not work anymore. If the plan is not to release some solution in the near future, is it at least possible to describe some easy workaround for this? For example I've tried manually edit all the billing.html files I'm able to find out on my system, but this does not help as it looks like billing.html itself is generated from something else (egg file?). So a way how to fix it at least locally would be appreciated here. Thanks! Karel
comment:5 Changed 7 years ago by
I'd be happy to test and push the fix if it's acceptable to the plugin author.
comment:6 Changed 7 years ago by
Sorry, I lost track of this in the blur of my other duties :) I will get a patch tested and published today. There are some other outstanding tickets related to trac moving forward as well, so hitting it all at once makes sense to me.
Sorry for the delay, and thanks Ryan for the offer of assistance.
comment:8 Changed 7 years ago by
Also Ryan J Ollos (rjollos) :
For future reference, you have my permission to make patches against any of my repositories for the good of ensuring continued compatibility. I will continue to try and provide support. At this point though I am something like 11 (2006ish) years into trac and most of the ego attachments I might have had to this code are gone and I would love for it to continue to work with or without my current attention.
Thanks again for your continued support and thanks to lkraav for the bug reports!
Cheers, Russ Tyndall
comment:9 Changed 7 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
comment:12 Changed 7 years ago by
Replying to Russ Tyndall:
16688 has not removed xi:include macros.html
as per 16683 so #12853 is not yet resolved for the permissions branch.
I don't see any macros from
macros.html
used inbilling.html
, so it appears the include can just be deleted (and also from any other templates where it is used).