EZP-29694: Draft conflict does not work as expected for a content ite…

…m with multiple translations
ezsystems · Apr 10, 2019 · d80d379 · d80d379
1 parent f4f0ac5
commit d80d379
Show file tree

Hide file tree

Showing 11 changed files with 81 additions and 34 deletions.
diff --git a/src/bundle/Controller/Content/VersionDraftConflictController.php b/src/bundle/Controller/Content/VersionDraftConflictController.php
@@ -59,24 +59,21 @@ public function __construct(
 
     /**
      * @param int $contentId
-     * @param int|null $locationId
+     * @param string $languageCode
      *
      * @return Response
      *
-     * @throws \eZ\Publish\API\Repository\Exceptions\UnauthorizedException
      * @throws \eZ\Publish\API\Repository\Exceptions\NotFoundException
-     * @throws \InvalidArgumentException
+     * @throws \eZ\Publish\API\Repository\Exceptions\UnauthorizedException
      */
-    public function draftHasNoConflictAction(int $contentId, ?int $locationId = null): Response
+    public function draftHasNoConflictAction(int $contentId, string $languageCode): Response
     {
         $content = $this->contentService->loadContent($contentId);
-        $location = $this->locationService->loadLocation(
-            $locationId ?? $content->contentInfo->mainLocationId
-        );
+        $location = $this->locationService->loadLocation($content->contentInfo->mainLocationId);
         $contentInfo = $content->contentInfo;
 
         try {
-            $contentDraftHasConflict = (new ContentDraftHasConflict($this->contentService))->isSatisfiedBy($contentInfo);
+            $contentDraftHasConflict = (new ContentDraftHasConflict($this->contentService, $languageCode))->isSatisfiedBy($contentInfo);
         } catch (UnauthorizedException $e) {
             $error = $this->translator->trans(
                 /** @Desc("Cannot check if the draft has no conflict with other drafts. %error%. ") */
@@ -91,7 +88,7 @@ public function draftHasNoConflictAction(int $contentId, ?int $locationId = null
         if ($contentDraftHasConflict) {
             $versionsDataset = $this->datasetFactory->versions();
             $versionsDataset->load($contentInfo);
-            $conflictedDrafts = $versionsDataset->getConflictedDraftVersions($contentInfo->currentVersionNo);
+            $conflictedDrafts = $versionsDataset->getConflictedDraftVersions($contentInfo->currentVersionNo, $languageCode);
 
             $modalContent = $this->renderView('@ezdesign/content/modal_draft_conflict.html.twig', [
                 'conflicted_drafts' => $conflictedDrafts,

diff --git a/src/bundle/Controller/ContentController.php b/src/bundle/Controller/ContentController.php
@@ -12,6 +12,7 @@
 use eZ\Publish\API\Repository\LocationService;
 use eZ\Publish\API\Repository\UserService;
 use eZ\Publish\API\Repository\Values\Content\Content;
+use eZ\Publish\API\Repository\Values\Content\Language;
 use eZ\Publish\API\Repository\Values\Content\Location;
 use eZ\Publish\Core\Base\Exceptions\BadStateException;
 use EzSystems\EzPlatformAdminUi\Exception\InvalidArgumentException as AdminInvalidArgumentException;
@@ -190,6 +191,7 @@ public function editAction(Request $request): Response
 
                 if (!$versionInfo->isDraft()) {
                     $contentDraft = $this->contentService->createContentDraft($contentInfo, $versionInfo);
+                    $this->updateDraftInitialLanguage($contentDraft, $language);
                     $versionNo = $contentDraft->getVersionInfo()->versionNo;
 
                     $this->notificationHandler->success(
@@ -456,4 +458,30 @@ public function updateVisibilityAction(Request $request): Response
 
         return $result instanceof Response ? $result : $this->redirectToRoute('ezplatform.dashboard');
     }
+
+    /**
+     * @param \eZ\Publish\API\Repository\Values\Content\Content $contentDraft
+     * @param \eZ\Publish\API\Repository\Values\Content\Language $language
+     *
+     * @throws \eZ\Publish\API\Repository\Exceptions\BadStateException
+     * @throws \eZ\Publish\API\Repository\Exceptions\ContentFieldValidationException
+     * @throws \eZ\Publish\API\Repository\Exceptions\ContentValidationException
+     * @throws \eZ\Publish\API\Repository\Exceptions\InvalidArgumentException
+     * @throws \eZ\Publish\API\Repository\Exceptions\UnauthorizedException
+     */
+    private function updateDraftInitialLanguage(
+        Content $contentDraft,
+        Language $language
+    ): void {
+        if ($language->languageCode === $contentDraft->versionInfo->initialLanguageCode) {
+            return;
+        }
+
+        $contentUpdateStruct = $this->contentService->newContentUpdateStruct();
+
+        $contentUpdateStruct->initialLanguageCode = $language->languageCode;
+        $contentUpdateStruct->creatorId = $contentDraft->versionInfo->creatorId;
+
+        $this->contentService->updateContent($contentDraft->versionInfo, $contentUpdateStruct);
+    }
 }
diff --git a/src/bundle/Controller/Version/VersionConflictController.php b/src/bundle/Controller/Version/VersionConflictController.php
@@ -36,23 +36,23 @@ public function __construct(ContentService $contentService)
      *
      * @param int $contentId
      * @param int $versionNo
+     * @param string $languageCode
      *
      * @return Response
      *
-     * @throws \eZ\Publish\API\Repository\Exceptions\UnauthorizedException
      * @throws \eZ\Publish\API\Repository\Exceptions\NotFoundException
-     * @throws \InvalidArgumentException
+     * @throws \eZ\Publish\API\Repository\Exceptions\UnauthorizedException
      * @throws \eZ\Publish\Core\Base\Exceptions\BadStateException
      */
-    public function versionHasNoConflictAction(int $contentId, int $versionNo): Response
+    public function versionHasNoConflictAction(int $contentId, int $versionNo, string $languageCode): Response
     {
         $versionInfo = $this->contentService->loadVersionInfoById($contentId, $versionNo);
 
         if (!$versionInfo->isDraft()) {
             throw new BadStateException('Version status', 'status is not draft');
         }
 
-        if ((new VersionHasConflict($this->contentService))->isSatisfiedBy($versionInfo)) {
+        if ((new VersionHasConflict($this->contentService, $languageCode))->isSatisfiedBy($versionInfo)) {
             return new Response('', Response::HTTP_CONFLICT);
         }
 

diff --git a/src/bundle/Resources/config/routing.yml b/src/bundle/Resources/config/routing.yml
@@ -486,7 +486,7 @@ ezplatform.version.has_no_conflict:
         languageCode: ~
 
 ezplatform.version_draft.has_no_conflict:
-    path: /version-draft/has-no-conflict/{contentId}
+    path: /version-draft/has-no-conflict/{contentId}/{languageCode}
     options:
         expose: true
     defaults:

diff --git a/src/bundle/Resources/public/js/scripts/admin.location.view.js b/src/bundle/Resources/public/js/scripts/admin.location.view.js
@@ -22,13 +22,14 @@
     };
     const handleEditItem = (content) => {
         const contentId = content._id;
-        const checkVersionDraftLink = Routing.generate('ezplatform.version_draft.has_no_conflict', { contentId });
+        const languageCode = content.mainLanguageCode;
+        const checkVersionDraftLink = Routing.generate('ezplatform.version_draft.has_no_conflict', { contentId, languageCode });
         const submitVersionEditForm = () => {
             doc.querySelector('#form_subitems_content_edit_content_info').value = contentId;
             doc.querySelector('#form_subitems_content_edit_version_info_content_info').value = contentId;
             doc.querySelector('#form_subitems_content_edit_version_info_version_no').value =
                 content.CurrentVersion.Version.VersionInfo.versionNo;
-            doc.querySelector(`#form_subitems_content_edit_language_${content.mainLanguageCode}`).checked = true;
+            doc.querySelector(`#form_subitems_content_edit_language_${languageCode}`).checked = true;
             doc.querySelector('#form_subitems_content_edit_create').click();
         };
         const addDraft = () => {

diff --git a/src/bundle/Resources/public/js/scripts/button.content.edit.js b/src/bundle/Resources/public/js/scripts/button.content.edit.js
@@ -15,7 +15,7 @@
             'input[name="' + versionEditFormName + '[version_info][version_no]"]'
         );
         const languageInput = versionEditForm.querySelector('#' + versionEditFormName + '_language_' + languageCode);
-        const checkVersionDraftLink = global.Routing.generate('ezplatform.version_draft.has_no_conflict', { contentId });
+        const checkVersionDraftLink = global.Routing.generate('ezplatform.version_draft.has_no_conflict', { contentId, languageCode });
         const submitVersionEditForm = () => {
             contentInfoInput.value = contentId;
             versionInfoContentInfoInput.value = contentId;

diff --git a/src/bundle/Resources/public/js/scripts/sidebar/btn/location.edit.js b/src/bundle/Resources/public/js/scripts/sidebar/btn/location.edit.js
@@ -4,7 +4,6 @@
     const form = editActions.querySelector('form');
     const contentIdInput = form.querySelector('#content_edit_content_info') || form.querySelector('#user_edit_content_info');
     const contentId = contentIdInput.value;
-    const checkVersionDraftLink = global.Routing.generate('ezplatform.version_draft.has_no_conflict', { contentId });
     const resetRadioButtons = () =>
         btns.forEach((btn) => {
             btn.checked = false;
@@ -49,6 +48,10 @@
         attachModalListeners(wrapper);
     };
     const changeHandler = () => {
+        const checkedBtn = btns.find((btn) => btn.checked);
+        const languageCode = checkedBtn.value;
+        const checkVersionDraftLink = global.Routing.generate('ezplatform.version_draft.has_no_conflict', { contentId, languageCode });
+
         fetch(checkVersionDraftLink, {
             credentials: 'same-origin',
         }).then((response) => {

diff --git a/src/bundle/Resources/views/content/tab/versions/table.html.twig b/src/bundle/Resources/views/content/tab/versions/table.html.twig
@@ -31,11 +31,11 @@
                 path('ez_user_update', {
                 'contentId': version.contentInfo.id,
                 'versionNo': version.versionNo,
-                'language': version.translations[0].languageCode,
+                'language': version.initialLanguageCode,
             }) : path('ez_content_draft_edit', {
                 'contentId': version.contentInfo.id,
                 'versionNo': version.versionNo,
-                'language': version.translations[0].languageCode,
+                'language': version.initialLanguageCode,
                 'locationId': location.id
             })
             %}
@@ -85,7 +85,7 @@
                             data-version-has-conflict-url="{{ path('ezplatform.version.has_no_conflict', {
                                 'contentId': version.contentInfo.id,
                                 'versionNo': version.versionNo,
-                                'languageCode': version.translations[0].languageCode
+                                'languageCode': version.initialLanguageCode
                             }) }}"
                             class="btn btn-icon mx-2 ez-btn--content-draft-edit"
                             title="{{ 'tab.versions.table.action.draft.edit'|trans|desc('Edit Draft') }}">

diff --git a/src/lib/Specification/Content/ContentDraftHasConflict.php b/src/lib/Specification/Content/ContentDraftHasConflict.php
@@ -10,26 +10,30 @@
 
 use eZ\Publish\API\Repository\Exceptions\UnauthorizedException;
 use eZ\Publish\API\Repository\ContentService;
-use eZ\Publish\API\Repository\Values\Content\ContentInfo;
 use EzSystems\EzPlatformAdminUi\Specification\AbstractSpecification;
 
 class ContentDraftHasConflict extends AbstractSpecification
 {
-    /** @var ContentService */
+    /** @var \eZ\Publish\API\Repository\ContentService */
     private $contentService;
 
+    /** @var string */
+    private $languageCode;
+
     /**
-     * @param ContentService $contentService
+     * @param \eZ\Publish\API\Repository\ContentService $contentService
+     * @param string $languageCode
      */
-    public function __construct(ContentService $contentService)
+    public function __construct(ContentService $contentService, string $languageCode)
     {
         $this->contentService = $contentService;
+        $this->languageCode = $languageCode;
     }
 
     /**
      * Checks if Content has draft conflict.
      *
-     * @param ContentInfo $contentInfo
+     * @param \eZ\Publish\API\Repository\Values\Content\ContentInfo $contentInfo
      *
      * @return bool
      *
@@ -40,7 +44,10 @@ public function isSatisfiedBy($contentInfo): bool
         $versions = $this->contentService->loadVersions($contentInfo);
 
         foreach ($versions as $checkedVersionInfo) {
-            if ($checkedVersionInfo->isDraft() && $checkedVersionInfo->versionNo > $contentInfo->currentVersionNo) {
+            if ($checkedVersionInfo->isDraft()
+                && $checkedVersionInfo->versionNo > $contentInfo->currentVersionNo
+                && $checkedVersionInfo->initialLanguageCode === $this->languageCode
+            ) {
                 return true;
             }
         }

diff --git a/src/lib/Specification/Version/VersionHasConflict.php b/src/lib/Specification/Version/VersionHasConflict.php
@@ -15,15 +15,20 @@
 
 class VersionHasConflict extends AbstractSpecification
 {
-    /** @var ContentService */
+    /** @var \eZ\Publish\API\Repository\ContentService */
     private $contentService;
 
+    /** @var string */
+    private $languageCode;
+
     /**
-     * @param ContentService $contentService
+     * @param \eZ\Publish\API\Repository\ContentService $contentService
+     * @param string $languageCode
      */
-    public function __construct(ContentService $contentService)
+    public function __construct(ContentService $contentService, string $languageCode)
     {
         $this->contentService = $contentService;
+        $this->languageCode = $languageCode;
     }
 
     /**
@@ -40,7 +45,10 @@ public function isSatisfiedBy($versionInfo): bool
         $versions = $this->contentService->loadVersions($versionInfo->getContentInfo());
 
         foreach ($versions as $checkedVersionInfo) {
-            if ($checkedVersionInfo->versionNo > $versionInfo->versionNo && $checkedVersionInfo->isPublished()) {
+            if ($checkedVersionInfo->versionNo > $versionInfo->versionNo
+                && $checkedVersionInfo->isPublished()
+                && $checkedVersionInfo->initialLanguageCode === $this->languageCode
+            ) {
                 return true;
             }
         }

diff --git a/src/lib/UI/Dataset/VersionsDataset.php b/src/lib/UI/Dataset/VersionsDataset.php
@@ -73,15 +73,18 @@ function (VersionInfo $versionInfo) {
 
     /**
      * @param int $currentVersionNo
+     * @param string $languageCode
      *
      * @return array
      */
-    public function getConflictedDraftVersions(int $currentVersionNo): array
+    public function getConflictedDraftVersions(int $currentVersionNo, string $languageCode): array
     {
         return $this->filterVersions(
             $this->data,
-            function (VersionInfo $versionInfo) use ($currentVersionNo) {
-                return $versionInfo->isDraft() && $versionInfo->versionNo > $currentVersionNo;
+            function (VersionInfo $versionInfo) use ($currentVersionNo, $languageCode) {
+                return $versionInfo->isDraft()
+                    && $versionInfo->versionNo > $currentVersionNo
+                    && $versionInfo->initialLanguageCode === $languageCode;
             }
         );
     }