Cancel buttons on batch marker allocate and workflow state pages still save changes

Check 1 - cancel should cancel (Make sure you have some teachers and students enrolled in the course before you begin). In an assignment activity with marking workflow enabled, view/grade all submissions and tick one or more submissions that have 'Not marked' state. From the 'With selected' options, choose 'Set marking workflow state' and click 'Go'. Accept the confirmation dialogue. Click the 'Cancel' button. Observe that the submission state has remained at 'Not marked'. In an assignment activity with marking allocation enabled, view/grade all submissions and tick one or more submissions that have no marker assigned. From the 'With selected' options, choose 'Set allocated marker' and click 'Go'. Accept the confirmation dialogue. Click the 'Cancel' button. Observe that the submission marker has remained unset. Check 2 - check that non-cancel code paths still work Check 1 - cancel should cancel In an assignment activity with marking workflow enabled, view/grade all submissions and tick one or more submissions that have 'Not marked' state. From the 'With selected' options, choose 'Set marking workflow state' and click 'Go'. Accept the confirmation dialogue. Change the state and click "Save changes" Observe that the submission state has updated correctly In an assignment activity with marking allocation enabled, view/grade all submissions and tick one or more submissions that have no marker assigned and a work state not in this list (ready for review, in review, ready for release, released) From the 'With selected' options, choose 'Set allocated marker' and click 'Go'. Accept the confirmation dialogue. Change the marker and click "Save changes" Observe that the submission marker has changed