Merge pull request #99 from tungleduyxyz/issue_89_sync_killbill_payme…

…nt_apis Issue 89 Support external key
killbill · Mar 26, 2024 · 7ce8d28 · 7ce8d28
2 parents 6b1a563 + 2597a09
commit 7ce8d28
Showing 1 changed file with 10 additions and 2 deletions.
diff --git a/lib/killbill_client/models/transaction.rb b/lib/killbill_client/models/transaction.rb
@@ -125,7 +125,11 @@ def void(user = nil, reason = nil, comment = nil, options = {}, refresh_options
                                 :reason  => reason,
                                 :comment => comment,
                             }.merge(options)
-          KillBillClient::Model::Payment.find_by_id(payment_id, false, false, options)
+          if payment_external_key
+            KillBillClient::Model::Payment.find_by_external_key(payment_external_key, false, false, options)
+          else
+            KillBillClient::Model::Payment.find_by_id(payment_id, false, false, options)
+          end
         end
       end
 
@@ -225,7 +229,11 @@ def complete_initial_transaction(user, reason, comment, options, refresh_options
                              :reason => reason,
                              :comment => comment
                          }.merge(options)
-          KillBillClient::Model::Payment.find_by_id(payment_id, false, false, options)
+          if payment_external_key
+            KillBillClient::Model::Payment.find_by_external_key(payment_external_key, false, false, options)
+          else
+            KillBillClient::Model::Payment.find_by_id(payment_id, false, false, options)
+          end
         end
       end