jekyll-spree-client 0.1.11 → 0.1.12

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 57f0f9a9615cb8ecd401ff1c4224b1c246b33489809187f1dacd99bc8efdaba0
-  data.tar.gz: f9279e9889b35a6e7ef42013ca44590796f1530178973d99da5b30d4bb6adcfd
+  metadata.gz: 29f4a2f251e21575d7f5b4ef34af064f8fb80a7a2e494267d19336f201288c3d
+  data.tar.gz: 77521c26bb015d395fa515adf4270385c28de024a57725d08123c8c7d0daebd5
 SHA512:
-  metadata.gz: 9f77636bc5292b48977e0e7d43de5609d3437568c93aaac1d80f3e9c485b98b87c368d2351770a0c6a047129a1ba2c4a4dc136e6b2ee83ecaf22e0243e37b2ed
-  data.tar.gz: 71243959c46fa41db5d1db9e9a3d661e9284688257b6b00f2a278a6c7174562a0cfdfdfd2197500d141571843f1c6af6290877cbcc70e6bb0590b7ded16ebd80
+  metadata.gz: 71480aa397d2fe3e0191833bc55194d6d287ac1825c5ebe01ecbb341a13d1de79f16d1686a2b6a4ebd879cc882584d9bab940ded155208988386110104d00a8b
+  data.tar.gz: f917c8e9cf3675801c8240ab88bc05f19368fd1597065ec7d789f4ec82ed5acab22733e5dc65ba7dfc63cc703ed197de4c09d1207f9e79f0d1663253a647b2dc

data/README.md

@@ -152,8 +152,18 @@ errors:
 
 ## Notes
 
-* Synchronization is unidirectional.  If the product doesn't exist on
-  Spree, it's created.  If it does, the Jekyll document is updated.
+* Synchronization follows this rules.
+
+  If the product doesn't exist on Spree, it's created.
+
+  If it does, the Jekyll document is updated with price, weight,
+  dimensions and stock.
+
+  It means once the product is created on Spree, Spree manages this
+  variables.
+
+  But Product properties like description and metadata are updated from
+  Jekyll to Spree.
 
   We want this to change in the near future but we need to think a way
   to prevent conflicts.

data/lib/jekyll/spree_client.rb

@@ -33,7 +33,21 @@ module Jekyll
     #
     # @return [Array]
     def variant_fields
-      @variant_fields ||= %w[price weight height width depth description cost_price].freeze
+      @variant_fields ||= %w[price weight height width depth cost_price].freeze
+    end
+
+    # Fields that can change locally and we need to sync to Spree
+    #
+    # @return [Array]
+    def product_fields
+      @product_fields ||= %w[name description meta_description meta_keywords meta_title].freeze
+    end
+
+    # All fields, using during creation
+    #
+    # @return [Array]
+    def create_fields
+      @create_fields ||= product_fields + variant_fields
     end
 
     # Spree Client
@@ -76,6 +90,10 @@ module Jekyll
         end
 
         products.each do |product|
+          # Products have names, not titles, so we save them for later
+          product.data['_name'] = product.data['name'] if product.data.key? 'name'
+          product.data['name'] = product.data['title']
+
           # Remove previous errors
           product.data.delete 'errors'
           # Find the corresponding Spree variant in the response
@@ -89,6 +107,8 @@ module Jekyll
           else
             create product
           end
+
+          product.data['name'] = product.data.delete('_name') if product.data.key? '_name'
         end
       end
 
@@ -159,6 +179,22 @@ module Jekyll
     # @param [Hash]
     # @return [true,false]
     def update(product, variant)
+      # Sync local changes to Spree
+      remote_product = product.data.slice(*product_fields).transform_keys(&:to_sym)
+      # Only when they haven't changed, so we don't waste time sending a
+      # request.
+      unless remote_product == variant.slice(*product_fields).transform_keys(&:to_sym)
+        # XXX: We don't have the product ID with the variant but we have
+        # the slug.
+        remote_product[:id] = variant['slug']
+        remote_product[:sku] = product.data[sku_field]
+
+        unless remote_products.update **remote_product
+          Jekyll.logger.error "Couldn't update #{product.data['title']}"
+          mark_with_error product, error_messages
+        end
+      end
+
       # XXX: The API replies with stringified numbers so we convert them
       # back to the type we're using locally.
       variant.slice(*variant_fields).each do |k, v|
@@ -180,9 +216,8 @@ module Jekyll
     # @param [Jekyll::Document]
     # @return [true,false]
     def create(product)
-      new_product = product.data.slice(*variant_fields).transform_keys(&:to_sym)
+      new_product = product.data.slice(*create_fields).transform_keys(&:to_sym)
       new_product[:sku] = product.data[sku_field]
-      new_product[:name] = product.data['title']
 
       unless remote_products.create **new_product
         Jekyll.logger.error "Couldn't create #{product.data['title']}"

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: jekyll-spree-client
 version: !ruby/object:Gem::Version
-  version: 0.1.11
+  version: 0.1.12
 platform: ruby
 authors:
 - f
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-03-03 00:00:00.000000000 Z
+date: 2021-04-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: spree-api-client