Merge pull request #13 from tanhongit/develop

remove redundant code in the send notification method
cslant · Oct 10, 2023 · 2688781 · 2688781
2 parents 8e91af2 + 379938c
commit 2688781
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 17 deletions.
diff --git a/common/helpers.php b/common/helpers.php
@@ -2,20 +2,6 @@
 
 use LbilTech\TelegramGitNotifier\Helpers\ConfigHelper;
 
-if (!function_exists('tgn_urlencoded_message')) {
-    /**
-     * Urlencoded message
-     *
-     * @param string $message
-     *
-     * @return array|string|string[]
-     */
-    function tgn_urlencoded_message(string $message): array|string
-    {
-        return str_replace(["\n"], ['%0A'], urlencode($message));
-    }
-}
-
 if (!function_exists('tgn_singularity')) {
     /**
      * The reverse of pluralizing, returns the singular form of a word in a string.

diff --git a/src/Services/NotificationService.php b/src/Services/NotificationService.php
@@ -14,9 +14,9 @@ class NotificationService implements NotificationInterface
 {
     use ActionEventTrait;
 
-    protected mixed $payload;
+    public mixed $payload;
 
-    protected string $message = '';
+    public string $message = '';
 
     public string $platform = EventConstant::DEFAULT_PLATFORM;
 
@@ -87,7 +87,7 @@ public function sendNotify(string $chatId, string $message = null): bool
             'chat_id'                  => $chatId,
             'disable_web_page_preview' => 1,
             'parse_mode'               => 'html',
-            'text'                     => tgn_urlencoded_message($this->message)
+            'text'                     => $this->message
         ];
 
         $url = 'https://api.telegram.org/bot'