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 https://github.com/activemerchant/active_merchant/blob/1d84209e9d37c12c3591a6372bf381ee08294251/lib/active_merchant/billing/gateways/braintree_blue.rb#L102

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