#
# Used for generating Atom feeds for web-accessible mailing list archives.
package PublicInbox::Feed;
use strict;
-use warnings;
+use v5.10.1;
use PublicInbox::View;
use PublicInbox::WwwAtomStream;
use PublicInbox::Smsg; # this loads w/o Search::Xapian
@@ -19,14 +19,14 @@ sub generate {
my ($ctx) = @_;
my $msgs = $ctx->{msgs} = recent_msgs($ctx);
return _no_thread() unless @$msgs;
- PublicInbox::WwwAtomStream->response($ctx, 200, \&generate_i);
+ PublicInbox::WwwAtomStream->response($ctx, \&generate_i);
}
sub generate_thread_atom {
my ($ctx) = @_;
- my $msgs = $ctx->{msgs} = $ctx->{-inbox}->over->get_thread($ctx->{mid});
+ my $msgs = $ctx->{msgs} = $ctx->{ibx}->over->get_thread($ctx->{mid});
return _no_thread() unless @$msgs;
- PublicInbox::WwwAtomStream->response($ctx, 200, \&generate_i);
+ PublicInbox::WwwAtomStream->response($ctx, \&generate_i);
}
sub generate_html_index {
@@ -34,7 +34,7 @@ sub generate_html_index {
# if the 'r' query parameter is given, it is a legacy permalink
# which we must continue supporting:
my $qp = $ctx->{qp};
- my $ibx = $ctx->{-inbox};
+ my $ibx = $ctx->{ibx};
if ($qp && !$qp->{r} && $ibx->over) {
return PublicInbox::View::index_topics($ctx);
}
@@ -49,12 +49,15 @@ sub generate_html_index {
sub new_html_i {
my ($ctx, $eml) = @_;
- $ctx->zmore($ctx->html_top) if exists $ctx->{-html_tip};
+ print { $ctx->zfh } $ctx->html_top if exists $ctx->{-html_tip};
- $eml and return PublicInbox::View::eml_entry($ctx, $eml);
+ if ($eml) {
+ $ctx->{smsg}->populate($eml) if !$ctx->{ibx}->{over};
+ return PublicInbox::View::eml_entry($ctx, $eml);
+ }
my $smsg = shift @{$ctx->{msgs}} or
- $ctx->zmore(PublicInbox::View::pagination_footer(
- $ctx, './new.html'));
+ print { $ctx->zfh } PublicInbox::View::pagination_footer(
+ $ctx, './new.html');
$smsg;
}
@@ -67,8 +70,9 @@ sub new_html {
}
$ctx->{-html_tip} = '';
$ctx->{-upfx} = '';
+ $ctx->{-spfx} = '' if $ctx->{ibx}->{coderepo};
$ctx->{-hr} = 1;
- PublicInbox::WwwStream::aresponse($ctx, 200, \&new_html_i);
+ PublicInbox::WwwStream::aresponse($ctx, \&new_html_i);
}
# private subs
@@ -79,8 +83,8 @@ sub _no_thread () {
sub recent_msgs {
my ($ctx) = @_;
- my $ibx = $ctx->{-inbox};
- my $max = $ibx->{feedmax};
+ my $ibx = $ctx->{ibx};
+ my $max = $ibx->{feedmax} // 25;
return PublicInbox::View::paginate_recent($ctx, $max) if $ibx->over;
# only for rare v1 inboxes which aren't indexed at all
@@ -88,7 +92,7 @@ sub recent_msgs {
my $hex = '[a-f0-9]';
my $addmsg = qr!^:000000 100644 \S+ (\S+) A\t${hex}{2}/${hex}{38}$!;
my $delmsg = qr!^:100644 000000 (\S+) \S+ D\t(${hex}{2}/${hex}{38})$!;
- my $refhex = qr/(?:HEAD|${hex}{4,40})(?:~[0-9]+)?/;
+ my $refhex = qr/(?:HEAD|${hex}{4,})(?:~[0-9]+)?/;
# revision ranges may be specified
my $range = 'HEAD';
@@ -108,13 +112,13 @@ sub recent_msgs {
my $last;
my $last_commit;
local $/ = "\n";
- my @oids;
+ my @ret;
while (defined(my $line = <$log>)) {
if ($line =~ /$addmsg/o) {
my $add = $1;
next if $deleted{$add}; # optimization-only
- push @oids, $add;
- if (scalar(@oids) >= $max) {
+ push(@ret, bless { blob => $add }, 'PublicInbox::Smsg');
+ if (scalar(@ret) >= $max) {
$last = 1;
last;
}
@@ -126,7 +130,7 @@ sub recent_msgs {
if ($last) {
local $/ = "\n";
while (my $line = <$log>) {
- if ($line =~ /^(${hex}{7,40})/) {
+ if ($line =~ /^(${hex}{7,})/) {
$last_commit = $1;
last;
}
@@ -136,8 +140,7 @@ sub recent_msgs {
$last_commit and
$ctx->{next_page} = qq[] .
'next (older)';
-
- [ map { bless {blob => $_ }, 'PublicInbox::Smsg' } @oids ];
+ \@ret;
}
1;