super_good-solidus_taxjar 0.15.2 → 0.16.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5b31e03589ed5699fa26ba2b8f2b5ccb99fe64af208fce5c15eba8ccec10366a
-  data.tar.gz: 63bb0d2f537723e15feaae9785090f6903c4bf6b7d2763915f22f5668dff6e89
+  metadata.gz: 501ae07a0037373ed9f5cca71c67d469e81f10b9ed8be735c6812401fdab9e02
+  data.tar.gz: 558cc49c0d712c4a14e74b5f9c98308084791fe98088f6737565a53e6303b8dd
 SHA512:
-  metadata.gz: a09644d7afc6976fc173bd000137424ac398057db518491bbfdfd544cf66b046347959388d3c913016ee3c1b0c43211adcb2a83692c96e2a4b66cf20dc5acc45
-  data.tar.gz: a723866885f456c418004f0642ccd7351e371c288a3f8d81cdb9b7a7179d655b738ac41f2a85b5878cc5da17ebf8a3af8697a3e02270008882d1da0ac0735a64
+  metadata.gz: c88ced44e52d7dbd7a90f2e6401d1e9cc01a085e1986ad836c7210e20df99e7ea061ba1129291bee9451107481a8d4f32bd833e4bd9997d960be839d72d1166c
+  data.tar.gz: 9aef0cf33ce2181eda0e7fb41ba704af88e3629ea4ed34a3f8063ddd8b61f67123edcf617099619d7c82c668eb07c85407873b327aad91345679f6b535d232eb

data/.travis.yml

@@ -1,4 +1,5 @@
 ---
+dist: trusty
 sudo: false
 language: ruby
 cache: bundler

data/CHANGELOG.md

@@ -2,6 +2,12 @@
 
 ## master
 
+## v0.16.0
+
+- Fix `#incomplete_address?` method to be friendly also to completely blank addresses.
+
+- Added `SuperGood::SolidusTaxJar::TaxRateCalculator` for retrieving the tax rate for a given `Spree::Address`. The calculator follows `TaxCalculator` conventions by relying on address validators and custom exception handling.
+
 ## v0.15.2
 
 - Add order number to param logging.

data/README.md

@@ -1,10 +1,8 @@
-## *Warning:* This extension is under active development and not yet at a v1.0 release.
-
 # `SuperGood::SolidusTaxJar` [![Build Status](https://travis-ci.com/SuperGoodSoft/solidus_taxjar.svg?token=rc5QTgHvLLF7cpqkmyfd&branch=master)](https://travis-ci.com/SuperGoodSoft/solidus_taxjar)
 
 `SuperGood::SolidusTaxJar` is a [Solidus](https://github.com/solidusio/solidus) extension that allows Solidus stores to use [TaxJar](https://www.taxjar.com/) for tax calculations.
 
-This is not a fork of [spree_taxjar](https://github.com/vinsol-spree-contrib/spree_taxjar), like [solidus_taxjar](https://github.com/boomerdigital/solidus_taxjar). Instead, `SuperGood::SolidusTaxJar` uses the new configurable tax system [introduced by @adammathys](https://github.com/solidusio/solidus/pull/1892) in Solidus 2.4.
+This is not a fork of [spree_taxjar](https://github.com/vinsol-spree-contrib/spree_taxjar), like [solidus_taxjar](https://github.com/boomerdigital/solidus_taxjar). Instead of using a custom calculator, `SuperGood::SolidusTaxJar` uses the new configurable tax system [by @adammathys](https://github.com/solidusio/solidus/pull/1892) introduced in Solidus v2.4. This maps better to how the TaxJar API itself works.
 
 ## Installation
 
@@ -33,7 +31,7 @@ This is not a fork of [spree_taxjar](https://github.com/vinsol-spree-contrib/spr
    ```ruby
    # Put this in config/initializers/taxjar.rb
 
-   SuperGood::SolidusTaxJar::TaxCalculator.exception_handler = ->(e) {
+   SuperGood::SolidusTaxJar.exception_handler = ->(e) {
      # Report exceptions in here. For example, if you were using the Sentry's
      # raven-ruby gem to report errors, you might do this:
      Raven.capture_exception(e)
@@ -42,6 +40,29 @@ This is not a fork of [spree_taxjar](https://github.com/vinsol-spree-contrib/spr
 
 4. Finally, make sure that the `TAXJAR_API_KEY` environment variable is set to a your TaxJar API key and make sure that you have a `Spree::TaxRate` with the name "Sales Tax". This will be used as the source for the tax adjustments that Solidus creates.
 
+## Project Status
+
+This extension is under active development and not yet at a v1.0 release, but it's currently being used in production by multiple Solidus stores.
+
+Requirements for TaxJar integrations vary as some stores also need reporting, which isn't provided out of the box by this extension. This is because individual stores will be using different background job frameworks or runners (Sidekiq, delayed_job, ActiveJob, etc.) and a reliable integration will rely on one of these. Because this part of the integration is small, we've chosen to provide the transaction reporting functionality, but have skipped directly integrating it.
+
+If you're having trouble integrating this extension with your store and would like some assistance, please reach out to Jared via e-mail at [jared@super.gd](mailto:jared@super.gd) or on the official Solidus as `@jared`.
+
+## Features
+
+The extension provides currently two high level `calculator` classes that wrap the low-level Ruby taxjar gem API calls:
+
+* tax calculator
+* tax rate calculator
+
+### TaxCalculator
+
+`SuperGood::SolidusTaxJar::TaxCalculator` allows calculating the full tax breakdown for a given `Spree::Order`. The breakdown includes separate line items taxes and shipment taxes.
+
+### TaxRateCalculator
+
+`SuperGood::SolidusTaxJar::TaxRateCalculator` allows calculating the tax rate for a given `Spree::Address`. It relies on the same low-level Ruby TaxJar API endpoint of the tax calculator in order to provide the most coherent and reliable results. TaxJar support recommends using this endpoint for live calculations.
+
 ## Development
 
 After checking out the repo, run `bin/setup` to install dependencies. Then, run `rake spec` to run the tests. You can also run `bin/console` for an interactive prompt that will allow you to experiment.

data/lib/super_good/solidus_taxjar.rb

@@ -5,7 +5,9 @@ require 'taxjar'
 require "super_good/solidus_taxjar/version"
 require "super_good/solidus_taxjar/api_params"
 require "super_good/solidus_taxjar/api"
+require "super_good/solidus_taxjar/calculator_helper"
 require "super_good/solidus_taxjar/tax_calculator"
+require "super_good/solidus_taxjar/tax_rate_calculator"
 require "super_good/solidus_taxjar/discount_calculator"
 
 module SuperGood
@@ -25,7 +27,10 @@ module SuperGood
     end
 
     self.cache_duration = 3.hours
-    self.cache_key = ->(order) { APIParams.order_params(order).to_json }
+    self.cache_key = ->(record) {
+      record_type = record.class.name.demodulize.underscore
+      APIParams.send("#{record_type}_params", record).to_json
+    }
     self.discount_calculator = ::SuperGood::SolidusTaxJar::DiscountCalculator
     self.exception_handler = ->(e) {
       Rails.logger.error "An error occurred while fetching TaxJar tax rates - #{e}: #{e.message}"

data/lib/super_good/solidus_taxjar/api.rb

@@ -22,6 +22,10 @@ module SuperGood
         end
       end
 
+      def tax_rate_for(address)
+        taxjar_client.tax_for_order(APIParams.tax_rate_address_params(address)).rate
+      end
+
       def tax_rates_for(address)
         taxjar_client.rates_for_location(*APIParams.address_params(address))
       end

data/lib/super_good/solidus_taxjar/api_params.rb

@@ -29,6 +29,13 @@ module SuperGood
           ]
         end
 
+        def tax_rate_address_params(address)
+          {
+            amount: 100,
+            shipping: 0,
+          }.merge(order_address_params(address))
+        end
+
         def transaction_params(order)
           {}
             .merge(customer_params(order))

data/lib/super_good/solidus_taxjar/calculator_helper.rb

@@ -0,0 +1,44 @@
+module SuperGood
+  module SolidusTaxJar
+    module CalculatorHelper
+      extend ActiveSupport::Concern
+
+      class_methods do
+        def default_api
+          ::SuperGood::SolidusTaxJar::API.new
+        end
+      end
+
+      def incomplete_address?(address)
+        return true if address.is_a?(Spree::Tax::TaxLocation)
+
+        [
+          address.address1,
+          address.city,
+          address.state&.abbr || address.state_name,
+          address.zipcode,
+          address.country&.iso
+        ].any?(&:blank?)
+      end
+
+      def taxable_address?(address)
+        SuperGood::SolidusTaxJar.taxable_address_check.(address)
+      end
+
+      def cache
+        if !Rails.env.test?
+          Rails.cache.fetch(
+            cache_key,
+            expires_in: SuperGood::SolidusTaxJar.cache_duration
+          ) { yield }
+        else
+          yield
+        end
+      end
+
+      def exception_handler
+        SuperGood::SolidusTaxJar.exception_handler
+      end
+    end
+  end
+end

data/lib/super_good/solidus_taxjar/tax_calculator.rb

@@ -1,9 +1,7 @@
 module SuperGood
   module SolidusTaxJar
     class TaxCalculator
-      def self.default_api
-        ::SuperGood::SolidusTaxJar::API.new
-      end
+      include CalculatorHelper
 
       def initialize(order, api: self.class.default_api)
         @order = order
@@ -115,33 +113,14 @@ module SuperGood
         Spree::TaxRate.find_by(name: "Sales Tax")
       end
 
-      def cache
-        if !Rails.env.test?
-          Rails.cache.fetch(
-            cache_key,
-            expires_in: SuperGood::SolidusTaxJar.cache_duration
-          ) { yield }
-        else
-          yield
-        end
-      end
-
       def cache_key
         SuperGood::SolidusTaxJar.cache_key.(order)
       end
 
-      def exception_handler
-        SuperGood::SolidusTaxJar.exception_handler
-      end
-
       def taxable_order?(order)
         SuperGood::SolidusTaxJar.taxable_order_check.(order)
       end
 
-      def taxable_address?(address)
-        SuperGood::SolidusTaxJar.taxable_address_check.(address)
-      end
-
       def shipping_tax_label(shipment, shipping_tax)
         SuperGood::SolidusTaxJar.shipping_tax_label_maker.(
           shipment,
@@ -152,18 +131,6 @@ module SuperGood
       def line_item_tax_label(taxjar_line_item, spree_line_item)
         SuperGood::SolidusTaxJar.line_item_tax_label_maker.(taxjar_line_item, spree_line_item)
       end
-
-      def incomplete_address?(tax_address)
-        return true if tax_address.is_a?(Spree::Tax::TaxLocation)
-
-        [
-          tax_address.address1,
-          tax_address.city,
-          tax_address&.state&.abbr || tax_address.state_name,
-          tax_address.zipcode,
-          tax_address.country.iso
-        ].any?(&:blank?)
-      end
     end
   end
 end

data/lib/super_good/solidus_taxjar/tax_rate_calculator.rb

@@ -0,0 +1,36 @@
+module SuperGood
+  module SolidusTaxJar
+    class TaxRateCalculator
+      include CalculatorHelper
+      def initialize(address, api: self.class.default_api)
+        @address = address
+        @api = api
+      end
+
+      def calculate
+        return no_rate if SuperGood::SolidusTaxJar.test_mode
+        return no_rate if incomplete_address?(address)
+        return no_rate unless taxable_address?(address)
+        cache do
+          api.tax_rate_for(address).to_d
+        end
+
+      rescue StandardError => e
+        exception_handler.(e)
+        no_rate
+      end
+
+      private
+
+      attr_reader :address, :api
+
+      def no_rate
+        BigDecimal(0)
+      end
+
+      def cache_key
+        SuperGood::SolidusTaxJar.cache_key.(address)
+      end
+    end
+  end
+end

data/lib/super_good/solidus_taxjar/version.rb

@@ -1,5 +1,5 @@
 module SuperGood
   module SolidusTaxJar
-    VERSION = "0.15.2"
+    VERSION = "0.16.0"
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: super_good-solidus_taxjar
 version: !ruby/object:Gem::Version
-  version: 0.15.2
+  version: 0.16.0
 platform: ruby
 authors:
 - Jared Norman
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2019-06-26 00:00:00.000000000 Z
+date: 2020-02-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: solidus_core
@@ -171,8 +171,10 @@ files:
 - lib/super_good/solidus_taxjar.rb
 - lib/super_good/solidus_taxjar/api.rb
 - lib/super_good/solidus_taxjar/api_params.rb
+- lib/super_good/solidus_taxjar/calculator_helper.rb
 - lib/super_good/solidus_taxjar/discount_calculator.rb
 - lib/super_good/solidus_taxjar/tax_calculator.rb
+- lib/super_good/solidus_taxjar/tax_rate_calculator.rb
 - lib/super_good/solidus_taxjar/version.rb
 - super_good-solidus_taxjar.gemspec
 homepage: https://github.com/SuperGoodSoft/solidus_taxjar