ruby-hackernews 1.3.1 → 1.3.2

data/README.rdoc

@@ -59,6 +59,9 @@ Each Entry instance has the following data:
 
     entry.time         # the elapsed time from submission
 
+    entry.text         # the text of the submission (ask/jobs only)
+                       # NOTE: it will fetch the inner page 
+
 After you've logged in (see below) you can do the following
 
     entry.upvote                                # votes the entry
@@ -74,6 +77,11 @@ You get an entry's comments with:
 
     entry.comments
 
+You can also get a specific comment by id with:
+
+    Comment.find("1234") # returns the comments with id 1234,
+                         # and its subcomments.
+
 Note that the method above will send a request to HN. If you just need the comments' count or url, you can instead use:
 
     entry.comments_count
@@ -93,6 +101,7 @@ Each Comment instance has the following data:
 
     comment = Entry.all.first.comments.first # gets the first comment of the first entry on HN's main page
 
+    comment.id             # comment's unique identifier
     comment.text           # comment's body
     comment.user.name      # poster's user name on HN
     comment.voting.score   # comment's score
@@ -158,3 +167,4 @@ Will return the HN comment url of the last submitted story of that user
 - Josh Ellington ( http://github.com/joshellington ) for reporting a bug about job entries
 - Wayne ( http://github.com/BlissOfBeing ) for adding User.submissions, Entry.comments_url and cleaning up the Rakefile
 - Daniel Da Cunha ( http://github.com/ddacunha ) for a fix on Entry#comments_count
+- Nathan Campos ( http://github.com/nathanpc) for adding #text to Entry

data/Rakefile

@@ -8,7 +8,7 @@ require 'rake/testtask'
 
 spec = Gem::Specification.new do |s|
   s.name = 'ruby-hackernews'
-  s.version = '1.3.1'
+  s.version = '1.3.2'
   s.add_dependency('require_all', '>= 1.1.0')
   s.add_dependency('mechanize', '>= 1.0.0')
   s.has_rdoc = false

data/lib/ruby-hackernews/domain/comment/comment.rb

@@ -9,14 +9,19 @@ module RubyHackernews
 
     attr_accessor :parent
 
-    def initialize(text, voting, user_info, reply_link)
+    def initialize(text, voting, user_info, reply_link, absolute_link)
       @text = text
       @voting = voting
       @user = user_info
       @reply_link = reply_link
+      @absolute_link = absolute_link
       @children = []
     end
 
+    def id
+      return @absolute_link.split("=")[1].to_i
+    end
+
     def <<(comment)
       comment.parent = self
       @children << comment
@@ -38,6 +43,10 @@ module RubyHackernews
       return CommentService.new.get_new_comments(pages)
     end
 
+    def self.find(id)
+      return CommentService.new.find_by_id(id)
+    end
+
     def reply(text)
       return false unless @reply_link
       CommentService.new.write_comment(@reply_link, text)

data/lib/ruby-hackernews/services/comment_service.rb

@@ -14,6 +14,15 @@ module RubyHackernews
       return get_comments_entities(table)
     end
 
+    def find_by_id(id)
+      page = agent.get(ConfigurationService.base_url + "item?id=#{id}")
+      comment = parse_comment(page.search("table")[2].search("tr").first)
+      get_comments_entities(page.search("table")[3]).each do |c|
+        comment << c
+      end
+      return comment
+    end
+
     def get_comments_entities(table)
       comments = []
       target   = comments
@@ -58,7 +67,9 @@ module RubyHackernews
       user_info = UserInfoParser.new(header).parse
       reply_link = element.search("td[@class='default']/p//u//a").first
       reply_url = reply_link['href'] if reply_link     
-      return Comment.new(text, voting, user_info, reply_url)
+      absolute_link_group = header.search("a")
+      absolute_url = absolute_link_group.count == 2 ? absolute_link_group[1]['href'] : nil
+      return Comment.new(text, voting, user_info, reply_url, absolute_url)
     end
 
     def write_comment(page_url, comment)

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: ruby-hackernews
 version: !ruby/object:Gem::Version
-  version: 1.3.1
+  version: 1.3.2
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-12-09 00:00:00.000000000 Z
+date: 2012-12-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: require_all
@@ -51,34 +51,34 @@ extra_rdoc_files: []
 files:
 - README.rdoc
 - Rakefile
-- lib/ruby-hackernews.rb
+- lib/ruby-hackernews/domain/comment/comment.rb
+- lib/ruby-hackernews/domain/entry/user_info.rb
+- lib/ruby-hackernews/domain/entry/time_info.rb
+- lib/ruby-hackernews/domain/entry/entry.rb
+- lib/ruby-hackernews/domain/entry/link_info.rb
+- lib/ruby-hackernews/domain/entry/voting_info.rb
+- lib/ruby-hackernews/domain/entry/comments_info.rb
+- lib/ruby-hackernews/domain/user.rb
 - lib/ruby-hackernews/services/comment_service.rb
-- lib/ruby-hackernews/services/entry_service.rb
-- lib/ruby-hackernews/services/voting_service.rb
 - lib/ruby-hackernews/services/text_service.rb
-- lib/ruby-hackernews/services/parsers/entry_page_parser.rb
-- lib/ruby-hackernews/services/parsers/comments_info_parser.rb
-- lib/ruby-hackernews/services/parsers/voting_info_parser.rb
-- lib/ruby-hackernews/services/parsers/entry_parser.rb
-- lib/ruby-hackernews/services/parsers/user_info_parser.rb
-- lib/ruby-hackernews/services/parsers/link_info_parser.rb
-- lib/ruby-hackernews/services/parsers/text_parser.rb
-- lib/ruby-hackernews/services/parsers/time_info_parser.rb
-- lib/ruby-hackernews/services/login_service.rb
-- lib/ruby-hackernews/services/signup_service.rb
-- lib/ruby-hackernews/services/not_authenticated_error.rb
 - lib/ruby-hackernews/services/user_info_service.rb
 - lib/ruby-hackernews/services/mechanize_context.rb
+- lib/ruby-hackernews/services/entry_service.rb
+- lib/ruby-hackernews/services/not_authenticated_error.rb
+- lib/ruby-hackernews/services/login_service.rb
 - lib/ruby-hackernews/services/page_fetcher.rb
+- lib/ruby-hackernews/services/parsers/user_info_parser.rb
+- lib/ruby-hackernews/services/parsers/link_info_parser.rb
+- lib/ruby-hackernews/services/parsers/entry_parser.rb
+- lib/ruby-hackernews/services/parsers/time_info_parser.rb
+- lib/ruby-hackernews/services/parsers/text_parser.rb
+- lib/ruby-hackernews/services/parsers/entry_page_parser.rb
+- lib/ruby-hackernews/services/parsers/comments_info_parser.rb
+- lib/ruby-hackernews/services/parsers/voting_info_parser.rb
 - lib/ruby-hackernews/services/configuration_service.rb
-- lib/ruby-hackernews/domain/user.rb
-- lib/ruby-hackernews/domain/entry/time_info.rb
-- lib/ruby-hackernews/domain/entry/link_info.rb
-- lib/ruby-hackernews/domain/entry/user_info.rb
-- lib/ruby-hackernews/domain/entry/voting_info.rb
-- lib/ruby-hackernews/domain/entry/comments_info.rb
-- lib/ruby-hackernews/domain/entry/entry.rb
-- lib/ruby-hackernews/domain/comment/comment.rb
+- lib/ruby-hackernews/services/voting_service.rb
+- lib/ruby-hackernews/services/signup_service.rb
+- lib/ruby-hackernews.rb
 homepage: http://github.com/bolthar/ruby-hackernews
 licenses: []
 post_install_message: