peddler 2.4.1 → 2.4.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 84233828507c69bc5a316cfbe5d6b7fb41382d4b1234ee7e66bf3ef56ee71755
-  data.tar.gz: a071d51905e4b223e4b3a731cd454b867193afe0ea21a9fbb6b4ea959742e14d
+  metadata.gz: f4a66be7a75ab648d2ad1780ee497a77849b7530943834db809822702989a310
+  data.tar.gz: 4fb3d4cc33073309224aa39f8689e5ef101109fb0aec1b2bded2e09d8626b7f4
 SHA512:
-  metadata.gz: 0e24d92c6c255d7145ee8ef95d8be3f1fcde9e22ddea8d4e1510cbef5bb0808d7340d2dff6652f96a3476d7d1c317a8cf1f96069097cf1450687b17ba69adbfc
-  data.tar.gz: be5e4557f6e73bceb02f4f1cae7ddad3a396add90d0269886072c97a58966ff6f795485369423424320f1b2ecc61a0ff2f1569add83c24260b1188b99532ed4a
+  metadata.gz: 2dc3c385ac768a25017570de392fca4000ee21428d0ede8a54c854867a4873001b30cd1d41ab1f01d762e260af31952982d756981a1e3d7dff90ef5f29326daa
+  data.tar.gz: 94d60b85d866ad6825ae6e7f41753b39ad6debe47e020934445698489e6baf4976076ec1548426fae6c9df197923386d56948228c6ea875a2d6df9bbe220cdd8

data/lib/mws/orders/client.rb

@@ -34,9 +34,7 @@ module MWS
       def list_orders(*marketplace_id)
         marketplace_id = marketplace_id.dup
         opts = extract_options(marketplace_id)
-        if opts.key?(:tfm_shipment_status)
-          opts['TFMShipmentStatus'] = opts.delete(:tfm_shipment_status)
-        end
+        opts['TFMShipmentStatus'] = opts.delete(:tfm_shipment_status) if opts.key?(:tfm_shipment_status)
 
         operation('ListOrders')
           .add(opts)

data/lib/mws/subscriptions/client.rb

@@ -139,7 +139,7 @@ module MWS
                               marketplace_id)
         operation('UpdateSubscription')
           .add('MarketplaceId' => marketplace_id)
-          .add(build_subscription(notification_type, sqs_queue_url, enabled))
+          .add(build_subscription(notification_type, sqs_queue_url, enabled: enabled))
 
         run
       end
@@ -163,7 +163,7 @@ module MWS
         }
       end
 
-      def build_subscription(notification_type, sqs_queue_url, enabled = true)
+      def build_subscription(notification_type, sqs_queue_url, enabled: true)
         {
           'Subscription.IsEnabled' => enabled,
           'Subscription.NotificationType' => notification_type,

data/lib/peddler/client.rb

@@ -22,6 +22,7 @@ module Peddler
       private
 
       def inherited(base)
+        super
         base.parser = parser
         base.params params
       end

data/lib/peddler/errors/class_generator.rb

@@ -21,7 +21,7 @@ module Peddler
         with_mutex do
           if Errors.const_defined?(name)
             error = Errors.const_get(name)
-            return error if error.ancestors.include?(Error)
+            return error if error <= Error
 
             raise TypeError, "#{name} must be a Peddler::Errors::Error"
           end
@@ -32,8 +32,8 @@ module Peddler
 
       private
 
-      def with_mutex
-        @mutex.synchronize { yield }
+      def with_mutex(&block)
+        @mutex.synchronize(&block)
       end
     end
   end

data/lib/peddler/headers.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require 'time'
+
 module Peddler
   # Parses useful metadata returned in response headers
   module Headers
@@ -47,7 +49,7 @@ module Peddler
       match_data = headers['Content-Type']&.match(/charset=(.*);?/)
       return unless match_data
 
-      string = match_data[1]
+      string = match_data[1].upcase
       string = 'UTF-8' if string == 'UTF8'
 
       Encoding.find(string)

data/lib/peddler/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Peddler
-  VERSION = '2.4.1'
+  VERSION = '2.4.3'
 end

data/lib/peddler/xml_response_parser.rb

@@ -5,7 +5,7 @@ require 'peddler/xml_parser'
 module Peddler
   # @!visibility private
   class XMLResponseParser < XMLParser
-    MATCHER = /^Message$|Report|Result/.freeze
+    MATCHER = /^Message$|^Node$|Report|Result/.freeze
     private_constant :MATCHER
 
     def next_token
@@ -14,6 +14,7 @@ module Peddler
 
     private
 
+    # FIXME: In next major version bump, stop assuming we know report internals.
     def find_data
       payload = xml.values.first
       found = payload.find { |k, _| k.match(MATCHER) }

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: peddler
 version: !ruby/object:Gem::Version
-  version: 2.4.1
+  version: 2.4.3
 platform: ruby
 authors:
 - Hakan Ensari
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-05-05 00:00:00.000000000 Z
+date: 2020-08-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: excon
@@ -52,7 +52,7 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: 0.5.0
-description: 
+description:
 email:
 - me@hakanensari.com
 executables: []
@@ -111,7 +111,7 @@ homepage: https://github.com/hakanensari/peddler
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -119,7 +119,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '2.4'
+      version: '2.5'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
@@ -127,7 +127,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubygems_version: 3.1.2
-signing_key: 
+signing_key:
 specification_version: 4
 summary: A Ruby wrapper to the Amazon MWS API
 test_files: []