Changes between Version 2 and Version 3 of TracWorkflow


Ignore:
Timestamp:
03/07/16 10:57:43 (9 years ago)
Author:
trac
Comment:

--

Legend:

Unmodified
Added
Removed
Modified
  • TracWorkflow

    v2 v3  
    22[[TracGuideToc]]
    33
    4 The Trac issue database provides a configurable workflow.
     4The Trac ticket system provides a configurable workflow.
    55
    66== The Default Ticket Workflow ==
     
    1111Graphically, that looks like this:
    1212
    13 [[Image(htdocs:../common/guide/original-workflow.png)]]
     13{{{#!Workflow width=500 height=240
     14leave = * -> *
     15leave.operations = leave_status
     16leave.default = 1
     17accept = new -> assigned
     18accept.permissions = TICKET_MODIFY
     19accept.operations = set_owner_to_self
     20resolve = new,assigned,reopened -> closed
     21resolve.permissions = TICKET_MODIFY
     22resolve.operations = set_resolution
     23reassign = new,assigned,reopened -> new
     24reassign.permissions = TICKET_MODIFY
     25reassign.operations = set_owner
     26reopen = closed -> reopened
     27reopen.permissions = TICKET_CREATE
     28reopen.operations = del_resolution
     29}}}
    1430
    1531There are some significant "warts" in this; such as accepting a ticket sets it to 'assigned' state, and assigning a ticket sets it to 'new' state.  Perfectly obvious, right?
     
    2137Graphically, it looks like this:
    2238
    23 [[Image(htdocs:../common/guide/basic-workflow.png)]]
     39{{{#!Workflow width=700 height=300
     40leave = * -> *
     41leave.operations = leave_status
     42leave.default = 1
     43accept = new,assigned,accepted,reopened -> accepted
     44accept.permissions = TICKET_MODIFY
     45accept.operations = set_owner_to_self
     46resolve = new,assigned,accepted,reopened -> closed
     47resolve.permissions = TICKET_MODIFY
     48resolve.operations = set_resolution
     49reassign = new,assigned,accepted,reopened -> assigned
     50reassign.permissions = TICKET_MODIFY
     51reassign.operations = set_owner
     52reopen = closed -> reopened
     53reopen.permissions = TICKET_CREATE
     54reopen.operations = del_resolution
     55}}}
    2456
    2557== Additional Ticket Workflows ==
     
    2759There are several example workflows provided in the Trac source tree; look in [trac:source:trunk/contrib/workflow contrib/workflow] for `.ini` config sections.  One of those may be a good match for what you want. They can be pasted into the `[ticket-workflow]` section of your `trac.ini` file. However if you have existing tickets then there may be issues if those tickets have states that are not in the new workflow.
    2860
    29 Here are some [http://trac.edgewall.org/wiki/WorkFlow/Examples diagrams] of the above examples.
     61Here are some [trac:WorkFlow/Examples diagrams] of the above examples.
    3062
    3163== Basic Ticket Workflow Customization ==
     
    3668Within this section, each entry is an action that may be taken on a ticket.
    3769For example, consider the `accept` action from `simple-workflow.ini`:
    38 {{{
     70{{{#!ini
    3971accept = new,accepted -> accepted
    4072accept.permissions = TICKET_MODIFY
     
    4678
    4779The available operations are:
    48  - del_owner -- Clear the owner field.
    49  - set_owner -- Sets the owner to the selected or entered owner.
    50    - ''actionname''`.set_owner` may optionally be set to a comma delimited list or a single value.
    51  - set_owner_to_self -- Sets the owner to the logged in user.
    52  - del_resolution -- Clears the resolution field
    53  - set_resolution -- Sets the resolution to the selected value.
    54    - ''actionname''`.set_resolution` may optionally be set to a comma delimited list or a single value.
    55 {{{
    56 Example:
    57 
     80- **del_owner** -- Clear the owner field.
     81- **set_owner** -- Sets the owner to the selected or entered owner. Defaults to the current user. When `[ticket] restrict_owner = true`, the select will be populated with users that have `TICKET_MODIFY` permission and an authenticated session.
     82 - ''actionname''`.set_owner` may optionally be set to a comma delimited list of users that will be used to populate the select, or a single user.
     83- **set_owner_to_self** -- Sets the owner to the logged in user.
     84- **del_resolution** -- Clears the resolution field
     85- **set_resolution** -- Sets the resolution to the selected value.
     86 - ''actionname''`.set_resolution` may optionally be set to a comma delimited list or a single value. Example:
     87 {{{#!ini
    5888resolve_new = new -> closed
    5989resolve_new.name = resolve
     
    6292resolve_new.set_resolution = invalid,wontfix
    6393}}}
    64  - leave_status -- Displays "leave as <current status>" and makes no change to the ticket.
     94- **leave_status** -- Displays "leave as <current status>" and makes no change to the ticket.
    6595'''Note:''' Specifying conflicting operations (such as `set_owner` and `del_owner`) has unspecified results.
    6696
    67 {{{
     97In this example, we see the `.name` attribute used.  The action here is `resolve_accepted`, but it will be presented to the user as `resolve`.
     98
     99{{{#!ini
    68100resolve_accepted = accepted -> closed
    69101resolve_accepted.name = resolve
     
    72104}}}
    73105
    74 In this example, we see the `.name` attribute used.  The action here is `resolve_accepted`, but it will be presented to the user as `resolve`.
    75 
    76106For actions that should be available in all states, `*` may be used in place of the state.  The obvious example is the `leave` action:
    77 {{{
     107{{{#!ini
    78108leave = * -> *
    79109leave.operations = leave_status
     
    85115There are a couple of hard-coded constraints to the workflow.  In particular, tickets are created with status `new`, and tickets are expected to have a `closed` state.  Further, the default reports/queries treat any state other than `closed` as an open state.
    86116
    87 While creating or modifying a ticket workfow, `contrib/workflow/workflow_parser.py` may be useful.  It can create `.dot` files that [http://www.graphviz.org GraphViz] understands to provide a visual description of the workflow.
    88 
    89 This can be done as follows (your install path may be different).
    90 {{{
     117Workflows can be visualized by rendering them on the wiki using the [WikiMacros#Workflow-macro Workflow macro].
     118
     119Workflows can also be visualized using the `contrib/workflow/workflow_parser.py` script.  The script outputs `.dot` files that [http://www.graphviz.org GraphViz] understands. The script can be used as follows (your install path may be different):
     120{{{#!sh
    91121cd /var/local/trac_devel/contrib/workflow/
    92122sudo ./showworkflow /srv/trac/PlannerSuite/conf/trac.ini
     
    94124And then open up the resulting `trac.pdf` file created by the script (it will be in the same directory as the `trac.ini` file).
    95125
    96 An online copy of the workflow parser is available at http://foss.wush.net/cgi-bin/visual-workflow.pl
    97 
    98126After you have changed a workflow, you need to restart apache for the changes to take effect. This is important, because the changes will still show up when you run your script, but all the old workflow steps will still be there until the server is restarted.
    99127
     
    102130By adding the following to your [ticket-workflow] section of trac.ini you get optional testing.  When the ticket is in new, accepted or needs_work status you can choose to submit it for testing.  When it's in the testing status the user gets the option to reject it and send it back to needs_work, or pass the testing and send it along to closed.  If they accept it then it gets automatically marked as closed and the resolution is set to fixed.  Since all the old work flow remains, a ticket can skip this entire section.
    103131
    104 {{{
     132{{{#!ini
    105133testing = new,accepted,needs_work,assigned,reopened -> testing
    106134testing.name = Submit to reporter for testing
     
    118146=== How to combine the `tracopt.ticket.commit_updater` with the testing workflow ===
    119147
    120 The [[source:trunk/tracopt/ticket/commit_updater.py|tracopt.ticket.commit_updater]] is the optional component that [[TracRepositoryAdmin#trac-post-commit-hook|replaces the old trac-post-commit-hook]], in Trac 0.12.
     148The [[trac:source:trunk/tracopt/ticket/commit_updater.py|tracopt.ticket.commit_updater]] is the optional component that [[TracRepositoryAdmin#trac-post-commit-hook|replaces the old trac-post-commit-hook]], in Trac 0.12.
    121149
    122150By default it reacts on some keywords found in changeset message logs like ''close'', ''fix'' etc. and performs the corresponding workflow action.
     
    124152If you have a more complex workflow, like the testing stage described above and you want the ''closes'' keyword to move the ticket to the ''testing'' status instead of the ''closed'' status, you need to adapt the code a bit.
    125153
    126 Have a look at the [[0.11/TracWorkflow#How-ToCombineSVNtrac-post-commit-hookWithTestWorkflow|Trac 0.11 recipe]] for the `trac-post-commit-hook`, this will give you some ideas about how to modify the component.
     154Have a look at the [[trac:wiki:0.11/TracWorkflow#How-ToCombineSVNtrac-post-commit-hookWithTestWorkflow|Trac 0.11 recipe]] for the `trac-post-commit-hook`, this will give you some ideas about how to modify the component.
    127155
    128156== Example: Add simple optional generic review state ==
     
    132160The new `reviewing` state along with its associated `review` action looks like this:
    133161
    134 {{{
     162{{{#!ini
    135163review = new,assigned,reopened -> reviewing
    136164review.operations = set_owner
     
    140168Then, to integrate this with the default Trac 0.11 workflow, you also need to add the `reviewing` state to the `accept` and `resolve` actions, like so:
    141169
    142 {{{
     170{{{#!ini
    143171accept = new,reviewing -> assigned
    144172[…]
     
    148176Optionally, you can also add a new action that allows you to change the ticket's owner without moving the ticket out of the `reviewing` state. This enables you to reassign review work without pushing the ticket back to the `new` status.
    149177
    150 {{{
     178{{{#!ini
    151179reassign_reviewing = reviewing -> *
    152180reassign_reviewing.name = reassign review
     
    157185The full `[ticket-workflow]` configuration will thus look like this:
    158186
    159 {{{
     187{{{#!ini
    160188[ticket-workflow]
    161189accept = new,reviewing -> assigned
     
    165193leave.default = 1
    166194leave.operations = leave_status
    167 reassign = new,assigned,reopened -> new
     195reassign = new,assigned,accepted,reopened -> assigned
    168196reassign.operations = set_owner
    169197reassign.permissions = TICKET_MODIFY
     
    185213== Example: Limit the resolution options for a new ticket ==
    186214
    187 The above resolve_new operation allows you to set the possible resolutions for a new ticket.  By modifying the existing resolve action and removing the new status from before the `->` we then get two resolve actions.  One with limited resolutions for new tickets, and then the regular one once a ticket is accepted.
    188 
    189 {{{
     215The above `resolve_new` operation allows you to set the possible resolutions for a new ticket.  By modifying the existing resolve action and removing the new status from before the `->` we then get two resolve actions.  One with limited resolutions for new tickets, and then the regular one once a ticket is accepted.
     216
     217{{{#!ini
    190218resolve_new = new -> closed
    191219resolve_new.name = resolve
     
    209237If you add additional states to your workflow, you may want to customize your milestone progress bars as well.  See [TracIni#milestone-groups-section TracIni].
    210238
    211 == some ideas for next steps ==
    212 
    213 New enhancement ideas for the workflow system should be filed as enhancement tickets against the `ticket system` component.  If desired, add a single-line link to that ticket here.  Also look at the [th:wiki:AdvancedTicketWorkflowPlugin] as it provides experimental operations.
    214 
    215 If you have a response to the comments below, create an enhancement ticket, and replace the description below with a link to the ticket.
    216 
    217  * the "operation" could be on the nodes, possible operations are:
    218    * '''preops''': automatic, before entering the state/activity
    219    * '''postops''': automatic, when leaving the state/activity
    220    * '''actions''': can be chosen by the owner in the list at the bottom, and/or drop-down/pop-up together with the default actions of leaving the node on one of the arrows.
    221 ''This appears to add complexity without adding functionality; please provide a detailed example where these additions allow something currently impossible to implement.''
    222 
    223  * operations could be anything: sum up the time used for the activity, or just write some statistical fields like
    224 ''A workflow plugin can add an arbitrary workflow operation, so this is already possible.''
    225 
    226  * set_actor should be an operation allowing to set the owner, e.g. as a "preop":
    227    * either to a role, a person
    228    * entered fix at define time, or at run time, e.g. out of a field, or select.
    229 ''This is either duplicating the existing `set_owner` operation, or needs to be clarified.''
    230 
    231  * Actions should be selectable based on the ticket type (different Workflows for different tickets)
    232 ''Look into the [th:wiki:AdvancedTicketWorkflowPlugin]'s `triage` operation.''
    233 
     239== Ideas for next steps ==
     240
     241New enhancement ideas for the workflow system should be filed as enhancement tickets against the `ticket system` component.  You can also document ideas on the [trac:TracIdeas/TracWorkflow TracIdeas/TracWorkflow] page.  Also look at the [http://trac-hacks.org/wiki/AdvancedTicketWorkflowPlugin AdvancedTicketWorkflowPlugin] as it provides experimental operations.