fb_graph 2.7.1 → 2.7.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 7655ead97fca704002f5dac9ef943dbd61e8dde6
-  data.tar.gz: 3166cea76ebd6bc8f439f0ecfe1080f294aa8f14
+  metadata.gz: e660328032ffbc8b0baa0033cf3099345ff336e0
+  data.tar.gz: 167ec5d9906e89ce2ada0e468e600241fb07978b
 SHA512:
-  metadata.gz: e42d8137dd19236c601b4a6cd1c85d185228b5c6cb90712c8dce246da36968410359e357f752aa8702de546197a65f0636e10b6bf059ba730a9834abf5fed926
-  data.tar.gz: 600d5eab48fae82d012047d911374ac8928a343b65777b8f22cabcd09d6749e1131672b6be4c59f9d4e632b39c92151f1af68d38781c08a1da5109ca39f54497
+  metadata.gz: 0fd73b3676e3b3b26c96ab0c1532f5bc41c9735f096991d154fe642ea365731fd942153221cfe555bb1b576673db0100a097b439e90948299d8ceb2b9bff6788
+  data.tar.gz: ed343eab7ee5fd8e682bb90e8ef85b0dee476130f8879cfb8cec76573f684cdefbbbf7afc82d9c3785ac3cffdb47f259b5617f2c0c2c2dc1c84498250472ae3f

data/README.rdoc

@@ -116,6 +116,16 @@ of the specific objects fields in the sidebar under "Objects".
   results.klass.search(results.query, results.collection.next)     # => same with results.next
   results.klass.search(results.query, results.collection.previous) # => same with results.previous
 
+
+==== Comments
+
+  # fetch comment replies
+  FbGraph::Comment.new("10151705618661509_29545180", :access_token => ACCESS_TOKEN).comments
+
+  # fetch if the comment accept replies
+  comment = FbGraph::Comment.fetch("10151705618661509_29545180", :access_token => ACCESS_TOKEN, :fields => "can_comment")
+  comment.can_comment
+
 === POST
 
 ==== Update status (wall post)
@@ -193,6 +203,14 @@ of the specific objects fields in the sidebar under "Objects".
     :message => 'Hello, where is photo?'
   )
 
+==== Reply a comment
+
+  comment = FbGraph::Comment.fetch("10151705618661509_29545180")
+  comment.reply!( 
+    :message => "Hello!",
+    :access_token => ACCESS_TOKEN
+  )
+
 === DELETE
 
 ==== Delete an object
@@ -209,6 +227,13 @@ of the specific objects fields in the sidebar under "Objects".
   post.unlike!(:access_token => ACCESS_TOKEN)
   post.destroy(:access_token => ACCESS_TOKEN)
 
+==== Delete a comment reply
+  
+  reply = comment.comment!( :access_token => ACCESS_TOKEN,
+    :message => "Hello!"
+  )
+  reply.destroy(:access_token => ACCESS_TOKEN)
+
 === Authentication
 
 Both Facebook JavaScript SDK and normal OAuth2 flow is supported.
@@ -288,4 +313,4 @@ https://github.com/nov/fb_graph/wiki
 
 == Copyright
 
-Copyright (c) 2010 nov matake. See LICENSE for details.
+Copyright (c) 2010 nov matake. See LICENSE for details.

data/VERSION

@@ -1 +1 @@
-2.7.1
+2.7.2

data/lib/fb_graph/comment.rb

@@ -1,9 +1,10 @@
 module FbGraph
   class Comment < Node
+    include Connections::Comments
     include Connections::Likes
     include Connections::Likes::Likable
 
-    attr_accessor :from, :message, :created_time, :like_count
+    attr_accessor :from, :message, :created_time, :like_count, :can_comment
 
     def initialize(identifier, attributes = {})
       super
@@ -16,6 +17,8 @@ module FbGraph
       end
       @message = attributes[:message]
       @like_count = attributes[:likes]
+      @can_comment = attributes[:can_comment]
+
       if attributes[:created_time]
         @created_time = Time.parse(attributes[:created_time]).utc
       end

data/lib/fb_graph/connections/comments.rb

@@ -16,6 +16,13 @@ module FbGraph
           :access_token => options[:access_token] || self.access_token
         )
       end
+
+      def reply!(options = {})
+        comment_id = post options.merge(:connection => :comments)
+        Comment.new comment_id, options.merge(
+          :access_token => options[:access_token] || self.access_token
+        )
+      end
     end
   end
 end

data/lib/fb_graph/node.rb

@@ -135,7 +135,18 @@ module FbGraph
       when 'null'
         nil
       else
-        _response_ = MultiJson.load(response.body).with_indifferent_access
+        _response_ = if response.body =~ /^"/
+          # NOTE:
+          #  Only for comment.reply!, which returns an identifier as String.
+          #  Once the API spec changed (I guess FB will do so), we can call "with_indifferent_access" for all response.
+          # NOTE:
+          #  When MultiJson.engine is JsonGem, parsing JSON String fails.
+          #  You should handle this case without MultiJson.
+          response.body.gsub('"', '')
+        else
+          MultiJson.load(response.body).with_indifferent_access
+        end
+
         if (200...300).include?(response.status)
           _response_
         else

data/lib/fb_graph.rb

@@ -4,6 +4,7 @@ require 'patch/rack/oauth2/util'
 require 'patch/rack/oauth2/client'
 require 'patch/rack/oauth2/access_token'
 
+
 module FbGraph
   VERSION = ::File.read(
     ::File.join(::File.dirname(__FILE__), '../VERSION')

data/spec/fb_graph/comment_spec.rb

@@ -28,4 +28,40 @@ describe FbGraph::Comment, '.new' do
     page_comment.from.should == FbGraph::Page.new('23456', :name => 'Smart.fm', :category => 'Web Site')
   end
 
+end
+
+describe FbGraph::Comment, '#fetch' do
+
+  context 'when access_token given' do
+    it 'gets the comment message, attributes, and user object' do
+      mock_graph :get, 'comment_id', 'comments/comment', :access_token => 'access_token' do
+        comment = FbGraph::Comment.fetch("comment_id", :access_token => 'access_token')
+        comment.identifier.should == '10151705618661509_29545180'
+        comment.can_comment == nil
+      end
+    end
+
+    it "gets the list of replies for the comment" do
+      mock_graph :get, 'comment_id/comments', 'comments/comments/with_cursor_paging_params', :access_token => 'access_token' do
+        replies = FbGraph::Comment.new("comment_id", :access_token => 'access_token').comments
+        replies.should be_instance_of FbGraph::Connection
+        replies.should be_a FbGraph::Collection
+        replies.collection.next.should include :limit, :after
+        replies.collection.previous.should include :limit, :before
+        replies.collection.cursors.should include :before, :after
+      end
+    end
+
+    context 'when can_comment is passed into the fields parameter' do
+      it 'gets the can_comment attribute' do
+        mock_graph :get, 'comment_id', 'comments/with_can_comment', :access_token => 'access_token', :params => { :fields => 'can_comment' } do
+          comment = FbGraph::Comment.fetch("comment_id", :access_token => 'access_token', :fields => 'can_comment')
+          comment.identifier.should == '10151705618661509_29545180'
+          comment.can_comment.should == true
+        end
+      end
+    end
+
+  end
+
 end

data/spec/fb_graph/connections/comments_spec.rb

@@ -72,4 +72,38 @@ describe FbGraph::Connections::Comments, '#comment!' do
       end
     end
   end
+end
+
+describe FbGraph::Comment, '#reply!' do
+  context 'when included by FbGraph::Post' do
+    context 'when no access_token given' do
+      it 'should raise FbGraph::Exception' do
+        mock_graph :post, '12345/comments', 'comments/comments/post_without_access_token', :status => [500, 'Internal Server Error'] do
+          lambda do
+            FbGraph::Post.new('12345').reply!(:message => 'hello')
+          end.should raise_exception(FbGraph::Exception)
+        end
+      end
+    end
+
+    context 'when invalid access_token is given' do
+      it 'should raise FbGraph::Exception' do
+        mock_graph :post, '12345/comments', 'comments/comments/post_with_invalid_access_token', :status => [500, 'Internal Server Error'] do
+          lambda do
+            FbGraph::Post.new('12345', :access_token => 'invalid').reply!(:message => 'hello')
+          end.should raise_exception(FbGraph::Exception)
+        end
+      end
+    end
+
+    context 'when valid access_token is given' do
+      it 'should return generated comment' do
+        mock_graph :post, '12345/comments', 'comments/comments/post_with_valid_access_token' do
+          reply = FbGraph::Comment.new('12345', :access_token => 'valid').reply!(:message => 'hello')
+          reply.should be_instance_of FbGraph::Comment
+          reply.identifier.should == '10151705618661509_29549300'
+        end
+      end
+    end
+  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: fb_graph
 version: !ruby/object:Gem::Version
-  version: 2.7.1
+  version: 2.7.2
 platform: ruby
 authors:
 - nov matake
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-05-26 00:00:00.000000000 Z
+date: 2013-06-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: httpclient
@@ -509,7 +509,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.0.3
+rubygems_version: 2.0.2
 signing_key: 
 specification_version: 4
 summary: A full-stack Facebook Graph API wrapper in Ruby.