fb_graph 0.4.1 → 0.4.2

data/VERSION

@@ -1 +1 @@
-0.4.1
+0.4.2

data/fb_graph.gemspec

@@ -5,7 +5,7 @@
 
 Gem::Specification.new do |s|
   s.name = %q{fb_graph}
-  s.version = "0.4.1"
+  s.version = "0.4.2"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["nov matake"]

data/lib/fb_graph/node.rb

@@ -28,26 +28,29 @@ module FbGraph
     protected
 
     def get(params = {})
-      _endpoint_ = build_endpoint(params.merge!(:method => :get))
+      _params_ = stringfy_access_token(params)
+      _endpoint_ = build_endpoint(_params_.merge!(:method => :get))
       handle_response do
         RestClient.get(_endpoint_)
       end
     end
 
     def post(params = {})
-      _endpoint_ = build_endpoint(params.merge!(:method => :post))
+      _params_ = stringfy_access_token(params)
+      _endpoint_ = build_endpoint(_params_.merge!(:method => :post))
       handle_response do
-        RestClient.post(_endpoint_, params)
+        RestClient.post(_endpoint_, _params_)
       end
     end
 
     def delete(params = {})
-      _endpoint_ = build_endpoint(params.merge!(:method => :delete))
+      _params_ = stringfy_access_token(params)
+      _endpoint_ = build_endpoint(_params_.merge!(:method => :delete))
       handle_response do
         # NOTE:
         # DELETE method didn't work for some reason.
         # Use POST with "method=delete" for now.
-        RestClient.post(_endpoint_, params.merge!(:method => :delete))
+        RestClient.post(_endpoint_, _params_.merge!(:method => :delete))
       end
     end
 
@@ -60,11 +63,6 @@ module FbGraph
         self.endpoint
       end
 
-      params[:access_token] ||= self.access_token
-      if params[:access_token].is_a?(OAuth2::AccessToken)
-        params[:access_token] = params[:access_token].token
-      end
-
       params.delete_if do |k, v|
         v.blank?
       end
@@ -74,6 +72,15 @@ module FbGraph
       _endpoint_
     end
 
+    def stringfy_access_token(params)
+      _params_ = params.dup
+      _params_[:access_token] ||= self.access_token
+      if access_token.is_a?(OAuth2::AccessToken)
+        _params_[:access_token] = _params_[:access_token].token
+      end
+      _params_
+    end
+
     def handle_response
       response = yield
       case response.body

data/spec/fb_graph/page_spec.rb

@@ -12,8 +12,4 @@ describe FbGraph::Page, '.fetch' do
     page.name.should       == 'Facebook Platform'
     page.category.should   == 'Technology'
   end
-
-  it 'should not require access_token' do
-    FbGraph::Page.fetch('platform', :access_token => 'access_token').should == FbGraph::Page.fetch('platform')
-  end
 end

metadata

@@ -5,8 +5,8 @@ version: !ruby/object:Gem::Version
   segments: 
   - 0
   - 4
-  - 1
-  version: 0.4.1
+  - 2
+  version: 0.4.2
 platform: ruby
 authors: 
 - nov matake