Ticket URL not showing in notification emails

[anonymous]

After adding or modifying a ticket the Ticket URL is not showing in the email sent.

[snip]

• status: new ⇒ closed
• resolution: ⇒ fixed

— Ticket URL: ≠

snip

RSS feed shows the url properly though.

[Matthew Good]

This is not a "critical" issue since that would mean that there was data loss or some other serious problem that made the system unusable.

[anonymous]

You can fix this by adding:

        self.hdf['ticket.link'] = self.env.abs_href.ticket(ticket.id)

To line 183 of Notify.py. changeset:1832 removed the link property which the email template was depending on, i think. See #1743 for formatting fixes as well.

[evan.s@…]

I still get no Ticket URL with

self.hdf['ticket.link'] = self.env.abs_href.ticket(ticket.id)

added to Notify.py

[evan.s@…]

Actually, it does work — had to wait for Python to catch up and begin using the modified Notify.py I suppose. Thanks for the fix :)

[linne@…]

Working patch for #1734

[Christopher Lenz]

Fixed in [2062].

[tiger@…]

Still missing in the trunk. Please apply the patch below:

Index: trac/Notify.py
===================================================================
--- trac/Notify.py      (revision 2067)
+++ trac/Notify.py      (working copy)
@@ -181,6 +181,7 @@
         self.hdf['email.ticket_body_hdr'] = self.format_hdr()
         self.hdf['ticket'] = self.ticket.values
         self.hdf['ticket.new'] = self.newticket
+        self.hdf['ticket.link'] = self.env.abs_href.ticket(ticket.id)
         subject = self.format_subj()
         if not self.newticket:
             subject = 'Re: ' + subject

[anonymous]

I am currently using the trunk and my notification emails look fine. They contain the correct ticket URL. The edgewall site is running the latest trunk and the ticket archives also show that the emails look fine.

[Matthew Good]

The value ticket.link is read from the ticket properties, so it should not need to be set directly. As the person above said, the notifications on the Trac project seem to have the link, and I can verify that this is working on my trunk installation as well, so the above patch is not necessary.

[Christian Boos]

(clearing report:35)