None high level module magic

Author: thaelathy

Adyen/__init__.py

@@ -22,13 +22,6 @@
 
 from .httpclient import HTTPClient
 
-username = None
-password = None
-platform = None
-merchant_account = None
-merchant_specific_url = None
-hmac = None
-
 
 class Adyen(AdyenBase):
     def __init__(self, **kwargs):

Adyen/client.py

@@ -79,6 +79,8 @@ def __init__(self, username=None, password=None,
         self.store_payout_username = store_payout_username
         self.store_payout_password = store_payout_password
         self.platform = platform
+        if not self.platform:
+            self.platform = 'test'
         self.merchant_specific_url = merchant_specific_url
         self.hmac = hmac
         self.merchant_account = merchant_account
@@ -353,17 +355,14 @@ def call_hpp(self, message, action, hmac_key="", **kwargs):
 
     def hpp_payment(self, request_data, action, hmac_key="", **kwargs):
 
-        from . import platform
-
         if not self.http_init:
             self.http_client = HTTPClient(self.app_name,
                                           self.USER_AGENT_SUFFIX,
                                           self.LIB_VERSION,
                                           self.http_force)
             self.http_init = True
 
-        if self.platform:
-            platform = self.platform
+        platform = self.platform
         if platform.lower() not in ['live', 'test']:
             errorstring = " 'platform' must be the value of 'live' or 'test' "
             raise ValueError(errorstring)
@@ -476,8 +475,7 @@ def _handle_http_error(self, url, response_obj, status_code, psp_ref,
         """
 
         if status_code == 404:
-            from . import merchant_specific_url
-            if url == merchant_specific_url:
+            if url == self.merchant_specific_url:
                 erstr = "Received a 404 for url:'{}'. Please ensure that" \
                         " the custom merchant specific url is correct" \
                     .format(url)
@@ -509,7 +507,6 @@ def _handle_http_error(self, url, response_obj, status_code, psp_ref,
                     " if the problem persists"
             raise AdyenAPIAuthenticationError(erstr)
         elif status_code == 403:
-            from . import username
 
             if response_obj.get("message") == "Invalid Merchant Account":
                 erstr = ("You provided the merchant account:'%s' that"
@@ -527,7 +524,7 @@ def _handle_http_error(self, url, response_obj, status_code, psp_ref,
                     " the PSP reference: %s" % (
                         response_obj["message"], self.username, psp_ref)
 
-            raise AdyenAPIInvalidPermission(erstr, username, psp_ref,
+            raise AdyenAPIInvalidPermission(erstr, self.username, psp_ref,
                                             raw_request=raw_request,
                                             raw_response=raw_response, url=url,
                                             psp=psp_ref, headers=headers)

Adyen/services.py

@@ -22,7 +22,7 @@ def __getattr__(self, attr):
 
 
 class AdyenServiceBase(AdyenBase):
-    def __init__(self, client=""):
+    def __init__(self, client=None):
         if client:
             self.client = client
         else:
@@ -41,7 +41,7 @@ class AdyenRecurring(AdyenServiceBase):
             use. If not provided, a new API client will be created.
     """
 
-    def __init__(self, client=""):
+    def __init__(self, client=None):
         super(AdyenRecurring, self).__init__(client=client)
         self.service = "Recurring"