Skip to content
This repository has been archived by the owner on Apr 14, 2023. It is now read-only.

Spree::Core::GatewayError generates error #254

Closed
octave opened this issue Mar 30, 2020 · 0 comments · Fixed by #266
Closed

Spree::Core::GatewayError generates error #254

octave opened this issue Mar 30, 2020 · 0 comments · Fixed by #266

Comments

@octave
Copy link

octave commented Mar 30, 2020

When Spree::Core::GatewayError is called in gateway.rb:

solidus_paypal_braintree/app/models/solidus_paypal_braintree/gateway.rb

Line 210 in 6d22350

fail Spree::Core::GatewayError, result.message unless result.success?

It triggers this error: uninitialized constant SolidusPaypalBraintree::Spree::Core

It seems like it is nesting it into SolidusPaypalBraintree.
seand7565 added a commit to seand7565/solidus_paypal_braintree that referenced this issue Oct 14, 2020
@seand7565
Fix Spree::Core lookup in gateway
cd3cbec 
Gateway assumes that Spree::Core lives at
SolidusPaypalBraintree::Spree::Core. Adding leading colons forces the
lookup to happen from the toplevel namespace, which grabs the correct
class.

Fixes solidusio#254
@seand7565 seand7565 mentioned this issue Oct 14, 2020
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix Spree::Core lookup in gateway seand7565/solidus_paypal_braintree
1 participant
@octave