python-amazon-mws · GriceTurrble · Feb 25, 2020 · May 27, 2018 · May 27, 2018 · May 27, 2018
diff --git a/docs/source/gettingStarted.rst b/docs/source/gettingStarted.rst
@@ -65,8 +65,7 @@ API Class Names
 ===============
 
 Each MWS API is covered by a separate class, all of which are subclasses
-of the base ``MWS`` class. These APIsA select few APIs are covered by
-classes of slightly different names. See here for all MWS APIs and the
+of the base ``MWS`` class. See here for all MWS APIs and the
 classes that cover them:
 
 +-------------------------------+---------------------------+
@@ -127,7 +126,10 @@ For example, let us check the status of the Orders API:
 *Note*: Technically, ``get_service_status`` calls will pass without
 checking credentials.
 
-The output for all request methods is either a ``DictWrapper`` (for XML
+The output for all request methods is a ``DictWrapper``. The default responses
+you can find in the attribute parsed:
+
+ (for XML
 content) or ``DataWrapper`` (for all others). The majority of our
 responses will be ``DictWrapper``\ s containing a few main attributes:
 
@@ -360,4 +362,4 @@ some patterns useful enough to put into a production setting.
 Be warned that there is no safe testing environment for MWS requests: any valid request
 has the potential to disrupt the production environment of that seller account. Please
 be cautious as to which requests are being sent, scrutinize and clean your input as
-best as possible, and know the potential outcomes for the requests being made.
+best as possible, and know the potential outcomes for the requests being made.
diff --git a/docs/source/reports.rst b/docs/source/reports.rst
@@ -10,13 +10,13 @@ using the python-amazon-mws wrapper.
 
     from mws import mws
 
-    access_key = 'accesskey' #replace with your access key
-    merchant_id = 'merchantid' #replace with your merchant id
-    secret_key = 'secretkey' #replace with your secret key
+    access_key = 'accesskey'  # replace with your access key
+    merchant_id = 'merchantid'  # replace with your merchant id
+    secret_key = 'secretkey'  # replace with your secret key
+
+    reportid = '123456'  # replace with report id
 
-    reportid = '123456' #replace with report id
-
     x = mws.Reports(access_key=access_key, secret_key=secret_key, account_id=merchant_id)
     report = x.get_report(report_id=reportid)
-    response_data = report.original
-    print response_data
+    response_data = report.parsed
+    print(response_data)
diff --git a/mws/apis/inbound_shipments.py b/mws/apis/inbound_shipments.py
@@ -92,7 +92,6 @@ class InboundShipments(MWS):
     """
     URI = "/FulfillmentInboundShipment/2010-10-01"
     VERSION = '2010-10-01'
-    NAMESPACE = '{http://mws.amazonaws.com/FulfillmentInboundShipment/2010-10-01/}'
     NEXT_TOKEN_OPERATIONS = [
         'ListInboundShipments',
         'ListInboundShipmentItems',

diff --git a/mws/apis/inventory.py b/mws/apis/inventory.py
@@ -17,7 +17,6 @@ class Inventory(MWS):
     """
     URI = '/FulfillmentInventory/2010-10-01'
     VERSION = '2010-10-01'
-    NAMESPACE = "{http://mws.amazonaws.com/FulfillmentInventory/2010-10-01}"
     NEXT_TOKEN_OPERATIONS = [
         'ListInventorySupply',
     ]

diff --git a/mws/apis/orders.py b/mws/apis/orders.py
@@ -17,7 +17,6 @@ class Orders(MWS):
     """
     URI = "/Orders/2013-09-01"
     VERSION = "2013-09-01"
-    NAMESPACE = '{https://mws.amazonservices.com/Orders/2013-09-01}'
     NEXT_TOKEN_OPERATIONS = [
         'ListOrders',
         'ListOrderItems',

diff --git a/mws/apis/products.py b/mws/apis/products.py
@@ -17,7 +17,6 @@ class Products(MWS):
     """
     URI = '/Products/2011-10-01'
     VERSION = '2011-10-01'
-    NAMESPACE = '{http://mws.amazonservices.com/schema/Products/2011-10-01}'
     # NEXT_TOKEN_OPERATIONS = []
 
     def list_matching_products(self, marketplace_id, query, context_id=None):

diff --git a/mws/apis/recommendations.py b/mws/apis/recommendations.py
@@ -16,7 +16,6 @@ class Recommendations(MWS):
     """
     URI = '/Recommendations/2013-04-01'
     VERSION = '2013-04-01'
-    NAMESPACE = "{https://mws.amazonservices.com/Recommendations/2013-04-01}"
     NEXT_TOKEN_OPERATIONS = [
         "ListRecommendations",
     ]

diff --git a/mws/apis/reports.py b/mws/apis/reports.py
@@ -169,7 +169,7 @@ def get_report(self, report_id):
             'Action': 'GetReport',
             'ReportId': report_id,
         }
-        return self.make_request(data)
+        return self.make_request(data, rootkey='ignore')
 
     # # TODO Add:
     # def manage_report_schedule(self):

diff --git a/mws/apis/sellers.py b/mws/apis/sellers.py
@@ -16,7 +16,6 @@ class Sellers(MWS):
     """
     URI = '/Sellers/2011-07-01'
     VERSION = '2011-07-01'
-    NAMESPACE = '{http://mws.amazonservices.com/schema/Sellers/2011-07-01}'
     NEXT_TOKEN_OPERATIONS = [
         'ListMarketplaceParticipations',
     ]

diff --git a/mws/apis/subscriptions.py b/mws/apis/subscriptions.py
@@ -5,7 +5,6 @@
 
 from ..mws import MWS
 from .. import utils
-# from .. import utils
 
 # TODO include NotificationType enumeration
 # TODO set up a basic object for Subscription and Destination types?
@@ -20,7 +19,6 @@ class Subscriptions(MWS):
     """
     URI = "/Subscriptions/2013-07-01"
     VERSION = "2013-07-01"
-    NAMESPACE = "{http://mws.amazonaws.com/Subscriptions/2013-07-01}"
 
     # TODO include a helper method for configuring and saving a destination to the object with a keyname
     # This might cut down on some time setting up all the values for the destination for each call,
@@ -47,7 +45,6 @@ def register_destination(self, marketplace_id, attributes=None, delivery_channel
 
         delivery_channel: Currently only supports SQS
         attributes: example {"sqsQueueUrl": "https://sqs.eu-west-2.amazonaws.com/378051892504/Amazon_MWS_Notify"}
-
         """
         if attributes is None:
             raise ValueError("attributes cannot be None")
@@ -57,8 +54,10 @@ def register_destination(self, marketplace_id, attributes=None, delivery_channel
             "MarketplaceId": marketplace_id,
             "Destination.DeliveryChannel": delivery_channel
         }
-        data.update(utils.enumerate_keyed_param("Destination.AttributeList.member", self._parse_attributes(attributes)))
-
+        data.update(utils.enumerate_keyed_param(
+            "Destination.AttributeList.member", 
+            self._parse_attributes(attributes),
+        ))
         return self.make_request(data, "POST")
 
     def deregister_destination(self, marketplace_id, attributes=None, delivery_channel="SQS"):
@@ -70,7 +69,6 @@ def deregister_destination(self, marketplace_id, attributes=None, delivery_chann
 
         delivery_channel: Currently only supports SQS
         attributes: example {"sqsQueueUrl": "https://sqs.eu-west-2.amazonaws.com/378051892504/Amazon_MWS_Notify"}
-
         """
         if attributes is None:
             raise ValueError("attributes cannot be None")
@@ -80,8 +78,10 @@ def deregister_destination(self, marketplace_id, attributes=None, delivery_chann
             "MarketplaceId": marketplace_id,
             "Destination.DeliveryChannel": delivery_channel
         }
-        data.update(utils.enumerate_keyed_param("Destination.AttributeList.member", self._parse_attributes(attributes)))
-
+        data.update(utils.enumerate_keyed_param(
+            "Destination.AttributeList.member",
+            self._parse_attributes(attributes),
+        ))
         return self.make_request(data, "POST")
 
     def list_registered_destinations(self, marketplace_id):
@@ -91,9 +91,10 @@ def list_registered_destinations(self, marketplace_id):
         Docs:
         http://docs.developer.amazonservices.com/en_US/subscriptions/Subscriptions_ListRegisteredDestinations.html
         """
-        data = {"Action": "ListRegisteredDestinations",
-                "MarketplaceId": marketplace_id}
-
+        data = {
+            "Action": "ListRegisteredDestinations",
+            "MarketplaceId": marketplace_id,
+        }
         return self.make_request(data, "POST")
 
     def send_test_notification_to_destination(self, marketplace_id, attributes=None, delivery_channel="SQS"):
@@ -105,16 +106,19 @@ def send_test_notification_to_destination(self, marketplace_id, attributes=None,
 
         delivery_channel: Currently only supports SQS
         attributes: example {"sqsQueueUrl": "https://sqs.eu-west-2.amazonaws.com/378051892504/Amazon_MWS_Notify"}
-
         """
         if attributes is None:
             raise ValueError("attributes cannot be None")
 
-        data = {"Action": "SendTestNotificationToDestination",
-                "MarketplaceId": marketplace_id,
-                "Destination.DeliveryChannel": delivery_channel}
-        data.update(utils.enumerate_keyed_param("Destination.AttributeList.member", self._parse_attributes(attributes)))
-
+        data = {
+            "Action": "SendTestNotificationToDestination",
+            "MarketplaceId": marketplace_id,
+            "Destination.DeliveryChannel": delivery_channel,
+        }
+        data.update(utils.enumerate_keyed_param(
+            "Destination.AttributeList.member",
+            self._parse_attributes(attributes),
+        ))
         return self.make_request(data, method="POST")
 
     def create_subscription(self, marketplace_id, attributes=None,
@@ -127,23 +131,23 @@ def create_subscription(self, marketplace_id, attributes=None,
 
         delivery_channel: Currently only supports SQS
         attributes: example {"sqsQueueUrl": "https://sqs.eu-west-2.amazonaws.com/378051892504/Amazon_MWS_Notify"}
-
         """
-
         if attributes is None:
             raise ValueError("attributes cannot be None")
 
         if notification_type is None:
             raise ValueError("notification_type cannot be None")
-        data = {"Action": "CreateSubscription",
-                "MarketplaceId": marketplace_id,
-                "Subscription.Destination.DeliveryChannel": delivery_channel,
-                "Subscription.IsEnabled": str(is_enabled).lower(),
-                "Subscription.NotificationType": notification_type}
-
-        data.update(utils.enumerate_keyed_param("Subscription.Destination.AttributeList.member",
-                                                self._parse_attributes(attributes)))
-
+        data = {
+            "Action": "CreateSubscription",
+            "MarketplaceId": marketplace_id,
+            "Subscription.Destination.DeliveryChannel": delivery_channel,
+            "Subscription.IsEnabled": str(is_enabled).lower(),
+            "Subscription.NotificationType": notification_type,
+        }
+        data.update(utils.enumerate_keyed_param(
+            "Subscription.Destination.AttributeList.member",
+            self._parse_attributes(attributes),
+        ))
         return self.make_request(data, "POST")
 
     def get_subscription(self, marketplace_id, attributes=None, notification_type=None, delivery_channel="SQS"):
@@ -155,25 +159,25 @@ def get_subscription(self, marketplace_id, attributes=None, notification_type=No
 
         delivery_channel: Currently only supports SQS
         attributes: example {"sqsQueueUrl": "https://sqs.eu-west-2.amazonaws.com/378051892504/Amazon_MWS_Notify"}
-
         """
-
         if attributes is None:
             raise ValueError("attributes cannot be None")
 
         if notification_type is None:
             raise ValueError("notification_type cannot be None")
-        data = {"Action": "GetSubscription",
-                "MarketplaceId": marketplace_id,
-                "Destination.DeliveryChannel": delivery_channel,
-                "NotificationType": notification_type}
-        data.update(utils.enumerate_keyed_param("Destination.AttributeList.member",
-                                                self._parse_attributes(attributes)))
-
+        data = {
+            "Action": "GetSubscription",
+            "MarketplaceId": marketplace_id,
+            "Destination.DeliveryChannel": delivery_channel,
+            "NotificationType": notification_type,
+        }
+        data.update(utils.enumerate_keyed_param(
+            "Destination.AttributeList.member",
+            self._parse_attributes(attributes),
+        ))
         return self.make_request(data, "POST")
 
     def delete_subscription(self, marketplace_id, attributes=None, notification_type=None, delivery_channel="SQS"):
-
         """
         Deletes the subscription for the specified notification type and destination.
 
@@ -182,21 +186,22 @@ def delete_subscription(self, marketplace_id, attributes=None, notification_type
 
         delivery_channel: Currently only supports SQS
         attributes: example {"sqsQueueUrl": "https://sqs.eu-west-2.amazonaws.com/378051892504/Amazon_MWS_Notify"}
-
         """
-
         if attributes is None:
             raise ValueError("attributes cannot be None")
 
         if notification_type is None:
             raise ValueError("notification_type cannot be None")
-        data = {"Action": "DeleteSubscription",
-                "MarketplaceId": marketplace_id,
-                "Destination.DeliveryChannel": delivery_channel,
-                    "NotificationType": notification_type}
-
-        data.update(utils.enumerate_keyed_param("Destination.AttributeList.member", self._parse_attributes(attributes)))
-
+        data = {
+            "Action": "DeleteSubscription",
+            "MarketplaceId": marketplace_id,
+            "Destination.DeliveryChannel": delivery_channel,
+            "NotificationType": notification_type,
+        }
+        data.update(utils.enumerate_keyed_param(
+            "Destination.AttributeList.member",
+            self._parse_attributes(attributes),
+        ))
         return self.make_request(data, "POST")
 
     def list_subscriptions(self, marketplace_id):
@@ -210,7 +215,6 @@ def list_subscriptions(self, marketplace_id):
             "Action": "ListSubscriptions",
             "MarketplaceId": marketplace_id,
         }
-
         return self.make_request(data, "POST")
 
     def update_subscription(self, marketplace_id, attributes=None,
@@ -230,12 +234,15 @@ def update_subscription(self, marketplace_id, attributes=None,
 
         if notification_type is None:
             raise ValueError("notification_type cannot be None")
-        data = {"Action": "UpdateSubscription",
-                "MarketplaceId": marketplace_id,
-                "Subscription.Destination.DeliveryChannel": delivery_channel,
-                "Subscription.IsEnabled": str(is_enabled).lower(),
-                "Subscription.NotificationType": notification_type}
-
-        data.update(utils.enumerate_keyed_param("Subscription.Destination.AttributeList.member",
-                                                self._parse_attributes(attributes)))
+        data = {
+            "Action": "UpdateSubscription",
+            "MarketplaceId": marketplace_id,
+            "Subscription.Destination.DeliveryChannel": delivery_channe
56BA
l,
+            "Subscription.IsEnabled": str(is_enabled).lower(),
+            "Subscription.NotificationType": notification_type,
+        }
+        data.update(utils.enumerate_keyed_param(
+            "Subscription.Destination.AttributeList.member",
+            self._parse_attributes(attributes),
+        ))
         return self.make_request(data, "POST")