Prevented modifications to the request object's original data (#454)

fixes #357

AUTHORS.rst

@@ -15,6 +15,7 @@ The following wonderful people contributed directly or indirectly to this projec
 - `ErgoZ Riftbit Vaper <https://github.com/ergoz>`_
 - `Eugene Lisitsky <https://github.com/lisitsky>`_
 - `franciscod <https://github.com/franciscod>`_
+- `Hugo Damer <https://github.com/HakimusGIT>`_
 - `Jacob Bom <https://github.com/bomjacob>`_
 - `JASON0916 <https://github.com/JASON0916>`_
 - `jh0ker <https://github.com/jh0ker>`_

telegram/animation.py

@@ -55,6 +55,8 @@ class Animation(TelegramObject):
         if not data:
             return None
 
+        data = super(Animation, Animation).de_json(data, bot)
+
         data['thumb'] = PhotoSize.de_json(data.get('thumb'), bot)
 
         return Animation(**data)

telegram/callbackquery.py

@@ -60,6 +60,8 @@ class CallbackQuery(TelegramObject):
         if not data:
             return None
 
+        data = super(CallbackQuery, CallbackQuery).de_json(data, bot)
+
         data['from_user'] = User.de_json(data.get('from'), bot)
         data['message'] = Message.de_json(data.get('message'), bot)
 

telegram/chatmember.py

@@ -59,6 +59,8 @@ class ChatMember(TelegramObject):
         if not data:
             return None
 
+        data = super(ChatMember, ChatMember).de_json(data, bot)
+
         data['user'] = User.de_json(data.get('user'), bot)
 
         return ChatMember(**data)

telegram/choseninlineresult.py

@@ -74,7 +74,8 @@ class ChosenInlineResult(TelegramObject):
         if not data:
             return None
 
-# Required
+        data = super(ChosenInlineResult, ChosenInlineResult).de_json(data, bot)
+        # Required
         data['from_user'] = User.de_json(data.pop('from'), bot)
         # Optionals
         data['location'] = Location.de_json(data.get('location'), bot)

telegram/document.py

@@ -63,6 +63,8 @@ class Document(TelegramObject):
         if not data:
             return None
 
+        data = super(Document, Document).de_json(data, bot)
+
         data['thumb'] = PhotoSize.de_json(data.get('thumb'), bot)
 
         return Document(**data)

telegram/game.py

@@ -73,6 +73,8 @@ class Game(TelegramObject):
         if not data:
             return None
 
+        data = super(Game, Game).de_json(data, bot)
+
         data['photo'] = PhotoSize.de_list(data.get('photo'), bot)
         data['text_entities'] = MessageEntity.de_list(data.get('text_entities'), bot)
         data['animation'] = Animation.de_json(data.get('animation'), bot)

telegram/gamehighscore.py

@@ -49,6 +49,8 @@ class GameHighScore(TelegramObject):
         if not data:
             return None
 
+        data = super(GameHighScore, GameHighScore).de_json(data, bot)
+
         data['user'] = User.de_json(data.get('user'), bot)
 
         return GameHighScore(**data)

telegram/message.py

@@ -200,6 +200,8 @@ class Message(TelegramObject):
         if not data:
             return None
 
+        data = super(Message, Message).de_json(data, bot)
+
         data['from_user'] = User.de_json(data.get('from'), bot)
         data['date'] = datetime.fromtimestamp(data['date'])
         data['chat'] = Chat.de_json(data.get('chat'), bot)

telegram/replykeyboardmarkup.py

@@ -67,6 +67,8 @@ class ReplyKeyboardMarkup(ReplyMarkup):
         if not data:
             return None
 
+        data = super(ReplyKeyboardMarkup, ReplyKeyboardMarkup).de_json(data, bot)
+
         data['keyboard'] = [KeyboardButton.de_list(keyboard, bot) for keyboard in data['keyboard']]
 
         return ReplyKeyboardMarkup(**data)

telegram/sticker.py

@@ -67,6 +67,8 @@ class Sticker(TelegramObject):
         if not data:
             return None
 
+        data = super(Sticker, Sticker).de_json(data, bot)
+
         data['thumb'] = PhotoSize.de_json(data.get('thumb'), bot)
 
         return Sticker(**data)

telegram/update.py

@@ -86,6 +86,8 @@ class Update(TelegramObject):
         if not data:
             return None
 
+        data = super(Update, Update).de_json(data, bot)
+
         data['message'] = Message.de_json(data.get('message'), bot)
         data['edited_message'] = Message.de_json(data.get('edited_message'), bot)
         data['inline_query'] = InlineQuery.de_json(data.get('inline_query'), bot)

telegram/user.py

@@ -77,6 +77,8 @@ class User(TelegramObject):
         if not data:
             return None
 
+        data = super(User, User).de_json(data, bot)
+
         return User(bot=bot, **data)
 
     def get_profile_photos(self, *args, **kwargs):

telegram/userprofilephotos.py

@@ -52,6 +52,8 @@ class UserProfilePhotos(TelegramObject):
         if not data:
             return None
 
+        data = super(UserProfilePhotos, UserProfilePhotos).de_json(data, bot)
+
         data['photos'] = [PhotoSize.de_list(photo, bot) for photo in data['photos']]
 
         return UserProfilePhotos(**data)

telegram/utils/request.py

@@ -31,8 +31,8 @@ import urllib3
 from urllib3.connection import HTTPConnection
 
 from telegram import (InputFile, TelegramError)
-from telegram.error import Unauthorized, InvalidToken, NetworkError, TimedOut, BadRequest, \
-    ChatMigrated, RetryAfter
+from telegram.error import (Unauthorized, NetworkError, TimedOut, BadRequest, ChatMigrated,
+                            RetryAfter, InvalidToken)
 
 logging.getLogger('urllib3').setLevel(logging.WARNING)
 

telegram/video.py

@@ -78,6 +78,8 @@ class Video(TelegramObject):
         if not data:
             return None
 
+        data = super(Video, Video).de_json(data, bot)
+
         data['thumb'] = PhotoSize.de_json(data.get('thumb'), bot)
 
         return Video(**data)

telegram/voice.py

@@ -61,4 +61,6 @@ class Voice(TelegramObject):
         if not data:
             return None
 
+        data = super(Voice, Voice).de_json(data, bot)
+
         return Voice(**data)