View Issue Details

IDProjectCategoryView StatusLast Update
0004537mantisbtbugtrackerpublic2004-11-06 06:50
Reporterbpfennig Assigned Tothraxisp  
PrioritynormalSeveritytweakReproducibilityalways
Status closedResolutionfixed 
Product Version0.19.0 
Fixed in Version0.19.1 
Summary0004537: "Change Status To" shows @@ sometimes
Description

If the user has the access rights to change the status but $g_status_enum_workflow is not defined for that status, the "Change Status To" popup shows @@.

TagsNo tags attached.
Attached Files
workflow_check.JPG (29,063 bytes)   
workflow_check.JPG (29,063 bytes)   

Activities

thraxisp

thraxisp

2004-09-14 08:14

reporter   ~0007564

I agree with the problem.

The root cause is really an incomplete implementation of a workflow. If a status has no next statuses, it should also be unreachable. Maybe we need a workflow checker in the admin section...

I can see that not changing the status should always be an option. What other status would you suggest if the workflow is undefined?

bpfennig

bpfennig

2004-09-14 08:47

reporter   ~0007567

I think it that case the "Change Status To" should be disabled or hidden.

thraxisp

thraxisp

2004-09-14 09:40

reporter   ~0007570

If the "Change Status To" is hidden, how will the bug ever get out of the bad state?

bpfennig

bpfennig

2004-09-14 12:14

reporter   ~0007576

Yes, you are right - then all available states should be shown.

thraxisp

thraxisp

2004-09-15 05:44

reporter   ~0007588

Two things need to be changed:

1) add a check in admin/check.php to validate the workflow (e.g., all states that are reachable are exitable)

2) add a default of all states in the change status dropdown if the workflow for a particular status is undefined.

thraxisp

thraxisp

2004-09-22 16:46

reporter   ~0007695

I attached a snap of the workflow checker. I was planning to show the access level required to reach the status as well. It lives under the Admin -> System Utilities.

When the relationship graph stuff is committed, I can use the same utilities to generate this graphically.

Comments are welcome.

thraxisp

thraxisp

2004-09-23 17:05

reporter   ~0007725

Fixed in CVS.