jekyll-spree-client 0.1.4 → 0.1.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8a40d8af711c0a63af1d4f40c7573a52b33d34dc8c3f4f8c895e14e6e5952f97
-  data.tar.gz: a8c8e002b61466f6fab8907c75e1243582511a2feb5ca77b1af1f1d2a0a0ae96
+  metadata.gz: 345d7f0e45d1c821288d7a4452601c163f2d82304d317dc72ec425d6abd5cb47
+  data.tar.gz: b74a132ff5c5592c2989bff62b2403e9a6cb10eacf4af715b5ee287c404f19a6
 SHA512:
-  metadata.gz: 69badade74e187bf6926ef4bb2e155c3720416e3411a384489c927a41559849d4e838b14fd28fc82a157a89b377934d088e748e7eebc4ff83d9757bd6951c2d9
-  data.tar.gz: a7ee374df482289ae15471665c3e784af2383f6b85c3b59444a16a88e17af31a8c21e8e6eda7e641cd3c90627064749ae6531a106a8431e13935fd16c921d84f
+  metadata.gz: a92e70c17670182523b32ad5e5f6038f3fa0611c23c731a61b30e9f36cb30df6f65dd406921830b35dda8fd303cb60f8738e2e895c73f180390a482762e6725f
+  data.tar.gz: f918174d0812624c71174e3c414160daa90c28539c1ae32859ce4c2046aa94ff9c222b5a551d3174009975c31939c7de75c30172a1a419290763b7249bf2418a

data/README.md

@@ -16,16 +16,6 @@ conversion unless you store prices as strings.
 
 You can use Integer and Floats now!
 
-### Update
-
-The issue is that Spree expects to receive prices in the locale format
-(ie, "200,00") but will return them on Ruby's `to_s` conversion
-("200.0") so we need to reconvert them to our current format and juggle
-between them.
-
-We'll deal with `Integer` for now until we find [a better
-solution](https://github.com/spree/spree/issues/10556).
-
 ## Installation
 
 Add this line to your site's Gemfile:
@@ -77,8 +67,10 @@ Add the required fields in their frontmatter:
 ---
 layout: post
 title: A product
+description: With a description
 sku: A unique ID
 price: 100.00
+cost_price: 50.0
 stock: 100
 weight: 100
 height: 100
@@ -90,18 +82,52 @@ depth: 100
 The layout doesn't matter, but we recommend to use a layout for
 products.
 
+### Hooks
+
+If you need to do some data manipulation before and after
+synchronization is done, you can use hooks.  In a `_plugins/spree.rb`
+file:
+
+```ruby
+Jekyll::Hooks.register :spree, :pre_render do |spree|
+  spree.local_products.each do |product|
+    product.data['_description'] = product.data['description']
+    product.data['description'] = product.data['productore']&.data&.dig('title')
+  end
+end
+
+Jekyll::Hooks.register :spree, :post_render do |spree|
+  spree.local_products.each do |product|
+    product.data['description'] = product.data.delete '_description'
+
+    product.data.delete 'description' unless product.data['description']
+  end
+end
+```
+
+This plugin gets the description value from another document (they're
+linked using
+[jekyll-linked-posts](https://0xacab.org/sutty/jekyll/jekyll-linked-posts/),
+it's a real example!), and then recovers it.
+
+* `pre_render` runs just before starting synchronization
+* `post_render` run after synchronization and before writing changes
+
 ### Building the site
 
 During site build, the plugin will communicate with the Spree API and
-create products when they don't exist or update their price, current
-stock and the other values.
+create products when they don't exist or locally update their price,
+current stock and the other values.
 
 Some fields will appear on the frontmatter:
 
 ```yaml
 variant_id: 1
 in_stock: true
-sync_error: 2020-10-10 00:00:00 UTC
+errors:
+  time: 2020-10-10 00:00:00 UTC
+  messages:
+  - Something went wrong
 ```
 
 * `variant_id` is the ID in the Spree database.
@@ -109,11 +135,17 @@ sync_error: 2020-10-10 00:00:00 UTC
 * `in_stock` is true or false according to availability (Spree decides
   this).
 
-* `sync_error` if the synchronization fails there will be a timestamp
-  here.
+* `errors` if the synchronization fails there will be a timestamp and
+  some messages here.
 
 ## Notes
 
+* Synchronization is unidirectional.  If the product doesn't exist on
+  Spree, it's created.  If it does, the Jekyll document is updated.
+
+  We want this to change in the near future but we need to think a way
+  to prevent conflicts.
+
 * All products are created as "master variants" on Spree.  There will be
   no pictures, taxonomies or other values, only basic product
   information.
@@ -126,11 +158,12 @@ sync_error: 2020-10-10 00:00:00 UTC
 * Stock is added to the default stock location, meaning the first active
   one.  The API doesn't show or allows to query the actual default.
 
-* The `price` is in the default currency.
+* `price` and `cost_price` are expressed in Spree's default currency.
 
 * If you install the `jekyll-write-and-commit-changes` plugin it will
-  save the data back to disk when you build with `JEKYLL_ENV=production`
-  environment variable.
+  save the data back to disk when you build.  Using the
+  `JEKYLL_ENV=production` environment variable will also commit changes
+  to git.
 
 * Check the shipping category ID on Spree, since there's no API for it,
   you need to provide it manually, default is `1`.

data/lib/jekyll-spree-client.rb

@@ -6,7 +6,9 @@ require_relative 'jekyll/spree_client'
 # matter.  The SKU needs to be unique, but this plugin doesn't ensure
 # that.  Duplicated SKUs will be silently ignored for now.
 Jekyll::Hooks.register :site, :post_read do |site|
-  config = site.config['spree']&.transform_keys(&:to_sym)
+  next unless ENV['SPREE_API_KEY']
+
+  config = site.config['spree']&.transform_keys(&:to_sym) || {}
   config.merge!(site: site, api_key: ENV['SPREE_API_KEY'], spree_url: ENV['SPREE_URL'])
 
   client = Jekyll::SpreeClient.new **config
@@ -16,4 +18,14 @@ Jekyll::Hooks.register :site, :post_read do |site|
   if ENV['JEKYLL_ENV'] == 'production' && site.respond_to?(:repository)
     site.repository.commit site.data.dig(site.config['locale'], 'spree', 'commit') || 'Spree synchronization'
   end
+
+  # Ordered articles
+  posts = site.site_payload['site']['posts']
+
+  # Make steps available to other articles
+  %w[cart shipment payment confirmation].each do |step|
+    site.config[step] = posts.find do |doc|
+      doc.data['layout'] == step
+    end
+  end
 end

data/lib/jekyll/spree_client.rb

@@ -33,7 +33,7 @@ module Jekyll
     #
     # @return [Array]
     def variant_fields
-      @variant_fields ||= %w[price weight height width depth].freeze
+      @variant_fields ||= %w[price weight height width depth description cost_price].freeze
     end
 
     # Spree Client
@@ -56,6 +56,8 @@ module Jekyll
     end
 
     def sync!
+      Jekyll::Hooks.trigger :spree, :pre_render, self
+
       # The API allows to query several SKUs at the same time, so we send
       # groups of 10 products.
       local_products.each_slice(10) do |products|
@@ -68,13 +70,14 @@ module Jekyll
           products.each do |p|
             mark_with_error p,
               i18n.dig('spree', 'errors', 'api') || "Couldn't obtain variants, the store may be down or API key is incorrect"
-            local_save p.save
           end
 
           next
         end
 
         products.each do |product|
+          # Remove previous errors
+          product.data.delete 'errors'
           # Find the corresponding Spree variant in the response
           variant = response['variants'].find do |v|
             v[sku_field] == product.data[sku_field]
@@ -88,6 +91,12 @@ module Jekyll
           end
         end
       end
+
+      Jekyll::Hooks.trigger :spree, :post_render, self
+
+      local_products.each do |product|
+        local_save product
+      end
     end
 
     private
@@ -152,19 +161,18 @@ module Jekyll
     def update(product, variant)
       # XXX: The API replies with stringified numbers so we convert them
       # back to the type we're using locally.
-      product.data.merge! variant.slice(*variant_fields).map do |k, v|
-        case product.data[k]
-        when Integer then v.to_i
-        when Float then v.to_f
-        else v
-        end
-      end.to_h
+      variant.slice(*variant_fields).each do |k, v|
+        product.data[k] = case product.data[k]
+                            when Integer then v.to_i
+                            when Float then v.to_f
+                            else v
+                          end
+      end
 
       product.data['in_stock'] = variant['in_stock']
       product.data['variant_id'] = variant['id']
-      product.data.delete 'errors'
 
-      local_save product
+      product
     end
 
     # If the product doesn't exist, create it.
@@ -179,16 +187,12 @@ module Jekyll
       unless remote_products.create **new_product
         Jekyll.logger.error "Couldn't create #{product.data['title']}"
         mark_with_error product, error_messages
-        local_save product
 
         return false
       end
 
       product.data['variant_id'] = remote_products.response.dig('master', 'id')
 
-      # Save the variant ID
-      local_save product
-
       # Add the initial stock
       spree.stock_items(id: product.data['variant_id'], stock_location_id: stock_location['id'])
            .stock_movements.create(quantity: product.data['stock'])

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: jekyll-spree-client
 version: !ruby/object:Gem::Version
-  version: 0.1.4
+  version: 0.1.9
 platform: ruby
 authors:
 - f
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-11-11 00:00:00.000000000 Z
+date: 2020-12-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: spree-api-client