external/aurweb
0
0
Fork 0
mirror of https://gitlab.archlinux.org/archlinux/aurweb.git synced 2025-02-03 10:43:03 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions

Fix pkgbase_votes_from_name()

Browse source 
In 676595f (Prefix package functions with pkg_/pkgbase_, 2014-04-05),
votes_for_pkgname() was renamed to pkgbase_votes_from_name() without
changing the semantics. Slightly adapt the implementation and interpret
the argument as a package base name instead of a package name. Also fix
the call site.

Reported-by: Felix Yan <felixonmars@gmail.com>
Signed-off-by: Lukas Fleischer <archlinux@cryptocrack.de>
This commit is contained in:
Lukas Fleischer 2014-07-16 17:21:08 +02:00
parent e9b13cef3f
commit d03f7a890f
2 changed files with 13 additions and 12 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
web/html/voters.php
View file

@ -1,11 +1,11 @@
<?php <?php
set_include_path(get_include_path() . PATH_SEPARATOR . '../lib'); set_include_path(get_include_path() . PATH_SEPARATOR . '../lib');
include_once('aur.inc.php'); include_once('aur.inc.php');
include_once('pkgfuncs.inc.php'); include_once('pkgbasefuncs.inc.php');
$SID = $_COOKIE['AURSID']; $SID = $_COOKIE['AURSID'];
$pkgname = htmlspecialchars($_GET['N']); $pkgbase_name = htmlspecialchars($_GET['N']);
$votes = pkgbase_votes_from_name($pkgname); $votes = pkgbase_votes_from_name($pkgbase_name);
html_header(__("Voters")); html_header(__("Voters"));
@ -13,7 +13,7 @@ if (has_credential(CRED_PKGBASE_LIST_VOTERS)):
?> ?>
<div class="box"> <div class="box">
<h2>Votes for <a href="<?= get_pkg_uri($pkgname); ?>"><?= $pkgname ?></a></h2> <h2>Votes for <a href="<?= get_pkgbase_uri($pkgbase_name); ?>"><?= $pkgbase_name ?></a></h2>
<div class="boxbody"> <div class="boxbody">
<ul> <ul>
<?php while (list($indx, $row) = each($votes)): ?> <?php while (list($indx, $row) = each($votes)): ?>

17
web/lib/pkgbasefuncs.inc.php
View file

@ -683,19 +683,20 @@ function pkgbase_vote ($base_ids, $action=true) {
} }
/** /**
* Get all usernames and IDs that voted for a specific package * Get all usernames and IDs that voted for a specific package base
* *
* @param string $pkgname The name of the package to retrieve votes for * @param string $pkgbase_name The package base to retrieve votes for
* *
* @return array User IDs and usernames that voted for a specific package * @return array User IDs and usernames that voted for a specific package base
*/ */
function pkgbase_votes_from_name($pkgname) { function pkgbase_votes_from_name($pkgbase_name) {
$dbh = DB::connect(); $dbh = DB::connect();
$q = "SELECT UsersID,Username,Name FROM PackageVotes "; $q = "SELECT UsersID, Username, Name FROM PackageVotes ";
$q.= "LEFT JOIN Users on (UsersID = Users.ID) "; $q.= "LEFT JOIN Users ON UsersID = Users.ID ";
$q.= "LEFT JOIN Packages on (PackageVotes.PackageBaseID = Packages.PackageBaseID) "; $q.= "LEFT JOIN PackageBases ";
$q.= "WHERE Name = ". $dbh->quote($pkgname) . " "; $q.= "ON PackageVotes.PackageBaseID = PackageBases.ID ";
$q.= "WHERE PackageBases.Name = ". $dbh->quote($pkgbase_name) . " ";
$q.= "ORDER BY Username"; $q.= "ORDER BY Username";
$result = $dbh->query($q); $result = $dbh->query($q);