Opened 2 years ago
Closed 2 years 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 2 years ago by
| Attachment: | python3.patch added |
|---|
comment:1 Changed 2 years 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 2 years 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 2 years ago by
| Owner: | changed from Tetsuya Morimoto to Jun Omae |
|---|---|
| Status: | new → assigned |
comment:7 Changed 2 years 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 2 years 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!



incomplete patch for python 3 support