feat: add toolbar action to copy article

sulu · Dec 28, 2022 · 695f139 · 695f139
1 parent c375e46
commit 695f139
Show file tree

Hide file tree

Showing 3 changed files with 36 additions and 8 deletions.
diff --git a/Admin/ArticleAdmin.php b/Admin/ArticleAdmin.php
@@ -228,14 +228,21 @@ function(Localization $localization) {
 
             if ($this->securityChecker->hasPermission(static::SECURITY_CONTEXT, PermissionTypes::LIVE)
                 && $this->securityChecker->hasPermission(static::getArticleSecurityContext($typeKey), PermissionTypes::LIVE)) {
+
+                $editDropdownToolbarActions = [
+                    new ToolbarAction('sulu_admin.delete_draft'),
+                    new ToolbarAction('sulu_admin.set_unpublished'),
+                    new ToolbarAction('sulu_admin.copy'),
+                ];
+
+                if (\count($locales) > 1) {
+                    $editDropdownToolbarActions[] = new ToolbarAction('sulu_admin.copy_locale');
+                }
+
                 $formToolbarActionsWithType[] = new DropdownToolbarAction(
                     'sulu_admin.edit',
                     'su-pen',
-                    [
-                        new ToolbarAction('sulu_admin.delete_draft'),
-                        new ToolbarAction('sulu_admin.set_unpublished'),
-                        new ToolbarAction('sulu_admin.copy')
-                    ]
+                    $editDropdownToolbarActions
                 );
             }
 

diff --git a/Controller/ArticleController.php b/Controller/ArticleController.php
@@ -528,9 +528,7 @@ public function postTriggerAction(string $id, Request $request): Response
     {
         // extract parameter
         $action = $this->getRequestParameter($request, 'action', true);
-
-        # TODO: remove default - just for testing
-        $locale = $this->getRequestParameter($request, 'locale', true, 'de');
+        $locale = $this->getRequestParameter($request, 'locale', false);
 
         // prepare vars
         $view = null;

diff --git a/Tests/Functional/Controller/ArticleControllerTest.php b/Tests/Functional/Controller/ArticleControllerTest.php
@@ -1193,6 +1193,29 @@ public function testCDelete()
         $this->assertNull($this->findViewDocument($article2['id'], 'de'));
     }
 
+    public function testCopy()
+    {
+        // create article
+        $sourceArticle = $this->testPost('Sulu ist toll - Artikel 1');
+        $this->publish($sourceArticle['id']);
+        $this->flush();
+
+        // trigger copy action
+        $this->client->jsonRequest('POST',
+            \sprintf('/api/articles/%s?action=copy', $sourceArticle['id'])
+        );
+        $response = $this->client->getResponse();
+
+        $result = \json_decode($response->getContent(), true);
+        $this->assertHttpStatusCode(200, $response);
+
+        $this->assertNotEquals($sourceArticle['id'], $result['id']);
+        $this->assertEquals($sourceArticle['title'], $result['title']);
+        $this->assertEquals($sourceArticle['articleType'], $result['articleType']);
+        $this->assertEquals(false, $result['publishedState']);
+    }
+
+
     public function testCopyLocaleWithoutSource()
     {
         // prepare vars