sports_south 0.15.0 → 1.0.0.pre

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: a79344ff1ad7199740dfb64fdf7b4bcf926b6221
-  data.tar.gz: 82f7781c63d7ce9e7a7deb4f1a4b225a0928eb4c
+  metadata.gz: 9e9ce1ac3fe3b6153a108f6cdf06a9d775a1ed5d
+  data.tar.gz: c1733e62462b1ce5c578c20a67cdbf0f6c8529ac
 SHA512:
-  metadata.gz: f87efe260a07df9fd5febede463f29b5b3f5f2874afc83512f7aa193ad888c02562c01ae5fc2bc69bc243ce88169053b13b9f18df4e7515be89847596765fb6d
-  data.tar.gz: 39ea202d9c745616f2a0b94dc0d615c11f4d2dd4da6e7e15d390cb055ba69118c98362cf4d1c826e70924c6ec047b7491eb6dcffff5ff5c35333df340b531d0c
+  metadata.gz: 71ff858afb5298a31484104fdd19c16ce993871accc302a077d9da794e9e14dafa2377b6aec2db278f9b0706db38f7f2c3209a6e197abe1333486aac81c6e810
+  data.tar.gz: 947dbd5e095c4609922fec14ba98a902ccb8cdbdcb1b4131f4015ede8146a671355f804b4d7b4ea5b608780bbd618302c04117796a5be6a5ee96539a9326a3ef

data/.gitignore

@@ -7,3 +7,4 @@
 /pkg/
 /spec/reports/
 /tmp/
+script/*

data/.ruby-version

@@ -1 +1 @@
-2.2.0
+2.3.1

data/lib/sports_south/base.rb

@@ -4,6 +4,7 @@ module SportsSouth
 
     TIMEOUT = 960 # seconds
     USER_AGENT = "sports_south rubygems.org/gems/sports_south v(#{SportsSouth::VERSION})"
+    CONTENT_TYPE = 'application/x-www-form-urlencoded'.freeze
 
     protected
 
@@ -44,7 +45,8 @@ module SportsSouth
       http = Net::HTTP.new(uri.host, uri.port)
       http.read_timeout = TIMEOUT
       request = Net::HTTP::Post.new(uri.request_uri)
-      request["User-Agent"] = USER_AGENT
+      request['User-Agent'] = USER_AGENT
+      request['Content-Type'] = CONTENT_TYPE
 
       return http, request
     end

data/lib/sports_south/brand.rb

@@ -7,26 +7,25 @@ module SportsSouth
       requires!(options, :username, :password, :source, :customer_number)
 
       http, request = get_http_and_request(API_URL, '/BrandUpdate')
-
       request.set_form_data(form_params(options))
+
       response = http.request(request)
-      body = sanitize_response(response)
-      xml_doc = Nokogiri::XML(body)
+      xml_doc  = Nokogiri::XML(sanitize_response(response))
 
       raise SportsSouth::NotAuthenticated if not_authenticated?(xml_doc)
 
-      brands = []
+      xml_doc.css('Table').map { |brand| map_hash(brand) }
+    end
 
-      xml_doc.css('Table').each do |brand|
-        brands << {
-          id: content_for(brand, 'BRDNO'),
-          name: content_for(brand, 'BRDNM'),
-          url: content_for(brand, 'BRDURL'),
-          item_count: content_for(brand, 'ITCOUNT'),
-        }
-      end
+    protected
 
-      brands
+    def self.map_hash(node)
+      {
+        id:         content_for(node, 'BRDNO'),
+        name:       content_for(node, 'BRDNM'),
+        url:        content_for(node, 'BRDURL'),
+        item_count: content_for(node, 'ITCOUNT')
+      }
     end
 
   end

data/lib/sports_south/category.rb

@@ -1,52 +1,51 @@
 module SportsSouth
   class Category < Base
+
     API_URL = 'http://webservices.theshootingwarehouse.com/smart/inventory.asmx'
 
     def self.all(options = {})
       requires!(options, :username, :password, :source, :customer_number)
 
       http, request = get_http_and_request(API_URL, '/CategoryUpdate')
-
       request.set_form_data(form_params(options))
 
       response = http.request(request)
-      body = sanitize_response(response)
-      xml_doc = Nokogiri::XML(body)
+      xml_doc  = Nokogiri::XML(sanitize_response(response))
 
       raise SportsSouth::NotAuthenticated if not_authenticated?(xml_doc)
 
-      categories = []
-
-      xml_doc.css('Table').each do |category|
-        categories << {
-          category_id:  content_for(category, 'CATID'),
-          description: content_for(category, 'CATDES'),
-          department_id: content_for(category, 'DEPID'),
-          department_description: content_for(category, 'DEP'),
-          attribute_1: content_for(category, 'ATTR1'),
-          attribute_2: content_for(category, 'ATTR2'),
-          attribute_3: content_for(category, 'ATTR3'),
-          attribute_4: content_for(category, 'ATTR4'),
-          attribute_5: content_for(category, 'ATTR5'),
-          attribute_6: content_for(category, 'ATTR6'),
-          attribute_7: content_for(category, 'ATTR7'),
-          attribute_8: content_for(category, 'ATTR8'),
-          attribute_9: content_for(category, 'ATTR9'),
-          attribute_10: content_for(category, 'ATTR0'),
-          attribute_11: content_for(category, 'ATTR11'),
-          attribute_12: content_for(category, 'ATTR12'),
-          attribute_13: content_for(category, 'ATTR13'),
-          attribute_14: content_for(category, 'ATTR14'),
-          attribute_15: content_for(category, 'ATTR15'),
-          attribute_16: content_for(category, 'ATTR16'),
-          attribute_17: content_for(category, 'ATTR17'),
-          attribute_18: content_for(category, 'ATTR18'),
-          attribute_19: content_for(category, 'ATTR19'),
-          attribute_20: content_for(category, 'ATTR20'),
-        }
-      end
-
-      categories
+      xml_doc.css('Table').map { |category| map_hash(category) }
+    end
+
+    protected
+
+    def self.map_hash(node)
+      {
+        category_id:            content_for(node, 'CATID'),
+        description:            content_for(node, 'CATDES'),
+        department_id:          content_for(node, 'DEPID'),
+        department_description: content_for(node, 'DEP'),
+        attribute_1:            content_for(node, 'ATTR1'),
+        attribute_2:            content_for(node, 'ATTR2'),
+        attribute_3:            content_for(node, 'ATTR3'),
+        attribute_4:            content_for(node, 'ATTR4'),
+        attribute_5:            content_for(node, 'ATTR5'),
+        attribute_6:            content_for(node, 'ATTR6'),
+        attribute_7:            content_for(node, 'ATTR7'),
+        attribute_8:            content_for(node, 'ATTR8'),
+        attribute_9:            content_for(node, 'ATTR9'),
+        attribute_10:           content_for(node, 'ATTR0'),
+        attribute_11:           content_for(node, 'ATTR11'),
+        attribute_12:           content_for(node, 'ATTR12'),
+        attribute_13:           content_for(node, 'ATTR13'),
+        attribute_14:           content_for(node, 'ATTR14'),
+        attribute_15:           content_for(node, 'ATTR15'),
+        attribute_16:           content_for(node, 'ATTR16'),
+        attribute_17:           content_for(node, 'ATTR17'),
+        attribute_18:           content_for(node, 'ATTR18'),
+        attribute_19:           content_for(node, 'ATTR19'),
+        attribute_20:           content_for(node, 'ATTR20')
+      }
     end
   end
 end

data/lib/sports_south/ffl.rb

@@ -13,8 +13,7 @@ module SportsSouth
       request.set_form_data(form_params(options).merge({ FFL: ffl }))
 
       response = http.request(request)
-      body = sanitize_response(response)
-      xml_doc = Nokogiri::XML(body)
+      xml_doc  = Nokogiri::XML(sanitize_response(response))
 
       raise SportsSouth::NotAuthenticated if not_authenticated?(xml_doc)
 

data/lib/sports_south/image.rb

@@ -13,12 +13,11 @@ module SportsSouth
       }))
 
       response = http.request(request)
-      body = sanitize_response(response)
-      xml_doc = Nokogiri::XML(body)
+      xml_doc  = Nokogiri::XML(sanitize_response(response))
 
       raise SportsSouth::NotAuthenticated if not_authenticated?(xml_doc)
 
-      images = {}
+      images = Hash.new
 
       xml_doc.css('Table').each do |image|
         size = content_for(image, 'ImageSize').to_sym

data/lib/sports_south/inventory.rb

@@ -24,83 +24,21 @@ module SportsSouth
     def self.all(options = {})
       requires!(options, :username, :password, :source, :customer_number)
 
-      options[:last_update] ||= '1/1/1990'  # Return full catalog.
-      options[:last_item]   ||= '-1'  # Return all items.
+      options[:last_update] ||= '1/1/1990' # Return full catalog.
+      options[:last_item]   ||= '-1'       # Return all items.
 
       http, request = get_http_and_request(API_URL, '/DailyItemUpdate')
 
       request.set_form_data(form_params(options).merge({
         LastUpdate: options[:last_update],
-        LastItem: options[:last_item],
+        LastItem:   options[:last_item].to_s,
       }))
 
-      response = http.request(request)
-      body = sanitize_response(response)
-      xml_doc = Nokogiri::XML(body)
+      xml_doc  = Nokogiri::XML(sanitize_response(http.request(request)))
 
       raise SportsSouth::NotAuthenticated if not_authenticated?(xml_doc)
 
-      items = []
-
-      xml_doc.css('Table').each do |item|
-        items << {
-          item_number: content_for(item, 'ITEMNO'),
-          description: content_for(item, 'IDESC'),
-          manufacturer_sequence: content_for(item, 'IMFSEQ'),
-          manufacturer_number: content_for(item, 'IMFGNO'),
-          catalog_sequence: content_for(item, 'CSEQ'),
-          item_type: ITEM_TYPES[content_for(item, 'ITYPE')],
-          short_description: content_for(item, 'SHDESC'),
-          unit_of_measure: content_for(item, 'UOM'),
-          catalog_price: content_for(item, 'PRC1'),
-          customer_price: content_for(item, 'CPRC'),
-          quantity_on_hand: content_for(item, 'QTYOH'),
-          weight_per_box: content_for(item, 'WTPBX'),
-          upc: content_for(item, 'ITUPC'),
-          manufacturer_item_number: content_for(item, 'MFGINO'),
-          scan_name_1: content_for(item, 'SCNAM1'),
-          scan_name_2: content_for(item, 'SCNAM2'),
-          catalog_code: CATALOG_CODES[content_for(item, 'CATCD')],
-          mapp_price_code: content_for(item, 'MFPRTYP'),
-          mapp_price: content_for(item, 'MFPRC'),
-          category_id: content_for(item, 'CATID'),
-          text_reference_number: content_for(item, 'TXTREF'),
-          picture_reference_number: content_for(item, 'PICREF'),
-          brand_id: content_for(item, 'ITBRDNO'),
-          item_model_number: content_for(item, 'IMODEL'),
-          item_purpose: content_for(item, 'IPURPOSE'),
-          series_description: content_for(item, 'SERIES'),
-          item_length: content_for(item, 'LENGTH'),
-          item_height: content_for(item, 'HEIGHT'),
-          item_width: content_for(item, 'WIDTH'),
-          item_ships_hazmat_air: content_for(item, 'HAZAIR'),
-          item_ships_hazmat_ground: content_for(item, 'HAZGRND'),
-          date_of_last_change: content_for(item, 'CHGDTE'),
-          date_added: content_for(item, 'CHGDTE'),
-          attribute_1: content_for(item, 'ITATR1'),
-          attribute_2: content_for(item, 'ITATR2'),
-          attribute_3: content_for(item, 'ITATR3'),
-          attribute_4: content_for(item, 'ITATR4'),
-          attribute_5: content_for(item, 'ITATR5'),
-          attribute_6: content_for(item, 'ITATR6'),
-          attribute_7: content_for(item, 'ITATR7'),
-          attribute_8: content_for(item, 'ITATR8'),
-          attribute_9: content_for(item, 'ITATR9'),
-          attribute_10: content_for(item, 'ITATR0'),
-          attribute_11: content_for(item, 'ITATR11'),
-          attribute_12: content_for(item, 'ITATR12'),
-          attribute_13: content_for(item, 'ITATR13'),
-          attribute_14: content_for(item, 'ITATR14'),
-          attribute_15: content_for(item, 'ITATR15'),
-          attribute_16: content_for(item, 'ITATR16'),
-          attribute_17: content_for(item, 'ITATR17'),
-          attribute_18: content_for(item, 'ITATR18'),
-          attribute_19: content_for(item, 'ITATR19'),
-          attribute_20: content_for(item, 'ITATR20'),
-        }
-      end
-
-      items
+      xml_doc.css('Table').map { |item| map_hash(item) }
     end
 
     def self.get_text(item_number, options = {})
@@ -111,12 +49,14 @@ module SportsSouth
       request.set_form_data(form_params(options).merge({ ItemNumber: item_number }))
 
       response = http.request(request)
-      body = sanitize_response(response)
-      xml_doc = Nokogiri::XML(body)
+      xml_doc = Nokogiri::XML(sanitize_response(response))
 
       raise SportsSouth::NotAuthenticated if not_authenticated?(xml_doc)
 
-      { catalog_text: content_for(xml_doc, 'CATALOGTEXT') }
+      {
+        item_number:  item_number,
+        catalog_text: content_for(xml_doc, 'CATALOGTEXT')
+      }
     end
 
     def self.inquiry(item_number, options = {})
@@ -127,16 +67,15 @@ module SportsSouth
       request.set_form_data(form_params(options).merge({ ItemNumber: item_number }))
 
       response = http.request(request)
-      body = sanitize_response(response)
-      xml_doc = Nokogiri::XML(body)
+      xml_doc  = Nokogiri::XML(sanitize_response(response))
 
       raise SportsSouth::NotAuthenticated if not_authenticated?(xml_doc)
 
       {
-        item_number: content_for(xml_doc, 'I'),
+        item_number:      content_for(xml_doc, 'I'),
         quantity_on_hand: content_for(xml_doc, 'Q').to_i,
-        catalog_price: content_for(xml_doc, 'P'),
-        customer_price: content_for(xml_doc, 'C'),
+        catalog_price:    content_for(xml_doc, 'P'),
+        customer_price:   content_for(xml_doc, 'C'),
       }
     end
 
@@ -149,22 +88,17 @@ module SportsSouth
       request.set_form_data(form_params(options).merge({ CSVItems: item_numbers.join(',') }))
 
       response = http.request(request)
-      body = sanitize_response(response)
-      xml_doc = Nokogiri::XML(body)
+      xml_doc  = Nokogiri::XML(sanitize_response(response))
 
       raise SportsSouth::NotAuthenticated if not_authenticated?(xml_doc)
 
-      items = []
-
-      xml_doc.css('Table').each do |item|
-        items << {
+      xml_doc.css('Table').map do |item|
+        {
           item_number: content_for(item, 'I'),
           quantity: content_for(item, 'Q'),
           price: content_for(item, 'P'),
         }
       end
-
-      items
     end
 
     # Pass an optional `:since` option (YYYY-MM-DDTHH:mm:ss.mss-HH:00) to get items updated since that timestamp.
@@ -177,25 +111,19 @@ module SportsSouth
 
       request.set_form_data(form_params(options).merge({ SinceDateTime: options[:since] }))
 
-      response = http.request(request)
-      body = sanitize_response(response)
-      xml_doc = Nokogiri::XML(body)
+      xml_doc = Nokogiri::XML(sanitize_response(http.request(request)))
 
       raise SportsSouth::NotAuthenticated if not_authenticated?(xml_doc)
 
-      items = []
-
-      xml_doc.css('Onhand').each do |item|
-        items << {
-          item_number: content_for(item, 'I'),
-          quantity: content_for(item, 'Q'),
+      xml_doc.css('Onhand').map do |item|
+        {
+          item_number:      content_for(item, 'I'),
+          quantity:         content_for(item, 'Q'),
           quantity_changed: content_for(item, 'D'),
-          catalog_price: content_for(item, 'P'),
-          customer_price: content_for(item, 'C'),
+          catalog_price:    content_for(item, 'P'),
+          customer_price:   content_for(item, 'C'),
         }
       end
-
-      items
     end
 
     def self.onhand_update(options = {})
@@ -206,23 +134,78 @@ module SportsSouth
       request.set_form_data(form_params(options))
 
       response = http.request(request)
-      body = sanitize_response(response)
-      xml_doc = Nokogiri::XML(body)
+      xml_doc  = Nokogiri::XML(sanitize_response(response))
 
       raise SportsSouth::NotAuthenticated if not_authenticated?(xml_doc)
 
-      items = []
-
-      xml_doc.css('Table').each do |item|
-        items << {
-          item_number: content_for(item, 'I'),
-          quantity: content_for(item, 'Q'),
-          catalog_price: content_for(item, 'P'),
+      xml_doc.css('Table').map do |item|
+        {
+          item_number:    content_for(item, 'I'),
+          quantity:       content_for(item, 'Q').to_i,
+          catalog_price:  content_for(item, 'P'),
           customer_price: content_for(item, 'C'),
         }
       end
+    end
+
+    protected
 
-      items
+    def self.map_hash(node)
+      {
+        item_number:              content_for(node, 'ITEMNO'),
+        description:              content_for(node, 'IDESC'),
+        manufacturer_sequence:    content_for(node, 'IMFSEQ'),
+        manufacturer_number:      content_for(node, 'IMFGNO'),
+        catalog_sequence:         content_for(node, 'CSEQ'),
+        item_type:                ITEM_TYPES[content_for(node, 'ITYPE')],
+        short_description:        content_for(node, 'SHDESC'),
+        unit_of_measure:          content_for(node, 'UOM'),
+        catalog_price:            content_for(node, 'PRC1'),
+        customer_price:           content_for(node, 'CPRC'),
+        quantity_on_hand:         content_for(node, 'QTYOH'),
+        weight_per_box:           content_for(node, 'WTPBX'),
+        upc:                      content_for(node, 'ITUPC'),
+        manufacturer_item_number: content_for(node, 'MFGINO'),
+        scan_name_1:              content_for(node, 'SCNAM1'),
+        scan_name_2:              content_for(node, 'SCNAM2'),
+        catalog_code:             CATALOG_CODES[content_for(node, 'CATCD')],
+        mapp_price_code:          content_for(node, 'MFPRTYP'),
+        mapp_price:               content_for(node, 'MFPRC'),
+        category_id:              content_for(node, 'CATID'),
+        text_reference_number:    content_for(node, 'TXTREF'),
+        picture_reference_number: content_for(node, 'PICREF'),
+        brand_id:                 content_for(node, 'ITBRDNO'),
+        item_model_number:        content_for(node, 'IMODEL'),
+        item_purpose:             content_for(node, 'IPURPOSE'),
+        series_description:       content_for(node, 'SERIES'),
+        item_length:              content_for(node, 'LENGTH'),
+        item_height:              content_for(node, 'HEIGHT'),
+        item_width:               content_for(node, 'WIDTH'),
+        item_ships_hazmat_air:    content_for(node, 'HAZAIR'),
+        item_ships_hazmat_ground: content_for(node, 'HAZGRND'),
+        date_of_last_change:      content_for(node, 'CHGDTE'),
+        date_added:               content_for(node, 'CHGDTE'),
+        attribute_1:              content_for(node, 'ITATR1'),
+        attribute_2:              content_for(node, 'ITATR2'),
+        attribute_3:              content_for(node, 'ITATR3'),
+        attribute_4:              content_for(node, 'ITATR4'),
+        attribute_5:              content_for(node, 'ITATR5'),
+        attribute_6:              content_for(node, 'ITATR6'),
+        attribute_7:              content_for(node, 'ITATR7'),
+        attribute_8:              content_for(node, 'ITATR8'),
+        attribute_9:              content_for(node, 'ITATR9'),
+        attribute_10:             content_for(node, 'ITATR0'),
+        attribute_11:             content_for(node, 'ITATR11'),
+        attribute_12:             content_for(node, 'ITATR12'),
+        attribute_13:             content_for(node, 'ITATR13'),
+        attribute_14:             content_for(node, 'ITATR14'),
+        attribute_15:             content_for(node, 'ITATR15'),
+        attribute_16:             content_for(node, 'ITATR16'),
+        attribute_17:             content_for(node, 'ITATR17'),
+        attribute_18:             content_for(node, 'ITATR18'),
+        attribute_19:             content_for(node, 'ITATR19'),
+        attribute_20:             content_for(node, 'ITATR20')
+      }
     end
 
   end

data/lib/sports_south/version.rb

@@ -1,3 +1,3 @@
 module SportsSouth
-  VERSION = "0.15.0"
+  VERSION = '1.0.0.pre'.freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: sports_south
 version: !ruby/object:Gem::Version
-  version: 0.15.0
+  version: 1.0.0.pre
 platform: ruby
 authors:
 - Dale Campbell
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2017-01-13 00:00:00.000000000 Z
+date: 2017-04-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: nokogiri
@@ -140,12 +140,12 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '2.0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">="
+  - - ">"
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 1.3.1
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.5
+rubygems_version: 2.6.7
 signing_key: 
 specification_version: 4
 summary: Sports South API Ruby library.