dato 0.8.0 → 0.8.1

data/lib/dato/repo.rb

@@ -1,18 +1,18 @@
 # frozen_string_literal: true
 
-require 'dato/json_api_serializer'
-require 'dato/json_api_deserializer'
-require 'dato/paginator'
+require "dato/json_api_serializer"
+require "dato/json_api_deserializer"
+require "dato/paginator"
 
 module Dato
   class Repo
     attr_reader :client, :type, :schema
 
-    IDENTITY_REGEXP = /\{\(.*?definitions%2F(.*?)%2Fdefinitions%2Fidentity\)}/
+    IDENTITY_REGEXP = /\{\(.*?definitions%2F(.*?)%2Fdefinitions%2Fidentity\)}/.freeze
 
     METHOD_NAMES = {
-      'instances' => :all,
-      'self' => :find
+      "instances" => :all,
+      "self" => :find,
     }.freeze
 
     def initialize(client, type, schema)
@@ -32,23 +32,23 @@ module Dato
     private
 
     def method_missing(method, *args, &block)
-      link = schema.links.find do |link|
-        METHOD_NAMES.fetch(link.rel, link.rel).to_sym == method.to_sym
+      link = schema.links.find do |ilink|
+        METHOD_NAMES.fetch(ilink.rel, ilink.rel).to_sym == method.to_sym
       end
 
-      return super if !link
+      return super unless link
 
       min_arguments_count = [
         link.href.scan(IDENTITY_REGEXP).size,
-        link.schema && link.method != :get ? 1 : 0
+        link.schema && link.method != :get ? 1 : 0,
       ].reduce(0, :+)
 
-      (args.size >= min_arguments_count) or
-        raise ArgumentError, "wrong number of arguments (given #{args.size}, expected #{min_arguments_count})"
+      (args.size >= min_arguments_count) ||
+        raise(ArgumentError, "wrong number of arguments (given #{args.size}, expected #{min_arguments_count})")
 
       placeholders = []
 
-      url = link['href'].gsub(IDENTITY_REGEXP) do |_stuff|
+      url = link["href"].gsub(IDENTITY_REGEXP) do |_stuff|
         placeholder = args.shift.to_s
         placeholders << placeholder
         placeholder
@@ -61,19 +61,16 @@ module Dato
         body = link.schema ? args.shift : {}
         query_string = args.shift || {}
 
-      elsif link.method == :delete
-        query_string = args.shift || {}
-
-      elsif link.method == :get
+      elsif %i[get delete].include?(link.method)
         query_string = args.shift || {}
       end
 
       options = args.any? ? args.shift.symbolize_keys : {}
 
       if link.schema && %i[post put].include?(link.method) && options.fetch(:serialize_response, true)
-        body = JsonApiSerializer.new(type, link).serialize(
+        body = JsonApiSerializer.new(link: link).serialize(
           body,
-          link.method == :post ? nil : placeholders.last
+          link.method == :post ? nil : placeholders.last,
         )
       end
 
@@ -92,21 +89,19 @@ module Dato
       if response && response[:data] && response[:data].is_a?(Hash) && response[:data][:type] == "job"
         job_result = nil
 
-        while !job_result do
+        until job_result
           begin
             sleep(1)
             job_result = client.job_result.find(response[:data][:id])
-          rescue ApiError => error
-            if error.response[:status] != 404
-              raise error
-            end
+          rescue ApiError => e
+            raise e if e.response[:status] != 404
           end
         end
 
         if job_result[:status] < 200 || job_result[:status] >= 300
           error = ApiError.new(
             status: job_result[:status],
-            body: JSON.dump(job_result[:payload])
+            body: JSON.dump(job_result[:payload]),
           )
 
           puts "===="
@@ -121,12 +116,10 @@ module Dato
         else
           job_result.payload
         end
+      elsif options.fetch(:deserialize_response, true)
+        JsonApiDeserializer.new(link.target_schema).deserialize(response)
       else
-        if options.fetch(:deserialize_response, true)
-          JsonApiDeserializer.new(link.target_schema).deserialize(response)
-        else
-          response
-        end
+        response
       end
     end
   end

data/lib/dato/site/client.rb

@@ -1,15 +1,15 @@
 # frozen_string_literal: true
 
-require 'dato/api_client'
-require 'dato/upload/file'
-require 'dato/upload/create_upload_path'
+require "dato/api_client"
+require "dato/upload/file"
+require "dato/upload/create_upload_path"
 
 module Dato
   module Site
     class Client
       include ApiClient
 
-      json_schema 'site-api'
+      json_schema "site-api"
 
       def create_upload_path(path_or_url)
         file = Upload::CreateUploadPath.new(self, path_or_url)
@@ -30,7 +30,7 @@ module Dato
         request(
           :post,
           "/pusher/authenticate",
-          { socket_id: socket_id, channel_name: channel}
+          { socket_id: socket_id, channel_name: channel },
         )
       end
     end

data/lib/dato/upload/create_upload_path.rb

@@ -1,9 +1,9 @@
 # frozen_string_literal: true
 
-require 'mime/types'
-require 'tempfile'
-require 'addressable'
-require 'net/http'
+require "mime/types"
+require "tempfile"
+require "addressable"
+require "net/http"
 
 module Dato
   module Upload
@@ -19,7 +19,7 @@ module Dato
         @file ||= if http_source?
                     uri = Addressable::URI.parse(source)
                     ext = ::File.extname(uri.path).downcase
-                    tempfile = Tempfile.new(['file', ext])
+                    tempfile = Tempfile.new(["file", ext])
                     tempfile.binmode
                     tempfile.write(download_file(source))
                     tempfile.rewind
@@ -31,7 +31,7 @@ module Dato
 
       def http_source?
         uri = Addressable::URI.parse(source)
-        uri.scheme == 'http' || uri.scheme == 'https'
+        uri.scheme == "http" || uri.scheme == "https"
       rescue Addressable::URI::InvalidURIError
         false
       end
@@ -51,9 +51,7 @@ module Dato
         mime_type = MIME::Types.of(filename).first
 
         request = Net::HTTP::Put.new(uri)
-        if mime_type
-          request.add_field("Content-Type", mime_type.to_s)
-        end
+        request.add_field("Content-Type", mime_type.to_s) if mime_type
         request.body = file.read
 
         http = Net::HTTP.new(uri.host, uri.port)
@@ -78,4 +76,3 @@ module Dato
     end
   end
 end
-

data/lib/dato/upload/file.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'dato/upload/create_upload_path'
+require "dato/upload/create_upload_path"
 
 module Dato
   module Upload
@@ -18,14 +18,14 @@ module Dato
         upload_path = CreateUploadPath.new(client, source).upload_path
 
         upload = client.uploads.create(
-          upload_attributes.merge(path: upload_path)
+          upload_attributes.merge(path: upload_path),
         )
 
         {
           alt: nil,
           title: nil,
           custom_data: {},
-        }.merge(field_attributes).merge(upload_id: upload['id'])
+        }.merge(field_attributes).merge(upload_id: upload["id"])
       end
     end
   end

data/lib/dato/upload/image.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'dato/upload/file'
+require "dato/upload/file"
 
 module Dato
   module Upload

data/lib/dato/utils/build_modular_block.rb

@@ -1,22 +1,22 @@
 # frozen_string_literal: true
 
-require 'dato/json_api_serializer'
+require "dato/json_api_serializer"
 
 module Dato
   module Utils
     module BuildModularBlock
       def self.build(unserialized_body)
-        json_api_serializer = JsonApiSerializer.new('item', nil)
+        json_api_serializer = JsonApiSerializer.new(type: "item")
         attributes = json_api_serializer.serialized_attributes(unserialized_body)
 
         payload = {
-          type: 'item',
+          type: "item",
           attributes: attributes,
           relationships: {
             item_type: {
               data: {
                 id: unserialized_body[:item_type],
-                type: 'item_type',
+                type: "item_type",
               },
             },
           },

data/lib/dato/utils/favicon_tags_builder.rb

@@ -20,7 +20,7 @@ module Dato
           build_apple_icon_tags,
           build_windows_tags,
           build_color_tags,
-          build_app_name_tag
+          build_app_name_tag,
         ].flatten.compact
       end
 
@@ -29,9 +29,9 @@ module Dato
 
         APPLE_TOUCH_ICON_SIZES.map do |size|
           link_tag(
-            'apple-touch-icon',
+            "apple-touch-icon",
             url(size),
-            sizes: "#{size}x#{size}"
+            sizes: "#{size}x#{size}",
           )
         end
       end
@@ -41,10 +41,10 @@ module Dato
 
         ICON_SIZES.map do |size|
           link_tag(
-            'icon',
+            "icon",
             url(size),
             sizes: "#{size}x#{size}",
-            type: "image/#{site.favicon.format}"
+            type: "image/#{site.favicon.format}",
           )
         end
       end
@@ -58,15 +58,15 @@ module Dato
       end
 
       def build_app_name_tag
-        meta_tag('application-name', site.name)
+        meta_tag("application-name", site.name)
       end
 
       def build_color_tags
         return unless theme_color
 
         [
-          meta_tag('theme-color', theme_color),
-          meta_tag('msapplication-TileColor', theme_color)
+          meta_tag("theme-color", theme_color),
+          meta_tag("msapplication-TileColor", theme_color),
         ]
       end
 
@@ -75,11 +75,11 @@ module Dato
       end
 
       def meta_tag(name, value)
-        { tag_name: 'meta', attributes: { name: name, content: value } }
+        { tag_name: "meta", attributes: { name: name, content: value } }
       end
 
       def link_tag(rel, href, attrs = {})
-        { tag_name: 'link', attributes: attrs.merge(rel: rel, href: href) }
+        { tag_name: "link", attributes: attrs.merge(rel: rel, href: href) }
       end
     end
   end

data/lib/dato/utils/locale_value.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'i18n/backend/fallbacks'
+require "i18n/backend/fallbacks"
 
 module Dato
   module Utils

data/lib/dato/utils/meta_tags/article_modified_time.rb

@@ -1,14 +1,14 @@
 # frozen_string_literal: true
 
-require 'dato/utils/meta_tags/base'
-require 'time'
+require "dato/utils/meta_tags/base"
+require "time"
 
 module Dato
   module Utils
     module MetaTags
       class ArticleModifiedTime < Base
         def build
-          og_tag('article:modified_time', item.updated_at.iso8601) if item
+          og_tag("article:modified_time", item.updated_at.iso8601) if item
         end
       end
     end

data/lib/dato/utils/meta_tags/article_publisher.rb

@@ -1,13 +1,13 @@
 # frozen_string_literal: true
 
-require 'dato/utils/meta_tags/base'
+require "dato/utils/meta_tags/base"
 
 module Dato
   module Utils
     module MetaTags
       class ArticlePublisher < Base
         def build
-          og_tag('article:publisher', facebook_page_url) if facebook_page_url
+          og_tag("article:publisher", facebook_page_url) if facebook_page_url
         end
 
         def facebook_page_url

data/lib/dato/utils/meta_tags/base.rb

@@ -1,8 +1,8 @@
 # frozen_string_literal: true
 
-require 'forwardable'
-require 'dato/local/field_type/seo'
-require 'active_support/core_ext/object/blank'
+require "forwardable"
+require "dato/local/field_type/seo"
+require "active_support/core_ext/object/blank"
 
 module Dato
   module Utils
@@ -19,7 +19,7 @@ module Dato
           fallback_seo = site.global_seo && site.global_seo.fallback_seo
 
           seo_field = item &&
-                      item.fields.detect { |f| f.field_type == 'seo' }
+                      item.fields.detect { |f| f.field_type == "seo" }
 
           item_seo_value = seo_field &&
                            item[seo_field.api_key] &&
@@ -36,11 +36,11 @@ module Dato
         end
 
         def meta_tag(name, content)
-          tag('meta', name: name, content: content)
+          tag("meta", name: name, content: content)
         end
 
         def og_tag(property, content)
-          tag('meta', property: property, content: content)
+          tag("meta", property: property, content: content)
         end
 
         def card_tag(name, content)

data/lib/dato/utils/meta_tags/description.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'dato/utils/meta_tags/base'
+require "dato/utils/meta_tags/base"
 
 module Dato
   module Utils
@@ -10,9 +10,9 @@ module Dato
           return unless description.present?
 
           [
-            meta_tag('description', description),
-            og_tag('og:description', description),
-            card_tag('twitter:description', description)
+            meta_tag("description", description),
+            og_tag("og:description", description),
+            card_tag("twitter:description", description),
           ]
         end
 

data/lib/dato/utils/meta_tags/image.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'dato/utils/meta_tags/base'
+require "dato/utils/meta_tags/base"
 
 module Dato
   module Utils
@@ -10,8 +10,8 @@ module Dato
           return unless image
 
           [
-            og_tag('og:image', image.url),
-            card_tag('twitter:image', image.url)
+            og_tag("og:image", image.url),
+            card_tag("twitter:image", image.url),
           ]
         end
 
@@ -21,11 +21,10 @@ module Dato
 
         def item_image
           item && item.fields
-                      .select { |field| field.field_type == 'file' }
+                      .select { |field| field.field_type == "file" }
                       .map { |field| item[field.api_key] }
                       .compact
                       .find do |image|
-
                         image.width && image.height &&
                           image.width >= 200 && image.height >= 200
                       end