Revert "workflows: allow marking a status as terminal"

This reverts commit af78aecfac.
This commit is contained in:
Thomas NOËL 2012-11-23 14:00:36 +01:00
parent 7c8e1ab0b8
commit 8230b00de6
2 changed files with 4 additions and 25 deletions

View File

@ -280,8 +280,7 @@ class WorkflowItemsDir(Directory):
class WorkflowStatusPage(Directory):
_q_exports = ['', 'delete', 'newitem', ('items', 'items_dir'),
'update_order', 'edit', 'reassign', 'visibility',
'force_endpoint', 'unforce_endpoint']
'update_order', 'edit', 'reassign', 'visibility']
def __init__(self, workflow, status_id):
self.workflow = workflow
@ -361,10 +360,6 @@ class WorkflowStatusPage(Directory):
'<ul>'
'<li><a href="edit">%s</a></li>' % _('Change Status Name')
'<li><a href="visibility" rel="popup">%s</a></li>' % _('Change Status Visibility')
if not self.status.forced_endpoint:
'<li><a href="force_endpoint">%s</a></li>' % _('Mark the Status as Terminal')
else:
'<li><a href="unforce_endpoint">%s</a></li>' % _('Unmark the Status as Terminal')
'<li><a href="delete" rel="popup">%s</a></li>' % _('Delete')
'</ul>'
'<div id="new-field">'
@ -564,16 +559,6 @@ class WorkflowStatusPage(Directory):
'<h2>%s</h2>' % _('Edit Workflow Status Visibility')
form.render()
def force_endpoint(self):
self.status.forced_endpoint = True
self.workflow.store()
return redirect('.')
def unforce_endpoint(self):
self.status.forced_endpoint = False
self.workflow.store()
return redirect('.')
class WorkflowStatusDirectory(Directory):
_q_exports = ['']

View File

@ -133,12 +133,9 @@ class Workflow(StorableObject):
for status in self.possible_status:
waitpoint = False
endpoint = True
if status.forced_endpoint:
endpoint = True
else:
for item in status.items:
endpoint = item.endpoint and endpoint
waitpoint = item.waitpoint or waitpoint
for item in status.items:
endpoint = item.endpoint and endpoint
waitpoint = item.waitpoint or waitpoint
if endpoint or waitpoint:
waitpoint_status.append(status)
return waitpoint_status
@ -151,8 +148,6 @@ class Workflow(StorableObject):
def get_not_endpoint_status(self):
not_endpoint_status = []
for status in self.possible_status:
if status.forced_endpoint:
continue
endpoint = True
for item in status.items:
endpoint = item.endpoint and endpoint
@ -340,7 +335,6 @@ class WorkflowStatus:
name = None
items = None
visibility = None
forced_endpoint = False
def __init__(self, name = None):
self.name = name