]> Sergey Matveev's repositories - public-inbox.git/blobdiff - lib/PublicInbox/Feed.pm
add various TODO items
[public-inbox.git] / lib / PublicInbox / Feed.pm
index 6b317ab0573fb298b0882187ddd65accaed823ea..6996f374e555a4f55d565fda80fe3b119369e3b0 100644 (file)
@@ -9,12 +9,10 @@ use Date::Parse qw(strptime str2time);
 use PublicInbox::Hval;
 use PublicInbox::GitCatFile;
 use constant {
-       DATEFMT => '%Y-%m-%dT%H:%M:%SZ',
-       MAX_PER_PAGE => 25,
+       DATEFMT => '%Y-%m-%dT%H:%M:%SZ', # atom standard
+       MAX_PER_PAGE => 25, # this needs to be tunable
 };
 
-# FIXME: workaround https://rt.cpan.org/Public/Bug/Display.html?id=22817
-
 # main function
 sub generate {
        my ($class, $args) = @_;
@@ -50,7 +48,7 @@ sub generate {
 
 sub generate_html_index {
        my ($class, $args) = @_;
-       require Mail::Thread;
+       require PublicInbox::Thread;
 
        my $max = $args->{max} || MAX_PER_PAGE;
        my $feed_opts = get_feedopts($args);
@@ -72,7 +70,7 @@ sub generate_html_index {
        });
        $git = undef; # destroy pipes.
 
-       my $th = Mail::Thread->new(@messages);
+       my $th = PublicInbox::Thread->new(@messages);
        $th->thread;
        my $html = "<html><head><title>$title</title>" .
                '<link rel="alternate" title="Atom feed" href="' .
@@ -245,7 +243,7 @@ sub add_to_feed {
        defined $mid or return 0;
        $mid = PublicInbox::Hval->new_msgid($mid);
        my $href = $mid->as_href . '.html';
-       my $content = PublicInbox::View->as_feed_entry($mime, $fullurl . $href);
+       my $content = PublicInbox::View->feed_entry($mime, $fullurl . $href);
        defined($content) or return 0;
 
        my $subject = mime_header($mime, 'Subject') or return 0;
@@ -301,7 +299,7 @@ sub dump_html_line {
 sub do_cat_mail {
        my ($git, $path) = @_;
        my $str = $git->cat_file("HEAD:$path");
-       Email::MIME->new($$str);
+       Email::MIME->new($str);
 }
 
 1;