about summary refs log tree commit homepage
path: root/lib
diff options
context:
space:
mode:
authorEric Wong <e@80x24.org>2022-10-08 08:24:48 +0000
committerEric Wong <e@80x24.org>2022-10-09 16:48:39 +0000
commit5290ed2889d7a62ce3e41e4ff08efaefd2727eda (patch)
tree8f9592335a45203f4cb543a5d31964db13689952 /lib
parentdda0b4ab620e1709c0e25b3d9d70745fe52d60d3 (diff)
downloadpublic-inbox-5290ed2889d7a62ce3e41e4ff08efaefd2727eda.tar.gz
I'm not sure how to best make a UI for one coderepo to many
inboxes/extindices, yet; but at least allow a simple 1:1
mapping, for now.  This ensures /$CODEREPO/$OID/s/ can work
as effectively as /$INBOX/$OID/s/ when looking for emails
associated with a git commit.
Diffstat (limited to 'lib')
-rw-r--r--lib/PublicInbox/Config.pm5
-rw-r--r--lib/PublicInbox/Inbox.pm3
-rw-r--r--lib/PublicInbox/ViewVCS.pm31
-rw-r--r--lib/PublicInbox/WwwCoderepo.pm14
4 files changed, 48 insertions, 5 deletions
diff --git a/lib/PublicInbox/Config.pm b/lib/PublicInbox/Config.pm
index a430cd5c..c27928de 100644
--- a/lib/PublicInbox/Config.pm
+++ b/lib/PublicInbox/Config.pm
@@ -404,6 +404,11 @@ sub repo_objs {
                         push @repo_objs, $repo if $repo;
                 }
                 if (scalar @repo_objs) {
+                        require Scalar::Util;
+                        for (@repo_objs) {
+                                push @{$_->{-ibxs}}, $ibxish;
+                                Scalar::Util::weaken($_->{-ibxs}->[-1]);
+                        }
                         $ibxish->{-repo_objs} = \@repo_objs;
                 } else {
                         delete $ibxish->{coderepo};
diff --git a/lib/PublicInbox/Inbox.pm b/lib/PublicInbox/Inbox.pm
index 3532bb58..cb98d2ad 100644
--- a/lib/PublicInbox/Inbox.pm
+++ b/lib/PublicInbox/Inbox.pm
@@ -205,7 +205,8 @@ sub base_url {
                 $url .= '/' if $url !~ m!/\z!;
                 return $url .= $self->{name} . '/';
         }
-        # called from a non-PSGI environment (e.g. NNTP/POP3):
+        # called from a non-PSGI environment or cross-inbox environment
+        # where multiple inboxes can have different domains
         my $url = $self->{url} // return undef;
         $url = $url->[0] // return undef;
         # expand protocol-relative URLs to HTTPS if we're
diff --git a/lib/PublicInbox/ViewVCS.pm b/lib/PublicInbox/ViewVCS.pm
index 6ada03e6..72b79ab7 100644
--- a/lib/PublicInbox/ViewVCS.pm
+++ b/lib/PublicInbox/ViewVCS.pm
@@ -25,7 +25,7 @@ use PublicInbox::ViewDiff qw(flush_diff uri_escape_path);
 use PublicInbox::View;
 use PublicInbox::Eml;
 use Text::Wrap qw(wrap);
-use PublicInbox::Hval qw(ascii_html to_filename);
+use PublicInbox::Hval qw(ascii_html to_filename prurl);
 my $hl = eval {
         require PublicInbox::HlMod;
         PublicInbox::HlMod->new;
@@ -152,6 +152,25 @@ sub show_commit_start { # ->psgi_qx callback
         }
 }
 
+sub ibx_url_for {
+        my ($ctx) = @_;
+        $ctx->{ibx} and return; # just fall back to $upfx
+        $ctx->{git} or return; # /$CODEREPO/$OID/s/ to (eidx|ibx)
+        if (my $ALL = $ctx->{www}->{pi_cfg}->ALL) {
+                $ALL->base_url // $ALL->base_url($ctx->{env});
+        } elsif (my $ibxs = $ctx->{git}->{-ibxs}) {
+                for my $ibx (@$ibxs) {
+                        if ($ibx->isrch) {
+                                return defined($ibx->{url}) ?
+                                        prurl($ctx->{env}, $ibx->{url}) :
+                                        "../../../$ibx->{name}/";
+                        }
+                }
+        } else {
+                undef;
+        }
+}
+
 sub cmt_finalize {
         my ($ctx) = @_;
         $ctx->{-linkify} //= PublicInbox::Linkify->new;
@@ -227,12 +246,16 @@ EOM
                         $q = wrap('', '', $q);
                         my $rows = ($q =~ tr/\n/\n/) + 1;
                         $q = ascii_html($q);
+                        my $ibx_url = ibx_url_for($ctx);
+                        my $alt = $ibx_url ? ' '.ascii_html($ibx_url) : '';
+                        $ibx_url = ascii_html($ibx_url) if defined $ibx_url;
+                        $ibx_url //= $upfx;
                         print $zfh <<EOM;
-<hr><form action=$upfx
+<hr><form action="$ibx_url"
 id=related><pre>find related emails, including ancestors/descendants/conflicts
 <textarea name=q cols=${\PublicInbox::View::COLS} rows=$rows>$q</textarea>
-<input type=submit value=search
-/>\t(<a href=${upfx}_/text/help/>help</a>)</pre></form>
+<input type=submit value="search$alt"
+/>\t(<a href="${ibx_url}_/text/help/">help</a>)</pre></form>
 EOM
                 }
         }
diff --git a/lib/PublicInbox/WwwCoderepo.pm b/lib/PublicInbox/WwwCoderepo.pm
index 2184e89d..99df39ef 100644
--- a/lib/PublicInbox/WwwCoderepo.pm
+++ b/lib/PublicInbox/WwwCoderepo.pm
@@ -35,6 +35,20 @@ sub prepare_coderepos {
                 $k = substr($k, length('coderepo.'), -length('.dir'));
                 $code_repos->{$k} //= $pi_cfg->fill_code_repo($k);
         }
+
+        # associate inboxes and extindices with coderepos for search:
+        for my $k (grep(/\Apublicinbox\.(?:.+)\.coderepo\z/, keys %$pi_cfg)) {
+                $k = substr($k, length('publicinbox.'), -length('.coderepo'));
+                my $ibx = $pi_cfg->lookup_name($k) // next;
+                $pi_cfg->repo_objs($ibx);
+                push @{$self->{-strong}}, $ibx; # strengthen {-ibxs} weakref
+        }
+        for my $k (grep(/\Aextindex\.(?:.+)\.coderepo\z/, keys %$pi_cfg)) {
+                $k = substr($k, length('extindex.'), -length('.coderepo'));
+                my $eidx = $pi_cfg->lookup_ei($k) // next;
+                $pi_cfg->repo_objs($eidx);
+                push @{$self->{-strong}}, $eidx; # strengthen {-ibxs} weakref
+        }
         while (my ($nick, $repo) = each %$code_repos) {
                 $self->{"\0$nick"} = $repo;
         }