From 640df1afc042a452c04a1ca86e26601a6982d5ff Mon Sep 17 00:00:00 2001 From: annatisch Date: Fri, 13 May 2016 15:37:43 -0700 Subject: [PATCH] Review feedback --- ClientRuntimes/Python/msrest/msrest/serialization.py | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/ClientRuntimes/Python/msrest/msrest/serialization.py b/ClientRuntimes/Python/msrest/msrest/serialization.py index c26596c95659d..9dbae75a676db 100644 --- a/ClientRuntimes/Python/msrest/msrest/serialization.py +++ b/ClientRuntimes/Python/msrest/msrest/serialization.py @@ -418,7 +418,7 @@ def serialize_data(self, data, data_type, **kwargs): enum_type = self.dependencies.get(data_type) if enum_type and issubclass(enum_type, Enum): - return self.serialize_enum(data, enum_obj=enum_type, **kwargs) + return Serializer.serialize_enum(data, enum_obj=enum_type) iter_type = data_type[0] + data_type[-1] if iter_type in self.serialize_type: @@ -540,7 +540,8 @@ def serialize_object(self, attr, **kwargs): else: return str(attr) - def serialize_enum(self, attr, enum_obj=None, **kwagrs): + @staticmethod + def serialize_enum(attr, enum_obj=None): try: return attr.value except AttributeError: @@ -728,7 +729,7 @@ def __call__(self, target_obj, response_data): if isinstance(response, basestring): return self.deserialize_data(data, response) - elif issubclass(response, Enum): + elif isinstance(response, type) and issubclass(response, Enum): return self.deserialize_enum(data, response) if data is None: