From: Eric Wong <e@80x24.org>
Date: Sat, 28 May 2016 01:57:12 +0000 (+0000)
Subject: www: remove footer_html support
X-Git-Tag: v1.0.0~480
X-Git-Url: http://www.git.stargrave.org/?a=commitdiff_plain;h=b8a44a8bf2356c1fc474b119b713d7e7df365a4a;p=public-inbox.git

www: remove footer_html support

I haven't used it in a while and the existing "description"
is probably good enough.

If we support it again, it should be plain-text + auto-linkified
for ease-of-maintenance and consistency.
---

diff --git a/lib/PublicInbox/Inbox.pm b/lib/PublicInbox/Inbox.pm
index 4bcab96f..c07aaa9f 100644
--- a/lib/PublicInbox/Inbox.pm
+++ b/lib/PublicInbox/Inbox.pm
@@ -64,16 +64,6 @@ sub cloneurl {
 	$self->{cloneurl} = \@url;
 }
 
-# TODO: can we remove this?
-sub footer_html {
-	my ($self) = @_;
-	my $footer = $self->{footer};
-	return $footer if defined $footer;
-	$footer = try_cat("$self->{mainrepo}/public-inbox/footer.html");
-	chomp $footer;
-	$self->{footer} = $footer;
-}
-
 sub base_url {
 	my ($self, $prq) = @_; # Plack::Request
 	if (defined $prq) {
diff --git a/lib/PublicInbox/WWW.pm b/lib/PublicInbox/WWW.pm
index 5b4d6c18..e8f1fbf0 100644
--- a/lib/PublicInbox/WWW.pm
+++ b/lib/PublicInbox/WWW.pm
@@ -252,8 +252,6 @@ sub footer {
 	my ($ctx) = @_;
 	return '' unless $ctx;
 	my $obj = $ctx->{-inbox} or return '';
-	my $footer = $obj->footer_html;
-	return $ctx->{footer} = $footer if $footer;
 
 	# auto-generate a footer
 	chomp(my $desc = $obj->description);