[Patch] Improvements for Xref action

[daniel]

I've made some enhancements to the xref action including:

1. Verification that the target ticket exists.
2. Including the comment change on the local ticket with the original save (so it appears in the change email and doesn't corrupt the ticket cnums)
3. Sending a change email for the target ticket.

[hju@…]

I'm facing the same problems.

I think we will try the patch.

Hope this will be included in a future release.

[anonymous]

chang the patch, by adding a second except clause.

            try:
                xticket = model.Ticket(self.env, ticketnum)
            except ResourceNotFound, e:
                #put in preview mode to prevent ticket being saved
                req.args['preview'] = True
                add_warning(req, "Unable to cross-reference Ticket #%s (%s)." % (ticketnum, e.message))
                return {}
            except ValueError, e:
                #put in preview mode to prevent ticket being saved
                req.args['preview'] = True
                add_warning(req, "Unable to cross-reference Ticket #%s (%s).>%s< is not a number." % (ticketnum, e.message, ticketnum))
                return {}

[Eli Carter]

(fixed formatting of description)

[Eli Carter]

While it breaks the intent of the customizable workflow interface, it does improve the behaviour. Modified version of the patch applied in [9958]. Thanks!