Removed support for tariffs

Author: marventhieme

CHANGELOG.md

@@ -9,6 +9,8 @@ All notable changes to `CMSMS` will be documented in this file
 #### Added
 - Added config value for encoding detection type
 - Two events for success and failure: `SMSSentSuccessfullyEvent` and `SMSSendingFailedEvent`
+#### Removed
+- Removed `tariff` support
 
 ## [3.3.0] - 2024-03-22
 #### Added

README.md

@@ -105,7 +105,6 @@ public function routeNotificationForCmsms()
 - `body('')`: Accepts a string value for the message body.
 - `originator('')`: Accepts a string value between 1 and 11 characters, used as the message sender name.
 - `reference('')`: Accepts a string value for your message reference. This information will be returned in a status report so you can match the message and it's status. Restrictions: 1 - 32 alphanumeric characters. Reference will not work for demo accounts.
-- `tariff()`: Accepts a integer value for your message tariff. The unit is eurocent. Requires the `originator` to be set to a specific value. Contact CM for this tariff value. CM also must enable this feature for your contract manually.
 - `multipart($minimum, $maximum)`: Accepts a 0 to 8 integer range which allows multipart messages. See the [documentation from CM](https://dashboard.onlinesmsgateway.com/docs#send-a-message-multipart) for more information.
 
 ### Available events

src/CmsmsClient.php

@@ -89,7 +89,6 @@ public function buildMessageJson(CmsmsMessage $message, string $recipient): stri
                 'authentication' => [
                     'productToken' => $this->productToken,
                 ],
-                'tariff' => $message->getTariff(),
                 'msg' => [[
                     'body' => $body,
                     'to' => [[

src/CmsmsMessage.php

@@ -12,8 +12,6 @@ class CmsmsMessage
 
     protected string $reference = '';
 
-    protected int $tariff = 0;
-
     protected ?int $minimumNumberOfMessageParts = null;
 
     protected ?int $maximumNumberOfMessageParts = null;
@@ -68,18 +66,6 @@ public function getReference(): string
         return $this->reference;
     }
 
-    public function tariff(int $tariff): self
-    {
-        $this->tariff = $tariff;
-
-        return $this;
-    }
-
-    public function getTariff(): int
-    {
-        return $this->tariff;
-    }
-
     public function multipart(int $minimum, int $maximum): self
     {
         if ($maximum > 8 || $minimum >= $maximum) {

tests/CmsmsClientTest.php

@@ -83,20 +83,6 @@ public function it_throws_exception_on_error_response()
         $this->client->send($this->message, '00301234');
     }
 
-    /** @test */
-    public function it_includes_tariff_data()
-    {
-        $message = clone $this->message;
-        $message->tariff(20);
-
-        $messageJson = $this->client->buildMessageJson($message, '00301234');
-
-        $messageJsonObject = json_decode($messageJson);
-
-        $this->assertTrue(isset($messageJsonObject->messages->tariff));
-        $this->assertEquals(20, $messageJsonObject->messages->tariff);
-    }
-
     /** @test */
     public function it_includes_multipart_data()
     {

tests/CmsmsMessageTest.php

@@ -97,22 +97,6 @@ public function it_cannot_set_a_reference_that_contains_non_alpha_numeric_values
         CmsmsMessage::create()->reference('@#$*A*Sjks87');
     }
 
-    /** @test */
-    public function it_can_set_tariff()
-    {
-        $message = CmsmsMessage::create()->tariff(12);
-
-        $this->assertEquals(12, $message->getTariff());
-    }
-
-    /** @test */
-    public function it_can_set_an_empty_tariff()
-    {
-        $message = CmsmsMessage::create()->tariff(0);
-
-        $this->assertEquals(0, $message->getTariff());
-    }
-
     public function it_can_set_multipart()
     {
         $message = CmsmsMessage::create()->multipart(1, 4);