0001-workflows-add-possibility-to-jump-to-previous-status.patch

Frédéric Péters, 25 mai 2017 10:48

Voir les différences: en ligne côte à côte

Subject: [PATCH] workflows: add possibility to jump to previous status
 (#16473)

 tests/test_backoffice_pages.py | 91 ++++++++++++++++++++++++++++++++++++++++++
 wcs/admin/workflows.py         |  3 +-
 wcs/formdata.py                | 15 +++++++
 wcs/wf/jump.py                 |  2 +-
 wcs/workflows.py               | 45 +++++++++++++--------
 5 files changed, 137 insertions(+), 19 deletions(-)

     import StringIO
     import time
     import hashlib
     import random
     import pytest
     from webtest import Upload
-...
         resp = app.get(formdata.get_url(backoffice=True))
         assert 'Backoffice Data' in resp.body
         assert 'Not set' in resp.body
     def test_workflow_jump_previous(pub):
         user = create_user(pub)
         create_environment(pub)
         wf = Workflow(name='jump around')
         #       North
         #     /      \
         # West <----> East
         #   |          |
         #   |         autojump
         #    |         |
         #     \       /
         #       South
         st1 = wf.add_status('North')
         st1.id = 'north'
         st2 = wf.add_status('West')
         st2.id = 'west'
         st3 = wf.add_status('East')
         st3.id = 'east'
         st4 = wf.add_status('Autojump')
         st4.id = 'autojump'
         st5 = wf.add_status('South')
         st5.id = 'south'
         button_by_id = {}
         def add_jump(label, src, dst_id):
             jump = ChoiceWorkflowStatusItem()
             jump.id = str(random.random())
             jump.label = label
             jump.by = ['logged-users']
             jump.status = dst_id
             src.items.append(jump)
             jump.parent = src
             button_by_id[label] = 'button%s' % jump.id
         add_jump('Go West', st1, st2.id)
         add_jump('Go East', st1, st3.id)
         add_jump('Go South', st2, st5.id)
         add_jump('Go Autojump', st3, st4.id)
         add_jump('Go Back', st5, '_previous')
         add_jump('Jump West', st3, st2.id)
         add_jump('Jump East', st2, st3.id)
         jump = JumpWorkflowStatusItem()
         jump.id = '_auto-jump'
         jump.status = st5.id
         st4.items.append(jump)
         jump.parent = st4
         wf.store()
         formdef = FormDef.get_by_urlname('form-title')
         formdef.data_class().wipe()
         formdef.workflow = wf
         formdef.store()
         formdata = formdef.data_class()()
         formdata.data = {}
         formdata.just_created()
         formdata.store()
         app = login(get_app(pub))
         resp = app.get('/backoffice/management/form-title/%s/' % formdata.id)
         # jump around using buttons
         resp = resp.form.submit(button_by_id['Go West']).follow()
         assert formdef.data_class().get(formdata.id).status == 'wf-%s' % st2.id
         resp = resp.form.submit(button_by_id['Go South']).follow()
         assert formdef.data_class().get(formdata.id).status == 'wf-%s' % st5.id
         resp = resp.form.submit(button_by_id['Go Back']).follow()
         assert formdef.data_class().get(formdata.id).status == 'wf-%s' % st2.id
         resp = resp.form.submit(button_by_id['Go South']).follow()
         assert formdef.data_class().get(formdata.id).status == 'wf-%s' % st5.id
         resp = resp.form.submit(button_by_id['Go Back']).follow()
         assert formdef.data_class().get(formdata.id).status == 'wf-%s' % st2.id
         resp = resp.form.submit(button_by_id['Jump East']).follow()
         assert formdef.data_class().get(formdata.id).status == 'wf-%s' % st3.id
         resp = resp.form.submit(button_by_id['Go Autojump']).follow()
         assert formdef.data_class().get(formdata.id).status == 'wf-%s' % st5.id
         resp = resp.form.submit(button_by_id['Go Back']).follow()
         assert formdef.data_class().get(formdata.id).status == 'wf-%s' % st3.id
         # and do a last jump using the API
         formdata = formdef.data_class().get(formdata.id)
         formdata.jump_status('_previous')
         assert formdata.status == 'wf-%s' % st5.id

         for status in workflow.possible_status:
             i = status.id
             for item in status.items:
                 next_status_ids = [x.id for x in item.get_target_status() if x.id != status.id]
                 next_status_ids = [x.id for x in item.get_target_status()
                                    if x.id and x.id != status.id]
                 if not next_status_ids:
                     next_status_ids = [status.id]
                 done = {}

                 return None
             return wf_status.handle_form(form, self, user)
         def get_previous_status(self):
             for evolution in reversed(self.evolution[:-1]):
                 status_id = evolution.status
                 if not status_id:
                     continue
                 status = self.formdef.workflow.get_status(status_id[3:])
                 if not status.is_waitpoint():
                     continue
                 return status
             return None
         def jump_status(self, status_id):
             if status_id == '_previous':
                 previous_status = self.get_previous_status()
                 assert previous_status, 'failed to compute previous status'
                 status_id = previous_status.id
             evo = Evolution()
             evo.time = time.localtime()
             evo.status = 'wf-%s' % status_id

                 return
             if self.must_jump(formdata):
                 wf_status = self.get_target_status()
                 wf_status = self.get_target_status(formdata)
                 if wf_status:
                     formdata.status = 'wf-%s' % wf_status[0].id

             # a waitpoint status is a status waiting for an event (be it user
             # interaction or something else), but can also be an endpoint (where
             # the user would wait, infinitely).
             waitpoint_status = []
             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
                 if endpoint or waitpoint:
                     waitpoint_status.append(status)
             return waitpoint_status
             return [x for x in self.possible_status if x.is_waitpoint()]
         def get_endpoint_status(self):
             not_endpoint_status = self.get_not_endpoint_status()
-...
                     return True
             return False
         def is_waitpoint(self):
             waitpoint = False
             endpoint = True
             if self.forced_endpoint:
                 endpoint = True
             else:
                 for item in self.items:
                     endpoint = item.endpoint and endpoint
                     waitpoint = item.waitpoint or waitpoint
             return bool(endpoint or waitpoint)
         def __getstate__(self):
             odict = self.__dict__.copy()
             if odict.has_key('parent'):
-...
         def get_substitution_variables(self, formdata):
             return {}
         def get_target_status(self):
         def get_target_status(self, formdata=None):
             """Returns a list of status this item can lead to."""
             if not getattr(self, 'status', None):
                 return []
             if self.status == '_previous':
                 if formdata is None:
                     # must be in a formdata to compute destination, just give a
                     # fake status for presentation purpose
                     return [WorkflowStatus(_('Previous Status'))]
                 previous_status = formdata.get_previous_status()
                 if previous_status:
                     return [previous_status]
                 return []
             try:
                 return [x for x in self.parent.parent.possible_status if x.id == self.status]
             except IndexError:
-...
         def add_parameters_widgets(self, form, parameters, prefix='', formdef=None):
             if 'status' in parameters:
                 form.add(SingleSelectWidget, '%sstatus' % prefix, title = _('Status'), value = self.status,
                     options = [(None, '---')] + [(x.id, x.name) for x in self.parent.parent.possible_status])
                     options = [(None, '---')] +
                               [(x.id, x.name) for x in self.parent.parent.possible_status] +
                               [('_previous', _('Previous Status'))])
         def get_parameters(self):
             return ('status',)
-...
         def submit_form(self, form, formdata, user, evo):
             if form.get_submit() == 'button%s' % self.id:
                 wf_status = self.get_target_status()
                 wf_status = self.get_target_status(formdata)
                 if wf_status:
                     evo.status = 'wf-%s' % wf_status[0].id
                     form.clear_errors()
-...
         def submit_form(self, form, formdata, user, evo):
             if form.is_submitted() and not form.has_errors():
                 wf_status = self.get_target_status()
                 wf_status = self.get_target_status(formdata)
                 if wf_status:
                     evo.status = 'wf-%s' % wf_status[0].id
+    -

Projet

Général

Profil

Produits Entr'ouvert » w.c.s.

0001-workflows-add-possibility-to-jump-to-previous-status.patch