[API] Remove unused PathHiderDocumentationModifier

GSadee · Sep 26, 2024 · b2c70df · b2c70df
1 parent e5c594a
commit b2c70df
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 64 deletions.
diff --git a/UPGRADE-API-2.0.md b/UPGRADE-API-2.0.md
@@ -8,7 +8,6 @@
 * The following services were removed:
     * `sylius.listener.api_postgresql_driver_exception_listener`
 
-
 * Non-prefix serialization groups in Sylius resources have been removed.
    If you have extended any of them, you must prefix them with `sylius:`, for example:
 
@@ -214,3 +213,7 @@ All the `setter` methods have been removed from the commands above and also ther
         array $context = [],
     ): void;
 ```
+
+* The `Sylius\Bundle\ApiBundle\OpenApi\Documentation\PathHiderDocumentationModifier` class and service have been removed.
+  The `sylius.api.paths_to_hide` parameter used in this class has also been removed. We recommend defining endpoints 
+  as NotExposed in case of the need to hide operations.
diff --git a/src/Sylius/Bundle/ApiBundle/OpenApi/Documentation/PathHiderDocumentationModifier.php b/src/Sylius/Bundle/ApiBundle/OpenApi/Documentation/PathHiderDocumentationModifier.php
diff --git a/src/Sylius/Bundle/ApiBundle/Resources/config/app/config.yaml b/src/Sylius/Bundle/ApiBundle/Resources/config/app/config.yaml
@@ -9,16 +9,6 @@ parameters:
         - "sylius_api_shop_order_payment_get_configuration"
         - "sylius_api_shop_order_payment_patch"
         - "sylius_api_shop_order_payment_patch_account"
-    sylius.api.paths_to_hide:
-        - "%sylius.security.new_api_route%/admin/address-log-entry/{id}"
-        - "%sylius.security.new_api_route%/admin/catalog-promotion-actions/{id}"
-        - "%sylius.security.new_api_route%/admin/catalog-promotion-scopes/{id}"
-        - "%sylius.security.new_api_route%/admin/channel-pricings/{id}"
-        - "%sylius.security.new_api_route%/admin/promotion-actions/{id}"
-        - "%sylius.security.new_api_route%/admin/promotion-rules/{id}"
-        - "%sylius.security.new_api_route%/admin/shipping-method-rules/{id}"
-        - "%sylius.security.new_api_route%/admin/shop-users/{id}"
-        - "%sylius.security.new_api_route%/shop/adjustments/{id}"
     sylius.security.new_api_route: "/api/v2"
     sylius.security.new_api_regex: "^%sylius.security.new_api_route%"
     sylius.security.new_api_admin_route: "%sylius.security.new_api_route%/admin"

diff --git a/src/Sylius/Bundle/ApiBundle/Resources/config/integrations/swagger.xml b/src/Sylius/Bundle/ApiBundle/Resources/config/integrations/swagger.xml
@@ -33,11 +33,6 @@
             <tag name="sylius.open_api.modifier" />
         </service>
 
-        <service id="Sylius\Bundle\ApiBundle\OpenApi\Documentation\PathHiderDocumentationModifier">
-            <argument>%sylius.api.paths_to_hide%</argument>
-            <tag name="sylius.open_api.modifier" />
-        </service>
-
         <service id="Sylius\Bundle\ApiBundle\OpenApi\Documentation\AttributeTypeDocumentationModifier">
             <argument type="service" id="sylius.registry.attribute_type" />
             <tag name="sylius.open_api.modifier" />