From df63b3c76d4b3de0dede51f90c864c05387d6a7f Mon Sep 17 00:00:00 2001 From: Asad Manzoor Date: Sat, 29 May 2021 12:19:04 +0500 Subject: [PATCH] Load payment processors config from json string instead of dict (#81) --- ecommerce/extensions/edly_ecommerce_app/api/v1/constants.py | 2 +- .../extensions/edly_ecommerce_app/api/v1/tests/test_views.py | 2 +- ecommerce/extensions/edly_ecommerce_app/helpers.py | 3 ++- ecommerce/extensions/edly_ecommerce_app/tests/test_helpers.py | 2 +- 4 files changed, 5 insertions(+), 4 deletions(-) diff --git a/ecommerce/extensions/edly_ecommerce_app/api/v1/constants.py b/ecommerce/extensions/edly_ecommerce_app/api/v1/constants.py index e02a0c8befd..12663c7b316 100644 --- a/ecommerce/extensions/edly_ecommerce_app/api/v1/constants.py +++ b/ecommerce/extensions/edly_ecommerce_app/api/v1/constants.py @@ -25,7 +25,7 @@ 'panel_notification_base_url', 'contact_mailing_address', 'theme_dir_name', - 'oauth_clients' + 'oauth2_clients' ] EDLY_PANEL_WORKER_USER = 'edly_panel_worker' diff --git a/ecommerce/extensions/edly_ecommerce_app/api/v1/tests/test_views.py b/ecommerce/extensions/edly_ecommerce_app/api/v1/tests/test_views.py index c7584b60bfd..f04f494c107 100644 --- a/ecommerce/extensions/edly_ecommerce_app/api/v1/tests/test_views.py +++ b/ecommerce/extensions/edly_ecommerce_app/api/v1/tests/test_views.py @@ -132,7 +132,7 @@ def setUp(self): colors=dict(primary='#00000'), platform_name='Edly', theme_dir_name='st-lutherx-ecommerce', - oauth_clients={ + oauth2_clients={ 'payments-sso': { 'id': 'payments-sso-id', 'secret': 'payments-sso-secret', diff --git a/ecommerce/extensions/edly_ecommerce_app/helpers.py b/ecommerce/extensions/edly_ecommerce_app/helpers.py index de3cb89ebeb..f97a3d02d6f 100644 --- a/ecommerce/extensions/edly_ecommerce_app/helpers.py +++ b/ecommerce/extensions/edly_ecommerce_app/helpers.py @@ -1,6 +1,7 @@ from django.conf import settings import jwt +import json from django.core.exceptions import ValidationError from django.utils.translation import ugettext_lazy as _ @@ -226,6 +227,6 @@ def get_payment_processors_names(request_data): Returns: (str): Payment Processors Comma-separated List """ - payment_processors = request_data.get('payment_processor_config', {}) + payment_processors = json.loads(request_data.get('payment_processor_config', {})) edly_slug = request_data.get('edly_slug', '') return ','.join(payment_processors.get(edly_slug, {}).keys()) diff --git a/ecommerce/extensions/edly_ecommerce_app/tests/test_helpers.py b/ecommerce/extensions/edly_ecommerce_app/tests/test_helpers.py index 18c248993b6..a59ab3b35e4 100644 --- a/ecommerce/extensions/edly_ecommerce_app/tests/test_helpers.py +++ b/ecommerce/extensions/edly_ecommerce_app/tests/test_helpers.py @@ -51,7 +51,7 @@ def setUp(self): colors=dict(primary='#00000'), platform_name='Edly', theme_dir_name='st-lutherx-ecommerce', - oauth_clients={ + oauth2_clients={ 'payments-sso': { 'id': 'payments-sso-id', 'secret': 'payments-sso-secret',