jekyll-spree-client 0.1.20 → 0.1.21

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 390af0f63aee218e9f828303c23352dfaddd6c08c0830476ae04788b7115f6b0
-  data.tar.gz: 81f4e247841705eaea3f73b2e0a941455435c65dd17d87cee68449324d9b8c0f
+  metadata.gz: 8f691ea2cebbd451a912b7dc7ea4ec6cea4a30c8a2b345dd0a2ab5d113ef763f
+  data.tar.gz: 9d56c15db95801773a18ad56f0a1263ad0693a007611211f211e7e4a3797744c
 SHA512:
-  metadata.gz: 99290d03c7daebf7b12cead21c69d0560530ab8bc50ab37312c26f15a6d377c6ae22b4e35b31a77cd1e8550990e7c4b4c11432b896134edb5c52daa5f4b44641
-  data.tar.gz: f846b9e1914977776df45da460794e48855bc5b6fb37dec363d4b6e264e0530a2782e13fee38af25f119457c2bd4d519e5cc3e66ab47b9d24101d94e73a1d771
+  metadata.gz: f7a0c45963d7ca0ee95f4448873dfe9d9eccc2050f579b00e5faeb215eca6051f1bad615216610fec97817d2e48e7bb3da79b7f537d8157d462ead7b690c8975
+  data.tar.gz: 865dbd12ff3369260a90ca5f9cc8e3bb2020209e4bac961d49f32695cd7b97a2a6e59e171d4265f7480fc0b062afb987ba016297a3aa33cddac1121ad1da6dab

data/lib/jekyll/spree_client.rb

@@ -188,11 +188,14 @@ module Jekyll
       remote_product = attributes_from(product, product_fields).tap do |a|
         a[:extra_attributes] ||= {}
       end
-      variant_comparison = attributes_from variant, product_fields
+
+      variant_comparison = attributes_from(variant, product_fields).tap do |v|
+        v[:extra_attributes] = v[:extra_attributes]&.transform_keys(&:to_sym)
+      end
 
       # Only when they haven't changed, so we don't waste time sending a
       # request.
-      unless remote_product.slice(*variant_comparison.keys) == variant_comparison
+      unless compare_hashes(remote_product, variant_comparison)
         # XXX: We don't have the product ID with the variant but we have
         # the slug.
         remote_variant = { id: variant['id'] }.merge!(attributes_from product, variant_only_fields)
@@ -268,5 +271,18 @@ module Jekyll
     def attributes_from(document_or_hash, fields)
       (document_or_hash.respond_to?(:data) ? document_or_hash.data : document_or_hash).slice(*fields).transform_keys(&:to_sym)
     end
+
+    # Compare two hashes by the keys they share.  Some fields we send
+    # are not available through the APIv1, like meta_description.
+    #
+    # @link https://stackoverflow.com/a/15503329
+    # @param :h1 [Hash]
+    # @param :h2 [Hash]
+    # @return [Boolean]
+    def compare_hashes(h1, h2)
+      (h1.keys & h2.keys).all? do |k|
+        h1[k] == h2[k]
+      end
+    end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: jekyll-spree-client
 version: !ruby/object:Gem::Version
-  version: 0.1.20
+  version: 0.1.21
 platform: ruby
 authors:
 - f
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-03-17 00:00:00.000000000 Z
+date: 2022-03-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: spree-api-client
@@ -96,7 +96,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.3.5
+rubygems_version: 3.3.8
 signing_key:
 specification_version: 4
 summary: Spree Ecommerce API client for Jekyll