standardrb --fix

Author: mamhoff

Gemfile

@@ -1,23 +1,23 @@
 # frozen_string_literal: true
 
-source 'https://rubygems.org'
+source "https://rubygems.org"
 git_source(:github) { |repo| "https://github.com/#{repo}.git" }
 
-branch = ENV.fetch('SOLIDUS_BRANCH', 'main')
-gem 'solidus', github: 'solidusio/solidus', branch: branch
+branch = ENV.fetch("SOLIDUS_BRANCH", "main")
+gem "solidus", github: "solidusio/solidus", branch: branch
 
-gem 'rails', "~> #{ENV.fetch('RAILS_VERSION', '7.0')}"
+gem "rails", "~> #{ENV.fetch("RAILS_VERSION", "7.0")}"
 
 # Provides basic authentication functionality for testing parts of your engine
-gem 'solidus_auth_devise'
+gem "solidus_auth_devise"
 
-case ENV.fetch('DB', nil)
-when 'mysql'
-  gem 'mysql2'
-when 'postgresql'
-  gem 'pg'
+case ENV.fetch("DB", nil)
+when "mysql"
+  gem "mysql2"
+when "postgresql"
+  gem "pg"
 else
-  gem 'sqlite3'
+  gem "sqlite3"
 end
 
 gemspec
@@ -27,4 +27,4 @@ gemspec
 #
 # We use `send` instead of calling `eval_gemfile` to work around an issue with
 # how Dependabot parses projects: https://github.com/dependabot/dependabot-core/issues/1658.
-send(:eval_gemfile, 'Gemfile-local') if File.exist? 'Gemfile-local'
+send(:eval_gemfile, "Gemfile-local") if File.exist? "Gemfile-local"

Rakefile

@@ -1,15 +1,15 @@
 # frozen_string_literal: true
 
-require 'bundler/gem_tasks'
-require 'solidus_dev_support/rake_tasks'
+require "bundler/gem_tasks"
+require "solidus_dev_support/rake_tasks"
 SolidusDevSupport::RakeTasks.install
 
 task :default do
-  require 'bundler'
+  require "bundler"
   Bundler.with_unbundled_env do
-    sh 'bin/rspec'
+    sh "bin/rspec"
   end
 end
 
-CLOBBER.include('dummy-app')
-CLOBBER.include('sandbox')
+CLOBBER.include("dummy-app")
+CLOBBER.include("sandbox")

app/decorators/models/solidus_braintree/spree_user_decorator.rb

@@ -3,7 +3,7 @@
 module SolidusBraintree
   module SpreeUserDecorator
     def self.prepended(base)
-      base.has_one :braintree_customer, class_name: 'SolidusBraintree::Customer', inverse_of: :user
+      base.has_one :braintree_customer, class_name: "SolidusBraintree::Customer", inverse_of: :user
     end
 
     ::Spree.user_class.prepend self

app/helpers/solidus_braintree/braintree_admin_helper.rb

@@ -4,7 +4,7 @@ module SolidusBraintree
   module BraintreeAdminHelper
     # Returns a link to the Braintree web UI for the given Braintree payment
     def braintree_transaction_link(payment)
-      environment = payment.payment_method.preferred_environment == 'sandbox' ? 'sandbox' : 'www'
+      environment = (payment.payment_method.preferred_environment == "sandbox") ? "sandbox" : "www"
       merchant_id = payment.payment_method.preferred_merchant_id
       response_code = payment.response_code
 
@@ -14,9 +14,9 @@ def braintree_transaction_link(payment)
       link_to(
         response_code,
         "https://#{environment}.braintreegateway.com/merchants/#{merchant_id}/transactions/#{response_code}",
-        title: 'Show payment on Braintree',
-        target: '_blank',
-        rel: 'noopener'
+        title: "Show payment on Braintree",
+        target: "_blank",
+        rel: "noopener"
       )
     end
   end

app/models/solidus_braintree/avs_result.rb

@@ -1,50 +1,50 @@
 # frozen_string_literal: true
 
-require 'active_merchant/billing/avs_result'
+require "active_merchant/billing/avs_result"
 
 module SolidusBraintree
   class AVSResult < ActiveMerchant::Billing::AVSResult
     # Mapping took from ActiveMerchant::Billing::BraintreeBlueGateway
     AVS_MAPPING = {
-      'M' => {
-        'M' => 'M',
-        'N' => 'A',
-        'U' => 'B',
-        'I' => 'B',
-        'A' => 'B'
+      "M" => {
+        "M" => "M",
+        "N" => "A",
+        "U" => "B",
+        "I" => "B",
+        "A" => "B"
       },
-      'N' => {
-        'M' => 'Z',
-        'N' => 'C',
-        'U' => 'C',
-        'I' => 'C',
-        'A' => 'C'
+      "N" => {
+        "M" => "Z",
+        "N" => "C",
+        "U" => "C",
+        "I" => "C",
+        "A" => "C"
       },
-      'U' => {
-        'M' => 'P',
-        'N' => 'N',
-        'U' => 'I',
-        'I' => 'I',
-        'A' => 'I'
+      "U" => {
+        "M" => "P",
+        "N" => "N",
+        "U" => "I",
+        "I" => "I",
+        "A" => "I"
       },
-      'I' => {
-        'M' => 'P',
-        'N' => 'C',
-        'U' => 'I',
-        'I' => 'I',
-        'A' => 'I'
+      "I" => {
+        "M" => "P",
+        "N" => "C",
+        "U" => "I",
+        "I" => "I",
+        "A" => "I"
       },
-      'A' => {
-        'M' => 'P',
-        'N' => 'C',
-        'U' => 'I',
-        'I' => 'I',
-        'A' => 'I'
+      "A" => {
+        "M" => "P",
+        "N" => "C",
+        "U" => "I",
+        "I" => "I",
+        "A" => "I"
       },
-      'B' => {
-        'B' => 'B'
+      "B" => {
+        "B" => "B"
       },
-      nil => { nil => nil }
+      nil => {nil => nil}
     }.freeze
 
     class << self

app/models/solidus_braintree/base_record.rb

@@ -2,7 +2,7 @@
 
 module SolidusBraintree
   def self.table_name_prefix
-    'solidus_paypal_braintree_'
+    "solidus_paypal_braintree_"
   end
 
   class BaseRecord < ::Spree::Base

app/models/solidus_braintree/configuration.rb

@@ -3,30 +3,30 @@
 module SolidusBraintree
   class Configuration < BaseRecord
     PAYPAL_BUTTON_PREFERENCES = {
-      color: { availables: %w[gold blue silver white black], default: 'white' },
-      shape: { availables: %w[pill rect], default: 'rect' },
-      label: { availables: %w[checkout credit pay buynow paypal installment], default: 'checkout' },
-      tagline: { availables: %w[true false], default: 'false' },
-      layout: { availables: %w[horizontal vertical], default: 'horizontal' },
-      messaging: { availables: %w[true false], default: 'false' }
+      color: {availables: %w[gold blue silver white black], default: "white"},
+      shape: {availables: %w[pill rect], default: "rect"},
+      label: {availables: %w[checkout credit pay buynow paypal installment], default: "checkout"},
+      tagline: {availables: %w[true false], default: "false"},
+      layout: {availables: %w[horizontal vertical], default: "horizontal"},
+      messaging: {availables: %w[true false], default: "false"}
     }.freeze
 
     include ::Spree::Preferences::Persistable
 
-    belongs_to :store, class_name: 'Spree::Store', optional: false
+    belongs_to :store, class_name: "Spree::Store", optional: false
 
     # Preferences for Paypal button
     PAYPAL_BUTTON_PREFERENCES.each do |name, desc|
-      preference_name = "paypal_button_#{name}".to_sym
-      attribute_name = "preferred_#{preference_name}".to_sym
+      preference_name = :"paypal_button_#{name}"
+      attribute_name = :"preferred_#{preference_name}"
 
       preference preference_name, :string, default: desc[:default]
 
       validates attribute_name, inclusion: desc[:availables]
     end
 
-    preference :venmo_button_color, :preference_select, default: 'blue'
-    preference :venmo_button_width, :preference_select, default: '320'
+    preference :venmo_button_color, :preference_select, default: "blue"
+    preference :venmo_button_width, :preference_select, default: "320"
 
     def preferred_venmo_button_color_options
       [["Blue", "blue"], ["White", "white"]]

app/models/solidus_braintree/gateway.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'solidus_braintree/request_protection'
+require "solidus_braintree/request_protection"
 
 module SolidusBraintree
   class Gateway < ::Spree::PaymentMethod
@@ -11,9 +11,9 @@ class TokenGenerationDisabledError < StandardError; end
     # Error message from Braintree that gets returned by a non voidable transaction
     NON_VOIDABLE_STATUS_ERROR_REGEXP = /can only be voided if status is authorized/
 
-    TOKEN_GENERATION_DISABLED_MESSAGE = 'Token generation is disabled. ' \
-                                        'To re-enable set the `token_generation_enabled` preference on the ' \
-                                        'gateway to `true`.'
+    TOKEN_GENERATION_DISABLED_MESSAGE = "Token generation is disabled. " \
+                                        "To re-enable set the `token_generation_enabled` preference on the " \
+                                        "gateway to `true`."
 
     ALLOWED_BRAINTREE_OPTIONS = [
       :device_data,
@@ -35,17 +35,17 @@ class TokenGenerationDisabledError < StandardError; end
     preference(:token_generation_enabled, :boolean, default: true)
 
     # Preferences for configuration of Braintree credentials
-    preference(:environment, :string, default: 'sandbox')
+    preference(:environment, :string, default: "sandbox")
     preference(:merchant_id, :string, default: nil)
-    preference(:public_key,  :string, default: nil)
+    preference(:public_key, :string, default: nil)
     preference(:private_key, :string, default: nil)
     preference(:http_open_timeout, :integer, default: 60)
     preference(:http_read_timeout, :integer, default: 60)
     preference(:merchant_currency_map, :hash, default: {})
     preference(:paypal_payee_email_map, :hash, default: {})
 
     # Which checkout flow to use (vault/checkout)
-    preference(:paypal_flow, :string, default: 'vault')
+    preference(:paypal_flow, :string, default: "vault")
 
     # A hash that gets passed to the `style` key when initializing the credit card fields.
     # See https://developers.braintreepayments.com/guides/hosted-fields/styling/javascript/v3
@@ -301,7 +301,7 @@ def reusable_sources(order)
     # Whether to store this payment method in the PayPal Vault. This only works when the checkout
     # flow is "vault", so make sure to call +super+ if you override it.
     def store_in_vault
-      preferred_paypal_flow == 'vault'
+      preferred_paypal_flow == "vault"
     end
 
     def logger
@@ -339,18 +339,18 @@ def transaction_options(source, options, submit_for_settlement: false)
       end
 
       params[:channel] = "Solidus"
-      params[:options] = { store_in_vault_on_success: store_in_vault }
+      params[:options] = {store_in_vault_on_success: store_in_vault}
 
       if submit_for_settlement
         params[:options][:submit_for_settlement] = true
       end
 
       if paypal_email = paypal_payee_email_for(source, options)
-        params[:options][:paypal] = { payee_email: paypal_email }
+        params[:options][:paypal] = {payee_email: paypal_email}
       end
 
       if source.venmo? && venmo_business_profile_id
-        params[:options][:venmo] = { profile_id: venmo_business_profile_id }
+        params[:options][:venmo] = {profile_id: venmo_business_profile_id}
       end
 
       if merchant_account_id = merchant_account_for(source, options)

app/models/solidus_braintree/response.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require 'active_merchant/billing/response'
-require_relative 'avs_result'
+require "active_merchant/billing/response"
+require_relative "avs_result"
 
 # Response object that all actions on the gateway should return
 module SolidusBraintree
@@ -57,21 +57,21 @@ def error_message(result)
       # Human readable error message for transaction responses
       def transaction_error_message(transaction)
         case transaction.status
-        when 'gateway_rejected'
+        when "gateway_rejected"
           I18n.t(transaction.gateway_rejection_reason,
-            scope: 'solidus_braintree.gateway_rejection_reasons',
+            scope: "solidus_braintree.gateway_rejection_reasons",
             default: "#{transaction.status.humanize} #{transaction.gateway_rejection_reason.humanize}")
-        when 'processor_declined'
+        when "processor_declined"
           I18n.t(transaction.processor_response_code,
-            scope: 'solidus_braintree.processor_response_codes',
+            scope: "solidus_braintree.processor_response_codes",
             default: "#{transaction.processor_response_text} (#{transaction.processor_response_code})")
-        when 'settlement_declined'
+        when "settlement_declined"
           I18n.t(transaction.processor_settlement_response_code,
-            scope: 'solidus_braintree.processor_settlement_response_codes',
+            scope: "solidus_braintree.processor_settlement_response_codes",
             default: "#{transaction.processor_settlement_response_text} (#{transaction.processor_settlement_response_code})") # rubocop:disable Layout/LineLength
         else
           I18n.t(transaction.status,
-            scope: 'solidus_braintree.transaction_statuses',
+            scope: "solidus_braintree.transaction_statuses",
             default: transaction.status.humanize)
         end
       end

app/models/solidus_braintree/source.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'solidus_braintree/request_protection'
+require "solidus_braintree/request_protection"
 
 module SolidusBraintree
   class Source < ::Spree::PaymentSource
@@ -20,7 +20,7 @@ class Source < ::Spree::PaymentSource
     }, suffix: :funding
 
     belongs_to :user, class_name: ::Spree::UserClassHandle.new, optional: true
-    belongs_to :payment_method, class_name: 'Spree::PaymentMethod'
+    belongs_to :payment_method, class_name: "Spree::PaymentMethod"
     has_many :payments, as: :source, class_name: "Spree::Payment", dependent: :destroy
 
     belongs_to :customer, class_name: "SolidusBraintree::Customer", optional: true
@@ -100,18 +100,18 @@ def display_number
       elsif venmo?
         username
       else
-        "XXXX-XXXX-XXXX-#{last_digits.to_s.rjust(4, 'X')}"
+        "XXXX-XXXX-XXXX-#{last_digits.to_s.rjust(4, "X")}"
       end
     end
 
     def display_paypal_funding_source
       I18n.t(paypal_funding_source,
-        scope: 'solidus_braintree.paypal_funding_sources',
+        scope: "solidus_braintree.paypal_funding_sources",
         default: paypal_funding_source)
     end
 
     def display_payment_type
-      "#{I18n.t('solidus_braintree.payment_type.label')}: #{friendly_payment_type}"
+      "#{I18n.t("solidus_braintree.payment_type.label")}: #{friendly_payment_type}"
     end
 
     private