Removing $contentVariables from TwilioSmsMessage #154, as it clashed with TwilioContentTemplateMessage implementation

Author: onlime

src/TwilioContentTemplateMessage.php

@@ -25,7 +25,9 @@ public function contentSid(string $contentSid): self
     }
 
     /**
-     * Set the content variables.
+     * For Content Editor/API only: Key-value pairs of Template variables and their substitution values.
+     * content_sid parameter must also be provided.
+     * If values are not defined in the content_variables parameter, the Template's default placeholder values are used.
      *
      * @param  array $contentVariables The variables to replace in the content template (i.e. ['1' => 'John Doe'])
      */

src/TwilioSmsMessage.php

@@ -34,8 +34,6 @@ class TwilioSmsMessage extends TwilioMessage
 
     public ?bool $sendAsMms = null;
 
-    public ?string $contentVariables = null;
-
     public ?string $riskCheck = null;
 
     /**
@@ -223,18 +221,6 @@ public function sendAsMms(bool $sendAsMms): self
         return $this;
     }
 
-    /**
-     * For Content Editor/API only: Key-value pairs of Template variables and their substitution values.
-     * content_sid parameter must also be provided.
-     * If values are not defined in the content_variables parameter, the Template's default placeholder values are used.
-     */
-    public function contentVariables(string $contentVariables): self
-    {
-        $this->contentVariables = $contentVariables;
-
-        return $this;
-    }
-
     /**
      * Include this parameter with a value of "disable" to skip any kind of risk check on the respective message request.
      * Possible values:

tests/Unit/TwilioSmsMessageTest.php

@@ -75,7 +75,6 @@ public function it_can_set_optional_parameters()
         $message->scheduleType('fixed');
         $message->sendAt('2021-01-01 00:00:00');
         $message->sendAsMms(true);
-        $message->contentVariables('{"name": "John"}');
         $message->riskCheck('disable');
 
         $this->assertEquals('http://example.com', $message->statusCallback);
@@ -92,7 +91,6 @@ public function it_can_set_optional_parameters()
         $this->assertEquals('fixed', $message->scheduleType);
         $this->assertEquals('2021-01-01 00:00:00', $message->sendAt);
         $this->assertEquals(true, $message->sendAsMms);
-        $this->assertEquals('{"name": "John"}', $message->contentVariables);
         $this->assertEquals('disable', $message->riskCheck);
     }
 }