about summary refs log tree commit homepage
path: root/lib/PublicInbox/WWW.pm
diff options
context:
space:
mode:
authorEric Wong <e@80x24.org>2016-06-30 02:35:21 +0000
committerEric Wong <e@80x24.org>2016-06-30 08:57:19 +0000
commitb30bcf12bdc0c3658e5e31c02b05c3609da04e7f (patch)
tree083db6ac69f75214e54e49c4bee9f2683b160d7b /lib/PublicInbox/WWW.pm
parentdfaff07b1578d5b32aae3225cb09c6a3f2177896 (diff)
downloadpublic-inbox-b30bcf12bdc0c3658e5e31c02b05c3609da04e7f.tar.gz
This hybrid view is better than the old flat, but can
still fall down compared to the old threaded view in
some cases.
Diffstat (limited to 'lib/PublicInbox/WWW.pm')
-rw-r--r--lib/PublicInbox/WWW.pm13
1 files changed, 8 insertions, 5 deletions
diff --git a/lib/PublicInbox/WWW.pm b/lib/PublicInbox/WWW.pm
index da5c1d30..cbd3142d 100644
--- a/lib/PublicInbox/WWW.pm
+++ b/lib/PublicInbox/WWW.pm
@@ -23,7 +23,7 @@ require PublicInbox::Git;
 use PublicInbox::GitHTTPBackend;
 our $INBOX_RE = qr!\A/([\w\.\-]+)!;
 our $MID_RE = qr!([^/]+)!;
-our $END_RE = qr!(t/|t\.mbox(?:\.gz)?|t\.atom|raw|)!;
+our $END_RE = qr!(T/|t/|t\.mbox(?:\.gz)?|t\.atom|raw|)!;
 our $ATTACH_RE = qr!(\d[\.\d]*)-([[:alnum:]][\w\.-]+[[:alnum:]])!i;
 
 sub new {
@@ -92,9 +92,10 @@ sub call {
                 invalid_inbox_mid($self, $ctx, $1, $2) ||
                         get_attach($ctx, $idx, $fn);
         # in case people leave off the trailing slash:
-        } elsif ($path_info =~ m!$INBOX_RE/$MID_RE/(?:T|T/|t)\z!o) {
-                my ($inbox, $mid) = ($1, $2);
-                r301($ctx, $inbox, $mid, 't/#u');
+        } elsif ($path_info =~ m!$INBOX_RE/$MID_RE/(T|t)\z!o) {
+                my ($inbox, $mid, $suffix) = ($1, $2, $3);
+                $suffix .= $suffix =~ /\A[tT]\z/ ? '/#u' : '/';
+                r301($ctx, $inbox, $mid, $suffix);
 
         } elsif ($path_info =~ m!$INBOX_RE/$MID_RE/R/?\z!o) {
                 my ($inbox, $mid) = ($1, $2);
@@ -241,8 +242,9 @@ sub get_mid_html {
 
 # /$INBOX/$MESSAGE_ID/t/
 sub get_thread {
-        my ($ctx) = @_;
+        my ($ctx, $flat) = @_;
         searcher($ctx) or return need_search($ctx);
+        $ctx->{flat} = $flat;
         require PublicInbox::View;
         PublicInbox::View::thread_html($ctx);
 }
@@ -416,6 +418,7 @@ sub msg_page {
         my $ret;
         $ret = invalid_inbox_mid($self, $ctx, $inbox, $mid) and return $ret;
         '' eq $e and return get_mid_html($ctx);
+        'T/' eq $e and return get_thread($ctx, 1);
         't/' eq $e and return get_thread($ctx);
         't.atom' eq $e and return get_thread_atom($ctx);
         't.mbox' eq $e and return get_thread_mbox($ctx);