Changeset 1620

Timestamp:

08/29/06 01:39:46

Author:

miyagawa

Message:

backout the call to utf8::decode in strip_html function. FeedParser? now set $XML::Atom::ForceUnicode? = 1 globally rather than local(), since this flag takes effect during the runtime access to $entry->body or other elements when XML::Atom accesses its DOM node. Nasty.

Files:

• branches/hackathon-summary/plagger (modified) (previous)
• branches/hackathon-summary/plagger/lib/Plagger/FeedParser.pm (modified) (2 diffs)
• branches/hackathon-summary/plagger/lib/Plagger/Util.pm (modified) (1 diff)

branches/hackathon-summary/plagger/lib/Plagger/FeedParser.pm

@@ -3 +3 @@
-
-use Feed::Find;
+use XML::Atom;
-use XML::Feed;
-use XML::Feed::RSS;
-$XML::Feed::RSS::PREFERRED_PARSER = "XML::RSS::LibXML";
+$XML::Atom::ForceUnicode = 1;
-
-use Plagger::Util;
@@ -20 +22 @@
-    }
-
-    local $XML::Atom::ForceUnicode = 1;
-    my $remote = eval { XML::Feed->parse($content_ref) }
-        or Carp::croak("Parsing content failed: " . ($@ || XML::Feed->errstr));

branches/hackathon-summary/plagger/lib/Plagger/Util.pm

@@ -28 +28 @@
-sub strip_html {
-    my $html = shift;
-    utf8::decode($html);
-
-    eval {