Changes between Version 2 and Version 3 of TracWorkflow


Ignore:
Timestamp:
Feb 5, 2019, 10:49:44 AM (5 years ago)
Author:
trac
Comment:

--

Legend:

Unmodified
Added
Removed
Modified
  • TracWorkflow

    v2 v3  
    1 = The Trac Ticket Workflow System =
     1= The Trac Ticket Workflow System
     2
     3[[PageOutline(2-5,Contents,pullout)]]
    24[[TracGuideToc]]
    3 
    45The Trac ticket system provides a configurable workflow.
    56
    6 == The Default Ticket Workflow ==
    7 === Environments upgraded from 0.10 ===
    8 When you run `trac-admin <env> upgrade`, your `trac.ini` will be modified to include a `[ticket-workflow]` section.
    9 The workflow configured in this case is the original workflow, so that ticket actions will behave like they did in 0.10.
    10 
    11 Graphically, that looks like this:
    12 
    13 {{{#!Workflow width=500 height=240
    14 leave = * -> *
    15 leave.operations = leave_status
    16 leave.default = 1
    17 accept = new -> assigned
    18 accept.permissions = TICKET_MODIFY
    19 accept.operations = set_owner_to_self
    20 resolve = new,assigned,reopened -> closed
    21 resolve.permissions = TICKET_MODIFY
    22 resolve.operations = set_resolution
    23 reassign = new,assigned,reopened -> new
    24 reassign.permissions = TICKET_MODIFY
    25 reassign.operations = set_owner
    26 reopen = closed -> reopened
    27 reopen.permissions = TICKET_CREATE
    28 reopen.operations = del_resolution
    29 }}}
    30 
    31 There 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?
    32 So you will probably want to migrate to "basic" workflow; [trac:source:trunk/contrib/workflow/migrate_original_to_basic.py contrib/workflow/migrate_original_to_basic.py] may be helpful.
    33 
    34 === Environments created with 0.11 ===
    35 When a new environment is created, a default workflow is configured in your trac.ini.  This workflow is the basic workflow (described in `basic-workflow.ini`), which is somewhat different from the workflow of the 0.10 releases.
    36 
    37 Graphically, it looks like this:
     7== The Default Ticket Workflow
     8
     9When a new environment is created, a default workflow is configured in your trac.ini. This workflow is the basic workflow, as specified in [trac:source:branches/1.2-stable/trac/ticket/workflows/basic-workflow.ini basic-workflow.ini]:
    3810
    3911{{{#!Workflow width=700 height=300
     
    4113leave.operations = leave_status
    4214leave.default = 1
     15
     16create = <none> -> new
     17create.default = 1
     18
     19create_and_assign = <none> -> assigned
     20create_and_assign.label = assign
     21create_and_assign.permissions = TICKET_MODIFY
     22create_and_assign.operations = may_set_owner
     23
    4324accept = new,assigned,accepted,reopened -> accepted
    4425accept.permissions = TICKET_MODIFY
    4526accept.operations = set_owner_to_self
     27
    4628resolve = new,assigned,accepted,reopened -> closed
    4729resolve.permissions = TICKET_MODIFY
    4830resolve.operations = set_resolution
     31
    4932reassign = new,assigned,accepted,reopened -> assigned
    5033reassign.permissions = TICKET_MODIFY
    5134reassign.operations = set_owner
     35
    5236reopen = closed -> reopened
    5337reopen.permissions = TICKET_CREATE
     
    5539}}}
    5640
    57 == Additional Ticket Workflows ==
    58 
    59 There 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.
     41== Additional Ticket Workflows
     42
     43There are example workflows provided in the Trac source tree, see [trac:source:branches/1.2-stable/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.
    6044
    6145Here are some [trac:WorkFlow/Examples diagrams] of the above examples.
    6246
    63 == Basic Ticket Workflow Customization ==
    64 
    65 Note: Ticket "statuses" or "states" are not separately defined. The states a ticket can be in are automatically generated by the transitions defined in a workflow. Therefore, creating a new ticket state simply requires defining a state transition in the workflow that starts or ends with that state.
    66 
    67 Create a `[ticket-workflow]` section in `trac.ini`.
    68 Within this section, each entry is an action that may be taken on a ticket.
     47== Basic Ticket Workflow Customization
     48
     49'''Note''': Ticket "statuses" or "states" are not separately defined. The states a ticket can be in are automatically generated by the transitions defined in a workflow. Therefore, creating a new ticket state simply requires defining a state transition in the workflow that starts or ends with that state.
     50
     51In the `[ticket-workflow]` section of `trac.ini`, each entry is an action that may be taken on a ticket.
    6952For example, consider the `accept` action from `simple-workflow.ini`:
     53
    7054{{{#!ini
    7155accept = new,accepted -> accepted
     
    7357accept.operations = set_owner_to_self
    7458}}}
     59
    7560The first line in this example defines the `accept` action, along with the states the action is valid in (`new` and `accepted`), and the new state of the ticket when the action is taken (`accepted`).
    76 The `accept.permissions` line specifies what permissions the user must have to use this action.
    77 The `accept.operations` line specifies changes that will be made to the ticket in addition to the status change when this action is taken.  In this case, when a user clicks on `accept`, the ticket owner field is updated to the logged in user.  Multiple operations may be specified in a comma separated list.
     61
     62The `accept.permissions` line specifies the permissions the user must have to use this action. [trac:ExtraPermissionsProvider] can define new permissions to be used here.
     63
     64The `accept.operations` line specifies changes that will be made to the ticket in addition to the status change when the action is taken.  In this case, when a user clicks on `accept`, the ticket owner field is updated to the logged in user.  Multiple operations may be specified in a comma separated list.
    7865
    7966The available operations are:
    80 - **del_owner** -- Clear the owner field.
     67- **del_owner** -- Clears the owner field.
    8168- **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.
     69 - ''actionname''`.set_owner` may optionally specify a comma delimited list of users that will be used to populate the select, or a single user. Groups and permissions may also be included in the list //(Since 1.1.3)//. When groups or permissions are specified the select is populated with all members of the group or all users that possess the permission.
    8370- **set_owner_to_self** -- Sets the owner to the logged in user.
    84 - **del_resolution** -- Clears the resolution field
     71- **may_set_owner** -- Sets the owner to the selected or entered owner. Defaults to the existing owner. //(Since 1.1.2)//.
     72- **del_resolution** -- Clears the resolution field.
    8573- **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:
     74 - ''actionname''`.set_resolution` may optionally be set to a comma delimited list or a single value. The resolution(s) specified in this attribute must be defined in the database. Example:
    8775 {{{#!ini
    8876resolve_new = new -> closed
    89 resolve_new.name = resolve
     77resolve_new.label = resolve
    9078resolve_new.operations = set_resolution
    9179resolve_new.permissions = TICKET_MODIFY
     
    9381}}}
    9482- **leave_status** -- Displays "leave as <current status>" and makes no change to the ticket.
    95 '''Note:''' Specifying conflicting operations (such as `set_owner` and `del_owner`) has unspecified results.
    96 
    97 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`.
     83- **reset_workflow** -- Resets the status of tickets that are in states no longer defined.
     84'''Note:''' Specifying conflicting operations, such as `set_owner` and `del_owner`, has unspecified results.
     85
     86The example that follows demonstrates the `.label` attribute. The action here is `resolve_accepted`, but it will be presented to the user as `resolve`.
    9887
    9988{{{#!ini
    10089resolve_accepted = accepted -> closed
    101 resolve_accepted.name = resolve
     90resolve_accepted.label = resolve
    10291resolve_accepted.permissions = TICKET_MODIFY
    10392resolve_accepted.operations = set_resolution
    10493}}}
    10594
    106 For actions that should be available in all states, `*` may be used in place of the state.  The obvious example is the `leave` action:
     95The `.label` attribute is new in Trac 1.1.3 and is functionally the same as the `.name` attribute, which is now deprecated. If neither `.label` or `.name` is specified, the action will be presented to the user as //resolve accepted//, the underscores having been replaced by whitespace (//Since 1.1.3//).
     96
     97For actions that should be available in all states, `*` may be used in place of the state. The obvious example is the `leave` action:
    10798{{{#!ini
    10899leave = * -> *
     
    110101leave.default = 1
    111102}}}
    112 This also shows the use of the `.default` attribute.  This value is expected to be an integer, and the order in which the actions are displayed is determined by this value.  The action with the highest `.default` value is listed first, and is selected by default.  The rest of the actions are listed in order of decreasing `.default` values.
    113 If not specified for an action, `.default` is 0.  The value may be negative.
    114 
    115 There 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.
     103
     104This also shows the use of the `.default` attribute. This value is expected to be an integer, and the order in which the actions are displayed is determined by this value. The action with the highest `.default` value is listed first, and is selected by default. The rest of the actions are listed in order of decreasing `.default` values.
     105If not specified for an action, `.default` is 0. The value may be negative.
     106
     107There is one hard-coded constraint to the workflow: tickets are expected to have a `closed` state. The default reports/queries treat any state other than `closed` as an open state.
     108
     109=== Ticket Create Action
     110
     111The ticket create actions are specified by a transition from the special `<none>` state. At least one create action must be available to the user in order for tickets to be created. The create actions defined in the default workflow are:
     112{{{#!ini
     113create = <none> -> new
     114create.default = 1
     115
     116create_and_assign = <none> -> assigned
     117create_and_assign.label = assign
     118create_and_assign.permissions = TICKET_MODIFY
     119create_and_assign.operations = may_set_owner
     120}}}
     121
     122=== Ticket Reset Action
     123
     124The special `_reset` action is added by default for tickets that are in states that are no longer defined. This allows tickets to be individually "repaired" after the workflow is changed, although it's recommended that the administrator perform the action by batch modifying the affected tickets. By default the `_reset` action is available to users with the `TICKET_ADMIN` permission and reset tickets are put in the //new// state. The default `_reset` action is equivalent to the following `[ticket-workflow]` action definition:
     125
     126{{{#!ini
     127_reset = -> new
     128_reset.label = reset
     129_reset.operations = reset_workflow
     130_reset.permissions = TICKET_ADMIN
     131_reset.default = 0
     132}}}
     133
     134Since [trac:milestone:1.0.3] the `_reset` action can be customized by redefining the implicit action. For example, to allow anyone with `TICKET_MODIFY` to perform the `_reset` action, the workflow action would need to be defined:
     135
     136{{{#!ini
     137_reset = -> new
     138_reset.label = reset
     139_reset.operations = reset_workflow
     140_reset.permissions = TICKET_MODIFY
     141_reset.default = 0
     142}}}
     143
     144== Workflow Visualization
    116145
    117146Workflows can be visualized by rendering them on the wiki using the [WikiMacros#Workflow-macro Workflow macro].
    118147
    119 Workflows 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):
     148Workflows 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):
     149
    120150{{{#!sh
    121 cd /var/local/trac_devel/contrib/workflow/
    122 sudo ./showworkflow /srv/trac/PlannerSuite/conf/trac.ini
    123 }}}
    124 And then open up the resulting `trac.pdf` file created by the script (it will be in the same directory as the `trac.ini` file).
    125 
    126 After 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.
    127 
    128 == Example: Adding optional Testing with Workflow ==
    129 
    130 By 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.
     151$ cd /var/local/trac_devel/contrib/workflow/
     152$ ./showworkflow /srv/trac/PlannerSuite/conf/trac.ini
     153}}}
     154The script outputs `trac.pdf` in the same directory as the `trac.ini` file.
     155
     156== Example: Adding optional Testing with Workflow
     157
     158The following adds a `testing` action. When the ticket has status `new`, `accepted` or `needs_work`, 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 is 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.
    131159
    132160{{{#!ini
    133161testing = new,accepted,needs_work,assigned,reopened -> testing
    134 testing.name = Submit to reporter for testing
     162testing.label = Submit to reporter for testing
    135163testing.permissions = TICKET_MODIFY
    136164
    137165reject = testing -> needs_work
    138 reject.name = Failed testing, return to developer
     166reject.label = Failed testing, return to developer
    139167
    140168pass = testing -> closed
    141 pass.name = Passes Testing
     169pass.label = Passes Testing
    142170pass.operations = set_resolution
    143171pass.set_resolution = fixed
    144172}}}
    145173
    146 === How to combine the `tracopt.ticket.commit_updater` with the testing workflow ===
    147 
    148 The [[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.
    149 
    150 By default it reacts on some keywords found in changeset message logs like ''close'', ''fix'' etc. and performs the corresponding workflow action.
    151 
    152 If 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.
    153 
    154 Have 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.
    155 
    156 == Example: Add simple optional generic review state ==
     174== Example: Add simple optional generic review state
    157175
    158176Sometimes Trac is used in situations where "testing" can mean different things to different people so you may want to create an optional workflow state that is between the default workflow's `assigned` and `closed` states, but does not impose implementation-specific details. The only new state you need to add for this is a `reviewing` state. A ticket may then be "submitted for review" from any state that it can be reassigned. If a review passes, you can re-use the `resolve` action to close the ticket, and if it fails you can re-use the `reassign` action to push it back into the normal workflow.
     
    166184}}}
    167185
    168 Then, 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:
     186To integrate this with the default workflow, you also need to add the `reviewing` state to the `accept` and `resolve` actions:
    169187
    170188{{{#!ini
     
    174192}}}
    175193
    176 Optionally, 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.
     194Optionally, 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:
    177195
    178196{{{#!ini
    179197reassign_reviewing = reviewing -> *
    180 reassign_reviewing.name = reassign review
     198reassign_reviewing.label = reassign review
    181199reassign_reviewing.operations = set_owner
    182200reassign_reviewing.permissions = TICKET_MODIFY
    183201}}}
    184202
    185 The full `[ticket-workflow]` configuration will thus look like this:
     203The full `[ticket-workflow]` configuration will be:
    186204
    187205{{{#!ini
    188206[ticket-workflow]
     207create = <none> -> new
     208create.default = 1
     209create_and_assign = <none> -> assigned
     210create_and_assign.label = assign
     211create_and_assign.permissions = TICKET_MODIFY
     212create_and_assign.operations = may_set_owner
    189213accept = new,reviewing -> assigned
    190214accept.operations = set_owner_to_self
     
    207231reassign_reviewing = reviewing -> *
    208232reassign_reviewing.operations = set_owner
    209 reassign_reviewing.name = reassign review
     233reassign_reviewing.label = reassign review
    210234reassign_reviewing.permissions = TICKET_MODIFY
    211235}}}
    212236
    213 == Example: Limit the resolution options for a new ticket ==
    214 
    215 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.
    216 
    217 {{{#!ini
    218 resolve_new = new -> closed
    219 resolve_new.name = resolve
    220 resolve_new.operations = set_resolution
    221 resolve_new.permissions = TICKET_MODIFY
    222 resolve_new.set_resolution = invalid,wontfix,duplicate
    223 
    224 resolve = assigned,accepted,reopened -> closed
    225 resolve.operations = set_resolution
    226 resolve.permissions = TICKET_MODIFY
    227 }}}
    228 
    229 == Advanced Ticket Workflow Customization ==
    230 
    231 If the customization above is not extensive enough for your needs, you can extend the workflow using plugins.  These plugins can provide additional operations for the workflow (like code_review), or implement side-effects for an action (such as triggering a build) that may not be merely simple state changes.  Look at [trac:source:trunk/sample-plugins/workflow sample-plugins/workflow] for a few simple examples to get started.
    232 
    233 But if even that is not enough, you can disable the !ConfigurableTicketWorkflow component and create a plugin that completely replaces it.
    234 
    235 == Adding Workflow States to Milestone Progress Bars ==
    236 
    237 If you add additional states to your workflow, you may want to customize your milestone progress bars as well.  See [TracIni#milestone-groups-section TracIni].
    238 
    239 == Ideas for next steps ==
    240 
    241 New 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.
     237== Advanced Ticket Workflow Customization
     238
     239If the customizations above do not meet your needs, you can extend the workflow with plugins. Plugins can provide additional operations for the workflow, like code review, or implement side-effects for an action, such as triggering a build, that may not be merely simple state changes. Look at [trac:source:branches/1.2-stable/sample-plugins/workflow sample-plugins/workflow] for a few examples to get started.
     240
     241But if even that is not enough, you can disable the !ConfigurableTicketWorkflow component and create a plugin that completely replaces it. See also the [https://trac-hacks.org/wiki/AdvancedTicketWorkflowPlugin AdvancedTicketWorkflowPlugin], which provides additional operations.
     242
     243== Adding Workflow States to Milestone Progress Bars
     244
     245If you add additional states to your workflow, you may want to customize your milestone progress bars as well. See the [TracIni#milestone-groups-section "[milestone-groups]"] section.
     246
     247== Ideas for next steps
     248
     249Enhancement ideas for the workflow system should be filed as enhancement tickets against the [trac:query:?status=assigned&status=new&status=reopened&keywords=~workflow&component=ticket+system ticket system] component.  You can also document ideas on the [trac:TracIdeas/TracWorkflow TracIdeas/TracWorkflow] page.