Merge pull request #137 from ToshY/feature/136

Split `addAndUpdateEdgeRule` into specific `addEdgeRule` and `updateEdgeRule` methods

Author: ToshY

docs/base-api.md

@@ -1436,22 +1436,12 @@ $baseApi->deletePullZone(
 );
 ```
 
-#### [Delete Edge Rule](https://docs.bunny.net/reference/pullzonepublic_deleteedgerule)
-
-```php
-$baseApi->deleteEdgeRule(
-    pullZoneId: 1,
-    edgeRuleId: 'c71d9594-3bc6-4639-9896-ba3e96217587',
-);
-```
-
-#### [Add/Update Edge Rule](https://docs.bunny.net/reference/pullzonepublic_addedgerule)
+#### [Add Edge Rule](https://docs.bunny.net/reference/pullzonepublic_addedgerule)
 
 ```php
-$baseApi->addOrUpdateEdgeRule(
+$baseApi->addEdgeRule(
     pullZoneId: 1,
     body: [
-        'Guid' => 'c71d9594-3bc6-4639-9896-ba3e96217587',
         'ActionType' => 4,
         'ActionParameter1' => '',
         'ActionParameter2' => '',
@@ -1515,6 +1505,27 @@ $baseApi->addOrUpdateEdgeRule(
         - `1` = Match All
         - `2` = Match None
 
+#### [Update Edge Rule](https://docs.bunny.net/reference/pullzonepublic_addedgerule)
+
+```php
+$baseApi->updateEdgeRule(
+    pullZoneId: 1,
+    body: [
+        'Guid' => 'c71d9594-3bc6-4639-9896-ba3e96217587',
+        'Triggers' => [
+            [
+                'Type' => 7,
+                'PatternMatches' => ['75']
+            ],
+        ],
+    ],
+);
+```
+
+!!! note
+
+    - The keys `Guid` and `Triggers` in the body are required parameters when updating an edge rule.
+
 #### [Set Edge Rule Enabled](https://docs.bunny.net/reference/pullzonepublic_setedgeruleenabled)
 
 ```php
@@ -1532,6 +1543,15 @@ $baseApi->setEdgeRuleEnabled(
 
     -  The key `Id` in the body denotes the pull zone ID (the same as the first argument) and is (for some reason) a required parameter.
 
+#### [Delete Edge Rule](https://docs.bunny.net/reference/pullzonepublic_deleteedgerule)
+
+```php
+$baseApi->deleteEdgeRule(
+    pullZoneId: 1,
+    edgeRuleId: 'c71d9594-3bc6-4639-9896-ba3e96217587',
+);
+```
+
 #### Set Zone Security Enabled
 
 ```php

src/BaseAPI.php

@@ -58,7 +58,8 @@
 use ToshY\BunnyNet\Model\API\Base\DRMCertificate\ListDRMCertificates;
 use ToshY\BunnyNet\Model\API\Base\PullZone\AddCustomCertificate;
 use ToshY\BunnyNet\Model\API\Base\PullZone\AddCustomHostname;
-use ToshY\BunnyNet\Model\API\Base\PullZone\AddOrUpdateEdgeRule;
+use ToshY\BunnyNet\Model\API\Base\PullZone\AddEdgeRule;
+use ToshY\BunnyNet\Model\API\Base\PullZone\UpdateEdgeRule;
 use ToshY\BunnyNet\Model\API\Base\PullZone\AddPullZone;
 use ToshY\BunnyNet\Model\API\Base\PullZone\CheckPullZoneAvailability;
 use ToshY\BunnyNet\Model\API\Base\PullZone\DeleteCertificate;
@@ -1693,12 +1694,58 @@ public function deleteEdgeRule(
      * @return BunnyClientResponseInterface
      * @param int $pullZoneId
      * @param array<string,mixed> $body
+     *
+     * @deprecated since 4.5 (to be removed in 5.0). Use {@link addEdgeRule} or {@link updateEdgeRule} instead.
      */
     public function addOrUpdateEdgeRule(
         int $pullZoneId,
         array $body,
     ): BunnyClientResponseInterface {
-        $endpoint = new AddOrUpdateEdgeRule();
+        return $this->addEdgeRule($pullZoneId, $body);
+    }
+
+    /**
+     * @throws ClientExceptionInterface
+     * @throws Exception\BunnyClientResponseException
+     * @throws Exception\JSONException
+     * @throws Exception\InvalidTypeForKeyValueException
+     * @throws Exception\InvalidTypeForListValueException
+     * @throws Exception\ParameterIsRequiredException
+     * @return BunnyClientResponseInterface
+     * @param int $pullZoneId
+     * @param array<string,mixed> $body
+     */
+    public function addEdgeRule(
+        int $pullZoneId,
+        array $body,
+    ): BunnyClientResponseInterface {
+        $endpoint = new AddEdgeRule();
+
+        ParameterValidator::validate($body, $endpoint->getBody());
+
+        return $this->client->request(
+            endpoint: $endpoint,
+            parameters: [$pullZoneId],
+            body: BodyContentHelper::getBody($body),
+        );
+    }
+
+    /**
+     * @throws ClientExceptionInterface
+     * @throws Exception\BunnyClientResponseException
+     * @throws Exception\JSONException
+     * @throws Exception\InvalidTypeForKeyValueException
+     * @throws Exception\InvalidTypeForListValueException
+     * @throws Exception\ParameterIsRequiredException
+     * @return BunnyClientResponseInterface
+     * @param int $pullZoneId
+     * @param array<string,mixed> $body
+     */
+    public function updateEdgeRule(
+        int $pullZoneId,
+        array $body,
+    ): BunnyClientResponseInterface {
+        $endpoint = new UpdateEdgeRule();
 
         ParameterValidator::validate($body, $endpoint->getBody());
 

src/Model/API/Base/PullZone/AddOrUpdateEdgeRule.php renamed to src/Model/API/Base/PullZone/AddEdgeRule.php

@@ -11,7 +11,7 @@
 use ToshY\BunnyNet\Model\EndpointBodyInterface;
 use ToshY\BunnyNet\Model\EndpointInterface;
 
-class AddOrUpdateEdgeRule implements EndpointInterface, EndpointBodyInterface
+class AddEdgeRule implements EndpointInterface, EndpointBodyInterface
 {
     public function getMethod(): Method
     {

src/Model/API/Base/PullZone/UpdateEdgeRule.php

@@ -0,0 +1,45 @@
+<?php
+
+declare(strict_types=1);
+
+namespace ToshY\BunnyNet\Model\API\Base\PullZone;
+
+use ToshY\BunnyNet\Enum\Header;
+use ToshY\BunnyNet\Enum\Method;
+use ToshY\BunnyNet\Enum\Type;
+use ToshY\BunnyNet\Model\AbstractParameter;
+use ToshY\BunnyNet\Model\EndpointBodyInterface;
+use ToshY\BunnyNet\Model\EndpointInterface;
+
+class UpdateEdgeRule implements EndpointInterface, EndpointBodyInterface
+{
+    public function getMethod(): Method
+    {
+        return Method::POST;
+    }
+
+    public function getPath(): string
+    {
+        return 'pullzone/%d/edgerules/addOrUpdate';
+    }
+
+    public function getHeaders(): array
+    {
+        return [
+            Header::CONTENT_TYPE_JSON,
+        ];
+    }
+
+    public function getBody(): array
+    {
+        return [
+            new AbstractParameter(name: 'Guid', type: Type::STRING_TYPE, required: true),
+            new AbstractParameter(name: 'Triggers', type: Type::ARRAY_TYPE, required: true, children: [
+                new AbstractParameter(name: 'Type', type: Type::INT_TYPE),
+                new AbstractParameter(name: 'PatternMatches', type: Type::ARRAY_TYPE, children: [
+                    new AbstractParameter(name: null, type: Type::STRING_TYPE),
+                ]),
+            ]),
+        ];
+    }
+}