]> Sergey Matveev's repositories - public-inbox.git/blobdiff - lib/PublicInbox/WwwStream.pm
www_stream: fix search for new.html endpoint
[public-inbox.git] / lib / PublicInbox / WwwStream.pm
index 62a4fe2607ee4fd3e17ad93dbd73cc87cf96b1be..d9abb5ab8068756d51c3f5aaae5b9098703744ce 100644 (file)
@@ -14,6 +14,12 @@ sub new {
        bless { nr => 0, cb => $cb, ctx => $ctx }, $class;
 }
 
+sub response {
+       my ($class, $ctx, $code, $cb) = @_;
+       [ $code, [ 'Content-Type', 'text/html; charset=UTF-8' ],
+         $class->new($ctx, $cb) ]
+}
+
 sub _html_top ($) {
        my ($self) = @_;
        my $ctx = $self->{ctx};
@@ -22,10 +28,21 @@ sub _html_top ($) {
        my $title = $ctx->{-title_html} || $desc;
        my $upfx = $ctx->{-upfx} || '';
        my $atom = $ctx->{-atom} || $upfx.'new.atom';
+       my $tip = $ctx->{-html_tip} || '';
        my $top = "<b>$desc</b> (<a\nhref=\"$atom\">Atom feed</a>)";
        if ($obj->search) {
-               $top = qq{<form\naction="$upfx"><pre>$top} .
-                         qq{ <input\nname=q\ntype=text />} .
+               my $q_val = $ctx->{-q_value_html};
+               if (defined $q_val && $q_val ne '') {
+                       $q_val = qq(\nvalue="$q_val" );
+               } else {
+                       $q_val = '';
+               }
+               # XXX gross, for SearchView.pm
+               my $extra = $ctx->{-extra_form_html} || '';
+               my $action = $upfx eq '' ? './' : $upfx;
+               $top = qq{<form\naction="$action"><pre>$top} .
+                         qq{ <input\nname=q\ntype=text$q_val/>} .
+                         $extra .
                          qq{<input\ntype=submit\nvalue=search />} .
                          q{</pre></form>}
        } else {
@@ -35,7 +52,7 @@ sub _html_top ($) {
                "<link\nrel=alternate\ntitle=\"Atom feed\"\n".
                "href=\"$atom\"\ntype=\"application/atom+xml\"/>" .
                PublicInbox::Hval::STYLE .
-               "</head><body>$top";
+               "</head><body>". $top . $tip;
 }
 
 sub _html_end {
@@ -44,8 +61,6 @@ sub _html_end {
        my $ctx = $self->{ctx};
        my $obj = $ctx->{-inbox};
        my $desc = ascii_html($obj->description);
-       my @urls = @{$obj->cloneurl};
-       my %seen = map { $_ => 1 } @urls;
 
        # FIXME: cleanup
        my $env = $ctx->{env};
@@ -54,9 +69,15 @@ sub _html_end {
                        "$env->{SERVER_NAME}:$env->{SERVER_PORT}";
        my $http = "$scheme://$host_port".($env->{SCRIPT_NAME} || '/');
        $http = URI->new($http . $obj->{name})->canonical->as_string;
-       $seen{$http} or unshift @urls, $http;
+       my %seen = ( $http => 1 );
+       my @urls = ($http);
+       foreach my $u (@{$obj->cloneurl}) {
+               next if $seen{$u};
+               $seen{$u} = 1;
+               push @urls, $u =~ /\Ahttps?:/ ? qq(<a\nhref="$u">$u</a>) : $u;
+       }
        if (scalar(@urls) == 1) {
-               $urls .= " git clone --mirror $urls[0]";
+               $urls .= " git clone --mirror $http";
        } else {
                $urls .= "\n" .
                        join("\n", map { "\tgit clone --mirror $_" } @urls);