Edgewall Software

Ticket #6516 (reopened enhancement)

Opened 13 months ago

Last modified 2 months ago

[PATCH] Including content of ticket fields in outgoing notification email headers

Reported by: qwp0 Owned by: eblot
Priority: normal Milestone: 0.13
Component: notification Version:
Severity: normal Keywords: ghop consider
Cc: doug.hellmann@…, androsis@…

Description

I (as 'androsis') have claimed a ticket in GHOP ( http://code.google.com/p/google-highly-open-participation-psf/issues/detail?id=255 ) -- "Update trac to include more ticket fields in outgoing email headers". There have been already posted some patchs, so it maybe useful for you to have a ticket here too.

I attach my most recent patch for this task (it can be found on the GHOP ticket of this issue too)...

P.S.: I have not found any existing ticket where I could post this, so if there is already a ticket about that I'm very sorry.

Attachments

trac-notification-headers_v3(tests).diff (2.3 KB) - added by qwp0 13 months ago.

Change History

Changed 13 months ago by qwp0

Changed 13 months ago by anonymous

  • cc doug.hellmann@… added

Changed 13 months ago by anonymous

  • summary changed from Including content of ticket fields in outgoing notification email headers to [PATCH] Including content of ticket fields in outgoing notification email headers

Changed 13 months ago by sid

  • type changed from defect to enhancement

Changed 13 months ago by eblot

  • status changed from new to closed
  • resolution set to duplicate

Duplicate of #1359

Changed 13 months ago by doug.hellmann

  • status changed from closed to reopened
  • resolution duplicate deleted

#1359 is not the same as this ticket.

In #1359, only changed fields are included in the header. In this ticket, all fields are always included in the headers.

The two patches could be merged or otherwise made compatible, but they don't provide the same functionality.

Changed 10 months ago by cboos

  • keywords consider added
  • milestone set to 0.12

Changed 3 months ago by androsis@…

  • cc androsis@… added

Changed 3 months ago by cboos

Closed #1359 as duplicate of this one.

We could eventually have something like a smtp_x_trac_ticket_headers option with the possible values of ALL, CHANGED or a list of fields.

Changed 2 months ago by qwp0

Hi, I created a patch which works with the configuration option mentioned by Christian, created against the trunk repository, exactly r7639. I don't know if it's ideal because this ticket should be done for milestone:0.13.

Unfortunaly, because of the too extensive unit testing I can't upload it here, sorry -- the file is available at http://tests.php5.sk/tmp/trac-notification-x_trac_ticket_headers.diff

I know it's none of my business now for the GHOP task (see the ticket description) is not actual but I hope the patch will be useful.

Add/Change #6516 ([PATCH] Including content of ticket fields in outgoing notification email headers)

Author



Change Properties
<Author field>
Action
as reopened
as The resolution will be set. Next status will be 'closed'
to The owner will change from eblot. Next status will be 'new'
 
Note: See TracTickets for help on using tickets.