diff --git a/src/DataObjects/WorkflowInstance.php b/src/DataObjects/WorkflowInstance.php
index d32631d0..a35d40e8 100644
--- a/src/DataObjects/WorkflowInstance.php
+++ b/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;
     }
diff --git a/src/Extensions/AdvancedWorkflowExtension.php b/src/Extensions/AdvancedWorkflowExtension.php
index 427d0e84..7a2f5409 100644
--- a/src/Extensions/AdvancedWorkflowExtension.php
+++ b/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);
diff --git a/src/Extensions/WorkflowEmbargoExpiryExtension.php b/src/Extensions/WorkflowEmbargoExpiryExtension.php
index b489051c..b7055558 100644
--- a/src/Extensions/WorkflowEmbargoExpiryExtension.php
+++ b/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