core: Problem with cron and runCommands

/show commands gets triggered by user manually, so there is incoming update containing his user_id so the bot knows who reply to.

What you want to do is write seperate command that iterates over all users in the database, checks if they got notification and then sends a message to them using user_id from the database row.

Best thing would be to extract the /show functionality to a separate class that you can call anywhere. Then, in your cron, simply loop the user list and do a Request::sendMessage(...) for each user that has notifications. I think it doesn’t make sense to create fake updates for all of these messages.

On the topic, just use -1 for update_id, otherwise there will probably be a problem with duplicate entries at some point.

@catsAND You must understand cron is meant for commands doing some kind of scheduled tasks. Commands run with runCommands() are executed as the bot itself, and because bot cannot message itself it will return errors.

In this case you replace sendMessage() part with:

if($this->getTelegram()->getBotId() != $chat_id) {
     return Request::emptyResponse();
} else {
     return Request::sendMessage($data);
}

(the code you pasted earlier does totally different thing, it makes it so that command only answers to bot)