Improve options parameter handling

Author: Manuel Huez

processout/activity.py

@@ -127,11 +127,13 @@ def fillWithData(self, data):
 
         return self
 
-    def all(self, options = None):
+    def all(self, options = {}):
         """Get all the project activities.
         Keyword argument:
         
         options -- Options for the request"""
+        self.fillWithData(options)
+
         request = Request(self._client)
         path    = "/activities"
         data    = {
@@ -152,13 +154,15 @@ def all(self, options = None):
 
 
 
-        return returnValues[0];
+        return returnValues[0]
 
-    def find(self, activityId, options = None):
+    def find(self, activityId, options = {}):
         """Find a specific activity and fetch its data.
         Keyword argument:
         activityId -- ID of the activity
         options -- Options for the request"""
+        self.fillWithData(options)
+
         request = Request(self._client)
         path    = "/activities/" + quote_plus(activityId) + ""
         data    = {
@@ -177,6 +181,6 @@ def find(self, activityId, options = None):
 
 
 
-        return returnValues[0];
+        return returnValues[0]
 
 

processout/authorizationrequest.py

@@ -233,11 +233,13 @@ def fillWithData(self, data):
 
         return self
 
-    def fetchCustomer(self, options = None):
+    def fetchCustomer(self, options = {}):
         """Get the customer linked to the authorization request.
         Keyword argument:
         
         options -- Options for the request"""
+        self.fillWithData(options)
+
         request = Request(self._client)
         path    = "/authorization-requests/" + quote_plus(self.id) + "/customers"
         data    = {
@@ -253,13 +255,15 @@ def fetchCustomer(self, options = None):
         returnValues.append(customer.fillWithData(body))
 
 
-        return returnValues[0];
+        return returnValues[0]
 
-    def create(self, customerId, options = None):
+    def create(self, customerId, options = {}):
         """Create a new authorization request for the given customer ID.
         Keyword argument:
         customerId -- ID of the customer
         options -- Options for the request"""
+        self.fillWithData(options)
+
         request = Request(self._client)
         path    = "/authorization-requests"
         data    = {
@@ -281,13 +285,15 @@ def create(self, customerId, options = None):
 
 
 
-        return returnValues[0];
+        return returnValues[0]
 
-    def find(self, authorizationRequestId, options = None):
+    def find(self, authorizationRequestId, options = {}):
         """Find an authorization request by its ID.
         Keyword argument:
         authorizationRequestId -- ID of the authorization request
         options -- Options for the request"""
+        self.fillWithData(options)
+
         request = Request(self._client)
         path    = "/authorization-requests/" + quote_plus(authorizationRequestId) + ""
         data    = {
@@ -306,6 +312,6 @@ def find(self, authorizationRequestId, options = None):
 
 
 
-        return returnValues[0];
+        return returnValues[0]
 
 

processout/coupon.py

@@ -239,11 +239,13 @@ def fillWithData(self, data):
 
         return self
 
-    def all(self, options = None):
+    def all(self, options = {}):
         """Get all the coupons.
         Keyword argument:
         
         options -- Options for the request"""
+        self.fillWithData(options)
+
         request = Request(self._client)
         path    = "/coupons"
         data    = {
@@ -264,13 +266,15 @@ def all(self, options = None):
 
 
 
-        return returnValues[0];
+        return returnValues[0]
 
-    def create(self, options = None):
+    def create(self, options = {}):
         """Create a new coupon.
         Keyword argument:
         
         options -- Options for the request"""
+        self.fillWithData(options)
+
         request = Request(self._client)
         path    = "/coupons"
         data    = {
@@ -295,13 +299,15 @@ def create(self, options = None):
 
 
 
-        return returnValues[0];
+        return returnValues[0]
 
-    def find(self, couponId, options = None):
+    def find(self, couponId, options = {}):
         """Find a coupon by its ID.
         Keyword argument:
         couponId -- ID of the coupon
         options -- Options for the request"""
+        self.fillWithData(options)
+
         request = Request(self._client)
         path    = "/coupons/" + quote_plus(couponId) + ""
         data    = {
@@ -320,13 +326,15 @@ def find(self, couponId, options = None):
 
 
 
-        return returnValues[0];
+        return returnValues[0]
 
-    def save(self, options = None):
+    def save(self, options = {}):
         """Save the updated coupon attributes.
         Keyword argument:
         
         options -- Options for the request"""
+        self.fillWithData(options)
+
         request = Request(self._client)
         path    = "/coupons/" + quote_plus(self.id) + ""
         data    = {
@@ -344,13 +352,15 @@ def save(self, options = None):
 
 
 
-        return returnValues[0];
+        return returnValues[0]
 
-    def delete(self, options = None):
+    def delete(self, options = {}):
         """Delete the coupon.
         Keyword argument:
         
         options -- Options for the request"""
+        self.fillWithData(options)
+
         request = Request(self._client)
         path    = "/coupons/" + quote_plus(self.id) + ""
         data    = {
@@ -363,6 +373,6 @@ def delete(self, options = None):
         returnValues.append(response.success)
 
 
-        return returnValues[0];
+        return returnValues[0]