fix: correctly replace cdata node in hpricot

improve: if user has error include appengine gems, do not import appengine agent

Rakefile

@@ -8,7 +8,7 @@ require "rake/gempackagetask"
 
 NAME = "fullfeed"
 SUMMARY = "Fullfeed RSS creator"
-GEM_VERSION = "0.3"
+GEM_VERSION = "0.3.1"
 
 spec = Gem::Specification.new do |s|
   s.name = NAME

fullfeed.gemspec

@@ -1,6 +1,6 @@
 NAME = "fullfeed"
 SUMMARY = "Fullfeed RSS creator"
-GEM_VERSION = "0.3.0"
+GEM_VERSION = "0.3.1"
 
 Gem::Specification.new do |s|
   s.name = NAME

lib/fullfeed/agent.rb

@@ -16,4 +16,5 @@
   gem('appengine-apis')
   require "#{File.dirname(__FILE__)}/agent/appengine_agent"
 rescue Gem::LoadError
+rescue NameError 
 end

lib/fullfeed/feed.rb

@@ -65,11 +65,12 @@ def process_item(item)
       if link && desc
         begin
           @logger.debug "  Extract item (#{guid}) link: #{link}"
-          desc.inner_html = extract_cached(guid, link)
+          desc.children.first.content = extract_cached(guid, link)
+
         rescue StandardError => e
-          @logger.error "Error fetching content: #{e.inspect}"
-        end
+          @logger.error "Error fetching/replacing content: #{e.inspect}"
 
+        end        
       else
         @logger.warn "No link or desc node found in item: #{item}"
 
@@ -105,9 +106,9 @@ def extract(link)
 
     def to_utf8(text)
       if @encoding == "UTF-8"
-        text
+        return text
       else
-        Iconv.conv("UTF-8", @encoding, text)
+        return Iconv.conv("UTF-8", @encoding, text)
       end
     end
   end

test/test_db_store.rb

@@ -0,0 +1,21 @@
+#!/usr/bin/env ruby
+
+require 'test/unit'
+require "#{File.dirname(__FILE__)}/../lib/fullfeed"
+require 'load_files'
+
+class TestDbStore < Test::Unit::TestCase
+  Fullfeed::Store::DbStore.setup("sqlite3:ynews.sqlite3")
+
+  def test_store
+    store = Fullfeed::Store::StoreFactory.store('http://test', 100, :db)
+
+    value = rand().to_s
+    store['/100'] = value
+    assert_equal store['/100'], value
+
+    value = "中文測試"
+    store['/200'] = value
+    assert_equal store['/200'], value
+  end
+end