[BUG] `Message.reply_text` leads to `Message thread not found` about python-telegram-bot HOT 9 CLOSED

See #4207. @aiastia maybe you can install fro that branch and double check if that fixes the problem for you?

from python-telegram-bot.

It worked. Thank you so much

from python-telegram-bot.

Hi. Thanks for reaching out. I tried to reproduce the behavior, but don't see an issue. What I've tried is the following.

Use this mwe:

import logging

from telegram import Update
from telegram.ext import (
    ApplicationBuilder,
    ContextTypes,
    MessageHandler,
    filters,
    Defaults,
)

logging.basicConfig(
    format="%(asctime)s - %(name)s - %(levelname)s - %(message)s", level=logging.WARNING
)
# set higher logging level for httpx to avoid all GET and POST requests being logged
logging.getLogger("telegram.Bot").setLevel(logging.DEBUG)
logging.getLogger("telegram.ext.ExtBot").setLevel(logging.DEBUG)


async def callback(update: Update, context: ContextTypes.DEFAULT_TYPE) -> None:
    chat_type = update.message.chat.type
    has_topics = bool(update.message.message_thread_id)
    await update.message.reply_text(
        f"Test reply for chat of type: {chat_type} with topics: {has_topics}",
        message_thread_id=None,
    )


if __name__ == "__main__":

    application = (
        ApplicationBuilder()
        .token("TOKEN")
        # I've tried with and without defaults.
        .defaults(Defaults(do_quote=True))
        .build()
    )
    application.add_handler(MessageHandler(filters.TEXT, callback))

    application.run_polling(allowed_updates=Update.ALL_TYPES)

Then I've sent messages to the bot in the following chats:

1. private chat
2. group chat
3. general topic of a forum group
4. special topic of a forum group

Everything worked as expected for me.

Can you please provide an MWE along with instructions on how to trigger the exception? Please mention group type, privacy settings of the bot, and any other relevant parameters.

from python-telegram-bot.

import logging

from telegram import Update
from telegram.ext import (
    ApplicationBuilder,
    ContextTypes,
    MessageHandler,
    filters,
    Defaults,CommandHandler,
)

logging.basicConfig(
    format="%(asctime)s - %(name)s - %(levelname)s - %(message)s", level=logging.WARNING
)
# set higher logging level for httpx to avoid all GET and POST requests being logged
logging.getLogger("telegram.Bot").setLevel(logging.DEBUG)
logging.getLogger("telegram.ext.ExtBot").setLevel(logging.DEBUG)


async def callback(update: Update, context: ContextTypes.DEFAULT_TYPE) -> None:
    chat_type = update.message.chat.type
    has_topics = bool(update.message.message_thread_id)
    await update.message.reply_text(
        f"Test reply for chat of type: {chat_type} with topics: {has_topics}",
        message_thread_id=None,
    )


if __name__ == "__main__":

    application = (
        ApplicationBuilder()
        .token("token")
        # I've tried with and without defaults.
        .defaults(Defaults(do_quote=True))
        .build()
    )
    #application.add_handler(MessageHandler(filters.TEXT, callback))
    application.add_handler(CommandHandler("ja", callback))
    application.run_polling(allowed_updates=Update.ALL_TYPES)

from python-telegram-bot.

Like this, when you use the ja command in this group, it works normally, but when you use ja to reply to a message in the group, an error message is reported.

from python-telegram-bot.

Can you see the picture? If you use the command like the picture, you will get an error. like this

from python-telegram-bot.

from python-telegram-bot.

Thanks for elaborating (though in the future, please send 1 comment instead of several and paste monospaced text instead of posting screenshots 😉 )

I think I found the issued

Message.message_thread_id can signify either the id of a forum topic. Or it can be the message_id of a message this message is a reply to. We have no way of knownig which one it is.
However, the parameter message_thread_id of send_message accepts only ids of forum topics. Passing a message id is not allowed (instead reply_parameters must be used).

My first idea was that we can fix that, by checking if Message.message_thread_id == Message.reply_to_message.message_id. however, this doesn't work either because:

Message 1

Message 2

Message 3

Here, Message3 will have message_thread_id==1 even though reply_to_message.message_id==2. I.e., 3 is a reply to 2, but since 2 is a reply to 1, the thread is the one of message 1.

TBH at first glance, I don't think how we can make this work reliably as long as we don't know if message_thread_id originates from a forum topic or from a message_id.

As a side note: if we get this to work, we should make reply_*.param.message_thread_id default to DEFAULT_NONE such that passing None actually removes the parameter.

from python-telegram-bot.

Poolitzer thankfully pointed me to Message.is_topic_message which is true exactly in the cases where message_thread_id is the id of a forum topic :) that way we can make it work. Have implemented most of the changes already and will open a PR shortly

from python-telegram-bot.