From 2fe007151dd2d7caec031cd1fca617eac47f2549 Mon Sep 17 00:00:00 2001
From: Sunil Joshi <joshi_sunil@in.com>
Date: Thu, 26 Sep 2013 12:02:01 +1000
Subject: Bug 621216 - Don't call GetQuip() if the user doesn't want quips
 r=simon, a=simon

---
 buglist.cgi | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

(limited to 'buglist.cgi')

diff --git a/buglist.cgi b/buglist.cgi
index 00d67c599..eda2e988d 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
-- 
cgit v1.2.3-24-g4f1b