ticketevolution-ruby 0.7.1 → 0.7.2

data/README.markdown

@@ -260,7 +260,7 @@ Click on the links next to each endpoint for more detail.
     @venue = @connection.venues.show(id)
 
 
-######ticketevolution-ruby v0.7.1
+######ticketevolution-ruby v0.7.2
 
 License
 -------

data/lib/ext/faraday/utils.rb

@@ -0,0 +1,15 @@
+module Faraday
+  module Utils
+    # Override Faraday's version so it sorts the params
+    alias :faradays_build_nested_query :build_nested_query
+    def build_nested_query(value, prefix = nil)
+      if value.is_a? Hash
+        value.to_ordered_hash.map { |k, v|
+          build_nested_query(v, prefix ? "#{prefix}%5B#{escape(k)}%5D" : escape(k))
+        }.join("&")
+      else
+        faradays_build_nested_query(value, prefix)
+      end
+    end
+  end
+end

data/lib/ext/hash.rb

@@ -0,0 +1,9 @@
+class Hash
+  def to_ordered_hash
+    ActiveSupport::OrderedHash.new.tap do |hash|
+      self.sort_by{|k,v| k.to_s}.each do |k, v|
+        hash[k] = v
+      end
+    end
+  end
+end

data/lib/ticket_evolution.rb

@@ -17,20 +17,12 @@ require 'active_support/core_ext/hash'
 require 'active_support/core_ext/module'
 require 'active_support/core_ext/object'
 require 'active_support/inflector'
+require 'active_support/ordered_hash'
 
-require 'faraday/response/verbose_logger'
-
-if ActiveSupport::VERSION::STRING < "3.1.0"
-  class Hash
-    def to_param(namespace = nil)
-      collect do |key, value|
-        value.to_query(namespace ? "#{namespace}[#{key}]" : key)
-      end.sort * '&'
-    end
+require 'ext/hash'
+require 'ext/faraday/utils'
 
-    alias :to_query :to_param
-  end
-end
+require 'faraday/response/verbose_logger'
 
 module TicketEvolution
   mattr_reader :root

data/lib/ticket_evolution/core/collection.rb

@@ -1,6 +1,6 @@
 module TicketEvolution
   class Collection
-    attr_accessor :total_entries, :per_page, :current_page, :entries
+    attr_accessor :total_entries, :per_page, :current_page, :entries, :code
 
     include Enumerable
 
@@ -15,6 +15,7 @@ module TicketEvolution
     def self.build_from_response(response, entries_key, singular_class)
       entries = response.body[entries_key] || []
       new(
+        :code => response.response_code,
         :total_entries => response.body['total_entries'],
         :per_page => response.body['per_page'],
         :current_page => response.body['current_page'],

data/lib/ticket_evolution/core/connection.rb

@@ -101,7 +101,7 @@ module TicketEvolution
       "#{uri}?#{if params.present?
         case method
         when :GET
-          params.to_query
+          Faraday::Utils.build_nested_query(params)
         else
           post_body(params)
         end

data/lib/ticket_evolution/core/endpoint/request_handler.rb

@@ -19,6 +19,7 @@ module TicketEvolution
       }
 
       def request(method, path, params = nil, &response_handler)
+        params = params.to_ordered_hash if params.is_a?(Hash)
         redirecting = caller.first =~ /request_handler/ ? false : true
         request = self.build_request(method, path, params, redirecting)
 

data/lib/ticket_evolution/version.rb

@@ -1,3 +1,3 @@
 module TicketEvolution
-  VERSION = '0.7.1'
+  VERSION = '0.7.2'
 end

data/spec/fixtures/net/endpoints/clients/update_fail.yml

@@ -956,3 +956,47 @@
       - nginx/1.0.11 + Phusion Passenger 3.0.11 (mod_rails/mod_rack)
     body: ! '{"updated_at":"2012-02-22T17:07:30Z","addresses":[],"email_addresses":[],"phone_numbers":[],"name":"foo","url":"/clients/2405","id":"2405"}'
     http_version: '1.1'
+- !ruby/struct:VCR::HTTPInteraction
+  request: !ruby/struct:VCR::Request
+    method: :put
+    uri: https://api.sandbox.ticketevolution.com:443/clients/2405
+    body: ! '{"addresses":[],"email_addresses":[],"name":"","phone_numbers":[],"server_message":"Generally
+      returned by successful POST requests. ","status_code":201,"updated_at":"2012-02-22T17:07:30Z","url":"/clients/2405"}'
+    headers:
+      x-signature:
+      - Xf/nsdiHiYbpLeGbgDDsOBfXJpGtlinClG/MoyzgkdI=
+      x-token:
+      - b2b5a7a33b1a78896ed1b53d81c5c9cc
+      accept:
+      - application/vnd.ticketevolution.api+json; version=8
+  response: !ruby/struct:VCR::Response
+    status: !ruby/struct:VCR::ResponseStatus
+      code: 422
+      message: Unprocessable Entity
+    headers:
+      content-type:
+      - application/vnd.ticketevolution.api+json; version=8; charset=utf-8
+      transfer-encoding:
+      - chunked
+      connection:
+      - keep-alive
+      status:
+      - '422'
+      x-powered-by:
+      - Phusion Passenger (mod_rails/mod_rack) 2.2.10
+      x-runtime:
+      - '0.028101'
+      x-ua-compatible:
+      - IE=Edge,chrome=1
+      strict-transport-security:
+      - max-age=31536000
+      cache-control:
+      - no-cache
+      server:
+      - nginx/0.7.65 + Phusion Passenger 2.2.10 (mod_rails/mod_rack)
+    body: ! '{"updated_at":"2012-02-22T17:07:30Z","phone_numbers":[],"url":"/clients/2405","errors":{"name":["can''t
+      be blank"]},"addresses":[{"postal_code":"77338","street_address":"21338 maple
+      harvest lane","extended_address":null,"locality":"Humble","latitude":null,"country_code":null,"region":"Tx","name":"Martil
+      martinez","longitude":null},{"postal_code":"77338","street_address":"21338 maple
+      harvest lane","extended_address":null,"locality":"Humble","latitude":null,"country_code":null,"region":"Tx","name":null,"longitude":null}],"name":null,"id":"2405","email_addresses":[]}'
+    http_version: '1.1'

data/spec/fixtures/net/endpoints/clients/update_success.yml

@@ -818,3 +818,53 @@
       - nginx/1.0.11 + Phusion Passenger 3.0.11 (mod_rails/mod_rack)
     body: ! '{"updated_at":"2012-02-22T17:07:29Z","addresses":[],"email_addresses":[],"phone_numbers":[],"name":"bar","url":"/clients/2404","id":"2404"}'
     http_version: '1.1'
+- !ruby/struct:VCR::HTTPInteraction
+  request: !ruby/struct:VCR::Request
+    method: :put
+    uri: https://api.sandbox.ticketevolution.com:443/clients/2404
+    body: ! '{"addresses":[],"email_addresses":[],"name":"bar","phone_numbers":[],"server_message":"Generally
+      returned by successful POST requests. ","status_code":201,"updated_at":"2012-02-22T17:07:29Z","url":"/clients/2404"}'
+    headers:
+      x-signature:
+      - 6qfTnv9NQPgeC4ZDAdzfRiTlYz0/VMBTQgX8SNPlYD0=
+      x-token:
+      - b2b5a7a33b1a78896ed1b53d81c5c9cc
+      accept:
+      - application/vnd.ticketevolution.api+json; version=8
+  response: !ruby/struct:VCR::Response
+    status: !ruby/struct:VCR::ResponseStatus
+      code: 200
+      message: OK
+    headers:
+      content-type:
+      - application/vnd.ticketevolution.api+json; version=8; charset=utf-8
+      transfer-encoding:
+      - chunked
+      connection:
+      - keep-alive
+      status:
+      - '200'
+      x-powered-by:
+      - Phusion Passenger (mod_rails/mod_rack) 2.2.10
+      etag:
+      - ! '"b58896cd9a50041a033626f2539aec18"'
+      x-runtime:
+      - '0.070331'
+      x-ua-compatible:
+      - IE=Edge,chrome=1
+      strict-transport-security:
+      - max-age=31536000
+      cache-control:
+      - max-age=0, private, must-revalidate
+      server:
+      - nginx/0.7.65 + Phusion Passenger 2.2.10 (mod_rails/mod_rack)
+    body: ! '{"updated_at":"2012-02-22T17:07:29Z","phone_numbers":[],"url":"/clients/2404","addresses":[{"postal_code":"92154","street_address":"1230
+      Ilexey Ave","extended_address":null,"locality":"San Diego","latitude":null,"country_code":null,"region":"CA","name":"Egbert
+      Purog","longitude":null},{"postal_code":"92154","street_address":"1230 Ilexey
+      Ave","extended_address":null,"locality":"San Diego","latitude":null,"country_code":null,"region":"CA","name":"E
+      Patrick Purog","longitude":null},{"postal_code":"92154","street_address":"1230
+      Ilexey Ave","extended_address":null,"locality":"San Diego","latitude":null,"country_code":null,"region":"CA","name":"Egbert
+      Purog","longitude":null},{"postal_code":"92154","street_address":"1230 Ilexey
+      Ave","extended_address":null,"locality":"San Diego","latitude":null,"country_code":null,"region":"CA","name":"E
+      Patrick Purog","longitude":null}],"name":"bar","id":"2404","email_addresses":[]}'
+    http_version: '1.1'

data/spec/lib/ext/hash_spec.rb

@@ -0,0 +1,28 @@
+require 'spec_helper'
+
+describe 'Hash' do
+  let(:hash) { { :one => 1, :two => 2, :three => 3 } }
+  let(:ordered_hash) { hash.to_ordered_hash }
+
+  it "should respond to #to_ordered_hash" do
+    hash.should respond_to :to_ordered_hash
+  end
+
+  describe "#to_ordered_hash" do
+    it "should return an instance of ActiveSupport::OrderedHash" do
+      ordered_hash.should be_an ActiveSupport::OrderedHash
+    end
+
+    it "should contain the same key value pairs as the original" do
+      hash.keys.should =~ ordered_hash.keys
+      hash.each do |k, v|
+        ordered_hash[k].should == v
+      end
+    end
+
+    it "should order it's keys alphabetically" do
+      keys = ordered_hash.keys.clone
+      keys.sort_by{|x|x.to_s}.should == ordered_hash.keys
+    end
+  end
+end

data/spec/lib/ticket_evolution/core/collection_spec.rb

@@ -10,6 +10,7 @@ describe TicketEvolution::Collection do
     its(:per_page) { should == 2 }
     its(:total_entries) { should == 1379 }
     its(:current_page) { should == 1 }
+    its(:code) { should == 200 }
   end
 
   context "#each" do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: ticketevolution-ruby
 version: !ruby/object:Gem::Version
-  version: 0.7.1
+  version: 0.7.2
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-02-23 00:00:00.000000000Z
+date: 2012-02-24 00:00:00.000000000Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
-  requirement: &70181057522720 !ruby/object:Gem::Requirement
+  requirement: &70213479646800 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,10 +21,10 @@ dependencies:
         version: 3.0.0
   type: :runtime
   prerelease: false
-  version_requirements: *70181057522720
+  version_requirements: *70213479646800
 - !ruby/object:Gem::Dependency
   name: faraday
-  requirement: &70181057521780 !ruby/object:Gem::Requirement
+  requirement: &70213480049520 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -32,10 +32,10 @@ dependencies:
         version: 0.7.3
   type: :runtime
   prerelease: false
-  version_requirements: *70181057521780
+  version_requirements: *70213480049520
 - !ruby/object:Gem::Dependency
   name: yajl-ruby
-  requirement: &70181057540900 !ruby/object:Gem::Requirement
+  requirement: &70213480475740 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -43,10 +43,10 @@ dependencies:
         version: 0.7.7
   type: :runtime
   prerelease: false
-  version_requirements: *70181057540900
+  version_requirements: *70213480475740
 - !ruby/object:Gem::Dependency
   name: multi_json
-  requirement: &70181057545340 !ruby/object:Gem::Requirement
+  requirement: &70213480492200 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -54,10 +54,10 @@ dependencies:
         version: 0.0.4
   type: :runtime
   prerelease: false
-  version_requirements: *70181057545340
+  version_requirements: *70213480492200
 - !ruby/object:Gem::Dependency
   name: nokogiri
-  requirement: &70181057549480 !ruby/object:Gem::Requirement
+  requirement: &70213481183520 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -65,10 +65,10 @@ dependencies:
         version: 1.4.3
   type: :runtime
   prerelease: false
-  version_requirements: *70181057549480
+  version_requirements: *70213481183520
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &70181057553060 !ruby/object:Gem::Requirement
+  requirement: &70213481182940 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -76,10 +76,10 @@ dependencies:
         version: 2.7.1
   type: :development
   prerelease: false
-  version_requirements: *70181057553060
+  version_requirements: *70213481182940
 - !ruby/object:Gem::Dependency
   name: vcr
-  requirement: &70181057556000 !ruby/object:Gem::Requirement
+  requirement: &70213481182200 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -87,10 +87,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70181057556000
+  version_requirements: *70213481182200
 - !ruby/object:Gem::Dependency
   name: webmock
-  requirement: &70181061812160 !ruby/object:Gem::Requirement
+  requirement: &70213481181120 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -101,10 +101,10 @@ dependencies:
         version: 1.8.0
   type: :development
   prerelease: false
-  version_requirements: *70181061812160
+  version_requirements: *70213481181120
 - !ruby/object:Gem::Dependency
   name: awesome_print
-  requirement: &70181062357560 !ruby/object:Gem::Requirement
+  requirement: &70213481179880 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -112,10 +112,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70181062357560
+  version_requirements: *70213481179880
 - !ruby/object:Gem::Dependency
   name: rake
-  requirement: &70181062709940 !ruby/object:Gem::Requirement
+  requirement: &70213481179020 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -123,7 +123,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70181062709940
+  version_requirements: *70213481179020
 description: Provides Ruby wrappers for the Ticket Evolution API (http://developer.ticketevolution.com).
   Ticket Evolution is the industry leader in software for the Ticket Broker industry.
 email:
@@ -147,6 +147,8 @@ files:
 - lib/docs/introduction.markdown
 - lib/docs/license.markdown
 - lib/docs/objects.markdown
+- lib/ext/faraday/utils.rb
+- lib/ext/hash.rb
 - lib/faraday/response/verbose_logger.rb
 - lib/ticket_evolution.rb
 - lib/ticket_evolution/account.rb
@@ -222,6 +224,7 @@ files:
 - spec/fixtures/net/endpoints/clients/update_fail.yml
 - spec/fixtures/net/endpoints/clients/update_success.yml
 - spec/fixtures/net/endpoints/search.yml
+- spec/lib/ext/hash_spec.rb
 - spec/lib/ticket_evolution/account_spec.rb
 - spec/lib/ticket_evolution/accounts_spec.rb
 - spec/lib/ticket_evolution/address_spec.rb
@@ -302,7 +305,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 3800414316783008053
+      hash: -2898500470310488560
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -326,6 +329,7 @@ test_files:
 - spec/fixtures/net/endpoints/clients/update_fail.yml
 - spec/fixtures/net/endpoints/clients/update_success.yml
 - spec/fixtures/net/endpoints/search.yml
+- spec/lib/ext/hash_spec.rb
 - spec/lib/ticket_evolution/account_spec.rb
 - spec/lib/ticket_evolution/accounts_spec.rb
 - spec/lib/ticket_evolution/address_spec.rb