Modify

Opened 10 years ago

Last modified 7 years ago

#11865 new defect

Notification doesn't work with SVN integration ticket changes

Reported by: RomNexus6 Owned by: ejucovy
Priority: normal Component: WorkflowNotificationPlugin
Severity: normal Keywords:
Cc: ntmlod Trac Release: 1.0

Description (last modified by Jun Omae)

Hello,

I have a TRAC installation (1.0) with status extended with "testing" value. I have your great plugin (0.4). It works very well with modifications done by the TRAC web pages, but it doens't work correctly if I set a ticket to "testing" usign SVN integration.

Example: SVN commit with message "my modification, test #1234"

This is my TRAC workflow setting:

[ticket-workflow]
accept = new,assigned,accepted,reopened -> accepted
accept.operations = set_owner_to_self
accept.permissions = TICKET_MODIFY
leave = * -> *
leave.default = 1
leave.operations = leave_status
pass = testing -> closed
pass.name = success: solved
pass.operations = set_resolution
pass.set_resolution = fixed
reassign = new,assigned,accepted,reopened,testing,needs_work -> *
reassign.operations = set_owner
reassign.permissions = TICKET_MODIFY
reject = testing -> needs_work
reject.name = test failed
reopen = closed -> reopened
reopen.operations = del_resolution
reopen.permissions = TICKET_CREATE
resolve = new,assigned,accepted,reopened -> closed
resolve.operations = set_resolution
resolve.permissions = TICKET_MODIFY
testing = new,accepted,needs_work,assigned,reopened -> testing
testing.name = test
testing.permissions = TICKET_MODIFY

This is my WorkflowNotificationPlugin setting:

[ticket-workflow-notifications]
new_ticket = @created
new_ticket.body = ${ticket.reporter} has created the ticket #${ticket.id}: "${ticket.summary}"\n\nDescription:\n\n${ticket.description}\n\nTicket URL: ${link}\n\n${project.name} <${project.url or abs_href()}>\n\n${project.descr}\n
new_ticket.recipients = r.signorelli
new_ticket.subject = [projects] NEW TICKET #${ticket.id}: ${ticket.summary}
when_fixed = resolve, pass
when_fixed.body = ${ticket.owner} has closed the ticket #${ticket.id}: "${ticket.summary}"\nWith resolution: ${ticket.resolution}\n\nTicket URL: ${link}\n\n${project.name} <${project.url or abs_href()}>\n\n${project.descr}\n
when_fixed.recipients = r.signorelli
when_fixed.subject = [projects] CLOSED TICKET #${ticket.id}: ${ticket.summary}
when_testing = testing
when_testing.body = ${ticket.owner} is testing the ticket #${ticket.id}: "${ticket.summary}"\n\nTicket URL: ${link}\n\n${project.name} <${project.url or abs_href()}>\n\n${project.descr}\n
when_testing.recipients = r.signorelli
when_testing.subject = [projects] TESTING TICKET #${ticket.id}: ${ticket.summary}

What do you think?

Thanks

Attachments (0)

Change History (4)

comment:1 Changed 10 years ago by Jun Omae

Description: modified (diff)

trac:CommitTicketUpdater doesn't respect ticket workflow. I guess the plugin doesn't work with repository commit hooks.

comment:2 Changed 10 years ago by anonymous

The ticket correctly changes to "testing" status. So I'm sure the trac:CommitTicketUpdater works well. Only the email notification doesn't work.

comment:3 Changed 10 years ago by ejucovy

I'll take a look as soon as I have a moment -- if I recall correctly, CommitTicketUpdater bypasses the workflow component, so this may require an adjustment to CommitTicketUpdater to work properly with this plugin.

comment:4 Changed 7 years ago by ntmlod

Cc: ntmlod added

Modify Ticket

Change Properties
Set your email in Preferences
Action
as new The owner will remain ejucovy.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.