From: travis%sedsystems.ca <>
Date: Fri, 28 Jan 2005 03:14:22 +0000 (+0000)
Subject: Bug 276605 : Ignore vote changes when usevotes = 0
X-Git-Tag: bugzilla-2.18.1~54
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=42338e8b18dd55ce8b09bd1d17217d26dec65f93;p=thirdparty%2Fbugzilla.git

Bug 276605 : Ignore vote changes when usevotes = 0
Patch by Frederic Buclin <LpSolit@gmail.com>   r=wurblzap  a=myk
---

diff --git a/template/en/default/bug/votes/list-for-user.html.tmpl b/template/en/default/bug/votes/list-for-user.html.tmpl
index 9bb3a1dd63..968c66862b 100644
--- a/template/en/default/bug/votes/list-for-user.html.tmpl
+++ b/template/en/default/bug/votes/list-for-user.html.tmpl
@@ -23,13 +23,16 @@
 
 [% IF !header_done %]
   [% h2 = voting_user.login FILTER html %]
-  [% PROCESS global/header.html.tmpl title = "Show Votes" %]
+  [% IF canedit %]
+    [% title = "Change Votes" %]
+  [% ELSE %]
+    [% title = "Show Votes" %]
+  [% END %]
+  [% PROCESS global/header.html.tmpl %]
 [% ELSE %]
   <hr>
 [% END %]
 
-[% canedit = 1 IF voting_user.login == user.login %]
-
 [% IF votes_recorded %]
   <p>
     <font color="red">
diff --git a/votes.cgi b/votes.cgi
index bbdbb34502..984d1f469f 100755
--- a/votes.cgi
+++ b/votes.cgi
@@ -76,7 +76,7 @@ elsif ($action eq "show_user") {
     show_user();
 }
 elsif ($action eq "vote") {
-    record_votes();
+    record_votes() if Param('usevotes');
     show_user();
 }
 else {
@@ -128,8 +128,7 @@ sub show_user {
     my $who = DBNameToIdAndCheck($name);
     my $userid = Bugzilla->user ? Bugzilla->user->id : 0;
     
-    my $canedit = 1 if (Bugzilla->user &&
-                        $name eq Bugzilla->user->login);
+    my $canedit = (Param('usevotes') && $userid == $who) ? 1 : 0;
     
     SendSQL("LOCK TABLES bugs READ, products READ, votes WRITE,
              cc READ, bug_group_map READ, user_group_map READ,
@@ -214,7 +213,8 @@ sub show_user {
 
     SendSQL("DELETE FROM votes WHERE vote_count <= 0");
     SendSQL("UNLOCK TABLES");
-    
+
+    $vars->{'canedit'} = $canedit;
     $vars->{'voting_user'} = { "login" => $name };
     $vars->{'products'} = \@products;