Update notes on move pop up

silverstripeltd · Aug 16, 2023 · c0aa5d8 · c0aa5d8
1 parent e3e67d8
commit c0aa5d8
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 6 deletions.
diff --git a/client/src/components/ElementActions/MoveAction.js b/client/src/components/ElementActions/MoveAction.js
@@ -44,7 +44,7 @@ const MoveAction = (MenuComponent) => (props) => {
     disabled,
     className: 'element-editor__actions-move',
     onClick: handleClick,
-    toggle: props.toggle, // todo: what is this?
+    toggle: props.toggle,
   };
 
   const modalSchemaUrl = `${getConfig().form.elementForm.moveModalSchemaUrl}/${id}`;

diff --git a/src/Forms/MoveElementHandler.php b/src/Forms/MoveElementHandler.php
@@ -4,6 +4,7 @@
 
 use DNADesign\Elemental\Extensions\ElementalPageExtension;
 use SilverStripe\CMS\Model\SiteTree;
+use SilverStripe\Control\Controller;
 use SilverStripe\Core\Injector\Injectable;
 use SilverStripe\Forms\FieldList;
 use SilverStripe\Forms\Form;
@@ -23,7 +24,7 @@ class MoveElementHandler
      *
      * @var Controller
      */
-    protected $controller;
+    protected ?Controller $controller;
 
     public function __construct($controller = null)
     {
@@ -35,7 +36,11 @@ public function Form($elementID)
         $fields = FieldList::create([
             LiteralField::create(
                 'MoveWarning',
-                '<p class="alert alert-info"><strong>Note</strong>: All published blocks will changed draft state once moved.</p>'
+                '<p class="alert alert-info">' .
+                '<strong>Note</strong>: All published blocks will changed draft state once moved. ' .
+                'If you would like to copy this block to another page, duplicate this block and then ' .
+                'move the duplicate.' .
+                '</p>'
             ),
             HiddenField::create(
                 'ElementID',
@@ -80,7 +85,8 @@ public function moveElement($element, $formData)
         if (!$page->ElementalArea()->exists()) {
             throw $this->validationResult(_t(
                 __CLASS__ . '.ElementalAreaNotFound',
-                'Could not find an elemental area on <strong>{PageName}</strong> to move <strong>{BlockName}</strong> to',
+                'Could not find an elemental area on <strong>{PageName}</strong> to move ' .
+                '<strong>{BlockName}</strong> to',
                 [
                     'PageName' => $page->Title,
                     'BlockName' => $element->Title
@@ -91,7 +97,8 @@ public function moveElement($element, $formData)
         if (!$page->canEdit() || !$element->canEdit()) {
             throw $this->validationResult(_t(
                 __CLASS__ . '.InsufficientPermissions',
-                'Can not move <strong>{PageName}</strong> to <strong>{BlockName}</strong> due to insufficient permissions',
+                'Can not move <strong>{PageName}</strong> to <strong>{BlockName}</strong> due to ' .
+                'insufficient permissions',
                 [
                     'PageName' => $page->Title,
                     'BlockName' => $element->Title
@@ -140,4 +147,4 @@ protected function validationResult($message, $field = null)
             ->addFieldError($field, $message);
         return new ValidationException($error);
     }
-}
+}