Merge pull request #9 from noplanman/selectchats_channels_extended

Use array for select-parameters, instead of individual parameters.

src/Commands/AdminCommands/ChatsCommand.php

@@ -35,7 +35,7 @@ class ChatsCommand extends AdminCommand
     /**
      * @var string
      */
-    protected $version = '1.1.0';
+    protected $version = '1.2.0';
 
     /**
      * @var bool
@@ -45,7 +45,7 @@ class ChatsCommand extends AdminCommand
     /**
      * Command execute method
      *
-     * @return mixed
+     * @return \Longman\TelegramBot\Entities\ServerResponse
      * @throws \Longman\TelegramBot\Exception\TelegramException
      */
     public function execute()
@@ -55,21 +55,18 @@ class ChatsCommand extends AdminCommand
         $chat_id = $message->getChat()->getId();
         $text    = trim($message->getText(true));
 
-        $results = DB::selectChats(
-            true, //Select groups (group chat)
-            true, //Select supergroups (super group chat)
-            true, //Select channels
-            true, //Select users (single chat)
-            null, //'yyyy-mm-dd hh:mm:ss' date range from
-            null, //'yyyy-mm-dd hh:mm:ss' date range to
-            null, //Specific chat_id to select
-            ($text === '' || $text === '*') ? null : $text //Text to search in user/group name
-        );
-
-        $user_chats        = 0;
-        $group_chats       = 0;
-        $super_group_chats = 0;
-        $channel_chats     = 0;
+        $results = DB::selectChats([
+            'groups'      => true,
+            'supergroups' => true,
+            'channels'    => true,
+            'users'       => true,
+            'text'        => ($text === '' || $text === '*') ? null : $text //Text to search in user/group name
+        ]);
+
+        $user_chats       = 0;
+        $group_chats      = 0;
+        $supergroup_chats = 0;
+        $channel_chats    = 0;
 
         if ($text === '') {
             $text_back = '';
@@ -102,7 +99,7 @@ class ChatsCommand extends AdminCommand
                         $text_back .= '- S ' . $chat->getTitle() . ' [' . $whois . ']' . PHP_EOL;
                     }
 
-                    ++$super_group_chats;
+                    ++$supergroup_chats;
                 } elseif ($chat->isGroupChat()) {
                     if ($text !== '') {
                         $text_back .= '- G ' . $chat->getTitle() . ' [' . $whois . ']' . PHP_EOL;
@@ -119,14 +116,14 @@ class ChatsCommand extends AdminCommand
             }
         }
 
-        if (($user_chats + $group_chats + $super_group_chats) === 0) {
+        if (($user_chats + $group_chats + $supergroup_chats) === 0) {
             $text_back = 'No chats found..';
         } else {
             $text_back .= PHP_EOL . 'Private Chats: ' . $user_chats;
             $text_back .= PHP_EOL . 'Groups: ' . $group_chats;
-            $text_back .= PHP_EOL . 'Super Groups: ' . $super_group_chats;
+            $text_back .= PHP_EOL . 'Super Groups: ' . $supergroup_chats;
             $text_back .= PHP_EOL . 'Channels: ' . $channel_chats;
-            $text_back .= PHP_EOL . 'Total: ' . ($user_chats + $group_chats + $super_group_chats);
+            $text_back .= PHP_EOL . 'Total: ' . ($user_chats + $group_chats + $supergroup_chats);
 
             if ($text === '') {
                 $text_back .= PHP_EOL . PHP_EOL . 'List all chats: /' . $this->name . ' *' . PHP_EOL . 'Search for chats: /' . $this->name . ' <search string>';

src/Commands/AdminCommands/SendtoallCommand.php

@@ -38,7 +38,7 @@ class SendtoallCommand extends AdminCommand
     /**
      * @var string
      */
-    protected $version = '1.3.0';
+    protected $version = '1.4.0';
 
     /**
      * @var bool
@@ -48,7 +48,7 @@ class SendtoallCommand extends AdminCommand
     /**
      * Execute command
      *
-     * @return boolean
+     * @return \Longman\TelegramBot\Entities\ServerResponse
      * @throws \Longman\TelegramBot\Exception\TelegramException
      */
     public function execute()
@@ -64,12 +64,12 @@ class SendtoallCommand extends AdminCommand
             $results = Request::sendToActiveChats(
                 'sendMessage', //callback function to execute (see Request.php methods)
                 ['text' => $text], //Param to evaluate the request
-                true, //Send to groups (group chat)
-                true, //Send to super groups chats (super group chat)
-                false, //Select channels
-                true, //Send to users (single chat)
-                null, //'yyyy-mm-dd hh:mm:ss' date range from
-                null  //'yyyy-mm-dd hh:mm:ss' date range to
+                [
+                    'groups'      => true,
+                    'supergroups' => true,
+                    'channels'    => false,
+                    'users'       => true,
+                ]
             );
 
             $total  = 0;

src/Commands/AdminCommands/WhoisCommand.php

@@ -42,7 +42,7 @@ class WhoisCommand extends AdminCommand
     /**
      * @var string
      */
-    protected $version = '1.2.0';
+    protected $version = '1.3.0';
 
     /**
      * @var bool
@@ -52,7 +52,7 @@ class WhoisCommand extends AdminCommand
     /**
      * Command execute method
      *
-     * @return mixed
+     * @return \Longman\TelegramBot\Entities\ServerResponse
      * @throws \Longman\TelegramBot\Exception\TelegramException
      */
     public function execute()
@@ -89,30 +89,25 @@ class WhoisCommand extends AdminCommand
             $result     = null;
 
             if (is_numeric($text)) {
-                $results = DB::selectChats(
-                    true, //Select groups
-                    true, //Select super groups
-                    true, //Select channels
-                    true, //Select users
-                    null, //'yyyy-mm-dd hh:mm:ss' date range from
-                    null, //'yyyy-mm-dd hh:mm:ss' date range to
-                    $user_id //Specific chat_id to select
-                );
+                $results = DB::selectChats([
+                    'groups'      => true,
+                    'supergroups' => true,
+                    'channels'    => true,
+                    'users'       => true,
+                    'chat_id'     => $user_id, //Specific chat_id to select
+                ]);
 
                 if (!empty($results)) {
                     $result = reset($results);
                 }
             } else {
-                $results = DB::selectChats(
-                    true, //Select groups
-                    true, //Select super groups
-                    true, //Select channels
-                    true, //Select users
-                    null, //'yyyy-mm-dd hh:mm:ss' date range from
-                    null, //'yyyy-mm-dd hh:mm:ss' date range to
-                    null, //Specific chat_id to select
-                    $text //Text to search in user/group name
-                );
+                $results = DB::selectChats([
+                    'groups'      => true,
+                    'supergroups' => true,
+                    'channels'    => true,
+                    'users'       => true,
+                    'text'        => $text //Text to search in user/group name
+                ]);
 
                 if (is_array($results) && count($results) === 1) {
                     $result = reset($results);

src/DB.php

@@ -979,35 +979,32 @@ class DB
     }
 
     /**
-     * Select Group and/or single Chats
+     * Select Groups, Supergroups, Channels and/or single user Chats (also by ID or text)
      *
-     * @param bool   $select_groups
-     * @param bool   $select_super_groups
-     * @param bool   $select_channels
-     * @param bool   $select_users
-     * @param string $date_from
-     * @param string $date_to
-     * @param int    $chat_id
-     * @param string $text
+     * @param $select_chats_params
      *
-     * @return array|bool (Selected chats or false if invalid arguments)
-     * @throws \Longman\TelegramBot\Exception\TelegramException
+     * @return array|bool
+     * @throws TelegramException
      */
-    public static function selectChats(
-        $select_groups = true,
-        $select_super_groups = true,
-        $select_channels = true,
-        $select_users = true,
-        $date_from = null,
-        $date_to = null,
-        $chat_id = null,
-        $text = null
-    ) {
+    public static function selectChats($select_chats_params)
+    {
         if (!self::isDbConnected()) {
             return false;
         }
 
-        if (!$select_groups && !$select_users && !$select_super_groups) {
+        // Set defaults for omitted values.
+        $select = array_merge([
+            'groups'      => true,
+            'supergroups' => true,
+            'channels'    => true,
+            'users'       => true,
+            'date_from'   => null,
+            'date_to'     => null,
+            'chat_id'     => null,
+            'text'        => null,
+        ], $select_chats_params);
+
+        if (!$select['groups'] && !$select['users'] && !$select['supergroups']) {
             return false;
         }
 
@@ -1019,7 +1016,7 @@ class DB
                 ' . TB_CHAT . '.`created_at` AS `chat_created_at`,
                 ' . TB_CHAT . '.`updated_at` AS `chat_updated_at`
             ';
-            if ($select_users) {
+            if ($select['users']) {
                 $query .= '
                     , ' . TB_USER . '.`id` AS `user_id`
                     FROM `' . TB_CHAT . '`
@@ -1034,34 +1031,34 @@ class DB
             $where  = [];
             $tokens = [];
 
-            if (!$select_groups || !$select_users || !$select_super_groups) {
+            if (!$select['groups'] || !$select['users'] || !$select['supergroups']) {
                 $chat_or_user = [];
 
-                $select_groups && $chat_or_user[] = TB_CHAT . '.`type` = "group"';
-                $select_super_groups && $chat_or_user[] = TB_CHAT . '.`type` = "supergroup"';
-                $select_channels && $chat_or_user[] = TB_CHAT . '.`type` = "channel"';
-                $select_users && $chat_or_user[] = TB_CHAT . '.`type` = "private"';
+                $select['groups'] && $chat_or_user[] = TB_CHAT . '.`type` = "group"';
+                $select['supergroups'] && $chat_or_user[] = TB_CHAT . '.`type` = "supergroup"';
+                $select['channels'] && $chat_or_user[] = TB_CHAT . '.`type` = "channel"';
+                $select['users'] && $chat_or_user[] = TB_CHAT . '.`type` = "private"';
 
                 $where[] = '(' . implode(' OR ', $chat_or_user) . ')';
             }
 
-            if (null !== $date_from) {
+            if (null !== $select['date_from']) {
                 $where[]              = TB_CHAT . '.`updated_at` >= :date_from';
-                $tokens[':date_from'] = $date_from;
+                $tokens[':date_from'] = $select['date_from'];
             }
 
-            if (null !== $date_to) {
+            if (null !== $select['date_to']) {
                 $where[]            = TB_CHAT . '.`updated_at` <= :date_to';
-                $tokens[':date_to'] = $date_to;
+                $tokens[':date_to'] = $select['date_to'];
             }
 
-            if (null !== $chat_id) {
+            if (null !== $select['chat_id']) {
                 $where[]            = TB_CHAT . '.`id` = :chat_id';
-                $tokens[':chat_id'] = $chat_id;
+                $tokens[':chat_id'] = $select['chat_id'];
             }
 
-            if (null !== $text) {
-                if ($select_users) {
+            if (null !== $select['text']) {
+                if ($select['users']) {
                     $where[] = '(
                         LOWER(' . TB_CHAT . '.`title`) LIKE :text
                         OR LOWER(' . TB_USER . '.`first_name`) LIKE :text
@@ -1071,7 +1068,7 @@ class DB
                 } else {
                     $where[] = 'LOWER(' . TB_CHAT . '.`title`) LIKE :text';
                 }
-                $tokens[':text'] = '%' . strtolower($text) . '%';
+                $tokens[':text'] = '%' . strtolower($select['text']) . '%';
             }
 
             if (!empty($where)) {

src/Request.php

@@ -961,40 +961,30 @@ class Request
     /**
      * Send message to all active chats
      *
-     * @param string  $callback_function
-     * @param array   $data
-     * @param boolean $send_groups
-     * @param boolean $send_super_groups
-     * @param boolean $send_channels
-     * @param boolean $send_users
-     * @param string  $date_from
-     * @param string  $date_to
+     * @param string $callback_function
+     * @param array  $data
+     * @param array  $select_chats_params
      *
      * @return array
-     * @throws \Longman\TelegramBot\Exception\TelegramException
+     * @throws TelegramException
      */
     public static function sendToActiveChats(
         $callback_function,
         array $data,
-        $send_groups = true,
-        $send_super_groups = true,
-        $send_channels = false,
-        $send_users = true,
-        $date_from = null,
-        $date_to = null
+        array $select_chats_params
     ) {
         $callback_path = __NAMESPACE__ . '\Request';
         if (!method_exists($callback_path, $callback_function)) {
             throw new TelegramException('Method "' . $callback_function . '" not found in class Request.');
         }
 
-        $chats = DB::selectChats($send_groups, $send_super_groups, $send_channels, $send_users, $date_from, $date_to);
+        $chats = DB::selectChats($select_chats_params);
 
         $results = [];
         if (is_array($chats)) {
             foreach ($chats as $row) {
                 $data['chat_id'] = $row['chat_id'];
-                $results[]       = call_user_func_array($callback_path . '::' . $callback_function, [$data]);
+                $results[]       = call_user_func($callback_path . '::' . $callback_function, $data);
             }
         }