Merge pull request #176 from amadeus4dev/update-typecheck

Replace type with isinstance

Author: minjikarin

amadeus/booking/_flight_orders.py

@@ -14,12 +14,12 @@ def post(self, flight, travelers):
         :raises amadeus.ResponseError: if the request could not be completed
         '''
         flight_offers = []
-        if type(flight) is not list:
+        if not isinstance(flight, list):
             flight_offers.append(flight)
         else:
             flight_offers.extend(flight)
         travelers_info = []
-        if type(travelers) is not list:
+        if not isinstance(travelers, list):
             travelers_info.append(travelers)
         else:
             travelers_info.extend(travelers)

amadeus/booking/_hotel_bookings.py

@@ -19,11 +19,11 @@ def post(self, hotel_offer_id, guests, payments):
         '''
         guests_info = []
         payment_info = []
-        if type(guests) is not list:
+        if not isinstance(guests, list):
             guests_info.append(guests)
         else:
             guests_info.extend(guests)
-        if type(payments) is not list:
+        if not isinstance(payments, list):
             payment_info.append(payments)
         else:
             payment_info.extend(payments)

amadeus/client/request.py

@@ -109,7 +109,7 @@ def __build_http_request(self):
             # Adds HTTP override in Header for the list of paths required
             if self.path in Request.list_httpoverride:
                 self.headers['X-HTTP-Method-Override'] = 'GET'
-            if type(self.params) is dict:
+            if isinstance(self.params, dict):
                 return HTTPRequest(self.url, headers=self.headers, method='POST',
                                    data=json.dumps(self.params).encode())
             else:
@@ -142,12 +142,12 @@ def _urlencode(self, d):
 
     # Flattens the hash keys, so page: { offset: 1 } becomes page[offet] = 1
     def _flatten_keys(self, d, key, out):
-        if type(d) is not dict:
+        if not isinstance(d, dict):
             raise TypeError('Only dicts can be encoded')
 
         for k in d:
             keystr = k if not key else '[{}]'.format(k)
-            if type(d[k]) is dict:
+            if isinstance(d[k], dict):
                 self._flatten_keys(d[k], str(key) + str(keystr), out)
             else:
                 out['{}{}'.format(key, keystr)] = d[k]

amadeus/shopping/flight_offers/_pricing.py

@@ -17,7 +17,7 @@ def post(self, body, **params):
         '''
         url = '/v1/shopping/flight-offers/pricing?'
         flight_offers = []
-        if type(body) is not list:
+        if not isinstance(body, list):
             flight_offers.append(body)
         else:
             flight_offers.extend(body)