Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update 0.82.0-0.83.0.sql #1468

Merged
merged 1 commit into from
May 30, 2024
Merged

Update 0.82.0-0.83.0.sql #1468

merged 1 commit into from
May 30, 2024

Conversation

r3bers
Copy link
Contributor

@r3bers r3bers commented May 30, 2024

Forgot to update schema to work with Boosts properly

? !
Type bug
BC Break yes / no
Fixed issues

Summary

Always has a warnings in DB.php on every update.

[30-May-2024 02:00:33 UTC] PHP Warning: PDOStatement::execute(): SQLSTATE[42S22]: Column not found: 1054 Unknown column 'chat_boost_updated_id' in 'field list' in /vendor/longman/telegram-bot/src/DB.php on line 404
[30-May-2024 02:00:33 UTC] PHP Stack trace:
[30-May-2024 02:00:33 UTC] PHP 1. {main}() /manager.php:0
[30-May-2024 02:00:33 UTC] PHP 2. TelegramBot\TelegramBotManager\BotManager->run() /manager.php:28
[30-May-2024 02:00:33 UTC] PHP 3. TelegramBot\TelegramBotManager\BotManager->handleRequest() /vendor/php-telegram-bot/telegram-bot-manager/src/BotManager.php:106
[30-May-2024 02:00:33 UTC] PHP 4. TelegramBot\TelegramBotManager\BotManager->handleWebhook() /vendor/php-telegram-bot/telegram-bot-manager/src/BotManager.php:266
[30-May-2024 02:00:33 UTC] PHP 5. Longman\TelegramBot\Telegram->handle() /vendor/php-telegram-bot/telegram-bot-manager/src/BotManager.php:418
[30-May-2024 02:00:33 UTC] PHP 6. Longman\TelegramBot\Telegram->processUpdate($update = class Longman\TelegramBot\Entities\Update { public $bot_username = 'some-bot'; public $raw_data = ['update_id' => 312745397, 'message' => [...]]; private ${Longman\TelegramBot\Entities\Entity}fields = ['update_id' => 312745397, 'message' => [...]] }) /vendor/longman/telegram-bot/src/Telegram.php:568
[30-May-2024 02:00:33 UTC] PHP 7. Longman\TelegramBot\DB::insertRequest($update = class Longman\TelegramBot\Entities\Update { public $bot_username = 'some-bot'; public $raw_data = ['update_id' => 312745397, 'message' => [...]]; private ${Longman\TelegramBot\Entities\Entity}fields = ['update_id' => 312745397, 'message' => [...]] }) /vendor/longman/telegram-bot/src/Telegram.php:655
[30-May-2024 02:00:33 UTC] PHP 8. Longman\TelegramBot\DB::insertTelegramUpdate($update_id = 312745397, $chat_id = -10000000000, $message_id = 000000, $edited_message_id = NULL, $channel_post_id = NULL, $edited_channel_post_id = NULL, $message_reaction_id = NULL, $message_reaction_count_id = NULL, $inline_query_id = NULL, $chosen_inline_result_id = NULL, $callback_query_id = NULL, $shipping_query_id = NULL, $pre_checkout_query_id = NULL, $poll_id = NULL, $poll_answer_poll_id = NULL, $my_chat_member_updated_id = NULL, $chat_member_updated_id = NULL, $chat_join_request_id = NULL, $chat_boost_updated_id = NULL, $chat_boost_removed_id = NULL) /vendor/longman/telegram-bot/src/DB.php:627
[30-May-2024 02:00:33 UTC] PHP 9. PDOStatement->execute() /vendor/longman/telegram-bot/src/DB.php:404

It seems just forgot to update schema

Forgot to update schema to work with Boosts properly
@TiiFuchs TiiFuchs merged commit 8f9cd11 into php-telegram-bot:develop May 30, 2024
4 checks passed
@noplanman
Copy link
Member

Thanks for this @r3bers!

@TiiFuchs we need to add this to structure.sql too.

@TiiFuchs
Copy link
Member

Have it on my todo, need another few hours until I get to it.

@TiiFuchs
Copy link
Member

TiiFuchs commented May 30, 2024

@noplanman Added it with commit ca9eeb1

@Worzaii
Copy link

Worzaii commented Jun 9, 2024

It seems the Packagist hasn't updated the structure.sql or the 0.82.0-0.83.0.sql from the latest commit.
Packagist reports latest update to be the 2024-05-25 19:57 UTC.

@r3bers
Copy link
Contributor Author

r3bers commented Jun 9, 2024

It was merged only in develop branch. Master brunch 8 commits behind

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants