appdb/include vote.php
WineHQ
wineowner at wine.codeweavers.com
Tue Jul 18 19:11:40 CDT 2006
ChangeSet ID: 26632
CVSROOT: /opt/cvs-commit
Module name: appdb
Changes by: wineowner at winehq.org 2006/07/18 19:11:40
Modified files:
include : vote.php
Log message:
Chris Morgan <cmorgan at alum.wpi.edu>
Fix voting(again) by changing all variables to use the new naming scheme
Patch: http://cvs.winehq.org/patch.py?id=26632
Old revision New revision Changes Path
1.22 1.23 +39 -39 appdb/include/vote.php
Index: appdb/include/vote.php
diff -u -p appdb/include/vote.php:1.22 appdb/include/vote.php:1.23
--- appdb/include/vote.php:1.22 19 Jul 2006 0:11:40 -0000
+++ appdb/include/vote.php 19 Jul 2006 0:11:40 -0000
@@ -7,18 +7,18 @@ define('MAX_VOTES',3);
/**
* count the number of votes for appId by userId
*/
-function vote_count($appId, $userId = null)
+function vote_count($iAppId, $iUserId = null)
{
- if(!$userId)
+ if(!$iUserId)
{
if($_SESSION['current']->isLoggedIn())
- $userId = $_SESSION['current']->iUserId;
+ $iUserId = $_SESSION['current']->iUserId;
else
return 0;
-}
+ }
$hResult = query_parameters("SELECT * FROM appVotes WHERE appId = '?' AND userId = '?'",
- $appId, $userId);
+ $iAppId, $iUserId);
return mysql_num_rows($hResult);
}
@@ -26,16 +26,16 @@ function vote_count($appId, $userId = nu
/**
* total votes by userId
*/
-function vote_count_user_total($userId = null)
+function vote_count_user_total($iUserId = null)
{
- if(!$userId)
+ if(!$iUserId)
{
if($_SESSION['current']->isLoggedIn())
- $userId = $_SESSION['current']->iUserId;
+ $iUserId = $_SESSION['current']->iUserId;
else
return 0;
}
- $hResult = query_parameters("SELECT * FROM appVotes WHERE userId = '?'", $userId);
+ $hResult = query_parameters("SELECT * FROM appVotes WHERE userId = '?'", $iUserId);
return mysql_num_rows($hResult);
}
@@ -43,9 +43,9 @@ function vote_count_user_total($userId =
/*
* total votes for appId
*/
-function vote_count_app_total($appId)
+function vote_count_app_total($iAppId)
{
- $hResult = query_parameters("SELECT * FROM appVotes WHERE appId = '?'", $appId);
+ $hResult = query_parameters("SELECT * FROM appVotes WHERE appId = '?'", $iAppId);
return mysql_num_rows($hResult);
}
@@ -53,55 +53,55 @@ function vote_count_app_total($appId)
/**
* add a vote for appId
*/
-function vote_add($appId, $slot, $userId = null)
+function vote_add($iAppId, $iSlot, $iUserId = null)
{
- if(!$userId)
+ if(!$iUserId)
{
if($_SESSION['current']->isLoggedIn())
- $userId = $_SESSION['current']->iUserId;
+ $iUserId = $_SESSION['current']->iUserId;
else
return;
}
- if($slot > MAX_VOTES)
+ if($iSlot > MAX_VOTES)
return;
- vote_remove($slot, $userId);
+ vote_remove($iSlot, $iUserId);
query_parameters("INSERT INTO appVotes (id, time, appId, userId, slot)
- VALUES (?, ?, '?', '?', '?')", "null", "null", $appId, $userId, $slot);
+ VALUES (?, ?, '?', '?', '?')", "null", "null", $iAppId, $iUserId, $iSlot);
}
/**
* remove vote for a slot
*/
-function vote_remove($slot, $userId = null)
+function vote_remove($iSlot, $iUserId = null)
{
- if(!$userId)
+ if(!$iUserId)
{
if($_SESSION['current']->isLoggedIn())
- $userId = $_SESSION['current']->iUserId;
+ $iUserId = $_SESSION['current']->iUserId;
else
return;
}
$sQuery = "DELETE FROM appVotes WHERE userId = '?' AND slot = '?'";
- query_parameters($sQuery, $userId, $slot);
+ query_parameters($sQuery, $iUserId, $iSlot);
}
-function vote_get_user_votes($userId = null)
+function vote_get_user_votes($iUserId = null)
{
- if(!$userId)
+ if(!$iUserId)
{
if($_SESSION['current']->isLoggedIn())
- $userId = $_SESSION['current']->iUserId;
- if(!$userId)
+ $iUserId = $_SESSION['current']->iUserId;
+ if(!$iUserId)
return array();
}
- $hResult = query_parameters("SELECT * FROM appVotes WHERE userId = '?'", $userId);
+ $hResult = query_parameters("SELECT * FROM appVotes WHERE userId = '?'", $iUserId);
if(!$hResult)
return array();
@@ -128,7 +128,7 @@ function vote_menu()
{
$sAppName = Application::lookup_name($votes[$i]->appId);
$str = "<a href='appview.php?iAppId=".$votes[$i]->appId."'> $sAppName</a>";
- $m->add("<input type=radio name=slot value='$i'> ".$str);
+ $m->add("<input type=radio name=iSlot value='$i'> ".$str);
}
else
$m->add("<input type=radio name=iSlot value='$i'> No App Selected");
@@ -156,7 +156,7 @@ function vote_update($vars)
if( !is_numeric($vars['iAppId']) OR !is_numeric($vars['iSlot']))
{
if(is_numeric($vars['iAppId']))
- util_redirect_and_exit(apidb_fullurl("appview.php?iAppId=".$vars["iAppId"]));
+ util_redirect_and_exit(apidb_fullurl("appview.php?iAppId=".$vars['iAppId']));
else
util_redirect_and_exit(apidb_fullurl("index.php"));
@@ -165,37 +165,37 @@ function vote_update($vars)
if($vars["sVote"])
{
- addmsg("Registered vote for App #".$vars["iAppId"], "green");
- vote_add($vars["iAppId"], $vars["slot"]);
- } else if($vars["sClear"])
+ addmsg("Registered vote for App #".$vars['iAppId'], "green");
+ vote_add($vars['iAppId'], $vars['iSlot']);
+ } else if($vars['sClear'])
{
/* see if we have a vote in this slot, if we don't there is */
/* little reason to remove it or even mention that we did anything */
- if(is_vote_in_slot($vars["slot"]))
+ if(is_vote_in_slot($vars['iSlot']))
{
- vote_remove($vars["slot"]);
- addmsg("Removed vote for App #".$vars["iAppId"], "green");
+ vote_remove($vars['iSlot']);
+ addmsg("Removed vote for App #".$vars['iAppId'], "green");
}
}
- util_redirect_and_exit(apidb_fullurl("appview.php?iAppId=".$vars["iAppId"]));
+ util_redirect_and_exit(apidb_fullurl("appview.php?iAppId=".$vars['iAppId']));
}
// tell us if there is a vote in a given slot so we don't
// display incorrect information to the user or go
// through the trouble of trying to remove a vote that doesn't exist
-function is_vote_in_slot($slot, $userId = null)
+function is_vote_in_slot($iSlot, $iUserId = null)
{
- if(!$userId)
+ if(!$iUserId)
{
if($_SESSION['current']->isLoggedIn())
- $userId = $_SESSION['current']->iUserId;
+ $iUserId = $_SESSION['current']->iUserId;
else
return;
}
$sQuery = "SELECT COUNT(*) as count from appVotes WHERE userId = '?' AND slot = '?'";
- if($hResult = query_parameters($sQuery, $userId, $slot))
+ if($hResult = query_parameters($sQuery, $iUserId, $iSlot))
{
$oRow = mysql_fetch_object($hResult);
if($oRow->count != 0)
More information about the wine-cvs
mailing list