Question about braintree blue refund implementation

[daisy1754]

Currently braintree blue refund is implemented in following way:

1. try refund
2. it it fails, try void
   

   active_merchant/lib/active_merchant/billing/gateways/braintree_blue.rb

   Line 102 in 1d84209

   response = response_from_result(@braintree_gateway.transaction.refund(transaction_id, money))

Is there any reason library is not fetching the transaction status and then do either refund or void? Current implementation is causing error response and seems a bit weird.

I'm happy to send a pull request to change behaviour but wanted to check if this is intentional. [EDIT: I no longer remember about this, so probably cannot send PR by myself]

[martinstreicher]

@daisy1754 What field in the Braintree Blue purchase transaction response represents the transaction_id to be used in the refund? Is it the network_transaction_id field found in the transaction response?

[daisy1754]

@martinstreicher quite honestly I don't remember

[martinstreicher]

No worries. I found it. It's the authorization field in the response.

[curiousepic]

Marking this "of interest" before a cleanup of stale issues