tigefa4u projects Trac

Fixing if/when confusion in the error template.

WorkFlow: some interface changes for the ITicketActionController

- `render_ticket_action_control` returns now a `(label, control, hint)` tuple.
The `hint` will be used to inform the user about what will happen if the action is performed.
- `get_ticket_changes` now returns simply the dictionary containing the ticket changes, instead of that dictionary plus a description of the change. That description was barely used (only by one sample plugin), and if there's a need to warn the user, the standard `req.warning` mechanism will achieve the same effect.

Also:
- further improvements to the documentation
- adapt the sample plugins to the above API changes (well, for those not already anticipating it!)
- the !CodeReview sample plugin has also been refactored a bit

WorkFlow: follow-up to the r5442 move. ''Note that those sample plugins are still not operational as they're already matching the future API for get_ticket_changes which should come shortly (in r5444 if all goes well)''

WorkFlow: renamed the '''!DefaultTicketActionController''' component to '''!ConfigurableTicketWorkflow'''

''Note that if you already have multiple controllers configured, you should do the same rename in your trac.ini file''

e.g. from:
{{{
[ticket]
workflow = DefaultTicketActionController,CodeReviewActionController,StatusFixerActionController
}}}

to:
{{{
[ticket]
workflow = ConfigurableTicketWorkflow,CodeReviewActionController,StatusFixerActionController
}}}

WorkFlow: move the default workfow implementation out of the API file.

WorkFlow: follow-up to r5439, the check for matching action controllers was not correct.

WorkFlow: the `get_ticket_changes` and `apply_action_side_effects` methods of the ITicketActionController interface, like the `render_ticket_action_control` one, where intended to be called only on the controllers that actually declared to handle the `action` being performed.

WorkFlow: enhanced the documentation and a few formatting cleanups.

In particular, I felt the need to clarify ''state'' vs. ''status''.
In many places, we were talking about ticket states referring to the ticket state as a whole, not only to its ''status'' field. The `ITicketActionController.get_all_states` method on the other hand was about getting the different values for the ''status'' fields, so I renamed the ITicketActionController method to `get_all_status`.

Fix typo in query RSS export method.

Add distinctive ''closed'' style for TracLinks to completed milestones.