solidus_backtracs 2.2.1 → 2.2.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a599802f23077392477db03b1e36eecf30633ef3b5892b85a35304d15b061fe3
-  data.tar.gz: cc197de83a593fbdd81872c61e27a69cc9a09ddf3ba359407e823caad22cc102
+  metadata.gz: d58f212b33488781331c666724e857474d6c60852d8acc845990b380c6727611
+  data.tar.gz: '09aacc1386429a22c7b953a7d09e192799fa8901c4a9b1fa26bb6fa9e744d995'
 SHA512:
-  metadata.gz: 5ddd0296522fa664cd9e92229438f25fa38698dfa9fa66316311b41b49d40bfc2a92bb7507a32d5789b5978ae71b71f3ac75c03f07c6ea657c440d8e960473fc
-  data.tar.gz: 60781ae6012c98e4a88d440e9d7faba29eb2f05b51106da5c3883fe8b12fe6616b4cd4260bc1ffc3a685221ed03d4c07c2b174befec222e999caf84c9287bba9
+  metadata.gz: d2b077b182bb182625eaa55567353d78c250e24f322dac1a6f31bad0ec262c5c5c8952a890b75914e212f55ae42f4a12b1fd19eddd9af0968a82fd3f986bddc2
+  data.tar.gz: bcf82e512fae4a40bc721aa7811440922815e0488feab5096d15e1a0da3c075935fb5a7e411b39a5e83ec9f44c048b1564028ec69aac6ac33456083899aaf366

data/CHANGELOG.md

@@ -1,2 +1,16 @@
 # Changelog
 
+## 2.2.0
+
+Support the Solidus Assembly gem, consider allowed SKUs within assemblies/bundles
+Retry logic for the authenticated request
+
+## 2.1.0
+
+New auth and API patterns for Backtracs
+Support specific SKUs to be mapped
+
+## 2.0.0
+
+Diverged the gem from Shipstation to the Bactracs API
+

data/lib/generators/solidus_backtracs/install/templates/initializer.rb

@@ -11,6 +11,7 @@ SolidusBacktracs.configure do |config|
 
   ## API Configuration
   config.api_base = ENV['BACKTRACS_API_BASE'] || 'https://bactracstest.andlor.com'
+  config.api_retries = ENV['BACKTRACS_API_RETRIES'] || 3
 
   # Backtracs expects the endpoint to be protected by HTTP Basic Auth.
   # Set the username and password you desire for Backtracs to use.
@@ -32,7 +33,7 @@ SolidusBacktracs.configure do |config|
   config.sku_map = {}
   config.default_rma_type = "W"
   config.default_carrier = "FedExGrnd"
-  config.default_ship_method = "GROUND"  
+  config.default_ship_method = "GROUND"
   config.default_status = "OPEN"
   config.default_rp_location = "FG-NEW"
   config.shippable_skus = []

data/lib/solidus_backtracs/api/request_runner.rb

@@ -10,27 +10,41 @@ module SolidusBacktracs
         @username = SolidusBacktracs.configuration.authentication_username
         @password = SolidusBacktracs.configuration.authentication_password
         @api_base = SolidusBacktracs.configuration.api_base
+        @retries  = SolidusBacktracs.configuration.api_retries
       end
 
-      def authenticated_call(method: nil, path: nil, serializer: nil, shipment: nil)
-        unless @username.present? || @password.present? || @api_base.present?
-          raise "Credentials not defined for Authentication"
-        end 
+      def authenticated_call(method: nil, path: nil, serializer: nil, shipment: nil, count: 0)
+        if count <= @retries
+          unless @username.present? || @password.present? || @api_base.present?
+            raise "Credentials not defined for Authentication"
+          end
 
-        Rails.cache.fetch("backtracks_cache_key", expires_in: 1.hour) do
-          @response = self.call(method: :get, path: "/webservices/user/Authentication.asmx/Login?sUserName=#{@username}&sPassword=#{@password}")
-        end
-
-        authenticted = parse_authentication_response(@response, "Result")
+          @response = Rails.cache.fetch("backtracks_cache_key", expires_in: 1.hour, skip_nil: true) do
+            self.call(method: :get, path: "/webservices/user/Authentication.asmx/Login?sUserName=#{@username}&sPassword=#{@password}")
+          end
 
-        if authenticted == 'false'
-          raise RequestError.from_response(@response)
-        else
+          raise RequestError.from_response(@response) unless @response # just try again for @retries?
+          authenticted = parse_authentication_response(@response, "Result")
+          raise RequestError.from_response(@response) if authenticted == "false"
           sguid = parse_authentication_response(@response, "Message")
-          params = serializer.call(shipment, sguid)
 
-          rma_response = call(method: :post, path: path, params: params)
-          sync_shipment(shipment, rma_response)
+          if authenticted == 'false'
+            clear_cache
+            raise "User Not Authenticated"
+          else
+            sguid = parse_authentication_response(@response, "Message")
+            params = serializer.call(shipment, sguid)
+
+            rma_response = call(method: :post, path: path, params: params)
+            unless parse_rma_creation_response(rma_response) == 'true'
+              clear_cache
+              count += 1
+              self.authenticated_call(method: :post, path: '/webservices/rma/rmaservice.asmx', serializer: serializer, shipment: shipment, count: count)
+            end
+            shipment_synced(shipment)
+          end
+        else
+          shipment_sync_failed(shipment)
         end
       end
 
@@ -62,25 +76,33 @@ module SolidusBacktracs
         end
       end
 
+      def clear_cache
+        Rails.cache.delete('backtracks_cache_key')
+        @response = nil
+      end
+
       def parse_authentication_response(response, type)
         response.dig("AuthenticationResponse", type)
       end
 
-      def sync_shipment(shipment, response)
-        result = response.dig("Envelope", "Body", "CreateNewResponse", "CreateNewResult", "Result")
-        if result == 'true'
-          shipment.update_column(:backtracs_synced_at, Time.zone.now)
+      def parse_rma_creation_response(response)
+        response.dig("Envelope", "Body", "CreateNewResponse", "CreateNewResult", "Result")
+      end
 
-          ::Spree::Event.fire(
-            'solidus_backtracs.api.sync_completed',
-            shipment: shipment
-          )
-        else
-          ::Spree::Event.fire(
-            'solidus_backtracs.api.sync_failed',
-            shipment: shipment
-          )
-        end
+      def shipment_synced(shipment)
+        shipment.update_column(:backtracs_synced_at, Time.zone.now)
+
+        ::Spree::Event.fire(
+          'solidus_backtracs.api.sync_completed',
+          shipment: shipment
+        )
+      end
+
+      def shipment_sync_failed(shipment)
+        ::Spree::Event.fire(
+          'solidus_backtracs.api.sync_failed',
+          shipment: shipment
+        )
       end
     end
   end

data/lib/solidus_backtracs/configuration.rb

@@ -19,7 +19,8 @@ module SolidusBacktracs
       :shipment_notice_class,
       :authentication_username,
       :authentication_password,
-      :api_base, 
+      :api_base,
+      :api_retries,
       :proxy_address,
       :proxy_port,
       :proxy_username,

data/lib/solidus_backtracs/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module SolidusBacktracs
-  VERSION = '2.2.1'
+  VERSION = '2.2.2'
 end

data/{solidus_shipstation.gemspec → solidus_bactracs.gemspec}

@@ -9,12 +9,12 @@ Gem::Specification.new do |spec|
   spec.email = 'zkhan@suvie.com'
 
   spec.summary = 'A Solidus extension for integrating the Backtracs API.'
-  spec.homepage = 'https://github.com/solidusio-contrib/solidus_backtracs'
+  spec.homepage = 'https://github.com/suvie-eng/solidus_backtracs'
   spec.license = 'BSD-3-Clause'
 
   spec.metadata['homepage_uri'] = spec.homepage
-  spec.metadata['source_code_uri'] = 'https://github.com/solidusio-contrib/solidus_backtracs'
-  spec.metadata['changelog_uri'] = 'https://github.com/solidusio-contrib/solidus_backtracs/blob/master/CHANGELOG.md'
+  spec.metadata['source_code_uri'] = 'https://github.com/suvie-eng/solidus_backtracs/'
+  spec.metadata['changelog_uri'] = 'https://github.com/suvie-eng/solidus_backtracs/blob/master/CHANGELOG.md'
 
   spec.required_ruby_version = Gem::Requirement.new('>= 2.5')
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: solidus_backtracs
 version: !ruby/object:Gem::Version
-  version: 2.2.1
+  version: 2.2.2
 platform: ruby
 authors:
 - Zeryab Ali
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2022-05-13 00:00:00.000000000 Z
+date: 2022-05-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: httparty
@@ -174,7 +174,7 @@ files:
 - lib/solidus_backtracs/shipment_notice.rb
 - lib/solidus_backtracs/testing_support/factories.rb
 - lib/solidus_backtracs/version.rb
-- solidus_shipstation.gemspec
+- solidus_bactracs.gemspec
 - spec/controllers/spree/backtracs_controller_spec.rb
 - spec/fixtures/backtracs_xml_schema.xsd
 - spec/jobs/solidus_backtracs/api/schedule_shipment_syncs_job_spec.rb
@@ -197,13 +197,13 @@ files:
 - spec/support/controllers.rb
 - spec/support/webmock.rb
 - spec/support/xsd.rb
-homepage: https://github.com/solidusio-contrib/solidus_backtracs
+homepage: https://github.com/suvie-eng/solidus_backtracs
 licenses:
 - BSD-3-Clause
 metadata:
-  homepage_uri: https://github.com/solidusio-contrib/solidus_backtracs
-  source_code_uri: https://github.com/solidusio-contrib/solidus_backtracs
-  changelog_uri: https://github.com/solidusio-contrib/solidus_backtracs/blob/master/CHANGELOG.md
+  homepage_uri: https://github.com/suvie-eng/solidus_backtracs
+  source_code_uri: https://github.com/suvie-eng/solidus_backtracs/
+  changelog_uri: https://github.com/suvie-eng/solidus_backtracs/blob/master/CHANGELOG.md
 post_install_message: 
 rdoc_options: []
 require_paths: