[Data Object Update] Add tasks "save" and "publish"

src/DataIndex/Query/DataObjectQuery.php

@@ -38,7 +38,7 @@
 
 final class DataObjectQuery implements DataObjectQueryInterface
 {
-    public const DATA_OBJECT_QUERY_ID = 'data_object_query';
+    public const string DATA_OBJECT_QUERY_ID = 'data_object_query';
 
     public function __construct(
         private readonly DataObjectSearch $search,

src/Element/Service/ElementSaveService.php

@@ -18,8 +18,12 @@
 
 use Exception;
 use Pimcore\Bundle\GenericDataIndexBundle\Service\SearchIndex\IndexQueue\SynchronousProcessingServiceInterface;
+use Pimcore\Bundle\StudioBackendBundle\Exception\Api\ForbiddenException;
+use Pimcore\Bundle\StudioBackendBundle\Security\Service\SecurityServiceInterface;
+use Pimcore\Bundle\StudioBackendBundle\Util\Constant\ElementPermissions;
 use Pimcore\Bundle\StudioBackendBundle\Util\Constant\ElementSaveTasks;
 use Pimcore\Model\DataObject\Concrete;
+use Pimcore\Model\Document;
 use Pimcore\Model\Element\DuplicateFullPathException;
 use Pimcore\Model\Element\ElementInterface;
 use Pimcore\Model\UserInterface;
@@ -30,15 +34,17 @@
  */
 final readonly class ElementSaveService implements ElementSaveServiceInterface
 {
-    public function __construct(private SynchronousProcessingServiceInterface $synchronousProcessingService)
-    {
+    public function __construct(
+        private SynchronousProcessingServiceInterface $synchronousProcessingService,
+        private SecurityServiceInterface $securityService
+    ) {
     }
 
     /**
      * @throws DuplicateFullPathException
      * @throws Exception
      */
-    public function save(ElementInterface $element, UserInterface $user, ?string $task): void
+    public function save(ElementInterface $element, UserInterface $user, ?string $task = null): void
     {
         $this->synchronousProcessingService->enable();
         $element->setUserModification($user->getId());
@@ -59,6 +65,16 @@ public function save(ElementInterface $element, UserInterface $user, ?string $ta
      */
     private function processTask(ElementInterface $element, UserInterface $user, string $task): void
     {
+        if ($task === ElementSaveTasks::PUBLISH->value) {
+            $this->processPublishTask($element, $user);
+        }
+
+        if ($task === ElementSaveTasks::SAVE->value || $task === ElementSaveTasks::PUBLISH->value) {
+            $element->save();
+
+            return;
+        }
+
         /**
          * @var Concrete $element
          */
@@ -74,4 +90,18 @@ private function processTask(ElementInterface $element, UserInterface $user, str
 
         $element->deleteAutoSaveVersions($user->getId());
     }
+
+    /**
+     * @throws ForbiddenException
+     */
+    private function processPublishTask(ElementInterface $element, UserInterface $user): void
+    {
+        if (!$element instanceof Concrete && !$element instanceof Document) {
+            return;
+        }
+
+        $this->securityService->hasElementPermission($element, $user, ElementPermissions::PUBLISH_PERMISSION);
+        $element->deleteAutoSaveVersions($user->getId());
+        $element->setPublished(true);
+    }
 }

src/Element/Service/ElementSaveServiceInterface.php

@@ -32,5 +32,5 @@ interface ElementSaveServiceInterface
      * @throws DuplicateFullPathException
      * @throws Exception
      */
-    public function save(ElementInterface $element, UserInterface $user, ?string $task): void;
+    public function save(ElementInterface $element, UserInterface $user, ?string $task = null): void;
 }

src/Util/Constant/ElementSaveTasks.php

@@ -25,6 +25,8 @@ enum ElementSaveTasks: string
 {
     use EnumToValueArrayTrait;
 
-    case VERSION = 'version';
     case AUTOSAVE = 'autoSave';
+    case PUBLISH = 'publish';
+    case SAVE = 'save';
+    case VERSION = 'version';
 }