mir/pyrogram
2
0
Fork 0
mirror of https://github.com/pyrogram/pyrogram synced 2025-08-23 10:28:00 +00:00
Code Issues Releases Wiki Activity

Make delete_messages return False when it fails to delete

Browse Source 
This is because there will be no exception raised, because Telegram
is not sending any RPCError when you try to delete a message you don't
have rights on to.
This commit is contained in:
Dan 2019-04-14 22:34:05 +02:00
parent 605b5f1b0f
commit 57be97566d
2 changed files with 8 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
pyrogram/client/methods/messages/delete_messages.py
View File

@ -48,7 +48,7 @@ class DeleteMessages(BaseClient):
Defaults to True.
Returns:
True on success.
True on success, False otherwise.
Raises:
:class:`RPCError <pyrogram.RPCError>` in case of a Telegram RPC error.
@ -57,18 +57,20 @@ class DeleteMessages(BaseClient):
message_ids = list(message_ids) if not isinstance(message_ids, int) else [message_ids]
if isinstance(peer, types.InputPeerChannel):
self.send(
r = self.send(
functions.channels.DeleteMessages(
channel=peer,
id=message_ids
)
)
else:
self.send(
r = self.send(
functions.messages.DeleteMessages(
id=message_ids,
revoke=revoke or None
)
)
return True
# Deleting messages you don't have right onto, won't raise any error.
# Check for pts_count, which is 0 in case deletes fail.
return bool(r.pts_count)

6
pyrogram/client/types/messages_and_media/message.py
View File

@ -2729,19 +2729,17 @@ class Message(PyrogramType, Update):
Defaults to True.
Returns:
True on success.
True on success, False otherwise.
Raises:
:class:`RPCError <pyrogram.RPCError>`
"""
self._client.delete_messages(
return self._client.delete_messages(
chat_id=self.chat.id,
message_ids=self.message_id,
revoke=revoke
)
return True
def click(self, x: int or str, y: int = None, quote: bool = None):
"""Bound method *click* of :obj:`Message <pyrogram.Message>`.