* lib/rss/rss.rb: improved type conversion.

* lib/rss/1.0.rb: ditto.
* lib/rss/0.9.rb: ditto.
* lib/rss/2.0.rb: ditto.
* lib/rss/image.rb: ditto.
* lib/rss/syndication.rb: ditto.

* test/rss/test_2.0.rb: added type conversion tests.
* test/rss/test_accessor.rb: ditto.
* test/rss/test_to_s.rb: ditto.
* test/rss/test_syndication.rb: ditto.
* test/rss/test_setup_maker_2.0.rb: ditto.
* test/rss/test_setup_maker_1.0.rb: ditto.
* test/rss/test_setup_maker_0.9.rb: ditto.
* test/rss/test_maker_sy.rb: ditto.
* test/rss/test_maker_image.rb: ditto.
* test/rss/test_maker_2.0.rb: ditto.
* test/rss/test_maker_0.9.rb: ditto.
* test/rss/test_image.rb: ditto.

* test/rss/test_maker_1.0.rb: use assert instead of assert_equal.

* test/rss/rss-assertions.rb: improved type conversion assertions.


git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@9610 b2dd03c8-39d4-4d8f-98ff-823fe69b080e

1 files changed, 6 insertions, 17 deletions

diff --git a/lib/rss/syndication.rb b/lib/rss/syndication.rb
index 56976085a1..8791ec24fc 100644
--- a/lib/rss/syndication.rb
+++ b/lib/rss/syndication.rb
@@ -17,8 +17,12 @@ module RSS
       super
       
       klass.module_eval(<<-EOC, *get_file_and_line_from_caller(1))
-        %w(updatePeriod updateFrequency).each do |name|
-          install_text_element("\#{SY_PREFIX}_\#{name}")
+        [
+          ["updatePeriod"],
+          ["updateFrequency", :positive_integer]
+        ].each do |name, type|
+          install_text_element("\#{SY_PREFIX}_\#{name}", type,
+                               "\#{SY_PREFIX}:\#{name}")
         end
 
         %w(updateBase).each do |name|
@@ -31,12 +35,6 @@ module RSS
           validate_sy_updatePeriod(new_value) if @do_validate
           self._sy_updatePeriod = new_value
         end
-
-        alias_method(:_sy_updateFrequency=, :sy_updateFrequency=)
-        def sy_updateFrequency=(new_value)
-          validate_sy_updateFrequency(new_value) if @do_validate
-          self._sy_updateFrequency = new_value.to_i
-        end
       EOC
     end
 
@@ -61,15 +59,6 @@ module RSS
         raise NotAvailableValueError.new("updatePeriod", value)
       end
     end
-
-    SY_UPDATEFREQUENCY_AVAILABLE_RE = /\A\s*\+?\d+\s*\z/
-    def validate_sy_updateFrequency(value)
-      value = value.to_s.strip
-      if SY_UPDATEFREQUENCY_AVAILABLE_RE !~ value
-        raise NotAvailableValueError.new("updateFrequency", value)
-      end
-    end
-
   end
 
   class RDF