IBX-7357: Adjusted Authors & Details tabs to modes specification

ibexa · Dec 14, 2023 · b93c683 · b93c683
1 parent 9c3fc93
commit b93c683
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 11 deletions.
diff --git a/src/lib/Tab/LocationView/AuthorsTab.php b/src/lib/Tab/LocationView/AuthorsTab.php
@@ -9,15 +9,12 @@
 namespace Ibexa\AdminUi\Tab\LocationView;
 
 use Ibexa\AdminUi\Specification\UserExists;
-use Ibexa\AdminUi\Specification\UserMode\IsUserModeEnabled;
-use Ibexa\AdminUi\UserSetting\UserMode;
 use Ibexa\Contracts\AdminUi\Tab\AbstractEventDispatchingTab;
 use Ibexa\Contracts\AdminUi\Tab\ConditionalTabInterface;
 use Ibexa\Contracts\AdminUi\Tab\OrderedTabInterface;
 use Ibexa\Contracts\Core\Repository\UserService;
 use Ibexa\Contracts\Core\Repository\Values\Content\ContentInfo;
 use Ibexa\Contracts\Core\Repository\Values\Content\VersionInfo;
-use Ibexa\User\UserSetting\UserSettingService;
 use JMS\TranslationBundle\Annotation\Desc;
 use Symfony\Component\EventDispatcher\EventDispatcherInterface;
 use Symfony\Contracts\Translation\TranslatorInterface;
@@ -29,19 +26,15 @@ class AuthorsTab extends AbstractEventDispatchingTab implements OrderedTabInterf
 
     private UserService $userService;
 
-    private UserSettingService $userSettingService;
-
     public function __construct(
         Environment $twig,
         TranslatorInterface $translator,
         UserService $userService,
-        UserSettingService $userSettingService,
         EventDispatcherInterface $eventDispatcher
     ) {
         parent::__construct($twig, $translator, $eventDispatcher);
 
         $this->userService = $userService;
-        $this->userSettingService = $userSettingService;
     }
 
     public function getIdentifier(): string
@@ -92,7 +85,7 @@ public function getTemplateParameters(array $contextParameters = []): array
 
     public function evaluate(array $parameters): bool
     {
-        return IsUserModeEnabled::fromUserSettings($this->userSettingService)->isSatisfiedBy(UserMode::SMART);
+        return true;
     }
 
     /**

diff --git a/src/lib/Tab/LocationView/DetailsTab.php b/src/lib/Tab/LocationView/DetailsTab.php
@@ -79,7 +79,7 @@ public function getName(): string
 
     public function getOrder(): int
     {
-        return 200;
+        return 750;
     }
 
     public function evaluate(array $parameters): bool

diff --git a/tests/lib/Tab/LocationView/AuthorsTabVisibilityTest.php b/tests/lib/Tab/LocationView/AuthorsTabVisibilityTest.php
@@ -25,14 +25,13 @@ protected function createTabForVisibilityInGivenUserModeTest(UserSettingService
             $this->createMock(Environment::class),
             $this->createMock(TranslatorInterface::class),
             $this->createMock(UserService::class),
-            $userSettingService,
             $this->createMock(EventDispatcherInterface::class)
         );
     }
 
     public function dataProviderForTestTabVisibilityInGivenUserMode(): iterable
     {
         yield 'smart mode' => [UserMode::SMART, [], true];
-        yield 'expert mode' => [UserMode::EXPERT, [], false];
+        yield 'expert mode' => [UserMode::EXPERT, [], true];
     }
 }