Skip to content

Commit

Permalink
Changes JSON parsing to use MultiJson
Browse files Browse the repository at this point in the history
  • Loading branch information
vesan committed Apr 29, 2012
1 parent 428ff97 commit b2c56ca
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 5 deletions.
2 changes: 1 addition & 1 deletion kippt.gemspec
Expand Up @@ -17,7 +17,7 @@ Gem::Specification.new do |gem|

gem.add_dependency "faraday", "~> 0.7.6"
gem.add_dependency "faraday_middleware", "~> 0.8.7"
gem.add_dependency "yajl-ruby", "~> 1.1.0"
gem.add_dependency "multi_json", "~> 1.3.4"

gem.add_development_dependency "rspec", "~> 2.9.0"
gem.add_development_dependency "webmock", "~> 1.8.6"
Expand Down
4 changes: 2 additions & 2 deletions lib/kippt/collection_resource.rb
Expand Up @@ -19,9 +19,9 @@ def collection_from_url(url)

def save_resource(object)
if object.id
response = @client.put("#{base_uri}/#{object.id}", writable_parameters_from(object))
response = @client.put("#{base_uri}/#{object.id}", data: writable_parameters_from(object))
else
response = @client.post("#{base_uri}", writable_parameters_from(object))
response = @client.post("#{base_uri}", data: writable_parameters_from(object))
end

save_response = {success: response.success?}
Expand Down
5 changes: 3 additions & 2 deletions lib/kippt/connection.rb
@@ -1,3 +1,5 @@
require "multi_json"

module Kippt::Connection
def get(url, options = {})
request(:get, url, options)
Expand All @@ -19,7 +21,6 @@ def delete(url, options = {})

def connection
@connection ||= Faraday.new("https://kippt.com/api") do |builder|
builder.use Faraday::Request::JSON
builder.use FaradayMiddleware::ParseJson
# builder.use Faraday::Response::Logger
builder.adapter Faraday.default_adapter
Expand All @@ -41,7 +42,7 @@ def request(method, url, options)
req.url url, options
else
req.url url
req.body = options
req.body = MultiJson.dump(options[:data])
end
end

Expand Down

0 comments on commit b2c56ca

Please sign in to comment.