[PATCH] Additional milestone operation for sample-plugins

[framay <franz.mayer@…>]

We have following workflow in my company:

If a ticket is rejected (which means not fixed / implemented) it will be moved to a milestone 'rejected'. Thus we have clean milestones with only implemented features / solved bug fixes.

It would be nice, if this could be parametrized using Trac workflow. I have implemented this in attached file MilestoneOperation.py. Maybe the attached file could be committed in sample-plugins, when there are no further comments regarding better / nicer implementation.

Plugin description:

---

Sets milestone for specific status.

Example

[ticket-workflow]
resolve.operations = set_resolution,set_milestone
resolve.milestone = invalid,wontfix,duplicate,worksforme->rejected

When setting status to duplicate the milestone will automaitcally change to rejected.

Note: if user has changed milestone manually, this workflow operation has no effect!

Configuration

Don't forget to add MilestoneOperation to the workflow option in [ticket] section. If there is no workflow option, the line will look like this:

[ticket]
workflow = ConfigurableTicketWorkflow,MilestoneOperation

[framay <franz.mayer@…>]

Plugin file for set_milestone operation

[Christian Boos]

Looks good, just found a typo: resoltions.

Also, you should add the

revision = "$Rev$"
url = "$URL$"

keys, and also author and author_email with your contact informations (same kind of policy as in contrib/ I'd say).

[framay <franz.mayer@…>]

improved version of component MilestoneOperation

[framay <franz.mayer@…>]

Replying to cboos:

Looks good, just found a typo: resoltions.

Replaced resoltions with resolutions in method __get_resolution_milestone_dict.

Also, you should add the

revision = "$Rev$"
url = "$URL$"

keys, and also author and author_email with your contact informations (same kind of policy as in contrib/ I'd say).

Added licence info with author and author email as well as revision and url. Is this OK as I did it?

[Christian Boos]

I added MilestoneOperation.py to trunk/sample-plugins/workflow in r11227.

While testing it I did the following changes:

• style: don't use dict.has_key(k) but k in dict or even dict.get(k) when appropriate; test against None with is/is not None
• in render_ticket_action_control, status was used when this was actually about resolutions
• the check for a Milestone existence was wrong… here's another victim of #4130 ;-)

A more subtle fix: if a milestone was set, and the ticket got reopened, then closing it again with a resolution which was not mapped would clear the milestone.

There's still a problem, not due to the plugin but rather to the well-known issue that we have trouble differentiating changes made by the user and changes made by the workflow: after an explicit "Preview" action, the new milestone set by the workflow becomes the "user defined" milestone, and as such it sticks. At least we have the "(revert)" action if needed, so it's not critical.