[BC] Introduce type specific interfaces (#224)

* Introduce type specific interfaces

* Apply php-cs-fixer changes

* fix namespace

* Import correct interfaces

* More interface related problems

* Update search provider

* Fix more name

* Fix elementservice

* Add upgrade notes

---------

Co-authored-by: mattamon <[email protected]>

Author: mattamon

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,