Add selector function for json_decode.

srmklive · Sep 30, 2023 · 6f66e66 · 6f66e66
1 parent 27e8f80
commit 6f66e66
Show file tree

Hide file tree

Showing 68 changed files with 237 additions and 214 deletions.
diff --git a/src/Services/Str.php b/src/Services/Str.php
@@ -3,9 +3,12 @@
 namespace Srmklive\PayPal\Services;
 
 use GuzzleHttp\Utils;
+use Srmklive\PayPal\Traits\JsonDecodeSelector;
 
 class Str extends \Illuminate\Support\Str
 {
+    use JsonDecodeSelector;
+
     /**
      * Determine if a given value is valid JSON.
      *
@@ -24,8 +27,8 @@ public static function isJson($value): bool
         }
 
         try {
-            Utils::jsonDecode($value, true, 512, 4194304);
-        } catch (\JsonException $jsonException) {
+            (new Str)->jsonDecodeFunction()($value, true, 512, 4194304);
+        } catch (\Exception $jsonException) {
             return false;
         }
 

diff --git a/src/Traits/JsonDecodeSelector.php b/src/Traits/JsonDecodeSelector.php
@@ -0,0 +1,16 @@
+<?php
+
+namespace Srmklive\PayPal\Traits;
+
+trait JsonDecodeSelector
+{
+    /**
+     * Decide the function to use for decoding json.
+     *
+     * @return string
+     */
+    protected function jsonDecodeFunction(): string
+    {
+        return class_exists(\GuzzleHttp\Utils::class) ? '\GuzzleHttp\Utils::jsonDecode' : '\GuzzleHttp\Utils::json_decode';
+    }
+}
diff --git a/src/Traits/PayPalHttpClient.php b/src/Traits/PayPalHttpClient.php
@@ -210,9 +210,9 @@ private function doPayPalRequest(bool $decode = true)
             // Perform PayPal HTTP API request.
             $response = $this->makeHttpRequest();
 
-            return ($decode === false) ? $response->getContents() : Utils::jsonDecode($response, true);
+            return ($decode === false) ? $response->getContents() : $this->jsonDecodeFunction()($response, true);
         } catch (RuntimeException $t) {
-            $error = ($decode === false) || (Str::isJson($t->getMessage()) === false) ? $t->getMessage() : Utils::jsonDecode($t->getMessage(), true);
+            $error = ($decode === false) || (Str::isJson($t->getMessage()) === false) ? $t->getMessage() : $this->jsonDecodeFunction()($t->getMessage(), true);
 
             return ['error' => $error];
         }

diff --git a/src/Traits/PayPalRequest.php b/src/Traits/PayPalRequest.php
@@ -6,6 +6,7 @@
 
 trait PayPalRequest
 {
+    use JsonDecodeSelector;
     use PayPalHttpClient;
     use PayPalAPI;
     use PayPalExperienceContext;

diff --git a/tests/MockClientClasses.php b/tests/MockClientClasses.php
@@ -10,9 +10,12 @@
 use GuzzleHttp\Utils;
 use Psr\Http\Message\ResponseInterface;
 use Srmklive\PayPal\Services\PayPal as PayPalClient;
+use Srmklive\PayPal\Traits\JsonDecodeSelector;
 
 trait MockClientClasses
 {
+    use JsonDecodeSelector;
+
     private function mock_http_client($response): HttpClient
     {
         $mock = new HttpMockHandler([

diff --git a/tests/Mocks/Requests/BillingPlans.php b/tests/Mocks/Requests/BillingPlans.php
@@ -11,7 +11,7 @@ trait BillingPlans
      */
     private function createPlanParams(): array
     {
-        return Utils::jsonDecode('{
+        return $this->jsonDecodeFunction()('{
   "product_id": "PROD-XXCD1234QWER65782",
   "name": "Video Streaming Service Plan",
   "description": "Video Streaming Service basic plan",
@@ -84,7 +84,7 @@ private function createPlanParams(): array
      */
     private function updatePlanParams(): array
     {
-        return Utils::jsonDecode('[
+        return $this->jsonDecodeFunction()('[
   {
     "op": "replace",
     "path": "/payment_preferences/payment_failure_threshold",
@@ -98,7 +98,7 @@ private function updatePlanParams(): array
      */
     private function updatePlanPricingParams(): array
     {
-        return Utils::jsonDecode('{
+        return $this->jsonDecodeFunction()('{
   "pricing_schemes": [
     {
       "billing_cycle_sequence": 2,

diff --git a/tests/Mocks/Requests/CatalogProducts.php b/tests/Mocks/Requests/CatalogProducts.php
@@ -11,7 +11,7 @@ trait CatalogProducts
      */
     private function createProductParams(): array
     {
-        return Utils::jsonDecode('{
+        return $this->jsonDecodeFunction()('{
           "name": "Video Streaming Service",
           "description": "Video streaming service",
           "type": "SERVICE",
@@ -26,7 +26,7 @@ private function createProductParams(): array
      */
     private function updateProductParams(): array
     {
-        return Utils::jsonDecode('[
+        return $this->jsonDecodeFunction()('[
           {
             "op": "replace",
             "path": "/description",

diff --git a/tests/Mocks/Requests/Disputes.php b/tests/Mocks/Requests/Disputes.php
@@ -11,7 +11,7 @@ trait Disputes
      */
     protected function updateDisputeParams(): array
     {
-        return Utils::jsonDecode('[
+        return $this->jsonDecodeFunction()('[
   {
     "op": "add",
     "path": "/partner_actions/-",

diff --git a/tests/Mocks/Requests/DisputesActions.php b/tests/Mocks/Requests/DisputesActions.php
@@ -11,7 +11,7 @@ trait DisputesActions
      */
     protected function acceptDisputeClaimParams(): array
     {
-        return Utils::jsonDecode('{
+        return $this->jsonDecodeFunction()('{
   "note": "Full refund to the customer.",
   "accept_claim_type": "REFUND"
 }', true);
@@ -22,7 +22,7 @@ protected function acceptDisputeClaimParams(): array
      */
     protected function acceptDisputeResolutionParams(): array
     {
-        return Utils::jsonDecode('{
+        return $this->jsonDecodeFunction()('{
   "note": "I am ok with the refund offered."
 }', true);
     }
@@ -32,7 +32,7 @@ protected function acceptDisputeResolutionParams(): array
      */
     protected function acknowledgeItemReturnedParams(): array
     {
-        return Utils::jsonDecode('{
+        return $this->jsonDecodeFunction()('{
   "note": "I have received the item back.",
   "acknowledgement_type": "ITEM_RECEIVED"
 }', true);

diff --git a/tests/Mocks/Requests/Identity.php b/tests/Mocks/Requests/Identity.php
@@ -8,7 +8,7 @@ trait Identity
 {
     private function mockCreateMerchantApplicationParams(): array
     {
-        return Utils::jsonDecode('{
+        return $this->jsonDecodeFunction()('{
   "redirect_uris": [
     "https://example.com/callback",
     "https://example.com/callback2"
@@ -29,7 +29,7 @@ private function mockCreateMerchantApplicationParams(): array
 
     private function mockSetAccountPropertiesParams(): array
     {
-        return Utils::jsonDecode('{
+        return $this->jsonDecodeFunction()('{
     "categories": [
       {
         "name": "PAYMENT",

diff --git a/tests/Mocks/Requests/Invoices.php b/tests/Mocks/Requests/Invoices.php
@@ -11,7 +11,7 @@ trait Invoices
      */
     private function createInvoiceParams(): array
     {
-        return Utils::jsonDecode('{
+        return $this->jsonDecodeFunction()('{
   "detail": {
     "invoice_number": "#123",
     "reference": "deal-ref",
@@ -175,7 +175,7 @@ private function createInvoiceParams(): array
      */
     private function updateInvoiceParams(): array
     {
-        return Utils::jsonDecode('{
+        return $this->jsonDecodeFunction()('{
   "id": "INV2-C82X-JNN9-Y6S5-CNXW",
   "status": "DRAFT",
   "detail": {
@@ -375,7 +375,7 @@ private function updateInvoiceParams(): array
      */
     private function cancelInvoiceParams(): array
     {
-        return Utils::jsonDecode('{
+        return $this->jsonDecodeFunction()('{
   "subject": "Invoice Cancelled",
   "note": "Cancelling the invoice",
   "send_to_invoicer": true,
@@ -391,7 +391,7 @@ private function cancelInvoiceParams(): array
      */
     private function generateQRCodeInvoiceParams(): array
     {
-        return Utils::jsonDecode('{
+        return $this->jsonDecodeFunction()('{
   "width": 400,
   "height": 400
 }', true);
@@ -402,7 +402,7 @@ private function generateQRCodeInvoiceParams(): array
      */
     private function registerInvoicePaymentParams(): array
     {
-        return Utils::jsonDecode('{
+        return $this->jsonDecodeFunction()('{
   "width": 400,
   "height": 400
 }', true);
@@ -413,7 +413,7 @@ private function registerInvoicePaymentParams(): array
      */
     private function refundInvoicePaymentParams(): array
     {
-        return Utils::jsonDecode('{
+        return $this->jsonDecodeFunction()('{
   "method": "BANK_TRANSFER",
   "refund_date": "2018-05-21",
   "amount": {
@@ -428,7 +428,7 @@ private function refundInvoicePaymentParams(): array
      */
     private function sendInvoiceParams(): array
     {
-        return Utils::jsonDecode('{
+        return $this->jsonDecodeFunction()('{
   "subject": "Payment due for the invoice #ABC-123",
   "note": "Please pay before the due date to avoid incurring late payment charges which will be adjusted in the next bill generated.",
   "send_to_invoicer": true,
@@ -444,7 +444,7 @@ private function sendInvoiceParams(): array
      */
     private function sendInvoiceReminderParams(): array
     {
-        return Utils::jsonDecode('{
+        return $this->jsonDecodeFunction()('{
   "subject": "Reminder: Payment due for the invoice #ABC-123",
   "note": "Please pay before the due date to avoid incurring late payment charges which will be adjusted in the next bill generated.",
   "send_to_invoicer": true,

diff --git a/tests/Mocks/Requests/InvoicesSearch.php b/tests/Mocks/Requests/InvoicesSearch.php
@@ -11,7 +11,7 @@ trait InvoicesSearch
      */
     private function invoiceSearchParams(): array
     {
-        return Utils::jsonDecode('{
+        return $this->jsonDecodeFunction()('{
             "total_amount_range": {
                 "lower_amount": {
                     "currency_code": "USD",

diff --git a/tests/Mocks/Requests/InvoicesTemplates.php b/tests/Mocks/Requests/InvoicesTemplates.php
@@ -11,7 +11,7 @@ trait InvoicesTemplates
      */
     private function mockCreateInvoiceTemplateParams(): array
     {
-        return Utils::jsonDecode('{
+        return $this->jsonDecodeFunction()('{
   "default_template": true,
   "template_info": {
     "configuration": {
@@ -241,7 +241,7 @@ private function mockCreateInvoiceTemplateParams(): array
      */
     private function mockUpdateInvoiceTemplateParams(): array
     {
-        return Utils::jsonDecode('{
+        return $this->jsonDecodeFunction()('{
   "default_template": true,
   "template_info": {
     "configuration": {

diff --git a/tests/Mocks/Requests/Orders.php b/tests/Mocks/Requests/Orders.php
@@ -11,7 +11,7 @@ trait Orders
     */
     private function createOrderParams(): array
     {
-        return Utils::jsonDecode('{
+        return $this->jsonDecodeFunction()('{
             "intent": "CAPTURE",
             "purchase_units": [
               {
@@ -29,7 +29,7 @@ private function createOrderParams(): array
     */
     private function updateOrderParams(): array
     {
-        return Utils::jsonDecode('[
+        return $this->jsonDecodeFunction()('[
         {
           "op": "replace",
           "path": "/purchase_units/@reference_id==\'PUHF\'/shipping/address",

diff --git a/tests/Mocks/Requests/PartnerReferrals.php b/tests/Mocks/Requests/PartnerReferrals.php
@@ -8,7 +8,7 @@ trait PartnerReferrals
 {
     private function mockCreatePartnerReferralParams(): array
     {
-        return Utils::jsonDecode('{
+        return $this->jsonDecodeFunction()('{
   "individual_owners": [
     {
       "names": [

diff --git a/tests/Mocks/Requests/PaymentAuthorizations.php b/tests/Mocks/Requests/PaymentAuthorizations.php
@@ -11,7 +11,7 @@ trait PaymentAuthorizations
      */
     private function mockCaptureAuthorizedPaymentParams(): array
     {
-        return Utils::jsonDecode('{
+        return $this->jsonDecodeFunction()('{
   "amount": {
     "value": "10.99",
     "currency_code": "USD"
@@ -27,7 +27,7 @@ private function mockCaptureAuthorizedPaymentParams(): array
      */
     private function mockReAuthorizeAuthorizedPaymentParams(): array
     {
-        return Utils::jsonDecode('{
+        return $this->jsonDecodeFunction()('{
   "amount": {
     "value": "10.99",
     "currency_code": "USD"

diff --git a/tests/Mocks/Requests/PaymentCaptures.php b/tests/Mocks/Requests/PaymentCaptures.php
@@ -11,7 +11,7 @@ trait PaymentCaptures
      */
     private function mockRefundCapturedPaymentParams(): array
     {
-        return Utils::jsonDecode('{
+        return $this->jsonDecodeFunction()('{
   "amount": {
     "value": "10.99",
     "currency_code": "USD"

diff --git a/tests/Mocks/Requests/PaymentExperienceWebProfiles.php b/tests/Mocks/Requests/PaymentExperienceWebProfiles.php
@@ -11,7 +11,7 @@ trait PaymentExperienceWebProfiles
      */
     private function mockCreateWebProfileParams(): array
     {
-        return Utils::jsonDecode('[
+        return $this->jsonDecodeFunction()('[
   {
     "id": "XP-GCUV-X35G-HNEY-5MJY",
     "name": "exampleProfile",
@@ -64,7 +64,7 @@ private function mockCreateWebProfileParams(): array
      */
     private function partiallyUpdateWebProfileParams(): array
     {
-        return Utils::jsonDecode('[
+        return $this->jsonDecodeFunction()('[
   {
     "op": "add",
     "path": "/presentation/brand_name",
@@ -82,7 +82,7 @@ private function partiallyUpdateWebProfileParams(): array
      */
     private function updateWebProfileParams(): array
     {
-        return Utils::jsonDecode('{
+        return $this->jsonDecodeFunction()('{
   "name": "exampleProfile",
   "presentation": {
     "logo_image": "https://example.com/logo_image/"

diff --git a/tests/Mocks/Requests/PaymentMethodsTokens.php b/tests/Mocks/Requests/PaymentMethodsTokens.php
@@ -11,7 +11,7 @@ trait PaymentMethodsTokens
      */
     private function mockCreatePaymentSetupTokensParams(): array
     {
-        return Utils::jsonDecode('{
+        return $this->jsonDecodeFunction()('{
             "payment_source": {
               "card": {
                 "number": "4111111111111111",
@@ -41,7 +41,7 @@ private function mockCreatePaymentSetupTokensParams(): array
      */
     private function mockCreatePaymentSetupPayPalParams(): array
     {
-        return Utils::jsonDecode('{
+        return $this->jsonDecodeFunction()('{
           "payment_source": {
               "paypal": {
                   "description": "Description for PayPal to be shown to PayPal payer",

diff --git a/tests/Mocks/Requests/Payouts.php b/tests/Mocks/Requests/Payouts.php
@@ -11,7 +11,7 @@ trait Payouts
      */
     private function mockCreateBatchPayoutParams(): array
     {
-        return Utils::jsonDecode('{
+        return $this->jsonDecodeFunction()('{
   "sender_batch_header": {
     "sender_batch_id": "Payouts_2018_100007",
     "email_subject": "You have a payout!",

diff --git a/tests/Mocks/Requests/ReferencedPayouts.php b/tests/Mocks/Requests/ReferencedPayouts.php
@@ -11,7 +11,7 @@ trait ReferencedPayouts
      */
     private function mockCreateReferencedBatchPayoutParams(): array
     {
-        return Utils::jsonDecode('{
+        return $this->jsonDecodeFunction()('{
   "referenced_payouts": [
     {
       "reference_id": "2KP03934U4415543C",
@@ -30,7 +30,7 @@ private function mockCreateReferencedBatchPayoutParams(): array
      */
     private function mockCreateReferencedBatchPayoutItemParams(): array
     {
-        return Utils::jsonDecode('{
+        return $this->jsonDecodeFunction()('{
   "reference_id": "CAPTURETXNID",
   "reference_type": "TRANSACTION_ID"
 }', true);