jekyll-spree-client 0.1.14 → 0.1.18

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 231ce6a54824c7e484336d5a80f90eecf82ccff491a0de8f80fcd7e81b8e0d9a
-  data.tar.gz: 5808e4bf0454d60ead7c62f2e2ecf8828f4417c97f4a0a5f907039dffdc73591
+  metadata.gz: 3d065883f0802c7520e29ebaa9fad7b624c22d903c190b06450543351603e389
+  data.tar.gz: 1810529172893e3c5feb437a27817e15d51e1b5bcd494d2b80de2dea44ffedea
 SHA512:
-  metadata.gz: 17d67e2e407cba9595c4218b01dc03512f75a4b959c1917ab0d95488cf045cd6e9296e11568a2aae01ab21ddd9fbf66d4e8c276d64b202fe67cfd8e834dce2ff
-  data.tar.gz: 0ef761a1c5fb123635e4e1fd9f6636e5ff2b3ae7297d8ebfc4b937d1337be7d46ccb7a2bd9729ea27f8f56706b3dad3b1688669633f52afc522bab5a0eee61f5
+  metadata.gz: a43430f248135eaad2522f0975af3c07cec45e4a4144f226435bfd2b3f097d9d525738d1401450ccbeb34cd3704023588c183a5391ccba1604ea4077d960b773
+  data.tar.gz: 16e594d12bbc16c8ae40a6b851065a3024897d66cf6585c461a8c6255ad9cc1590276c79cf9ff2920103259b36615f6a9bd13787342cf1586b4c9cae1d285b04

data/README.md

@@ -43,6 +43,7 @@ spree:
   sku_field: sku
   api_url: http://localhost:3000
   shipping_category_id: 1
+  store: https://your.store
 ```
 
 You can get the API key on your user profile.  Pass it as the
@@ -52,7 +53,7 @@ You can get the API key on your user profile.  Pass it as the
 SPREE_API_KEY="some random value generated by spree" bundle exec jekyll build
 ```
 
-Since v0.1.4 you can pass the store URL too, and it'll take precedence
+Since v0.1.4 you can pass the API URL too, and it'll take precedence
 over `_config.yml`:
 
 ```bash

data/lib/jekyll/spree_client.rb

@@ -5,7 +5,7 @@ require 'spree_client'
 
 # 
 module Jekyll
-  SpreeClient = Struct.new :site, :sku_field, :api_key, :spree_url, :shipping_category_id, keyword_init: true do
+  SpreeClient = Struct.new :site, :sku_field, :api_key, :spree_url, :shipping_category_id, :store, keyword_init: true do
     # Products are posts with the SKU field
     #
     # @return [Array] Jekyll::Document
@@ -33,28 +33,33 @@ module Jekyll
     #
     # @return [Array]
     def variant_fields
-      @variant_fields ||= %w[price weight height width depth cost_price pay_what_you_can].freeze
+      @variant_fields ||= %w[sku 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
+      @product_fields ||= %w[name description meta_description meta_keywords meta_title pay_what_you_can].freeze
     end
 
     # All fields, using during creation
     #
     # @return [Array]
     def create_fields
-      @create_fields ||= product_fields + variant_fields
+      @create_fields ||= (product_fields + variant_fields).freeze
+    end
+
+    # Fields which are only updated on the Variant.
+    def variant_only_fields
+      @variant_only_fields ||= %w[track_inventory].freeze
     end
 
     # Spree Client
     #
     # @return [SpreeClient::API::V1]
     def spree
-      @spree ||= ::SpreeClient::API::V1.new **to_h.slice(:api_key, :spree_url)
+      @spree ||= ::SpreeClient::API::V1.new **to_h.slice(:api_key, :spree_url, :store)
     end
 
     # Localization with jekyll-locales
@@ -180,18 +185,23 @@ module Jekyll
     # @return [true,false]
     def update(product, variant)
       # Sync local changes to Spree
-      remote_product = product.data.slice(*product_fields).transform_keys(&:to_sym)
-      variant_comparison = variant.slice(*product_fields).transform_keys(&:to_sym)
+      remote_product = attributes_from product, product_fields
+      variant_comparison = attributes_from variant, product_fields
 
       # Only when they haven't changed, so we don't waste time sending a
       # request.
       unless remote_product.slice(*variant_comparison.keys) == 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)
+
+        remote_product.delete_if do |k, v|
+          remote_variant.key? k
+        end
+
         remote_product[:id] = variant['slug']
-        remote_product[:sku] = product.data[sku_field]
 
-        unless remote_products.update **remote_product
+        unless remote_products.update(**remote_product) && spree.variants.update(**remote_variant)
           Jekyll.logger.error "Couldn't update #{product.data['title']}"
           mark_with_error product, error_messages
         end
@@ -218,20 +228,22 @@ module Jekyll
     # @param [Jekyll::Document]
     # @return [true,false]
     def create(product)
-      new_product = product.data.slice(*create_fields).transform_keys(&:to_sym)
-      new_product[:sku] = product.data[sku_field]
+      new_product = attributes_from(product, create_fields)
 
-      unless remote_products.create **new_product
+      unless remote_products.create(**new_product)
         Jekyll.logger.error "Couldn't create #{product.data['title']}"
         mark_with_error product, error_messages
 
         return false
       end
 
-      product.data['variant_id'] = remote_products.response.dig('master', 'id')
+      variant_id = remote_products.response.dig('master', 'id')
+      spree.variants(id: variant_id).update(attributes_from(product, variant_only_fields))
+      product.data['variant_id'] = variant_id
+      product.data['in_stock'] = product.data['stock'].to_i > 0
 
       # Add the initial stock
-      spree.stock_items(id: product.data['variant_id'], stock_location_id: stock_location['id'])
+      spree.stock_items(id: variant_id, stock_location_id: stock_location['id'])
            .stock_movements.create(quantity: product.data['stock'])
 
       true
@@ -245,5 +257,14 @@ module Jekyll
         end
       end&.flatten
     end
+
+    # Converts fields from a Jekyll::Document into an attribute Hash.
+    #
+    # @param [Jekyll::Document,Hash] Document or Hash to extract data from
+    # @param [Array] An Array of field names
+    # @return [Hash] A symbolized Hash of fields and values
+    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
   end
 end

data/lib/jekyll-spree-client.rb

@@ -22,7 +22,7 @@ Jekyll::Hooks.register :site, :post_read do |site|
   end
 
   config = site.config['spree']&.transform_keys(&:to_sym) || {}
-  config.merge!(site: site, api_key: ENV['SPREE_API_KEY'], spree_url: ENV['SPREE_URL'])
+  config.merge!(site: site, api_key: ENV['SPREE_API_KEY'], spree_url: ENV['SPREE_URL'], store: site.config['url'])
 
   client = Jekyll::SpreeClient.new **config
 

metadata

@@ -1,29 +1,29 @@
 --- !ruby/object:Gem::Specification
 name: jekyll-spree-client
 version: !ruby/object:Gem::Version
-  version: 0.1.14
+  version: 0.1.18
 platform: ruby
 authors:
 - f
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-05-11 00:00:00.000000000 Z
+date: 2021-09-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: spree-api-client
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '0.2'
+        version: 0.2.3
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '0.2'
+        version: 0.2.3
 - !ruby/object:Gem::Dependency
   name: fast_blank
   requirement: !ruby/object:Gem::Requirement