Modify ↓
#7250 closed defect (fixed)
[Patch] Wiki to HTML rendering of Description, and more robustness for Description and Comment rendering
Reported by: | pipern | Owned by: | Robert Corsaro |
---|---|---|---|
Priority: | normal | Component: | AnnouncerPlugin |
Severity: | normal | Keywords: | patch |
Cc: | Trac Release: | 0.12 |
Description
If the change comment for a ticket include hyperlinks, then it failed with this exception:
... File "...trac/wiki/formatter.py", line 1147, in handle_match return internal_handler(match, fullmatch) File "...trac/wiki/formatter.py", line 529, in _shref_formatter return self._make_link(ns, target, match, match, fullmatch) File "...trac/wiki/formatter.py", line 590, in _make_link return self._make_ext_link(ns+':'+target, label) File "...trac/wiki/formatter.py", line 653, in _make_ext_link (self.req or self.env).abs_href.base AttributeError: 'str' object has no attribute 'base'
because we were passing in a string for abs_href to Mock req. I've changed that to pass in the real env.abs_href, and also started to render the Description as HTML too.
Patch attached.
Attachments (1)
Change History (5)
Changed 14 years ago by
Attachment: | th-7250.diff added |
---|
comment:1 Changed 14 years ago by
I have to errors with announcer plugin at the moment.
2010-06-19 12:12:34,481 Trac[ticket_email] ERROR:. Traceback (most recent call last): File "build/bdist.linux-x86_64/egg/announcerplugin/formatters/ticket_email.py", line 189, in _format_html temp = formatter.generate(True) File "/usr/local/lib/python2.6/dist-packages/Trac-0.11.7.1dev_r0-py2.6.egg/trac/wiki/formatter.py", line 1082, in generate escape_newlines) File "/usr/local/lib/python2.6/dist-packages/Trac-0.11.7.1dev_r0-py2.6.egg/trac/wiki/formatter.py", line 857, in format for line in text.splitlines(): AttributeError: 'NoneType' object has no attribute 'splitlines' 2010-06-19 12:12:34,729 Trac[api] ERROR: AnnouncementSystem failed. Traceback (most recent call last): File "build/bdist.linux-x86_64/egg/announcerplugin/api.py", line 369, in _real_send distributor.distribute(transport, packages[transport], evt) File "build/bdist.linux-x86_64/egg/announcerplugin/distributors/email_distributor.py", line 233, in distribute self._do_send(transport, event, k, v, fmtdict[k]) File "build/bdist.linux-x86_64/egg/announcerplugin/distributors/email_distributor.py", line 328, in _do_send rootMessage['X-Trac-Announcement-ID'] = self._event_id(event) File "build/bdist.linux-x86_64/egg/announcerplugin/distributors/email_distributor.py", line 298, in _event_id self.log.warning("%08d"%event.target.id) TypeError: %d format: a number is required, not unicode 2010-06-19 12:13:08,389 Trac[email_distributor] WARNING: 00000864 2010-06-19 12:13:08,389 Trac[email_distributor] WARNING: 00000864
The first seems to have gone with your patch. Thank you!
It occurs when adding an attachment to a ticket without writing a description.
The second error persits. Perhaps I will open a new ticket for it.
comment:2 Changed 14 years ago by
Summary: | Wiki to HTML rendering of Description, and more robustness for Description and Comment rendering [PATCH] → [Patch] Wiki to HTML rendering of Description, and more robustness for Description and Comment rendering |
---|
comment:3 Changed 14 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
Note: See
TracTickets for help on using
tickets.
Patch to render description and use env.abs_href in the Mock req rather than string returned by env.abs_ref()