From: Sunil Joshi <joshi_sunil@in.com>
Date: Thu, 26 Sep 2013 02:02:01 +0000 (+1000)
Subject: Bug 621216 - Don't call GetQuip() if the user doesn't want quips
X-Git-Tag: bugzilla-4.5.1~29
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=2fe007151dd2d7caec031cd1fca617eac47f2549;p=thirdparty%2Fbugzilla.git

Bug 621216 - Don't call GetQuip() if the user doesn't want quips
r=simon, a=simon
---

diff --git a/buglist.cgi b/buglist.cgi
index 00d67c599e..eda2e988df 100755
--- a/buglist.cgi
+++ b/buglist.cgi
@@ -905,7 +905,10 @@ if (scalar(@bugowners) > 1 && $user->in_group('editbugs')) {
 # the list more compact.
 $vars->{'splitheader'} = $cgi->cookie('SPLITHEADER') ? 1 : 0;
 
-$vars->{'quip'} = GetQuip();
+if ($user->settings->{'display_quips'}->{'value'} eq 'on') {
+    $vars->{'quip'} = GetQuip();
+}
+
 $vars->{'currenttime'} = localtime(time());
 
 # See if there's only one product in all the results (or only one product