Opened 10 months ago
Closed 10 months ago
#14274 closed enhancement (fixed)
Support for Trac 1.6
Reported by: | Joachim Mairböck | Owned by: | Jun Omae |
---|---|---|---|
Priority: | normal | Component: | TracTicketReferencePlugin |
Severity: | normal | Keywords: | |
Cc: | Trac Release: | 1.6 |
Description
It would be nice if this plugin was updated to support Trac 1.6.
I understand that this uses ITemplateStreamFilter, so porting it is likely non-trivial.
I have a (probably incomplete) patch with some changes for python 3 compatibility but the actual Trac changes are still missing.
Attachments (1)
Change History (11)
Changed 10 months ago by
Attachment: | python3.patch added |
---|
comment:1 Changed 10 months ago by
I do not use trac now, so it's difficult to maintain it in the future. I want to delegate maintenance privileges to someone if possible.
comment:2 Changed 10 months ago by
Okay. I'll push the proposed patch after reviewing it and testing.
In the patch, I'd suggest using explicit relative import (e.g. from .model import ...
) rather than absolute import for Python 2 and 3.
-from model import CUSTOM_FIELDS, TICKETREF, TicketLinks -from utils import cnv_sorted_refs +from ticketref.model import CUSTOM_FIELDS, TICKETREF, TicketLinks +from ticketref.utils import cnv_sorted_refs -from utils import cnv_list2text, cnv_sorted_refs, cnv_text2list +from ticketref.utils import cnv_list2text, cnv_sorted_refs, cnv_text2list
comment:3 Changed 10 months ago by
Owner: | changed from Tetsuya Morimoto to Jun Omae |
---|---|
Status: | new → assigned |
comment:7 Changed 10 months ago by
Resolution: | fixed |
---|---|
Status: | closed → reopened |
Thank you for (nearly) fixing this so fast!
One minor thing is still missing: web_ui.py
is using reduce()
, so it is missing a from functools import reduce
. Can you fix this please?
comment:9 Changed 10 months ago by
Oh, I missed it. I've ran tests and confirmed tests passing, however I didn't notice no tests for web_ui.py. Please try the latest. Thanks!
comment:10 Changed 10 months ago by
Resolution: | → fixed |
---|---|
Status: | reopened → closed |
Thanks for the fix!
incomplete patch for python 3 support