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 00d67c599ec9bdb6bcafd122a53ae8c7a09243f9..eda2e988dfff1efdcdca08799735e7f476ce4863 100755 (executable)
--- 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