diff options
| author | kou <kou@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2008-02-11 08:24:18 +0000 |
|---|---|---|
| committer | kou <kou@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2008-02-11 08:24:18 +0000 |
| commit | 44b86ef7ec1a9ba6eb65effd44541e4e3ab22ab0 (patch) | |
| tree | 93eaecbf2b894bf946ef21140348b613be7daefb /lib/rss/maker/0.9.rb | |
| parent | 69002ff0b5cd4fdb2321b80f19961a94ec189c3a (diff) | |
| download | ruby-44b86ef7ec1a9ba6eb65effd44541e4e3ab22ab0.tar.gz ruby-44b86ef7ec1a9ba6eb65effd44541e4e3ab22ab0.tar.xz ruby-44b86ef7ec1a9ba6eb65effd44541e4e3ab22ab0.zip | |
* lib/rss/rss.rb (RSS::VERSION), test/rss/test_version.rb:
0.2.3 -> 0.2.4.
* lib/rss/maker.rb, lib/rss/maker/, test/rss/test_maker_2.0.rb:
fixed a bug that RSS::Maker.make("0.9")'s item doesn't make some
elements if description is missed.
Reported by Michael Auzenne. Thanks!!!
* lib/rss/maker/0.9.rb, test/rss/test_maker_0.9.rb:
RSS::Maker.make("0.9") generates RSS 0.92 not RSS 0.91.
git-svn-id: http://svn.ruby-lang.org/repos/ruby/trunk@15431 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
Diffstat (limited to 'lib/rss/maker/0.9.rb')
| -rw-r--r-- | lib/rss/maker/0.9.rb | 22 |
1 files changed, 14 insertions, 8 deletions
diff --git a/lib/rss/maker/0.9.rb b/lib/rss/maker/0.9.rb index c83597dfd..72b14dc97 100644 --- a/lib/rss/maker/0.9.rb +++ b/lib/rss/maker/0.9.rb @@ -7,7 +7,7 @@ module RSS class RSS09 < RSSBase - def initialize(feed_version="0.91") + def initialize(feed_version="0.92") super @feed_type = "rss" end @@ -258,23 +258,27 @@ module RSS def to_feed(rss) item = Rss::Channel::Item.new set = setup_values(item) - if set or title {|t| t.have_required_values?} + _not_set_required_variables = not_set_required_variables + if _not_set_required_variables.empty? rss.items << item set_parent(item, rss.channel) setup_other_elements(rss, item) elsif variable_is_set? - raise NotSetError.new("maker.items", not_set_required_variables) + raise NotSetError.new("maker.items", _not_set_required_variables) end end private def required_variable_names - %w(link) + [] end def not_set_required_variables vars = super - vars << "title" unless title {|t| t.have_required_values?} + if @maker.feed_version == "0.91" + vars << "title" unless title {|t| t.have_required_values?} + vars << "link" unless link {|l| l.have_required_values?} + end vars end @@ -454,8 +458,10 @@ module RSS end end - add_maker("0.9", RSS09) - add_maker("0.91", RSS09) - add_maker("rss0.91", RSS09) + add_maker("0.9", "0.92", RSS09) + add_maker("0.91", "0.91", RSS09) + add_maker("0.92", "0.92", RSS09) + add_maker("rss0.91", "0.91", RSS09) + add_maker("rss0.92", "0.92", RSS09) end end |
