[BC] Introduce type specific interfaces

doc/01_Installation/02_Upgrade.md

@@ -6,3 +6,24 @@ Following steps are necessary during updating to newer versions.
 - Execute the following command to reindex all elements to be able to use all new features:
 
   ```bin/console generic-data-index:update:index```
+
+## Upgrade to 2.0.0
+
+### BC-Breaks
+
+#### Interface changes
+-  Added `PermissionTypes $permissionType` parameter with default type `PermissionTypes::LIST` to
+- `AssetSearchServiceInterface::search` method
+- `DocumentSearchServiceInterface::search` method
+- `DataObjectSearchServiceInterface::search` method
+- `ElementSearchServiceInterface::search` method
+- Search services `byId` methods now return elements based on the `PermissionTypes::VIEW` permission
+- Added type specific interfaces for searches to avoid mixing up different search types in search services
+  - `AssetSearch` now implements `AssetSearchInterface`
+  - `DocumentSearch` now implements `DocumentSearchInterface`
+  - `ElementSearch` now implements `ElementSearchInterface`
+- Search services now require the specific search type for the search
+  - `AssetSearchServiceInterface::search` now requires a `AssetSearchInterface`
+  - `DocumentSearchServiceInterface::search`  now requires a `DocumentSearchInterface`
+  - `ElementSearchServiceInterface::search`  now requires a `ElementSearchInterface`
+- `SearchProviderInterface` now returns type specific search interfaces

src/Model/Search/Asset/AssetSearch.php

@@ -18,6 +18,6 @@
 
 use Pimcore\Bundle\GenericDataIndexBundle\Model\Search\BaseSearch;
 
-final class AssetSearch extends BaseSearch
+final class AssetSearch extends BaseSearch implements AssetSearchInterface
 {
 }

src/Model/Search/Asset/AssetSearchInterface.php

@@ -0,0 +1,23 @@
+<?php
+declare(strict_types=1);
+
+/**
+ * Pimcore
+ *
+ * This source file is available under two different licenses:
+ * - GNU General Public License version 3 (GPLv3)
+ * - Pimcore Commercial License (PCL)
+ * Full copyright and license information is available in
+ * LICENSE.md which is distributed with this source code.
+ *
+ *  @copyright  Copyright (c) Pimcore GmbH (http://www.pimcore.org)
+ *  @license    http://www.pimcore.org/license     GPLv3 and PCL
+ */
+
+namespace Pimcore\Bundle\GenericDataIndexBundle\Model\Search\Asset;
+
+use Pimcore\Bundle\GenericDataIndexBundle\Model\Search\Interfaces\SearchInterface;
+
+interface AssetSearchInterface extends SearchInterface
+{
+}

src/Model/Search/Document/DocumentSearch.php

@@ -18,6 +18,6 @@
 
 use Pimcore\Bundle\GenericDataIndexBundle\Model\Search\BaseSearch;
 
-final class DocumentSearch extends BaseSearch
+final class DocumentSearch extends BaseSearch implements DocumentSearchInterface
 {
 }

src/Model/Search/Document/DocumentSearchInterface.php

@@ -0,0 +1,23 @@
+<?php
+declare(strict_types=1);
+
+/**
+ * Pimcore
+ *
+ * This source file is available under two different licenses:
+ * - GNU General Public License version 3 (GPLv3)
+ * - Pimcore Commercial License (PCL)
+ * Full copyright and license information is available in
+ * LICENSE.md which is distributed with this source code.
+ *
+ *  @copyright  Copyright (c) Pimcore GmbH (http://www.pimcore.org)
+ *  @license    http://www.pimcore.org/license     GPLv3 and PCL
+ */
+
+namespace Pimcore\Bundle\GenericDataIndexBundle\Model\Search\Document;
+
+use Pimcore\Bundle\GenericDataIndexBundle\Model\Search\Interfaces\SearchInterface;
+
+interface DocumentSearchInterface extends SearchInterface
+{
+}

src/Model/Search/Element/ElementSearch.php

@@ -18,6 +18,6 @@
 
 use Pimcore\Bundle\GenericDataIndexBundle\Model\Search\BaseSearch;
 
-final class ElementSearch extends BaseSearch
+final class ElementSearch extends BaseSearch implements ElementSearchInterface
 {
 }

src/Model/Search/Element/ElementSearchInterface.php

@@ -0,0 +1,23 @@
+<?php
+declare(strict_types=1);
+
+/**
+ * Pimcore
+ *
+ * This source file is available under two different licenses:
+ * - GNU General Public License version 3 (GPLv3)
+ * - Pimcore Commercial License (PCL)
+ * Full copyright and license information is available in
+ * LICENSE.md which is distributed with this source code.
+ *
+ *  @copyright  Copyright (c) Pimcore GmbH (http://www.pimcore.org)
+ *  @license    http://www.pimcore.org/license     GPLv3 and PCL
+ */
+
+namespace Pimcore\Bundle\GenericDataIndexBundle\Model\Search\Element;
+
+use Pimcore\Bundle\GenericDataIndexBundle\Model\Search\Interfaces\SearchInterface;
+
+interface ElementSearchInterface extends SearchInterface
+{
+}

src/Service/Search/SearchService/Asset/AssetSearchService.php

@@ -19,9 +19,9 @@
 use Exception;
 use Pimcore\Bundle\GenericDataIndexBundle\Enum\Permission\UserPermissionTypes;
 use Pimcore\Bundle\GenericDataIndexBundle\Exception\AssetSearchException;
+use Pimcore\Bundle\GenericDataIndexBundle\Model\Search\Asset\AssetSearchInterface;
 use Pimcore\Bundle\GenericDataIndexBundle\Model\Search\Asset\SearchResult\AssetSearchResult;
 use Pimcore\Bundle\GenericDataIndexBundle\Model\Search\Asset\SearchResult\AssetSearchResultItem;
-use Pimcore\Bundle\GenericDataIndexBundle\Model\Search\Interfaces\SearchInterface;
 use Pimcore\Bundle\GenericDataIndexBundle\Model\Search\Modifier\Filter\Basic\IdFilter;
 use Pimcore\Bundle\GenericDataIndexBundle\Permission\Workspace\AssetWorkspace;
 use Pimcore\Bundle\GenericDataIndexBundle\SearchIndexAdapter\Search\Pagination\PaginationInfoServiceInterface;
@@ -47,7 +47,7 @@ public function __construct(
     /**
      * @throws AssetSearchException
      */
-    public function search(SearchInterface $assetSearch): AssetSearchResult
+    public function search(AssetSearchInterface $assetSearch): AssetSearchResult
     {
         $assetSearch = $this->searchHelper->addSearchRestrictions(
             search: $assetSearch,

src/Service/Search/SearchService/Asset/AssetSearchServiceInterface.php

@@ -17,17 +17,18 @@
 namespace Pimcore\Bundle\GenericDataIndexBundle\Service\Search\SearchService\Asset;
 
 use Pimcore\Bundle\GenericDataIndexBundle\Exception\AssetSearchException;
+use Pimcore\Bundle\GenericDataIndexBundle\Model\Search\Asset\AssetSearchInterface;
 use Pimcore\Bundle\GenericDataIndexBundle\Model\Search\Asset\SearchResult\AssetSearchResult;
 use Pimcore\Bundle\GenericDataIndexBundle\Model\Search\Asset\SearchResult\AssetSearchResultItem;
-use Pimcore\Bundle\GenericDataIndexBundle\Model\Search\Interfaces\SearchInterface;
+
 use Pimcore\Model\User;
 
 interface AssetSearchServiceInterface
 {
     /**
      * @throws AssetSearchException
      */
-    public function search(SearchInterface $assetSearch): AssetSearchResult;
+    public function search(AssetSearchInterface $assetSearch): AssetSearchResult;
 
     /**
      * @throws AssetSearchException

src/Service/Search/SearchService/Document/DocumentSearchService.php

@@ -19,9 +19,9 @@
 use Exception;
 use Pimcore\Bundle\GenericDataIndexBundle\Enum\Permission\UserPermissionTypes;
 use Pimcore\Bundle\GenericDataIndexBundle\Exception\DocumentSearchException;
+use Pimcore\Bundle\GenericDataIndexBundle\Model\Search\Document\DocumentSearchInterface;
 use Pimcore\Bundle\GenericDataIndexBundle\Model\Search\Document\SearchResult\DocumentSearchResult;
 use Pimcore\Bundle\GenericDataIndexBundle\Model\Search\Document\SearchResult\DocumentSearchResultItem;
-use Pimcore\Bundle\GenericDataIndexBundle\Model\Search\Interfaces\SearchInterface;
 use Pimcore\Bundle\GenericDataIndexBundle\Model\Search\Modifier\Filter\Basic\IdFilter;
 use Pimcore\Bundle\GenericDataIndexBundle\Permission\Workspace\DocumentWorkspace;
 use Pimcore\Bundle\GenericDataIndexBundle\SearchIndexAdapter\Search\Pagination\PaginationInfoServiceInterface;
@@ -47,7 +47,7 @@ public function __construct(
     /**
      * @throws DocumentSearchException
      */
-    public function search(SearchInterface $documentSearch): DocumentSearchResult
+    public function search(DocumentSearchInterface $documentSearch): DocumentSearchResult
     {
         $documentSearch = $this->searchHelper->addSearchRestrictions(
             search: $documentSearch,

src/Service/Search/SearchService/Document/DocumentSearchServiceInterface.php

@@ -17,17 +17,17 @@
 namespace Pimcore\Bundle\GenericDataIndexBundle\Service\Search\SearchService\Document;
 
 use Pimcore\Bundle\GenericDataIndexBundle\Exception\DocumentSearchException;
+use Pimcore\Bundle\GenericDataIndexBundle\Model\Search\Document\DocumentSearchInterface;
 use Pimcore\Bundle\GenericDataIndexBundle\Model\Search\Document\SearchResult\DocumentSearchResult;
 use Pimcore\Bundle\GenericDataIndexBundle\Model\Search\Document\SearchResult\DocumentSearchResultItem;
-use Pimcore\Bundle\GenericDataIndexBundle\Model\Search\Interfaces\SearchInterface;
 use Pimcore\Model\User;
 
 interface DocumentSearchServiceInterface
 {
     /**
      * @throws DocumentSearchException
      */
-    public function search(SearchInterface $documentSearch): DocumentSearchResult;
+    public function search(DocumentSearchInterface $documentSearch): DocumentSearchResult;
 
     /**
      * @throws DocumentSearchException

src/Service/Search/SearchService/Element/ElementSearchService.php

@@ -19,9 +19,9 @@
 use Exception;
 use Pimcore\Bundle\GenericDataIndexBundle\Enum\SearchIndex\ElementType;
 use Pimcore\Bundle\GenericDataIndexBundle\Exception\ElementSearchException;
+use Pimcore\Bundle\GenericDataIndexBundle\Model\Search\Element\ElementSearchInterface;
 use Pimcore\Bundle\GenericDataIndexBundle\Model\Search\Element\SearchResult\ElementSearchResult;
 use Pimcore\Bundle\GenericDataIndexBundle\Model\Search\Interfaces\ElementSearchResultItemInterface;
-use Pimcore\Bundle\GenericDataIndexBundle\Model\Search\Interfaces\SearchInterface;
 use Pimcore\Bundle\GenericDataIndexBundle\SearchIndexAdapter\Search\Pagination\PaginationInfoServiceInterface;
 use Pimcore\Bundle\GenericDataIndexBundle\Service\Search\SearchService\Asset\AssetSearchServiceInterface;
 use Pimcore\Bundle\GenericDataIndexBundle\Service\Search\SearchService\DataObject\DataObjectSearchServiceInterface;
@@ -44,7 +44,7 @@ public function __construct(
     ) {
     }
 
-    public function search(SearchInterface $elementSearch): ElementSearchResult
+    public function search(ElementSearchInterface $elementSearch): ElementSearchResult
     {
         $elementSearch = $this->searchHelper->addSearchRestrictions($elementSearch);
 

src/Service/Search/SearchService/Element/ElementSearchServiceInterface.php

@@ -18,17 +18,17 @@
 
 use Pimcore\Bundle\GenericDataIndexBundle\Enum\SearchIndex\ElementType;
 use Pimcore\Bundle\GenericDataIndexBundle\Exception\ElementSearchException;
+use Pimcore\Bundle\GenericDataIndexBundle\Model\Search\Element\ElementSearchInterface;
 use Pimcore\Bundle\GenericDataIndexBundle\Model\Search\Element\SearchResult\ElementSearchResult;
 use Pimcore\Bundle\GenericDataIndexBundle\Model\Search\Interfaces\ElementSearchResultItemInterface;
-use Pimcore\Bundle\GenericDataIndexBundle\Model\Search\Interfaces\SearchInterface;
 use Pimcore\Model\User;
 
 interface ElementSearchServiceInterface
 {
     /**
      * @throws ElementSearchException
      */
-    public function search(SearchInterface $elementSearch): ElementSearchResult;
+    public function search(ElementSearchInterface $elementSearch): ElementSearchResult;
 
     /**
      * @throws ElementSearchException

src/Service/Search/SearchService/SearchProvider.php

@@ -17,18 +17,20 @@
 namespace Pimcore\Bundle\GenericDataIndexBundle\Service\Search\SearchService;
 
 use Pimcore\Bundle\GenericDataIndexBundle\Model\Search\Asset\AssetSearch;
+use Pimcore\Bundle\GenericDataIndexBundle\Model\Search\Asset\AssetSearchInterface;
 use Pimcore\Bundle\GenericDataIndexBundle\Model\Search\DataObject\DataObjectSearch;
 use Pimcore\Bundle\GenericDataIndexBundle\Model\Search\DataObject\DataObjectSearchInterface;
 use Pimcore\Bundle\GenericDataIndexBundle\Model\Search\Document\DocumentSearch;
+use Pimcore\Bundle\GenericDataIndexBundle\Model\Search\Document\DocumentSearchInterface;
 use Pimcore\Bundle\GenericDataIndexBundle\Model\Search\Element\ElementSearch;
-use Pimcore\Bundle\GenericDataIndexBundle\Model\Search\Interfaces\SearchInterface;
+use Pimcore\Bundle\GenericDataIndexBundle\Model\Search\Element\ElementSearchInterface;
 
 /**
  * @internal
  */
 final class SearchProvider implements SearchProviderInterface
 {
-    public function createAssetSearch(): SearchInterface
+    public function createAssetSearch(): AssetSearchInterface
     {
         return new AssetSearch();
     }
@@ -38,12 +40,12 @@ public function createDataObjectSearch(): DataObjectSearchInterface
         return new DataObjectSearch();
     }
 
-    public function createDocumentSearch(): SearchInterface
+    public function createDocumentSearch(): DocumentSearchInterface
     {
         return new DocumentSearch();
     }
 
-    public function createElementSearch(): SearchInterface
+    public function createElementSearch(): ElementSearchInterface
     {
         return new ElementSearch();
     }

src/Service/Search/SearchService/SearchProviderInterface.php

@@ -16,16 +16,18 @@
 
 namespace Pimcore\Bundle\GenericDataIndexBundle\Service\Search\SearchService;
 
+use Pimcore\Bundle\GenericDataIndexBundle\Model\Search\Asset\AssetSearchInterface;
 use Pimcore\Bundle\GenericDataIndexBundle\Model\Search\DataObject\DataObjectSearchInterface;
-use Pimcore\Bundle\GenericDataIndexBundle\Model\Search\Interfaces\SearchInterface;
+use Pimcore\Bundle\GenericDataIndexBundle\Model\Search\Document\DocumentSearchInterface;
+use Pimcore\Bundle\GenericDataIndexBundle\Model\Search\Element\ElementSearchInterface;
 
 interface SearchProviderInterface
 {
-    public function createAssetSearch(): SearchInterface;
+    public function createAssetSearch(): AssetSearchInterface;
 
     public function createDataObjectSearch(): DataObjectSearchInterface;
 
-    public function createDocumentSearch(): SearchInterface;
+    public function createDocumentSearch(): DocumentSearchInterface;
 
-    public function createElementSearch(): SearchInterface;
+    public function createElementSearch(): ElementSearchInterface;
 }