API Standardise extension hooks (#547)

Author: GuySartorelli

src/DataObjects/WorkflowInstance.php

@@ -669,7 +669,6 @@ public function getWorkflowFields()
         $fields->push(HiddenField::create('TransitionID', ''));
         // Let the Active Action update the fields that the user can interact with so that data can be
         // stored for the workflow.
-        $action->updateWorkflowFields($fields);
         $action->invokeWithExtensions('updateWorkflowFields', $fields);
         return $fields;
     }

src/Extensions/AdvancedWorkflowExtension.php

@@ -62,7 +62,7 @@ public function startworkflow($data, $form, $request)
      *
      * @param Form $form
      */
-    public function updateEditForm(Form $form)
+    protected function updateEditForm(Form $form)
     {
         Requirements::javascript('symbiote/silverstripe-advancedworkflow:client/dist/js/advancedworkflow.js');
         $service = singleton(WorkflowService::class);

src/Extensions/WorkflowEmbargoExpiryExtension.php

@@ -547,7 +547,7 @@ public function getIsUnPublishScheduled()
      * @param Member $member
      * @return bool|null
      */
-    public function canEdit($member)
+    protected function canEdit($member)
     {
         if (!Permission::check('EDIT_EMBARGOED_WORKFLOW') && // not given global/override permission to edit
             !$this->owner->AllowEmbargoedEditing) { // item flagged as not editable