huginn_acumen_product_agent 1.5.1 → 1.7.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 96b074d16608b20299c0c1bd7292bd060c28bef20290938b240cf96b15543d2d
-  data.tar.gz: 1bbaabd459e91d72240568816d9c4df03c7fb6dfee73f2adedaa01dbe2df2c47
+  metadata.gz: f139af2f44d6618c028e02f0e32392f4d86b9d93bd671fd4d30f97cfe5d285a3
+  data.tar.gz: 389101ff173b00ba5e995826ce02ff0f04d6bf0b14059a1d68998ef882b3e7ac
 SHA512:
-  metadata.gz: 04cd9b20490eefafff31faa38afc4500a90f54d9cf53f2acf33181d652657b5fe90ef7a60ed6c21c5f58451aa7a27ec9e7bb606ee7cdfcfe81fe64294ff08e74
-  data.tar.gz: 35688020d94feb470257944770b2bd8b9d22334e1a67a70da12b41e26a962abc8bcd3f594c659901b6068afbd8a8aa05242f4bbce57a105f2d4e83fb028f7e04
+  metadata.gz: '079581a58cddd69561dc20c7ed8b9e131aad0ec78c76b365ca6ecd7cacd1b114dd21ed9533b24beec13534fa6a0373d12da9b6e01274a33a540a0877553f3185'
+  data.tar.gz: d3623463c67be3cd512f2f0e2199f8c51249abc99c461b502738e7b4fac0e5afb2886eb8fe15b6096920df2d8e856b8836c38767aabf586b71b4d30b63d0681c

data/lib/huginn_acumen_product_agent/acumen_client.rb

@@ -95,6 +95,8 @@ class AcumenClient
                   <column_name>Inv_Product.Category</column_name>
                   <column_name>Inv_Product.Next_Release</column_name>
                   <column_name>Inv_Product.BO_Reason</column_name>
+                  <column_name>Inv_Product.Not_On_Website</column_name>
+                  <column_name>Inv_Product.Not_Active</column_name>
               </requested_output>
           </acusoapRequest>
       XML

data/lib/huginn_acumen_product_agent/acumen_product_agent.rb

@@ -11,7 +11,28 @@ module Agents
         default_schedule 'never'
 
         description <<-MD
-      Huginn agent for sane ACUMEN product data.
+      Huginn agent for retrieving sane ACUMEN product data.
+
+      ## Agent Options
+      The following outlines the available options in this agent
+
+      ### Acumen Connection
+      * endpoint: The root URL for the Acumen API
+      * site_code: The site code from Acumen
+      * password: The Acumen API password
+
+      ### Variant Settings
+      * physical_formats: A list of the formats associated with a physical product
+      * digital_formats: A list of the formats associated with a digital product
+
+      ### Product Attributes
+      * attribute_to_property: An optional map linking Acumen attributes to Schema.org
+        product properties.
+
+      ### Other Options
+      * ignore_skus: An optional array of Acumen product skus that will be intentionally
+        excluded from any output.
+
         MD
 
         def default_options
@@ -22,7 +43,6 @@ module Agents
                 'physical_formats' => [],
                 'digital_formats' => [],
                 'attribute_to_property' => {},
-                'contributor_types_map' => {},
             }
         end
 
@@ -51,8 +71,10 @@ module Agents
                 errors.add(:base, "if provided, attribute_to_property must be a hash")
             end
 
-            unless options['contributor_types_map'].is_a?(Hash)
-                errors.add(:base, "if provided, contributor_types_map must be a hash")
+            if options['ignore_skus']
+              unless options['ignore_skus'].is_a?(Array)
+                  errors.add(:base, "if provided, ignore_skus must be an array")
+              end
             end
         end
 
@@ -78,6 +100,7 @@ module Agents
             password = interpolated['password']
             physical_formats = interpolated['physical_formats']
             digital_formats = interpolated['digital_formats']
+            ignore_skus = interpolated['ignore_skus'] ? interpolated['ignore_skus'] : []
 
             auth = {
                 'site_code' => site_code,
@@ -104,7 +127,9 @@ module Agents
             end
 
             products.each do |product|
-                create_event payload: product
+                unless ignore_skus.include?(product['sku'])
+                  create_event payload: product
+                end
             end
         end
 

data/lib/huginn_acumen_product_agent/concerns/acumen_product_query_concern.rb

@@ -10,6 +10,8 @@ module AcumenProductQueryConcern
 
     def get_products_by_ids(acumen_client, ids)
         response = acumen_client.get_products(ids)
+        products = []
+
         products = parse_product_request(response)
 
         response = acumen_client.get_products_marketing(ids)
@@ -20,7 +22,16 @@ module AcumenProductQueryConcern
 
     def get_variants_for_ids(acumen_client, ids)
         result = get_linked_products_by_ids(acumen_client, ids)
-        result.select { |link| link['alt_format'].to_s != 0.to_s }
+
+        # Filtering out duplicate links getting sent from acumen
+        filter = []
+        result.each do |link|
+          if (link['alt_format'].to_s != 0.to_s && !link.in?(filter))
+            filter.push(link)
+          end
+        end
+
+        return filter
     end
 
     def get_linked_products_by_ids(acumen_client, ids)
@@ -77,20 +88,24 @@ module AcumenProductQueryConcern
 
     def get_product_categories(acumen_client, products)
         # fetch categories
-        skus = products.map { |product| product['sku'] }
+
+        skus = products.map { |product| product['model'].map { |m| m['sku'] } }[0]
         response = acumen_client.get_product_categories(skus)
         categories = process_product_categories_query(response)
 
         # map categories to products
         products.each do |product|
-            sku = product['sku']
-            if categories[sku]
-                active = categories[sku].select { |c| c['inactive'] == '0' }
-                product['categories'] = active.map do |category|
-                  {
-                    '@type' => 'Thing',
-                    'identifier' => category['category_id']
-                  }
+            product['model'].each do |variant|
+                variant['categories'] = []
+                sku = variant['sku']
+                if categories[sku]
+                    active = categories[sku].select { |c| c['inactive'] == '0' }
+                    active.map do |category|
+                      variant['categories'].push({
+                        '@type' => 'Thing',
+                        'identifier' => category['category_id']
+                      })
+                    end
                 end
             end
         end
@@ -111,6 +126,7 @@ module AcumenProductQueryConcern
             variant['@type'] = 'ProductModel'
             variant['isDefault'] = false
             variant['isTaxable'] = field_value(p, 'Inv_Product.Taxable') == '1'
+            variant['isAvailableForPurchase'] = field_value(p, 'Inv_Product.Not_On_Website') == '0'
             variant['acumenAttributes'] = {
               'is_master' => field_value(p, 'Inv_Product.OnWeb_LinkOnly') == '0'
             }
@@ -145,6 +161,7 @@ module AcumenProductQueryConcern
                     'info_alpha_1' => field_value(p, 'Inv_Product.Info_Alpha_1'),
                     'info_boolean_1' => field_value(p, 'Inv_Product.Info_Boolean_1'),
                 },
+                'isAvailableForPurchase' => field_value(p, variant['isAvailableForPurchase']),
             }
 
             category = field_value(p, 'Inv_Product.Category')
@@ -388,6 +405,7 @@ module AcumenProductQueryConcern
         result.each do |product|
           if product['model'].length == 1
             product['model'][0]['isDefault'] = true
+            set_base_sku(product, product['model'][0]['sku'])
             next
           else
             physical_formats.each do |val|
@@ -408,6 +426,17 @@ module AcumenProductQueryConcern
               end
             end
           end
+
+          model_ids = product['model'].map { |m| m['identifier'] }
+          primary_variant = product['model'].select { |m| m['identifier'] == model_ids.min }.first
+
+          # Set the base SKU to the SKU of the oldest record.
+          # The base_sku property is designed to be a system value specific to the inegration using this agent.
+          # As a result, we don't particularly care what that value is so long as we can retrieve it consistently
+          # across executions. If a paperback product is created first, this will always return that product's SKU
+          # as the base. This gives us a consistent way to link Acumen products to an external system where database
+          # IDs may not match.
+          set_base_sku(product, primary_variant ? primary_variant['sku'] : product['model'][0]['sku'])
         end
         result
     end
@@ -427,6 +456,18 @@ module AcumenProductQueryConcern
         field[key]['__content__'] if field[key]
     end
 
+    def set_base_sku(product, sku)
+
+      product['additionalProperty'].push({
+          '@type' => 'PropertyValue',
+          'propertyID' => 'baseSku',
+          'name' => 'Base SKU',
+          'value' => sku,
+      })
+
+      product
+    end
+
     def quantitative_value(value, unit)
       {
         '@type' => 'QuantitativeValue',

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: huginn_acumen_product_agent
 version: !ruby/object:Gem::Version
-  version: 1.5.1
+  version: 1.7.1
 platform: ruby
 authors:
 - Jacob Spizziri
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-07-15 00:00:00.000000000 Z
+date: 2020-11-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler