Fix inline keyboard buttons with empty values

delivrance · delivrance · commit 1d55eaa1ba6e · 2021-03-21T21:52:37.000+01:00
diff --git a/pyrogram/types/bots_and_keyboards/inline_keyboard_button.py b/pyrogram/types/bots_and_keyboards/inline_keyboard_button.py
@@ -129,7 +129,7 @@ def read(b: "raw.base.KeyboardButton"):
             )
 
     async def write(self, client: "pyrogram.Client"):
-        if self.callback_data:
+        if self.callback_data is not None:
             # Telegram only wants bytes, but we are allowed to pass strings too, for convenience.
             data = bytes(self.callback_data, "utf-8") if isinstance(self.callback_data, str) else self.callback_data
 
@@ -138,32 +138,32 @@ async def write(self, client: "pyrogram.Client"):
                 data=data
             )
 
-        if self.url:
+        if self.url is not None:
             return raw.types.KeyboardButtonUrl(
                 text=self.text,
                 url=self.url
             )
 
-        if self.login_url:
+        if self.login_url is not None:
             return self.login_url.write(
                 text=self.text,
                 bot=await client.resolve_peer(self.login_url.bot_username)
             )
 
-        if self.switch_inline_query:
+        if self.switch_inline_query is not None:
             return raw.types.KeyboardButtonSwitchInline(
                 text=self.text,
                 query=self.switch_inline_query
             )
 
-        if self.switch_inline_query_current_chat:
+        if self.switch_inline_query_current_chat is not None:
             return raw.types.KeyboardButtonSwitchInline(
                 text=self.text,
                 query=self.switch_inline_query_current_chat,
                 same_peer=True
             )
 
-        if self.callback_game:
+        if self.callback_game is not None:
             return raw.types.KeyboardButtonGame(
                 text=self.text
             )

Original file line number	Diff line number	Diff line change
`@@ -129,7 +129,7 @@ def read(b: "raw.base.KeyboardButton"):`
`129`	`129`	`)`
`130`	`130`
`131`	`131`	`async def write(self, client: "pyrogram.Client"):`
`132`		`- if self.callback_data:`
	`132`	`+ if self.callback_data is not None:`
`133`	`133`	`# Telegram only wants bytes, but we are allowed to pass strings too, for convenience.`
`134`	`134`	`data = bytes(self.callback_data, "utf-8") if isinstance(self.callback_data, str) else self.callback_data`
`135`	`135`
`@@ -138,32 +138,32 @@ async def write(self, client: "pyrogram.Client"):`
`138`	`138`	`data=data`
`139`	`139`	`)`
`140`	`140`
`141`		`- if self.url:`
	`141`	`+ if self.url is not None:`
`142`	`142`	`return raw.types.KeyboardButtonUrl(`
`143`	`143`	`text=self.text,`
`144`	`144`	`url=self.url`
`145`	`145`	`)`
`146`	`146`
`147`		`- if self.login_url:`
	`147`	`+ if self.login_url is not None:`
`148`	`148`	`return self.login_url.write(`
`149`	`149`	`text=self.text,`
`150`	`150`	`bot=await client.resolve_peer(self.login_url.bot_username)`
`151`	`151`	`)`
`152`	`152`
`153`		`- if self.switch_inline_query:`
	`153`	`+ if self.switch_inline_query is not None:`
`154`	`154`	`return raw.types.KeyboardButtonSwitchInline(`
`155`	`155`	`text=self.text,`
`156`	`156`	`query=self.switch_inline_query`
`157`	`157`	`)`
`158`	`158`
`159`		`- if self.switch_inline_query_current_chat:`
	`159`	`+ if self.switch_inline_query_current_chat is not None:`
`160`	`160`	`return raw.types.KeyboardButtonSwitchInline(`
`161`	`161`	`text=self.text,`
`162`	`162`	`query=self.switch_inline_query_current_chat,`
`163`	`163`	`same_peer=True`
`164`	`164`	`)`
`165`	`165`
`166`		`- if self.callback_game:`
	`166`	`+ if self.callback_game is not None:`
`167`	`167`	`return raw.types.KeyboardButtonGame(`
`168`	`168`	`text=self.text`
`169`	`169`	`)`