plate_api 1.2.0 → 1.2.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c4adf706bf694ce9826fd9ed0e2b6a5376038cb7efd99167bebddee74bf2ff2b
-  data.tar.gz: a90b502c510ca13f87e464ba026ab53ad16737f77ad6954d23b075afcfbc0bea
+  metadata.gz: 4d7353c1eddabde505f9f31f8f0f3939b439a0988d7e758357370a7b49c43e7b
+  data.tar.gz: 5e21a75f33de4346ec8da239d568ec3434b370d1515b1d5bd8b5b3922695937e
 SHA512:
-  metadata.gz: a84c77342116a513c6e8f16b62b9059ef069afe29910d7e5aa74abfc14e94da3328f1f80a31c4633ff9320c4e454dd81e63e48a0c29919664636af92e5ebdd1d
-  data.tar.gz: b0a78eab66e3da0a6e0b7350c80bb624f35b01e56bdd703b63997404a2e929282155acd9b5d2c86ba1d9d5d06909a04c44286dafb050c7b6d57921dd96790546
+  metadata.gz: b530d2fa2aff8acf661d333dd6559dc3d8974af6bcc50d19f770ebf603af12b7e193e74e575693d48d0ccf56f046ecc3730b48e99609dd17e06d300a34b1c5c9
+  data.tar.gz: 98c2873014f3aedfb12aeb889e7647a931449aef26a81a07eebb8a2c75c6c14fe63e0686cdc80f21a8e4a9102d0c6286b0d7ec72509dd45c0474e51b2632803d

data/.travis.yml

@@ -6,7 +6,7 @@ sudo: false
 language: ruby
 rvm:
   - 2.5.1
-before_install: gem install bundler -v 1.16.1
+before_install: gem install bundler -v 2.1.4
 before_script:
   - curl -L https://codeclimate.com/downloads/test-reporter/test-reporter-latest-linux-amd64 > ./cc-test-reporter
   - chmod +x ./cc-test-reporter

data/Dockerfile

@@ -0,0 +1,6 @@
+FROM ruby:2.7
+
+RUN mkdir /plate_api
+WORKDIR /plate_api
+COPY . /plate_api
+RUN bin/setup

data/Gemfile.lock

@@ -1,9 +1,9 @@
 PATH
   remote: .
   specs:
-    plate_api (1.2.0)
+    plate_api (1.2.5)
       faraday (~> 1.0.1)
-      mimemagic (~> 0.3.3)
+      mime-types (~> 3.3.1)
 
 GEM
   remote: https://rubygems.org/
@@ -30,7 +30,9 @@ GEM
     hashdiff (1.0.1)
     i18n (1.8.2)
       concurrent-ruby (~> 1.0)
-    mimemagic (0.3.5)
+    mime-types (3.3.1)
+      mime-types-data (~> 3.2015)
+    mime-types-data (3.2021.1115)
     minitest (5.14.1)
     multipart-post (2.1.1)
     public_suffix (4.0.5)

data/README.md

@@ -145,6 +145,12 @@ After checking out the repo, run `bin/setup` to install dependencies. Then, run
 
 To install this gem onto your local machine, run `bundle exec rake install`. To release a new version, update the version number in `version.rb`, and then run `bundle exec rake release`, which will create a git tag for the version, push git commits and tags, and push the `.gem` file to [rubygems.org](https://rubygems.org).
 
+## Run Specs with Docker
+```bash
+docker build -t plate_api_gem .
+docker run --rm -v $(pwd):/plate_api -it plate_api_gem rspec
+```
+
 ## License
 
 The gem is available as open source under the terms of the [MIT License](https://opensource.org/licenses/MIT).

data/lib/plate_api/object_handler.rb

@@ -7,38 +7,42 @@ module PlateApi
     def initialize(handling_class, api_connector)
       raise ArgumentError, "`handling_class` given for #new is not valid" unless handling_class
       raise ArgumentError, "`api_connector` given for #new is not valid" unless api_connector
+
       @handling_class = handling_class
       @api_connector = api_connector
     end
 
     def find(id)
       raise ArgumentError, "`id` given for #find is not valid" unless id
+
       result = @api_connector.get(resource_path(id))
       if result["data"]
-        return new_object(result["data"])
+        new_object(result["data"])
       else
-        raise Error::Response, result
+        raise ResponseError, result
       end
     end
 
     def update(id, attributes)
       raise ArgumentError, "`id` given for #update is not valid" unless id
-      raise ArgumentError, "`attributes` given for #update is not valid" unless attributes.is_a? Hash
-      result = @api_connector.put(resource_path(id), {"data" => attributes})
+      raise ArgumentError, "`attributes` given for #update is not valid" unless attributes.is_a?(Hash)
+
+      result = @api_connector.put(resource_path(id), { "data" => attributes })
 
       if result["data"]
-        return new_object(result["data"])
+        new_object(result["data"])
       else
-        raise Error::Response, result
+        raise ResponseError, result
       end
     end
 
-    def create(parent, attributes, create_method=:post)
+    def create(parent, attributes, create_method = :post)
       raise ArgumentError, "`parent` given for #create is not valid" unless parent
-      raise ArgumentError, "`attributes` given for #create is not valid" unless attributes.is_a? Hash
+      raise ArgumentError, "`attributes` given for #create is not valid" unless attributes.is_a?(Hash)
+
       parameters = case create_method
       when :post
-        {"data" => attributes}
+        { "data" => attributes }
       when :post_multipart
         attributes
       end
@@ -46,45 +50,70 @@ module PlateApi
       result = @api_connector.send(create_method, collection_path(parent.class, parent.id), parameters)
 
       if result["data"]
-        return new_object(result["data"])
+        new_object(result["data"])
       else
-        raise Error::Response, result
+        raise ResponseError, result
       end
     end
 
     def delete(id)
       raise ArgumentError, "`id` given for #find is not valid" unless id
+
       result = @api_connector.delete(resource_path(id))
       if result["data"]
-        return new_object(result["data"])
+        new_object(result["data"])
       else
-        raise Error::Response, result
+        raise ResponseError, result
       end
     end
 
-    def index(parent_class, parent_id, extra_params={})
+    def index(parent_class, parent_id, extra_params = {}, return_raw = false)
       raise ArgumentError, "`parent_id` given for #index is not valid" unless parent_id
       raise ArgumentError, "`parent_class` given for #index is not valid" unless parent_class
 
       result = @api_connector.get(collection_path(parent_class, parent_id), extra_params)
-      if result["data"]
-        return result["data"].map{|x| new_object(x)}
-      else
-        raise Error::Response, result
+      raise ResponseError, result unless result["data"]
+      return result if return_raw
+
+      result["data"].map { |x| new_object(x) }
+    end
+
+    def index_block(parent_class, parent_id, extra_params = {}, &block)
+      extra_params[:page] = extra_params.delete("page") if extra_params["page"]
+      extra_params[:page] = 1 unless extra_params[:page]
+
+      pagination = index_block_iteration(parent_class, parent_id, extra_params, &block)
+      return unless pagination
+
+      while pagination["page"] < pagination["total_pages"]
+        extra_params.merge!(page: (pagination["page"] + 1))
+        pagination = index_block_iteration(parent_class, parent_id, extra_params, &block)
+        break unless pagination
       end
     end
 
     def index_total_count(parent)
       result = @api_connector.get(collection_path(parent.class, parent.id), per_page: 1)
       if result["meta"]
-        return result["meta"]["pagination"]["total_records"]
+        result["meta"]["pagination"]["total_records"]
       else
-        raise Error::Response, result
+        raise ResponseError, result
       end
     end
 
     private
 
+    def index_block_iteration(parent_class, parent_id, params)
+      result = index(parent_class, parent_id, params, true)
+      objects = result["data"].map { |x| new_object(x) }
+      meta = result["meta"]
+      yield(objects, meta)
+
+      # Returns pagination metadata so it can be used to
+      # iterate through the pages.
+      meta["pagination"] if meta
+    end
+
     # Construct a new object of @handling_class, given a succesful api_response
     def new_object(api_response)
       @handling_class.new(
@@ -99,15 +128,15 @@ module PlateApi
       "#{@handling_class.api_name}/#{id}"
     end
 
-    def collection_path(parent_class=nil, parent_id=nil)
-      if (parent_class != nil) ^ (parent_id != nil)
+    def collection_path(parent_class = nil, parent_id = nil)
+      if (!parent_class.nil?) ^ (!parent_id.nil?)
         raise ArgumentError, "An invalid combination `parent_class` and `parent_id` is given. Provide both or none."
       end
 
       if parent_class
         "#{parent_class.api_name}/#{parent_id}/#{@handling_class.api_name}"
       else
-        "#{@handling_class.api_name}"
+        @handling_class.api_name.to_s
       end
     end
   end

data/lib/plate_api/plate_object/base.rb

@@ -6,7 +6,7 @@ module PlateApi::PlateObject
     HasOneRelations = {}
     HasManyRelations = {}
 
-    def initialize(id, attributes, relations, object_handler=nil)
+    def initialize(id, attributes, relations, object_handler = nil)
       @id = id
       @object_handler = object_handler
       initialize_state(attributes, relations)
@@ -17,24 +17,27 @@ module PlateApi::PlateObject
     end
 
     def reload
-      raise ArgumentError.new("No object_handler is set.") unless @object_handler
+      raise ArgumentError, "No object_handler is set." unless @object_handler
+
       reinitialize(@object_handler.find(@id))
-      return self
+      self
     end
 
     def update(attributes)
-      raise ArgumentError.new("Input `attributes` is not a Hash") unless attributes.is_a? Hash
-      raise ArgumentError.new("No object_handler is attached to this object") unless @object_handler
+      raise ArgumentError, "Input `attributes` is not a Hash" unless attributes.is_a?(Hash)
+      raise ArgumentError, "No object_handler is attached to this object" unless @object_handler
+
       if new_object = @object_handler.update(@id, attributes)
         reinitialize(new_object)
       else
-        raise ArgumentError.new("The update was unsuccesful.")
+        raise ArgumentError, "The update was unsuccesful."
       end
-      return self
+      self
     end
 
     def delete
-      raise ArgumentError.new("No object_handler is attached to this object") unless @object_handler
+      raise ArgumentError, "No object_handler is attached to this object" unless @object_handler
+
       @object_handler.delete(@id)
     end
 
@@ -46,18 +49,25 @@ module PlateApi::PlateObject
       to_s
     end
 
-    def method_missing(m, *args, &block)
-      if attributes[m.to_s]
-        return attributes[m.to_s]
-      elsif attributes["content"] && attributes["content"][m.to_s]
-        return attributes["content"][m.to_s]["value"]
+    def method_missing(method_name, *args, &block)
+      if attributes[method_name.to_s]
+        attributes[method_name.to_s]
+      elsif attributes["content"] && attributes["content"][method_name.to_s]
+        attributes["content"][method_name.to_s]["value"]
       else
         super
       end
     end
 
+    def respond_to_missing?(method_name, include_private = false)
+      return true if attributes[method_name.to_s]
+      return true if attributes["content"] && attributes["content"][method_name.to_s]
+
+      super
+    end
+
     def ==(other)
-      return other.id == @id && other.class == self.class
+      other.id == @id && other.class == self.class
     end
 
     private
@@ -72,56 +82,68 @@ module PlateApi::PlateObject
       @relations = relations
     end
 
-    def self.has_one(name, klass)
-      HasOneRelations[self.name] ||= {}
-      self.attr_accessor "#{name}_id"
-      HasOneRelations[self.name][name.to_s] = klass
-      define_has_one_method(name, klass)
-    end
+    def set_relation_ids(relations_attributes)
+      HasOneRelations[self.class.name] ||= {}
 
-    def self.define_has_one_method(name, klass)
-      define_method(name.to_s) do
-        id = self.send("#{name}_id")
-        return nil unless id
-        @object_handler.api_connector.handler(Object.const_get(klass)).find(id)
+      return unless relations_attributes
+
+      self.class::HasOneRelations[self.class.name].keys.each do |relation_name|
+        val = relations_attributes["#{relation_name}_id"]
+        if val
+          send("#{relation_name}_id=", val)
+        end
       end
     end
 
-    def self.has_many(plural_name, singular_name, klass, define_create_method=false)
-      HasManyRelations[self.name] ||= {}
-      HasManyRelations[self.name][plural_name.to_s] = klass
-      define_has_many_methods(plural_name, klass)
-      define_create_method(singular_name, klass) if define_create_method
-    end
+    class << self
+      private
 
-    def self.define_has_many_methods(plural_name, klass)
-      define_method(plural_name.to_s) do |params={}|
-        @object_handler.api_connector.handler(
-          Object.const_get(klass)
-        ).index(self.class, @id, params)
+      def has_one(name, klass)
+        HasOneRelations[self.name] ||= {}
+        attr_accessor("#{name}_id")
+        HasOneRelations[self.name][name.to_s] = klass
+        define_has_one_method(name, klass)
       end
 
-      define_method("#{plural_name}_total_count") do
-        @object_handler.api_connector.handler(
-          Object.const_get(klass)
-        ).index_total_count(self)
+      def define_has_one_method(name, klass)
+        define_method(name.to_s) do
+          id = send("#{name}_id")
+          return nil unless id
+
+          @object_handler.api_connector.handler(Object.const_get(klass)).find(id)
+        end
       end
-    end
 
-    def self.define_create_method(singular_name, klass)
-      define_method("create_#{singular_name}") do |create_attributes|
-        @object_handler.api_connector.handler(Object.const_get(klass)).create(self, create_attributes)
+      def has_many(plural_name, singular_name, klass, define_create_method = false)
+        HasManyRelations[name] ||= {}
+        HasManyRelations[name][plural_name.to_s] = klass
+        define_has_many_methods(plural_name, klass)
+        define_create_method(singular_name, klass) if define_create_method
       end
-    end
 
-    def set_relation_ids(relations_attributes)
-      HasOneRelations[self.class.name] ||= {}
+      def define_has_many_methods(plural_name, klass)
+        define_method(plural_name.to_s) do |params = {}|
+          @object_handler.api_connector.handler(
+            Object.const_get(klass)
+          ).index(self.class, @id, params)
+        end
 
-      return unless relations_attributes
-      self.class::HasOneRelations[self.class.name].keys.each do |relation_name|
-        val = relations_attributes["#{relation_name}_id"]
-        if val
-          send("#{relation_name}_id=", val)
+        define_method("#{plural_name}_total_count") do
+          @object_handler.api_connector.handler(
+            Object.const_get(klass)
+          ).index_total_count(self)
+        end
+
+        define_method("all_#{plural_name}") do |params = {}, &block|
+          @object_handler.api_connector.handler(
+            Object.const_get(klass)
+          ).index_block(self.class, @id, params, &block)
+        end
+      end
+
+      def define_create_method(singular_name, klass)
+        define_method("create_#{singular_name}") do |create_attributes|
+          @object_handler.api_connector.handler(Object.const_get(klass)).create(self, create_attributes)
         end
       end
     end

data/lib/plate_api/post_multipart_request.rb

@@ -1,11 +1,8 @@
-require 'mimemagic'
-
 module PlateApi
   class PostMultipartRequest < Request
     HttpAdapter = :net_http
-    MimeMagic.add('image/jpeg', extensions: "jfif")
 
-    def initialize(public_key, secret, path, parameters={}, custom_server=nil)
+    def initialize(public_key, secret, path, parameters = {}, custom_server = nil)
       super(public_key, secret, "POST", path, custom_server)
 
       @post_parameters = map_parameters(parameters)
@@ -13,19 +10,32 @@ module PlateApi
 
     def extra_builder_options(builder)
       builder.request :multipart
-	    builder.request :url_encoded
+      builder.request :url_encoded
     end
 
     def extra_request_options(request)
       request.body = @post_parameters
     end
 
+    def mime_type(full_path)
+      begin
+        IO.popen(["file", "--brief", "--mime-type", full_path], in: :close, err: :close) { |io| io.read.chomp }
+      rescue SystemCallError
+        # determine mime_type based on extension as a fallback, in case `file` is not installed on the client machine
+        mime_type_fallback(full_path)
+      end
+    end
+
+    def mime_type_fallback(full_path)
+      MIME::Types.type_for(full_path).first.content_type
+    end
+
     def map_parameters(parameters)
       parameters.keys.each do |key|
         val = parameters[key]
         if val.is_a? File
           full_path = File.expand_path(val)
-          mime_type = MimeMagic.by_path(full_path).type
+          mime_type = mime_type(full_path)
           parameters[key] = Faraday::UploadIO.new(full_path, mime_type)
         end
       end

data/lib/plate_api/request.rb

@@ -1,22 +1,22 @@
 require "faraday"
 # require 'faraday_middleware'
-require 'time'
+require "time"
 require "base64"
 require "openssl"
 require "json"
-
+require "mime-types"
 
 module PlateApi
   class Request
     DefaultApiBaseEndpoint = "https://www.startwithplate.com/api/v2"
     HttpAdapter = Faraday.default_adapter
 
-    def initialize(public_key, secret, method, path, custom_server=nil)
+    def initialize(public_key, secret, method, path, custom_server = nil)
       base_api_endpoint = custom_server ? custom_server : DefaultApiBaseEndpoint
 
-      @connection = ::Faraday.new(url: base_api_endpoint) do |faraday|
+      @connection = ::Faraday.new(url: base_api_endpoint, request: { timeout: 600 }) do |faraday|
         extra_builder_options(faraday)
-        faraday.adapter     HttpAdapter
+        faraday.adapter HttpAdapter
       end
 
       @public_key = public_key
@@ -25,20 +25,20 @@ module PlateApi
       @path = strip_path(path)
     end
 
-    def execute(response_type=:raw)
+    def execute(response_type = :raw)
       response = @connection.send(@method.downcase) do |request|
         request.url url_path
-        request.headers['Date'] = request_date
-        request.headers['Authorization'] = calculate_signature
+        request.headers["Date"] = request_date
+        request.headers["Authorization"] = calculate_signature
         extra_request_options(request)
       end
 
       return case response_type
-      when :raw
-        return response.body
-      when :json
-        return JSON.parse(response.body)
-      end
+             when :raw
+               return response.body
+             when :json
+               return JSON.parse(response.body)
+             end
     end
 
     def request_date
@@ -72,7 +72,7 @@ module PlateApi
     end
 
     def strip_path(path)
-      path.gsub(/^\/|\/$/, '')
+      path.gsub(/^\/|\/$/, "")
     end
   end
 end

data/lib/plate_api/response_error.rb

@@ -0,0 +1,5 @@
+module PlateApi
+  class ResponseError < StandardError
+
+  end
+end

data/lib/plate_api/version.rb

@@ -1,3 +1,3 @@
 module PlateApi
-  VERSION = "1.2.0"
+  VERSION = "1.2.6"
 end

data/lib/plate_api.rb

@@ -9,7 +9,7 @@ require "plate_api/delete_request"
 require "plate_api/post_request"
 require "plate_api/post_multipart_request"
 require "plate_api/put_request"
-require "plate_api/error"
+require "plate_api/response_error"
 
 
 module PlateApi

data/plate_api.gemspec

@@ -1,20 +1,19 @@
-
 lib = File.expand_path("../lib", __FILE__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 require "plate_api/version"
 
 Gem::Specification.new do |spec|
-  spec.name        = 'plate_api'
-  spec.version     = PlateApi::VERSION
-  spec.date        = '2019-01-05'
-  spec.summary     = "Connector for the Plate API"
+  spec.name = "plate_api"
+  spec.version = PlateApi::VERSION
+  spec.date = "2019-01-05"
+  spec.summary = "Connector for the Plate API"
   spec.description = "This gem can be used to connect to the Plate API. It takes care 
   of the authentication procedure. It also provides a basic wrapper around the Plate API,
   so objects in Plate can be manipulated as local objects."
-  spec.authors     = ["David Kortleven", "Harmen Fuite"]
-  spec.email       = ['david@getplate.com', 'harmen@getplate.com']
-  spec.files       = ["lib/plate_api.rb"]
-  spec.license       = 'MIT'
+  spec.authors = ["David Kortleven", "Harmen Fuite"]
+  spec.email = ["david@getplate.com", "harmen@getplate.com"]
+  spec.files = ["lib/plate_api.rb"]
+  spec.license = "MIT"
 
   # Prevent pushing this gem to RubyGems.org. To allow pushes either set the 'allowed_push_host'
   # to allow pushing to a single host or delete this section to allow pushing to any host.
@@ -22,19 +21,20 @@ Gem::Specification.new do |spec|
     spec.metadata["allowed_push_host"] = "https://rubygems.org"
   else
     raise "RubyGems 2.0 or newer is required to protect against " \
-      "public gem pushes."
+          "public gem pushes."
   end
 
-  spec.files         = `git ls-files -z`.split("\x0").reject do |f|
+  spec.files = `git ls-files -z`.split("\x0").reject do |f|
     f.match(%r{^(test|spec|features)/})
   end
-  spec.bindir        = "exe"
-  spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
+  spec.bindir = "exe"
+  spec.executables = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
   spec.require_paths = ["lib"]
 
   spec.add_dependency "faraday", "~> 1.0.1"
+  spec.add_dependency "mime-types", "~> 3.3.1"
   # spec.add_dependency "faraday_middleware", "~> 0.13.1"
-  spec.add_dependency "mimemagic", "~> 0.3.3"
+  # spec.add_dependency "mimemagic", "~> 0.3.10"
 
   spec.add_development_dependency "bundler", "~> 2.1.4"
   spec.add_development_dependency "rake", "~> 13.0"
@@ -44,5 +44,4 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency "simplecov"
   spec.add_development_dependency "simplecov-console"
   spec.add_development_dependency "byebug"
-
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: plate_api
 version: !ruby/object:Gem::Version
-  version: 1.2.0
+  version: 1.2.6
 platform: ruby
 authors:
 - David Kortleven
@@ -26,19 +26,19 @@ dependencies:
       - !ruby/object:Gem::Version
         version: 1.0.1
 - !ruby/object:Gem::Dependency
-  name: mimemagic
+  name: mime-types
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.3.3
+        version: 3.3.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.3.3
+        version: 3.3.1
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -165,6 +165,7 @@ files:
 - ".gitignore"
 - ".rspec"
 - ".travis.yml"
+- Dockerfile
 - Gemfile
 - Gemfile.lock
 - LICENSE.txt
@@ -175,7 +176,6 @@ files:
 - lib/plate_api.rb
 - lib/plate_api/connector.rb
 - lib/plate_api/delete_request.rb
-- lib/plate_api/error.rb
 - lib/plate_api/get_request.rb
 - lib/plate_api/object_handler.rb
 - lib/plate_api/plate_object/attachment.rb
@@ -196,6 +196,7 @@ files:
 - lib/plate_api/post_request.rb
 - lib/plate_api/put_request.rb
 - lib/plate_api/request.rb
+- lib/plate_api/response_error.rb
 - lib/plate_api/version.rb
 - plate_api.gemspec
 homepage: 

data/lib/plate_api/error.rb

@@ -1,5 +0,0 @@
-module Error
-  class Response < StandardError
-
-  end
-end