noplanman / php-telegram-bot

src/ConversationDB.php

Spacing +1 added lines, -1 removed lines

@@ -84,7 +84,7 @@
         }
 
         try {
-            $sth    = self::$pdo->prepare('INSERT INTO `' . TB_CONVERSATION . '`
+            $sth = self::$pdo->prepare('INSERT INTO `' . TB_CONVERSATION . '`
                 (
                 `status`, `user_id`, `chat_id`, `command`, `notes`, `created_at`, `updated_at`
                 )

src/Commands/AdminCommands/SendtochannelCommand.php

Doc Comments +2 added lines, -2 removed lines

@@ -55,7 +55,7 @@ 
     /**
      * Command execute method
      *
-     * @return \Longman\TelegramBot\Entities\ServerResponse|mixed
+     * @return \Longman\TelegramBot\Entities\ServerResponse
      * @throws \Longman\TelegramBot\Exception\TelegramException
      */
     public function execute()
@@ -329,7 +329,7 @@ 
      *
      * @todo Why send just to the first found channel?
      *
-     * @return mixed
+     * @return \Longman\TelegramBot\Entities\ServerResponse
      * @throws \Longman\TelegramBot\Exception\TelegramException
      */
     public function executeNoDb()

Unused Use Statements +3 added lines, -3 removed lines

@@ -10,12 +10,12 @@
 
 namespace Longman\TelegramBot\Commands\AdminCommands;
 
-use Longman\TelegramBot\Entities\Keyboard;
-use Longman\TelegramBot\Request;
-use Longman\TelegramBot\Conversation;
 use Longman\TelegramBot\Commands\AdminCommand;
+use Longman\TelegramBot\Conversation;
+use Longman\TelegramBot\Entities\Keyboard;
 use Longman\TelegramBot\Entities\Message;
 use Longman\TelegramBot\Exception\TelegramException;
+use Longman\TelegramBot\Request;
 
 class SendtochannelCommand extends AdminCommand
 {

src/DB.php

Spacing +1 added lines, -1 removed lines

@@ -262,7 +262,7 @@
         }
 
         //Convert each Entity item into an object based on its JSON reflection
-        $json_entities = array_map(function ($entity) {
+        $json_entities = array_map(function($entity) {
             return json_decode($entity, true);
         }, $entities);
 

Doc Comments +1 added lines, -1 removed lines

@@ -251,7 +251,7 @@
      * @todo Find a better way, as json_* functions are very heavy
      *
      * @param array|null $entities
-     * @param mixed      $default
+     * @param string|null      $default
      *
      * @return mixed
      */

src/Request.php

Doc Comments +1 added lines, -1 removed lines

@@ -238,7 +238,7 @@
      *
      * @param \Longman\TelegramBot\Entities\File $file
      *
-     * @return boolean
+     * @return boolean|string
      * @throws \Longman\TelegramBot\Exception\TelegramException
      */
     public static function downloadFile(File $file)

Spacing +6 added lines, -6 removed lines

@@ -260,7 +260,7 @@ 
                 'first_name' => 'botname',
                 'username'   => 'namebot',
             ];
-            $data['chat']       = ['id' => $data['chat_id']];
+            $data['chat'] = ['id' => $data['chat_id']];
 
             $fake_response['result'] = $data;
         }
@@ -284,14 +284,14 @@ 
         $multipart    = [];
 
         // Convert any nested arrays into JSON strings.
-        array_walk($data, function (&$item) {
+        array_walk($data, function(&$item) {
             is_array($item) && $item = json_encode($item);
         });
 
         //Reformat data array in multipart way if it contains a resource
         foreach ($data as $key => $item) {
             $has_resource |= (is_resource($item) || $item instanceof \GuzzleHttp\Psr7\Stream);
-            $multipart[]  = ['name' => $key, 'contents' => $item];
+            $multipart[] = ['name' => $key, 'contents' => $item];
         }
         if ($has_resource) {
             return ['multipart' => $multipart];
@@ -325,7 +325,7 @@ 
                 '/bot' . self::$telegram->getApiKey() . '/' . $action,
                 $request_params
             );
-            $result   = (string) $response->getBody();
+            $result = (string) $response->getBody();
 
             //Logging getUpdates Update
             if ($action === 'getUpdates') {
@@ -651,8 +651,8 @@ 
                     $requests = DB::getTelegramRequestCount($chat_id, $inline_message_id);
 
                     $chat_per_second   = ($requests['LIMIT_PER_SEC'] == 0); // No more than one message per second inside a particular chat
-                    $global_per_second = ($requests['LIMIT_PER_SEC_ALL'] < 30);    // No more than 30 messages per second to different chats
-                    $groups_per_minute = (((is_numeric($chat_id) && $chat_id > 0) || !is_null($inline_message_id)) || ((!is_numeric($chat_id) || $chat_id < 0) && $requests['LIMIT_PER_MINUTE'] < 20));    // No more than 20 messages per minute in groups and channels
+                    $global_per_second = ($requests['LIMIT_PER_SEC_ALL'] < 30); // No more than 30 messages per second to different chats
+                    $groups_per_minute = (((is_numeric($chat_id) && $chat_id > 0) || !is_null($inline_message_id)) || ((!is_numeric($chat_id) || $chat_id < 0) && $requests['LIMIT_PER_MINUTE'] < 20)); // No more than 20 messages per minute in groups and channels
 
                     if ($chat_per_second && $global_per_second && $groups_per_minute) {
                         break;

src/Entities/InlineKeyboardButton.php

Indentation +2 added lines, -2 removed lines

@@ -42,11 +42,11 @@
     {
         return is_array($data) &&
                array_key_exists('text', $data) && (
-                   array_key_exists('url', $data) ||
+                    array_key_exists('url', $data) ||
                    array_key_exists('callback_data', $data) ||
                    array_key_exists('switch_inline_query', $data) ||
                    array_key_exists('switch_inline_query_current_chat', $data)
-               );
+                );
     }
 
     /**

src/Entities/ServerResponse.php

Spacing +5 added lines, -5 removed lines

@@ -100,11 +100,11 @@
 
         $result_object_types = [
             'total_count' => 'UserProfilePhotos', //Response from getUserProfilePhotos
-            'file_id'     => 'File',              //Response from getFile
-            'title'       => 'Chat',              //Response from getChat
-            'username'    => 'User',              //Response from getMe
-            'user'        => 'ChatMember',        //Response from getChatMember
-            'url'         => 'WebhookInfo',       //Response from getWebhookInfo
+            'file_id'     => 'File', //Response from getFile
+            'title'       => 'Chat', //Response from getChat
+            'username'    => 'User', //Response from getMe
+            'user'        => 'ChatMember', //Response from getChatMember
+            'url'         => 'WebhookInfo', //Response from getWebhookInfo
         ];
         foreach ($result_object_types as $type => $object_class) {
             if (isset($result[$type])) {

src/Entities/Entity.php

Spacing +2 added lines, -2 removed lines

@@ -199,8 +199,8 @@
     public function escapeMarkdown($string)
     {
         return str_replace(
-            ['[', '`', '*', '_',],
-            ['\[', '\`', '\*', '\_',],
+            ['[', '`', '*', '_', ],
+            ['\[', '\`', '\*', '\_', ],
             $string
         );
     }

src/Telegram.php

Doc Comments +1 added lines, -1 removed lines

@@ -801,7 +801,7 @@
     /**
      * Delete any assigned webhook
      *
-     * @return mixed
+     * @return ServerResponse
      * @throws \Longman\TelegramBot\Exception\TelegramException
      */
     public function deleteWebhook()

Spacing +4 added lines, -4 removed lines

@@ -787,7 +787,7 @@ 
             throw new TelegramException('Hook url is empty!');
         }
 
-        $data        = array_intersect_key($data, array_flip([
+        $data = array_intersect_key($data, array_flip([
             'certificate',
             'max_connections',
             'allowed_updates',
@@ -902,7 +902,7 @@ 
         }
 
         $this->run_commands  = true;
-        $this->botan_enabled = false;   // Force disable Botan.io integration, we don't want to track self-executed commands!
+        $this->botan_enabled = false; // Force disable Botan.io integration, we don't want to track self-executed commands!
 
         $result = Request::getMe();
 
@@ -919,8 +919,8 @@ 
         }
 
 
-        $this->enableAdmin($bot_id);    // Give bot access to admin commands
-        $this->getCommandsList();       // Load full commands list
+        $this->enableAdmin($bot_id); // Give bot access to admin commands
+        $this->getCommandsList(); // Load full commands list
 
         foreach ($commands as $command) {
             $this->update = new Update(

src/Commands/AdminCommands/CleanupCommand.php

Spacing +3 added lines, -3 removed lines

@@ -101,7 +101,7 @@ 
         }
 
         // Convert numeric-only values to days.
-        array_walk($clean_older_than, function (&$time) use ($custom_time) {
+        array_walk($clean_older_than, function(&$time) use ($custom_time) {
             if (!empty($custom_time)) {
                 $time = $custom_time;
             }
@@ -357,9 +357,9 @@ 
                 }
             }
 
-            $pdo->commit();     // commit changes to the database and end transaction
+            $pdo->commit(); // commit changes to the database and end transaction
         } catch (PDOException $e) {
-            $pdo->rollBack();   // rollback changes on exception (useful if you want to track down error - you can't replicate it when some of the data is already deleted...)
+            $pdo->rollBack(); // rollback changes on exception (useful if you want to track down error - you can't replicate it when some of the data is already deleted...)
 
             $data['text'] = '*Database cleanup failed!* _(check your error logs)_';
             Request::sendMessage($data);