Opened 22 months ago
Closed 22 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 22 months ago by
| Attachment: | python3.patch added |
|---|
comment:1 Changed 22 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 22 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 22 months ago by
| Owner: | changed from Tetsuya Morimoto to Jun Omae |
|---|---|
| Status: | new → assigned |
comment:7 Changed 22 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 22 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 22 months ago by
| Resolution: | → fixed |
|---|---|
| Status: | reopened → closed |
Thanks for the fix!



incomplete patch for python 3 support