Bug/missing square bracket escape gsm7 (#367)

* Fix regex

* Fixed regex by using character encoding

* Composer didn't like php-http discovery

Author: SecondeJK

composer.json

@@ -35,7 +35,10 @@
   },
   "config": {
     "optimize-autoloader": true,
-    "preferred-install": "dist"
+    "preferred-install": "dist",
+    "allow-plugins": {
+      "php-http/discovery": true
+    }
   },
   "autoload": {
     "psr-4": {

src/SMS/Message/SMS.php

@@ -13,7 +13,7 @@
 
 class SMS extends OutboundMessage
 {
-    public const GSM_7_PATTERN = '/\A[\n\f\r !\"\#$%&\'()*+,-.\/0123456789:;<=>?@ABCDEFGHIJKLMNOPQRSTUVWXYZ[\\\\\^_abcdefghijklmnopqrstuvwxyz{\|}~ ¡£¤¥§¿ÄÅÆÇÉÑÖØÜßàäåæèéìñòöøùüΓΔΘΛΞΠΣΦΨΩ€]*\z/m';
+    public const GSM_7_CHARSET = "\n\f\r !\"\#$%&'()*+,-./0123456789:;<=>?@ABCDEFGHIJKLMNOPQRSTUVWXYZ[\\]^_abcdefghijklmnopqrstuvwxyz{|}~ ¡£¤¥§¿ÄÅÆÇÉÑÖØÜßàäåæèéìñòöøùüΓΔΘΛΞΠΣΦΨΩ€";
 
     protected ?string $contentId;
 
@@ -32,7 +32,8 @@ public function __construct(string $to, string $from, protected string $message,
 
     public static function isGsm7(string $message): bool
     {
-        return (bool)preg_match(self::GSM_7_PATTERN, $message);
+        $fullPattern = "/\A[" . preg_quote(self::GSM_7_CHARSET, '/') . "]*\z/";
+        return (bool)preg_match($fullPattern, $message);
     }
 
     public function getContentId(): string
@@ -68,7 +69,7 @@ public function getErrorMessage(): ?string
 
         if ($this->getType() === 'text' && ! self::isGsm7($this->getMessage())) {
             $this->setErrorMessage("You are sending a message as `text` when contains unicode only 
-            characters. This could result in encoding problems with the target device or increased billing - See 
+            characters. This could result in encoding problems with the target device - See 
             https://developer.vonage.com/messaging/sms for details, or email support@vonage.com if you have any 
             questions.");
         }

test/SMS/ClientTest.php

@@ -413,7 +413,7 @@ public function testThrowsWarningWhenSendingUnicodeAsText(): void
 
         $this->expectWarning();
         $this->expectErrorMessage("You are sending a message as `text` when contains unicode only 
-            characters. This could result in encoding problems with the target device or increased billing - See 
+            characters. This could result in encoding problems with the target device - See 
             https://developer.vonage.com/messaging/sms for details, or email support@vonage.com if you have any 
             questions.");
 

test/SMS/Message/SMSTest.php

@@ -182,6 +182,7 @@ public function unicodeStringDataProvider(): array
     {
         return [
             ['this is a text', true],
+            ['This is a text with some tasty characters: [test]', true],
             ['This is also a GSM7 text', true],
             ['This is a Çotcha', true],
             ['This is also a çotcha', true],