diff --git a/about.php b/about.php index 295609d..c5ebdc8 100644 --- a/about.php +++ b/about.php @@ -1,11 +1,11 @@ - +

About

@@ -34,6 +34,6 @@

- + \ No newline at end of file diff --git a/candidate.php b/candidate.php index 77664fc..d731791 100644 --- a/candidate.php +++ b/candidate.php @@ -2,12 +2,12 @@ ob_start(); session_start(); - require_once("util/user_input.php"); - require_once("util/db.php"); - require_once("util/ALD.php"); + require_once('util/user_input.php'); + require_once('util/db.php'); + require_once('util/ALD.php'); - require_once("config/constants.php"); - require_once("util/privilege.php"); + require_once('config/constants.php'); + require_once('util/privilege.php'); require_once('modules/semver/semver.php'); require_once('util/get_privilege_symbols.php'); @@ -21,28 +21,28 @@ $error = true; # assume error here, reset on success $page_title = 'ERROR'; - if (isset($_GET["id"])) + if (isset($_GET['id'])) { - $id = mysql_real_escape_string($_GET["id"], $db_connection); - $logged_in = isset($_SESSION["userID"]); - $can_close = $logged_in && hasPrivilege($_SESSION["privileges"], PRIVILEGE_STDLIB); + $id = mysql_real_escape_string($_GET['id'], $db_connection); + $logged_in = isset($_SESSION['userID']); + $can_close = $logged_in && hasPrivilege($_SESSION['privileges'], PRIVILEGE_STDLIB); $diff = false; if (!empty($_POST) && $logged_in) { - if (isset($_POST["newcomment"])) + if (isset($_POST['newcomment'])) { - if (isset($_POST["vote"])) + if (isset($_POST['vote'])) { - $vote = (int)(mysql_real_escape_string($_POST["vote"])); + $vote = (int)(mysql_real_escape_string($_POST['vote'])); if (in_array($vote, array(-1, 0, 1))) { - $db_query = "SELECT COUNT(*) FROM $db_table_candidate_comments WHERE id = '$id' AND vote != '0' AND user = UNHEX('{$_SESSION["userID"]}')"; + $db_query = 'SELECT COUNT(*) FROM ' . DB_TABLE_CANDIDATE_COMMENTS . ' WHERE id = "' . $id . '" AND vote != "0" AND user = UNHEX("' . $_SESSION['userID'] . '")'; $db_result = mysql_query($db_query, $db_connection); if (!$db_result) { - $error_message = "Failed to get previous votes: MySQL error"; - $error_description = "Could not check if current user has already voted. MySQL error was: '" . mysql_error() . "'"; + $error_message = 'Failed to get previous votes: MySQL error'; + $error_description = 'Could not check if current user has already voted. MySQL error was: "' . mysql_error() . '"'; break; } $can_vote = mysql_fetch_object($db_result)->{'COUNT(*)'} == 0; # set to false if there's already a comment by the current user with a vote @@ -54,60 +54,60 @@ $vote = 0; } - $db_query = "INSERT INTO $db_table_candidate_comments (id, user, comment, vote) VALUES ($id, UNHEX('{$_SESSION["userID"]}'), '" . mysql_real_escape_string($_POST["newcomment"]) . "', '" . $vote . "')"; + $db_query = 'INSERT INTO ' . DB_TABLE_CANDIDATE_COMMENTS . ' (id, user, comment, vote) VALUES (' . $id . ', UNHEX("' . $_SESSION['userID'] . '"), "' . mysql_real_escape_string($_POST['newcomment']) . '", "' . $vote . '")'; $db_result = mysql_query($db_query, $db_connection); if (!$db_result) { - $error_message = "Failed to save comment: MySQL error"; - $error_description = "Could not save your last comment on this thread. MySQL error was: '" . mysql_error() . "'"; + $error_message = 'Failed to save comment: MySQL error'; + $error_description = 'Could not save your last comment on this thread. MySQL error was: "' . mysql_error() . '"'; break; } } - else if (isset($_POST["accept"]) || isset($_POST["reject"])) + else if (isset($_POST['accept']) || isset($_POST['reject'])) { if ($can_close) { - $db_query = "UPDATE $db_table_candidates Set closed = '1', closed-by = UNHEX('{$_SESSION["userID"]}'), closed-date = NOW(), closed-comment = '" . mysql_real_escape_string($_POST["closecomment"]) . "' WHERE id = '$id'"; + $db_query = 'UPDATE ' . DB_TABLE_CANDIDATES . ' Set closed = "1", closed-by = UNHEX("' . $_SESSION['userID'] . '"), closed-date = NOW(), closed-comment = "' . mysql_real_escape_string($_POST['closecomment']) . '" WHERE id = "' . $id . '"'; $db_result = mysql_query($db_query, $db_connection); if (!$db_result) { - $error_message = "Failed to close this thread: MySQL error"; - $error_description = "Could not close the thread. MySQL error was: '" . mysql_error() . "'"; + $error_message = 'Failed to close this thread: MySQL error'; + $error_description = 'Could not close the thread. MySQL error was: "' . mysql_error() . '"'; break; } - $db_query = "UPDATE $db_table_main Set default_include = '1' WHERE id = UNHEX('')"; # todo + $db_query = 'UPDATE $db_table_main Set default_include = "1" WHERE id = UNHEX("")'; # todo # TODO } } - header("Location: " . $_SERVER["REQUEST_URI"]); # reload to clear POST data and avoid repost of comment + header('Location: ' . $_SERVER['REQUEST_URI']); # reload to clear POST data and avoid repost of comment } - $db_query = "SELECT *, HEX(libid), HEX(userid), HEX(`closed-by`) FROM $db_table_candidates WHERE id = '$id'"; + $db_query = 'SELECT *, HEX(libid), HEX(userid), HEX(`closed-by`) FROM ' . DB_TABLE_CANDIDATES . ' WHERE id = "' . $id . '"'; $db_result = mysql_query($db_query, $db_connection); if (!$db_result) { - $error_message = "Failed to retrieve thread: MySQL error"; - $error_description = "Could not retrieve data on this thread. MySQL error was: '" . mysql_error() . "'"; + $error_message = 'Failed to retrieve thread: MySQL error'; + $error_description = 'Could not retrieve data on this thread. MySQL error was: "' . mysql_error() . '"'; break; } if (mysql_num_rows($db_result) != 1) { - $error_message = "Failed to retrieve thread: not found"; - $error_description = "Could not find this thread. Most likely, the URL is incorrect."; + $error_message = 'Failed to retrieve thread: not found'; + $error_description = 'Could not find this thread. Most likely, the URL is incorrect.'; break; } $candidate = mysql_fetch_assoc($db_result); - $lib = $api->getItemById($candidate["HEX(libid)"]); - $candidate["libname"] = $lib["name"]; - $candidate["libversion"] = $lib["version"]; - $temp = $api->getUserById($candidate["HEX(userid)"]); - $candidate["username"] = $temp["name"]; - if ($candidate["closed"]) + $lib = $api->getItemById($candidate['HEX(libid)']); + $candidate['libname'] = $lib['name']; + $candidate['libversion'] = $lib['version']; + $temp = $api->getUserById($candidate['HEX(userid)']); + $candidate['username'] = $temp['name']; + if ($candidate['closed']) { - $temp = $api->getUserById($candidate["HEX(`closed-by`)"]); - $candidate["closed-by"] = $temp["name"]; + $temp = $api->getUserById($candidate['HEX(`closed-by`)']); + $candidate['closed-by'] = $temp['name']; } else { @@ -116,45 +116,45 @@ if (count($list) > 0) { $diff = true; - usort($list, "semver_sort"); # sort by version numbers (descending) + usort($list, 'semver_sort'); # sort by version numbers (descending) $diff_base = $list[0]['version']; } } $comments = array(); - $db_query = "SELECT *, HEX(user) FROM $db_table_candidate_comments WHERE id = '$id'"; + $db_query = 'SELECT *, HEX(user) FROM ' . DB_TABLE_CANDIDATE_COMMENTS . ' WHERE id = "' . $id . '"'; $db_result = mysql_query($db_query, $db_connection); if (!$db_result) { - $error_message = "Failed to retrieve comments: MySQL error"; - $error_description = "Could not read the comments on this thread. MySQL error was: '" . mysql_error() . "'"; + $error_message = 'Failed to retrieve comments: MySQL error'; + $error_description = 'Could not read the comments on this thread. MySQL error was: "' . mysql_error() . '"'; break; } while ($comment = mysql_fetch_assoc($db_result)) { - $temp = $api->getUserById($comment["HEX(user)"]); - $comment["user"] = $temp["name"]; - $comment["user-mail"] = $temp["mail-md5"]; + $temp = $api->getUserById($comment['HEX(user)']); + $comment['user'] = $temp['name']; + $comment['user-mail'] = $temp['mail-md5']; $comment['user-privilege'] = $temp['privileges']; $comments[] = $comment; } - $db_query = "SELECT COUNT(*) FROM $db_table_candidate_comments WHERE id = '$id' AND vote > '0'"; # get upvote count + $db_query = 'SELECT COUNT(*) FROM ' . DB_TABLE_CANDIDATE_COMMENTS . ' WHERE id = "' . $id . '" AND vote > 0'; # get upvote count $db_result = mysql_query($db_query, $db_connection); if (!$db_result) { - $error_message = "Failed to retrieve upvote count: MySQL error"; - $error_description = "The number of upvotes could not be read. MySQL error was: '" . mysql_error() . "'"; + $error_message = 'Failed to retrieve upvote count: MySQL error'; + $error_description = 'The number of upvotes could not be read. MySQL error was: "' . mysql_error() . '"'; break; } $up_vote_count = mysql_fetch_object($db_result)->{'COUNT(*)'}; - $db_query = "SELECT COUNT(*) FROM $db_table_candidate_comments WHERE id = '$id' AND vote < '0'"; # get downvote count + $db_query = 'SELECT COUNT(*) FROM ' . DB_TABLE_CANDIDATE_COMMENTS . ' WHERE id = "' . $id . '" AND vote < 0'; # get downvote count $db_result = mysql_query($db_query, $db_connection); if (!$db_result) { - $error_message = "Failed to retrieve downvote count: MySQL error"; - $error_description = "The number of downvotes could not be read. MySQL error was: '" . mysql_error() . "'"; + $error_message = 'Failed to retrieve downvote count: MySQL error'; + $error_description = 'The number of downvotes could not be read. MySQL error was: "' . mysql_error() . '"'; break; } $down_vote_count = mysql_fetch_object($db_result)->{'COUNT(*)'}; @@ -163,55 +163,55 @@ if ($logged_in) { - $db_query = "SELECT COUNT(*) FROM $db_table_candidate_comments WHERE id = '$id' AND vote != '0' AND user = UNHEX('{$_SESSION["userID"]}')"; + $db_query = 'SELECT COUNT(*) FROM ' . DB_TABLE_CANDIDATE_COMMENTS . ' WHERE id = "' . $id . '" AND vote != 0 AND user = UNHEX("' . $_SESSION['userID'] . '")'; $db_result = mysql_query($db_query, $db_connection); if (!$db_result) { - $error_message = "Failed to get previous votes: MySQL error"; - $error_description = "Could not check if current user has already voted. MySQL error was: '" . mysql_error() . "'"; + $error_message = 'Failed to get previous votes: MySQL error'; + $error_description = 'Could not check if current user has already voted. MySQL error was: "' . mysql_error() . '"'; break; } $can_vote = mysql_fetch_object($db_result)->{'COUNT(*)'} == 0; # set to false if there's already a comment by the current user with a vote } - $page_title = ($candidate["closed"] ? "closed: " : "") . $candidate["libname"] . " v" . $candidate["libversion"] . " | Candidate for stdlib"; + $page_title = ($candidate['closed'] ? 'closed: ' : '') . $candidate['libname'] . ' v' . $candidate['libversion'] . ' | Candidate for stdlib'; } else { - $page_title = "Candidates for the standard library"; + $page_title = 'Candidates for the standard library'; - $db_cond = "closed != '1'"; - if (isset($_GET["mode"])) + $db_cond = 'closed != 1'; + if (isset($_GET['mode'])) { - if (strtolower($_GET["mode"]) == "closed") + if (strtolower($_GET['mode']) == 'closed') { - $db_cond = "closed = '1'"; - $page_title .= " (closed)"; + $db_cond = 'closed = 1'; + $page_title .= ' (closed)'; } - else if (strtolower($_GET["mode"]) == "all") + else if (strtolower($_GET['mode']) == 'all') { - $db_cond = "'1' = '1'"; + $db_cond = '1 = 1'; } } - $db_query = "SELECT id, HEX(libid), HEX(userid), date, closed FROM $db_table_candidates WHERE $db_cond"; + $db_query = 'SELECT id, HEX(libid), HEX(userid), date, closed FROM ' . DB_TABLE_CANDIDATES . ' WHERE ' . $db_cond; $db_result = mysql_query($db_query, $db_connection); if (!$db_result) { - $error_message = "Failed to retrieve list of candidates: MySQL error"; - $error_description = "The list of candidates could not be read. MySQL error was: '" . mysql_error() . "'"; + $error_message = 'Failed to retrieve list of candidates: MySQL error'; + $error_description = 'The list of candidates could not be read. MySQL error was: "' . mysql_error() . '"'; break; } $candidates = array(); while ($candidate = mysql_fetch_assoc($db_result)) { - $lib = $api->getItemById($candidate["HEX(libid)"]); - $candidate["lib-name"] = $lib["name"]; - $candidate["lib-version"] = $lib["version"]; + $lib = $api->getItemById($candidate['HEX(libid)']); + $candidate['lib-name'] = $lib['name']; + $candidate['lib-version'] = $lib['version']; - $temp = $api->getUserById($candidate["HEX(userid)"]); - $candidate["user"] = $temp["name"]; + $temp = $api->getUserById($candidate['HEX(userid)']); + $candidate['user'] = $temp['name']; $candidates[] = $candidate; } @@ -222,7 +222,7 @@ - + @@ -249,15 +249,15 @@ Library: - "> (v) + (v) User: - /profile"> + Applied: - + @@ -266,11 +266,11 @@ -
+
-
+
-
0 ? "+" : "-") . $total_vote_count; ?> votes
+
+
-
0 ? '+' : '-') . $total_vote_count; ?> votes

Comments

@@ -278,14 +278,14 @@ foreach ($comments AS $comment) { $symbols = get_privilege_symbols($comment['user-privilege']); - echo "" - . ""; + echo '' + . ''; } - if (!$candidate["closed"] && $logged_in) + if (!$candidate['closed'] && $logged_in) { ?> - + " - . ""; + echo '' + . ''; /* if ($can_close && !$in_standard) { @@ -346,14 +346,14 @@ { $status = $cand['closed'] ? 'closed' : 'open'; echo '
' - . "

{$cand['lib-name']} (v{$cand['lib-version']})

" + . '

' . $cand['lib-name'] . ' (v' . $cand['lib-version'] . ')

' . '
' - . "
Name
{$cand['lib-name']}
" - . "
Version
{$cand['lib-version']}
" - . "
User
$cand[user]
" - . "
Date
$cand[date]
" - . "
Status
$status
" - . "
Link
▶ Go to discussion thread ▶
" + . '
Name
' . $cand['lib-name'] . '
' + . '
Version
' . $cand['lib-version'] . '
' + . '
User
' . $cand['user'] . '
' + . '
Date
' . $cand['date'] . '
' + . '
Status
' . $status . '
' + . '
Link
▶ Go to discussion thread ▶
' . '
'; } ?> @@ -362,11 +362,11 @@ } ?> - + diff --git a/config/constants.php b/config/constants.php index 8ae8d13..0160431 100644 --- a/config/constants.php +++ b/config/constants.php @@ -1,16 +1,16 @@ \ No newline at end of file diff --git a/create-account.php b/create-account.php index 0740d1b..a13eecc 100644 --- a/create-account.php +++ b/create-account.php @@ -1,21 +1,21 @@ 0) { - throw new HttpException(409, NULL, "Account already exists"); + throw new HttpException(409, NULL, 'Account already exists'); } # check if user is correct $api = new ALD( API_URL ); try { - $user = $api->getUserById($_POST["id"]); + $user = $api->getUserById($_POST['id']); } catch (HttpException $e) { - throw new HttpException(404, NULL, "User not found in backend: '{$e->getMessage()}'."); + throw new HttpException(404, NULL, 'User not found in backend: "' . $e->getMessage() . '")'; } - if ($user["name"] != $_POST["user"] || $user["mail"] != md5($_POST["mail"])) + if ($user['name'] != $_POST['user'] || $user['mail'] != md5($_POST['mail'])) { - throw new HttpException(400, NULL, "data invalid"); + throw new HttpException(400, NULL, 'data invalid'); } # create account - $id = mysql_real_escape_string($_POST["id"]); - $mail = mysql_real_escape_string($_POST["mail"]); + $id = mysql_real_escape_string($_POST['id']); + $mail = mysql_real_escape_string($_POST['mail']); - $db_query = "INSERT INTO $db_table_user_profile (id, mail) VALUES (UNHEX('$id'), '$mail')"; + $db_query = 'INSERT INTO ' . DB_TABLE_USER_PROFILE . ' (id, mail) VALUES (UNHEX("' . $id . '"), "' . $mail . '")'; $db_result = mysql_query($db_query, $db_connection); if (!$db_result) { throw new HttpException(500, NULL, mysql_error()); } - header("HTTP/1.1 204 " . HttpException::getStatusMessage(204)); + header('HTTP/1.1 204 ' . HttpException::getStatusMessage(204)); exit; } catch (HttpException $e) { - header("HTTP/1.1 {$e->getCode()} " . HttpException::getStatusMessage($e->getCode())); + header('HTTP/1.1 ' . $e->getCode() . ' ' . HttpException::getStatusMessage($e->getCode())); echo $e->getMessage(); } ?> \ No newline at end of file diff --git a/help.php b/help.php index 757eaf6..a58594f 100644 --- a/help.php +++ b/help.php @@ -1,11 +1,11 @@ - + @@ -45,6 +45,6 @@

- + \ No newline at end of file diff --git a/index.php b/index.php index 0a6f07a..1b0fb6c 100644 --- a/index.php +++ b/index.php @@ -1,15 +1,15 @@ - + - +
@@ -40,6 +40,6 @@ apache
- + \ No newline at end of file diff --git a/internal/mailimage.php b/internal/mailimage.php index 42e2a67..15444bc 100644 --- a/internal/mailimage.php +++ b/internal/mailimage.php @@ -1,28 +1,28 @@ @@ -206,10 +206,10 @@ foreach ($meta_actions AS $action) { switch ($action['type']) { case ACT_ADD: $prefix = 'Added:'; - $details = "Property $action[property] has been added to the package metadata."; + $details = 'Property ' . $action['property'] . ' has been added to the package metadata.'; break; case ACT_DEL: $prefix = 'Deleted:'; - $details = "Property $action[property] has been removed from the package metadata."; + $details = 'Property ' . $action['property'] . ' has been removed from the package metadata.'; break; default: case ACT_MOD: $prefix = 'Modified:'; @@ -226,7 +226,7 @@ \ No newline at end of file diff --git a/items/list.php b/items/list.php index f0369e3..7294af0 100644 --- a/items/list.php +++ b/items/list.php @@ -2,54 +2,54 @@ ob_start(); session_start(); - require_once("../util/ALD.php"); - require_once("../config/constants.php"); - require_once("../util/sortArray.php"); + require_once('../util/ALD.php'); + require_once('../config/constants.php'); + require_once('../util/sortArray.php'); require_once('../partials/Notice.php'); $api = new ALD( API_URL ); - $logged_in = isset($_SESSION["user"]); + $logged_in = isset($_SESSION['user']); $error = true; for ($i = 0; $i < 1; $i++) { - $page_title = "Browse "; + $page_title = 'Browse '; - if ($type = (!empty($_GET["type"]) && in_array(strtolower($_GET["type"]), array("app", "lib"))) ? strtolower($_GET["type"]) : NULL) + if ($type = (!empty($_GET['type']) && in_array(strtolower($_GET['type']), array('app', 'lib'))) ? strtolower($_GET['type']) : NULL) { - $page_title .= ($type == "app") ? "applications" : ($type == "lib" ? "libraries" : "libraries and applications"); + $page_title .= ($type == 'app') ? 'applications' : ($type == 'lib' ? 'libraries' : 'libraries and applications'); } else # probably remove unknown type and reload? { - $page_title .= "libraries and applications"; + $page_title .= 'libraries and applications'; } - $user = !empty($_GET["user"]) ? $_GET["user"] : NULL - AND $page_title .= " by $user"; - $stdlib = !empty($_GET["stdlib"]) ? $_GET["stdlib"] : "both" - AND $page_title .= !empty($_GET["stdlib"]) ? " (lib standard)" : ""; - $tags = isset($_GET["tags"]) ? explode("|", $_GET["tags"]) : NULL - AND $page_title .= " (tags: " . implode($tags, ", ") . ")"; + $user = !empty($_GET['user']) ? $_GET['user'] : NULL + AND $page_title .= ' by ' . $user; + $stdlib = !empty($_GET['stdlib']) ? $_GET['stdlib'] : 'both' + AND $page_title .= !empty($_GET['stdlib']) ? ' (lib standard)' : ''; + $tags = isset($_GET['tags']) ? explode('|', $_GET['tags']) : NULL + AND $page_title .= ' (tags: ' . implode($tags, ', ') . ')'; - $page_index = !empty($_GET["page"]) ? (int)$_GET["page"] : 0; - $page_itemcount = !empty($_GET["items"]) ? (int)$_GET["items"] : 20; + $page_index = !empty($_GET['page']) ? (int)$_GET['page'] : 0; + $page_itemcount = !empty($_GET['items']) ? (int)$_GET['items'] : 20; $start_index = $page_index * $page_itemcount; try { - $items = $api->getItemList($start_index, $page_itemcount + 1, $type, $user, NULL, $tags, "latest", $stdlib); + $items = $api->getItemList($start_index, $page_itemcount + 1, $type, $user, NULL, $tags, 'latest', $stdlib); } catch (HttpException $e) { - $error_message = "Failed to get item list: API error"; - $error_description = "The requested list of items could not be retrieved. API error was: '{$e->getMessage()}'"; + $error_message = 'Failed to get item list: API error'; + $error_description = 'The requested list of items could not be retrieved. API error was: "' . $e->getMessage() . '"'; break; } if (count($items) > 0) { - $items = sortArray($items, "name"); + $items = sortArray($items, 'name'); } $error = false; } @@ -57,7 +57,7 @@ - + @@ -76,7 +76,7 @@ } else { - $last_letter = ""; + $last_letter = ''; $i = 0; foreach ($items as $item) { @@ -89,46 +89,46 @@ $current_letter = strtoupper(substr($item['name'], 0, 1)); if (!ctype_alpha($current_letter)) { - $current_letter = ".#?1"; + $current_letter = '.#?1'; } if ($current_letter != $last_letter) { - if ($last_letter != "") + if ($last_letter != '') { - echo ""; + echo ''; } - echo "

$current_letter

'; } else { - echo "No items found that match your query."; + echo 'No items found that match your query.'; } if ($page_index > 0) { - echo ""; + echo ''; } if (count($items) > $page_itemcount) { - echo ""; + echo ''; } } ?>
- + \ No newline at end of file diff --git a/items/view.php b/items/view.php index 748225a..1bbfc61 100644 --- a/items/view.php +++ b/items/view.php @@ -2,35 +2,35 @@ session_start(); ob_start(); - if (!isset($_GET["id"]) && (!isset($_GET["name"]) || !isset($_GET["version"]))) + if (!isset($_GET['id']) && (!isset($_GET['name']) || !isset($_GET['version']))) { - header("Location: ."); + header('Location: .'); exit; } - require_once("../util/ALD.php"); - require_once("../config/constants.php"); + require_once('../util/ALD.php'); + require_once('../config/constants.php'); require_once('../modules/semver/semver.php'); require_once('../modules/HttpException/HttpException.php'); require_once('../partials/Notice.php'); - $logged_in = isset($_SESSION["user"]); + $logged_in = isset($_SESSION['user']); $api = new ALD( API_URL ); $error = false; try { - if (isset($_GET["id"])) + if (isset($_GET['id'])) { - $id = $_GET["id"]; + $id = $_GET['id']; $item = $api->getItemById($id); } else { - $item = $api->getItem($_GET["name"], $_GET["version"]); - $id = $item["id"]; + $item = $api->getItem($_GET['name'], $_GET['version']); + $id = $item['id']; } - $page_title = "\"{$item['name']}\" (v{$item['version']})"; + $page_title = '"' . $item['name'] . '" (v' . $item['version'] . ')'; } catch (HttpException $e) { @@ -43,7 +43,7 @@ - + @@ -70,57 +70,57 @@ $tag "; + echo '' . $tag . ' '; } ?> - +
\"avatar\"
{$comment["user"]}$symbols
{$comment["date"]}
" . user_input_process($comment["comment"]) . '
' . (!empty($comment["vote"]) ? "
+1
" : "") . "
avatar
' . $comment['user'] . '' . $symbols . '
' . $comment['date'] . '
' . user_input_process($comment['comment']) . '
' . (!empty($comment['vote']) ? '
+1
' : '') . '
/profile">You
Now		You
Now
@@ -320,10 +320,10 @@
{$candidate["closed-by"]}
{$candidate["closed-date"]}
" . user_input_process($candidate["closed-comment"]) . "
' . $candidate['closed-by'] . '
' . $candidate['closed-date'] . '
' . user_input_process($candidate['closed-comment']) . '
Reviewed:Yes" : "red\">No") . ""; ?>Yes' : 'red">No') . ''; ?>

Description

getItemList(0, "all", NULL, NULL, $item['name']); + $versions = $api->getItemList(0, 'all', NULL, NULL, $item['name']); # remove the current item from the array - require_once("../util/searchSubArray.php"); - $index = searchSubArray($versions, array("id" => $item["id"])); + require_once('../util/searchSubArray.php'); + $index = searchSubArray($versions, array('id' => $item['id'])); if ($index !== NULL) { unset($versions[$index]); } - usort($versions, "semver_sort"); # sort by "version" field, following semver rules + usort($versions, 'semver_sort'); # sort by "version" field, following semver rules if (count($versions) > 0) { - echo "

Other versions:

'; } } ?> - + diff --git a/login.php b/login.php index 06e3196..6ad5b16 100644 --- a/login.php +++ b/login.php @@ -1,17 +1,17 @@ getUserList(); } - else if ($mode == "logout") + else if ($mode == 'logout') { - $page_title = "Logged out"; + $page_title = 'Logged out'; clearSession(); $should_redirect = true; } } else { - if (isset($_POST["name"]) && isset($_POST["password"])) + if (isset($_POST['name']) && isset($_POST['password'])) { for ($i = 0; $i < 1; $i++) { - if ($mode == "login") + if ($mode == 'login') { - if (isset($_POST["keepLoggedIn"]) && $_POST["keepLoggedIn"]) + if (isset($_POST['keepLoggedIn']) && $_POST['keepLoggedIn']) { session_set_cookie_params(8640000); # 100 Tage } - $page_title = "Login failed"; # assume failure, reset on success + $page_title = 'Login failed'; # assume failure, reset on success $should_redirect = false; $api = new ALD( SECURE_API_URL ); @@ -64,12 +64,12 @@ break; } - $_SESSION["user"] = $_POST['name']; - $_SESSION["userID"] = $user["id"]; - $_SESSION["password"] = $_POST["password"]; - $_SESSION["privileges"] = $user["privileges"]; + $_SESSION['user'] = $_POST['name']; + $_SESSION['userID'] = $user['id']; + $_SESSION['password'] = $_POST['password']; + $_SESSION['privileges'] = $user['privileges']; - $page_title = "Successfully logged in!"; + $page_title = 'Successfully logged in!'; $error = false; $should_redirect = true; } @@ -87,11 +87,11 @@ "; + echo ''; ?> - > + >

- " method="post"> + Enter your personal information below: required="required"/> '; foreach ($user_list AS $user) { - echo ""; + echo ''; } echo ''; } @@ -144,13 +144,13 @@ } if ($should_redirect) { - echo "

Redirecting to $redirect in 10 seconds...

"; + echo '

Redirecting to ' . $redirect . ' in 10 seconds...

'; } } ?>
- + $_SESSION[user]!
Logout"; + echo 'Welcome
' . $_SESSION['user'] . '!
Logout'; } else { diff --git a/register.php b/register.php index 1b61eb9..4d6c8d4 100644 --- a/register.php +++ b/register.php @@ -2,27 +2,27 @@ session_start(); ob_start(); - require_once("config/constants.php"); - require_once("util/ALD.php"); - require_once("util/secure_redirect.php"); + require_once('config/constants.php'); + require_once('util/ALD.php'); + require_once('util/secure_redirect.php'); require_once('partials/Notice.php'); secure_redirect(); - $page_title = "Registration failed"; - $mode = isset($_GET["mode"]) ? strtolower($_GET["mode"]) : "init"; + $page_title = 'Registration failed'; + $mode = isset($_GET['mode']) ? strtolower($_GET['mode']) : 'init'; $error = true; $api = new ALD( API_URL ); if (empty($_POST)) { - if ($mode == "init") + if ($mode == 'init') { - $page_title = "Register"; + $page_title = 'Register'; } - else if ($mode == "verify") + else if ($mode == 'verify') { - $page_title = "Complete your registration"; + $page_title = 'Complete your registration'; } $error = false; } @@ -30,48 +30,48 @@ { for ($i = 0; $i < 1; $i++) { - if ($mode == "init") + if ($mode == 'init') { - if (empty($_POST["name"]) || empty($_POST["mail"]) || empty($_POST["password"]) || empty($_POST["password_alt"])) + if (empty($_POST['name']) || empty($_POST['mail']) || empty($_POST['password']) || empty($_POST['password_alt'])) { - $message = "Data missing"; - $error_description = "Not all data required for a registration is present."; + $message = 'Data missing'; + $error_description = 'Not all data required for a registration is present.'; break; } try { - $api->initRegistration( $_POST["name"], $_POST["mail"], $_POST["password"], $_POST["password_alt"] ); + $api->initRegistration( $_POST['name'], $_POST['mail'], $_POST['password'], $_POST['password_alt'] ); } catch (HttpException $e) { - $message = "Failed to initiate registration"; - $error_description = "The attempt to initiate a registration failed. Error code was: '{$e->getCode()}'. Error message was: '{$e->getMessage()}'."; + $message = 'Failed to initiate registration'; + $error_description = 'The attempt to initiate a registration failed. Error code was: "' . $e->getCode() . '". Error message was: "' . $e->getMessage() . '".'; break; } - $message = "Registration has been initiated successfully. Check your email account for further details."; - $page_title = "Registration initiated"; + $message = 'Registration has been initiated successfully. Check your email account for further details.'; + $page_title = 'Registration initiated'; } - else if ($mode == "verify" && isset($_GET["id"])) + else if ($mode == 'verify' && isset($_GET['id'])) { - if (empty($_POST["token"])) + if (empty($_POST['token'])) { - $message = "Data missing"; - $error_description = "Not all data required for completing your rgistration is present."; + $message = 'Data missing'; + $error_description = 'Not all data required for completing your rgistration is present.'; break; } try { - $api->completeRegistration( $_GET["id"], $_POST["token"] ); + $api->completeRegistration( $_GET['id'], $_POST['token'] ); } catch (HttpException $e) { - $message = "Failed to complete registration"; - $error_description = "Completing the registration with the given token failed. Error message was: '{$e->getMessage()}'."; + $message = 'Failed to complete registration'; + $error_description = 'Completing the registration with the given token failed. Error message was: "' . $e->getMessage() . '"'; break; } - $message = "Your registration has successfully been completed."; - $page_title = "Registration successful"; + $message = 'Your registration has successfully been completed.'; + $page_title = 'Registration successful'; } $error = false; @@ -81,7 +81,7 @@ - + @@ -90,21 +90,21 @@
Look at the image below and type the displayed letters in the box. -
Token"/> -
Image available at: ">/users/register/token/ +
Token +
Image available at: /users/register/token/
- " method="post"> - + + - + Enter the data for the account you want to create: @@ -138,11 +138,11 @@ } ?> - + \ No newline at end of file diff --git a/review.php b/review.php index 9926adf..19ab455 100644 --- a/review.php +++ b/review.php @@ -2,48 +2,48 @@ ob_start(); session_start(); - require_once("util/ALD.php"); - require_once("config/constants.php"); + require_once('util/ALD.php'); + require_once('config/constants.php'); require_once('partials/Notice.php'); $api = new ALD( API_URL ); - $logged_in = isset($_SESSION["user"]); + $logged_in = isset($_SESSION['user']); - if (isset($_GET["id"])) + if (isset($_GET['id'])) { - require_once("util/db.php"); + require_once('util/db.php'); require_once('modules/semver/semver.php'); require_once('util/get_privilege_symbols.php'); $db_connection = db_ensure_connection(); - $id = mysql_real_escape_string($_GET["id"], $db_connection); + $id = mysql_real_escape_string($_GET['id'], $db_connection); $error = true; for ($i = 0; $i < 1; $i++) { - if (isset($_POST["newcomment"])) + if (isset($_POST['newcomment'])) { if ($logged_in) { - $db_query = "INSERT INTO $db_table_review_comments (id, user, comment) VALUES (UNHEX('$id'), UNHEX('{$_SESSION["userID"]}'), '" . mysql_real_escape_string($_POST["newcomment"]) . "')"; + $db_query = 'INSERT INTO ' . DB_TABLE_REVIEW_COMMENTS . ' (id, user, comment) VALUES (UNHEX("' . $id . '"), UNHEX("' . $_SESSION['userID'] . '"), "' . mysql_real_escape_string($_POST['newcomment']) . '")'; $db_result = mysql_query($db_query, $db_connection); if (!$db_result) { - $error_message = "Failed to post comment: MySQL error"; - $error_description = "Could not insert new comment. MySQL error was: '" . mysql_error() . "'"; + $error_message = 'Failed to post comment: MySQL error'; + $error_description = 'Could not insert new comment. MySQL error was: "' . mysql_error() . '"'; break; } } - header("Location: " . $_SERVER["REQUEST_URI"]); + header('Location: ' . $_SERVER['REQUEST_URI']); } - require_once("util/user_input.php"); + require_once('util/user_input.php'); $item = $api->getItemById($id); - $page_title = $item["name"] . " (v{$item["version"]}) | Code review"; + $page_title = $item['name'] . ' (v' . $item['version'] . ') | Code review'; $list = $api->getItemList(0, 'all', NULL, NULL, $item['name'], NULL, NULL, 'both', 'yes'); - usort($list, "semver_sort"); + usort($list, 'semver_sort'); for ($j = 0; $j < count($list); $j++) { @@ -53,21 +53,21 @@ } } - $db_query = "SELECT HEX(user), comment, date FROM $db_table_review_comments WHERE id = UNHEX('$id')"; + $db_query = 'SELECT HEX(user), comment, date FROM ' . DB_TABLE_REVIEW_COMMENTS . ' WHERE id = UNHEX("' . $id . '")'; $db_result = mysql_query($db_query, $db_connection); if (!$db_result) { - $error_message = "Failed to read comments: MySQL error"; - $error_description = "Comments could not be read from database. MySQL error was: '" . mysql_error() . "'"; + $error_message = 'Failed to read comments: MySQL error'; + $error_description = 'Comments could not be read from database. MySQL error was: "' . mysql_error() . '"'; break; } $comments = array(); while ($comment = mysql_fetch_assoc($db_result)) { - $temp = $api->getUserById($comment["HEX(user)"]); - $comment["user"] = $temp["name"]; - $comment["user-mail"] = $temp["mail-md5"]; + $temp = $api->getUserById($comment['HEX(user)']); + $comment['user'] = $temp['name']; + $comment['user-mail'] = $temp['mail-md5']; $comment['user-privilege'] = $temp['privileges']; $comments[] = $comment; } @@ -77,14 +77,14 @@ } else { - $page_title = "Unreviewed items"; - $items = $api->getItemList(0, "all", NULL, NULL, NULL, NULL, NULL, "both", "no"); + $page_title = 'Unreviewed items'; + $items = $api->getItemList(0, 'all', NULL, NULL, NULL, NULL, NULL, 'both', 'no'); } ?> - + @@ -134,13 +134,13 @@ foreach ($comments AS $comment) { $symbols = get_privilege_symbols($comment['user-privilege']); - echo "\"avatar\"
{$comment["user"]}$symbols
{$comment["date"]}
" . user_input_process($comment["comment"]) . "
"; + echo 'avatar
' . $comment['user'] . '' . $symbols . '
' . $comment['date'] . '
' . user_input_process($comment['comment']) . '
'; } - if (!$item["reviewed"] && $logged_in) + if (!$item['reviewed'] && $logged_in) { ?> - /profile">You
Now + You
Now @@ -164,12 +164,12 @@ foreach ($items AS $item) { echo '
' - . "

$item[name]

" + . '

' . $item['name'] . '

' . '
' - . "
Library
$item[name]
" - . "
Version
$item[version]
" - . "
User
{$item["user"]["name"]}
" - . "
Link
▶ Go to discussion thread ▶
" + . '
Library
' . $item['name'] . '
' + . '
Version
' . $item['version'] . '
' + . '
User
' . $item['user']['name'] . '
' + . '
Link
▶ Go to discussion thread ▶
' . '
'; } ?> @@ -178,11 +178,11 @@ } ?> - + diff --git a/sitemap.php b/sitemap.php index 4e01f2e..6a88da4 100644 --- a/sitemap.php +++ b/sitemap.php @@ -1,58 +1,58 @@ "libba.net code listing", - "link" => "http://libba.net/items/", - "description" => "Displays a list of all uploaded code"), - array("title" => "libba.net users", - "link" => "http://libba.net/users/", - "description" => "The list of registered libba.net users"), - array("title" => "libba.net help", - "link" => "http://libba.net/help", - "description" => "help / FAQ for libba.net"), - array("title" => "libba.net code upload", - "link" => "http://libba.net/upload", - "description" => "Here you can upload new code"), - array("title" => "about libba.net", - "link" => "http://libba.net/about", - "description" => "libba.net about page") + array('title' => 'libba.net code listing', + 'link' => 'http://libba.net/items/', + 'description' => 'Displays a list of all uploaded code'), + array('title' => 'libba.net users', + 'link' => 'http://libba.net/users/', + 'description' => 'The list of registered libba.net users'), + array('title' => 'libba.net help', + 'link' => 'http://libba.net/help', + 'description' => 'help / FAQ for libba.net'), + array('title' => 'libba.net code upload', + 'link' => 'http://libba.net/upload', + 'description' => 'Here you can upload new code'), + array('title' => 'about libba.net', + 'link' => 'http://libba.net/about', + 'description' => 'libba.net about page') ); $api = new ALD( API_URL ); foreach ($api->getItemList() AS $item) { - $item_data = $api->getItemById($item["id"]); - $items[] = array("title" => "$item[name] v$item[version]", - "link" => "http://libba.net/items/$item[id]", - "description" => htmlentities($item_data["description"], 0, "UTF-8"), - "pubDate" => $item_data["uploaded"]); + $item_data = $api->getItemById($item['id']); + $items[] = array('title' => $item['name'] . ' v' . $item['version'], + 'link' => 'http://libba.net/items/' . $item['id'], + 'description' => htmlentities($item_data['description'], 0, 'UTF-8'), + 'pubDate' => $item_data['uploaded']); } foreach ($api->getUserList() AS $user) { - $url_name = urlencode($user["name"]); - $items[] = array("title" => "profile for $user[name]", - "link" => "http://libba.net/users/$url_name/profile", - "description" => "the libba.net profile page for $user[name]"); - $items[] = array("title" => "activity for $user[name]", - "link" => "http://libba.net/users/$url_name/activity", - "description" => "recent activity by $user[name] on libba.net"); - $items[] = array("title" => "items by $user[name]", - "link" => "http://libba.net/users/$url_name/items", - "description" => "libraries and apps uploaded by $user[name] to libba.net"); - $items[] = array("title" => "achievements by $user[name]", - "link" => "http://libba.net/users/$url_name/achievements", - "description" => "achievements made by $user[name] on libba.net"); + $url_name = urlencode($user['name']); + $items[] = array('title' => 'profile for ' . $user['name'], + 'link' => 'http://libba.net/users/' . $url_name . '/profile', + 'description' => 'the libba.net profile page for ' . $user['name']); + $items[] = array('title' => 'activity for ' . $user['name'], + 'link' => 'http://libba.net/users/' . $url_name . '/activity', + 'description' => 'recent activity by ' . $user['name'] . ' on libba.net'); + $items[] = array('title' => 'items by ' . $user['name'], + 'link' => 'http://libba.net/users/' . $url_name . '/items', + 'description' => 'libraries and apps uploaded by ' . $user['name'] . ' to libba.net'); + $items[] = array('title' => 'achievements by ' . $user['name'], + 'link' => 'http://libba.net/users/' . $url_name . '/achievements', + 'description' => 'achievements made by ' . $user['name'] . ' on libba.net'); } # todo: fetch more items, such as reviews and candidates - echo ""; + echo ''; ?> @@ -69,12 +69,12 @@ "; + echo ''; foreach ($item AS $tag => $value) { - echo "<$tag>$value"; + echo '<' . $tag . '>' . $value . ''; } - echo ""; + echo ''; } ?> diff --git a/upload.php b/upload.php index 3855863..adb1eb4 100644 --- a/upload.php +++ b/upload.php @@ -1,24 +1,24 @@ getUserList(); } @@ -26,7 +26,7 @@ - + @@ -37,10 +37,10 @@

- " method="post" enctype="multipart/form-data"> + To upload your code, please fill in the following fields:
Package information @@ -57,7 +57,7 @@ "; + echo ''; } ?> @@ -70,22 +70,22 @@ uploadItem($_FILES["package"]["tmp_name"], $user, $password); + $id = $conn->uploadItem($_FILES['package']['tmp_name'], $user, $password); } catch (HttpException $e) { - echo "Failed to upload: {$e->getCode()}

{$e->getMessage()}

"; + echo 'Failed to upload: ' . $e->getCode() . '

' . $e->getMessage() . '

'; $error = true; } if (!isset($error)) @@ -98,10 +98,10 @@ } } else - echo "Failed to upload: required data is missing."; + echo 'Failed to upload: required data is missing.'; } ?>
- + \ No newline at end of file diff --git a/users/achievements.php b/users/achievements.php index 4eb8a1f..d02d9e8 100644 --- a/users/achievements.php +++ b/users/achievements.php @@ -2,31 +2,31 @@ ob_start(); session_start(); - if (!isset($_GET["user"])) + if (!isset($_GET['user'])) { - header("Location: ."); + header('Location: .'); } - require_once("../util/sortArray.php"); - require_once("../util/ALD.php"); - require_once("../config/constants.php"); + require_once('../util/sortArray.php'); + require_once('../util/ALD.php'); + require_once('../config/constants.php'); require_once('../partials/Notice.php'); $api = new ALD( API_URL ); - $logged_in = isset($_SESSION["user"]); + $logged_in = isset($_SESSION['user']); $error = true; for ($i = 0; $i < 1; $i++) { - $user = $_GET["user"]; + $user = $_GET['user']; try { $user_data = $api->getUser($user); } catch (HttpException $e) { - $error_message = "Failed to retrieve user: API error"; - $error_description = "User data could not be retrieved. API error was: '{$e->getMessage()}' (code: {$e->getCode()})"; + $error_message = 'Failed to retrieve user: API error'; + $error_description = 'User data could not be retrieved. API error was: "' . $e->getMessage() . '" (code: ' . $e->getCode() . ')'; break; } $page_title = $user; @@ -35,21 +35,21 @@ try { - $libs = $api->getItemList(0, "all", "lib", $user, NULL, NULL, NULL, "yes"); + $libs = $api->getItemList(0, 'all', 'lib', $user, NULL, NULL, NULL, 'yes'); } catch (HttpException $e) { - $error_message = "Failed to retrieve acheivements: API error"; - $error_description = "Libraries in stdlb coud not be retrieved. API error was: '{$e->getMessage()}' (code: {$e->getCode()})"; + $error_message = 'Failed to retrieve achievements: API error'; + $error_description = 'Libraries in stdlib could not be retrieved. API error was: "' . $e->getMessage() . '" (code: ' . $e->getCode() . ')'; break; } - $libs = sortArray($libs, array("name" => false, "version" => true)); + $libs = sortArray($libs, array('name' => false, 'version' => true)); foreach ($libs as $lib) { - $achievements[] = array("text" => "$user's library {$lib["name"]} v{$lib["version"]} is part of the standard lib for AutoHotkey", - "image" => "images/achievements/stdlib.png", - "link" => "items/" . $lib["id"]); + $achievements[] = array('text' => $user . '\'s library ' . $lib['name'] . ' v' . $lib['version'] . ' is part of the standard lib for AutoHotkey', + 'image' => 'images/achievements/stdlib.png', + 'link' => 'items/' . $lib['id']); } $error = false; } @@ -57,14 +57,14 @@ - +

"; + echo '' . $user . '\'s avatar'; echo $page_title; ?>

@@ -86,16 +86,16 @@ ?> \ No newline at end of file diff --git a/users/activity.php b/users/activity.php index 67f3149..f362748 100644 --- a/users/activity.php +++ b/users/activity.php @@ -4,97 +4,97 @@ ob_start(); session_start(); - if (!isset($_GET["user"])) + if (!isset($_GET['user'])) { - header("Location: ."); + header('Location: .'); } - require_once("../util/sortArray.php"); - require_once("../util/ALD.php"); - require_once("../config/constants.php"); - require_once("../util/user_input.php"); - require_once("../util/privilege.php"); - require_once("../util/db.php"); + require_once('../util/sortArray.php'); + require_once('../util/ALD.php'); + require_once('../config/constants.php'); + require_once('../util/user_input.php'); + require_once('../util/privilege.php'); + require_once('../util/db.php'); require_once('../partials/Notice.php'); $api = new ALD( API_URL ); - $logged_in = isset($_SESSION["user"]); + $logged_in = isset($_SESSION['user']); $error = true; for ($i = 0; $i < 1; $i++) { - $user = $_GET["user"]; + $user = $_GET['user']; try { $user_data = $api->getUser($user); } catch (HttpException $e) { - $error_message = "Failed to retrieve user: API error"; - $error_description = "User data could not be retrieved. API error was: '{$e->getMessage()}' (code: {$e->getCode()})"; + $error_message = 'Failed to retrieve user: API error'; + $error_description = 'User data could not be retrieved. API error was: "' . $e->getMessage() . '" (code: ' . $e->getCode() . ')'; break; } $page_title = $user; $db_connection = db_ensure_connection(); - $activity_item_count = !empty($_GET["items"]) ? strtolower($_GET["items"]) : 15; - $db_limit = $activity_item_count == "all" ? "" : " LIMIT $activity_item_count"; + $activity_item_count = !empty($_GET['items']) ? strtolower($_GET['items']) : 15; + $db_limit = $activity_item_count == 'all' ? '' : ' LIMIT ' . $activity_item_count; $activity = array(); $retrieved_items = array(); # joined - $activity[] = array("header" => "$user joined libba.net", - "text" => "Welcome to libba.net, $user! If you have any questions, consult the help or contact us!", - "image" => "images/activity/joined.png", - "date" => $user_data["joined"], - "link" => "./../$user/profile"); + $activity[] = array('header' => $user . ' joined libba.net', + 'text' => 'Welcome to libba.net, ' . $user . '! If you have any questions, consult the help or contact us!', + 'image' => 'images/activity/joined.png', + 'date' => $user_data['joined'], + 'link' => './../' . $user . '/profile'); # get items uploaded try { - $items = $api->getItemList(0, "all", NULL, $user); + $items = $api->getItemList(0, 'all', NULL, $user); } catch (HttpException $e) { - $error_message = "Failed to retrieve activity: API error"; - $error_description = "Could not get items uploaded. API error was: '{$e->getMessage()}' (code: {$e->getCode()})"; + $error_message = 'Failed to retrieve activity: API error'; + $error_description = 'Could not get items uploaded. API error was: "' . $e->getMessage() . '" (code: ' . $e->getCode() . ')'; break; } foreach ($items AS $item) { - $id = $item["id"]; + $id = $item['id']; $retrieved_items[$id] = $api->getItemById($id); - $activity[] = array("header" => "$user uploaded {$item["name"]} (v{$item["version"]})", - "text" => $retrieved_items[$id]["description"], - "image" => "images/activity/upload.png", - "date" => $retrieved_items[$id]["uploaded"], - "link" => "items/$id"); + $activity[] = array('header' => $user . ' uploaded ' . $item['name'] . ' (v' . $item['version'] . ')', + 'text' => $retrieved_items[$id]['description'], + 'image' => 'images/activity/upload.png', + 'date' => $retrieved_items[$id]['uploaded'], + 'link' => 'items/' . $id); } # get review comments - $db_query = "SELECT HEX(id), comment, date FROM $db_table_review_comments WHERE user = UNHEX('{$user_data["id"]}') ORDER BY date DESC $db_limit"; + $db_query = 'SELECT HEX(id), comment, date FROM ' . DB_TABLE_REVIEW_COMMENTS . ' WHERE user = UNHEX("' . $user_data['id'] . '") ORDER BY date DESC ' . $db_limit; $db_result = mysql_query($db_query, $db_connection); if (!$db_result) { - $error_message = "Failed to retrieve activity: MySQL error"; - $error_description = "Review comments could not be read. MySQL error was: '" . mysql_error() . "'"; + $error_message = 'Failed to retrieve activity: MySQL error'; + $error_description = 'Review comments could not be read. MySQL error was: "' . mysql_error() . '"'; break; } while ($comment = mysql_fetch_assoc($db_result)) { - $id = $comment["HEX(id)"]; + $id = $comment['HEX(id)']; $item = isset($retrieved_items[$id]) ? $retrieved_items[$id] : ($retrieved_items[$id] = $api->getItemById($id)); - $activity[] = array("header" => "$user commented on Code Review for {$item["name"]} v{$item["version"]}", - "text" => $comment["comment"], - "image" => "images/activity/review-comment.png", - "date" => $comment["date"], - "link" => "reviews/$id"); # todo: anchor for comment + $activity[] = array('header' => $user . 'commented on Code Review for ' . $item['name'] . ' v' . $item['version'] . '', + 'text' => $comment['comment'], + 'image' => 'images/activity/review-comment.png', + 'date' => $comment['date'], + 'link' => 'reviews/' . $id); # todo: anchor for comment } - if (hasPrivilege($user_data["privileges"], PRIVILEGE_REVIEW)) + if (hasPrivilege($user_data['privileges'], PRIVILEGE_REVIEW)) { # get reviews closed by user # todo! @@ -103,70 +103,70 @@ $retrieved_candidates = array(); # get candidates opened and closed - $db_query = "SELECT *, HEX(libid), HEX(userid), HEX(`closed-by`) FROM $db_table_candidates WHERE userid = UNHEX('{$user_data["id"]}') ORDER BY date DESC $db_limit"; + $db_query = 'SELECT *, HEX(libid), HEX(userid), HEX(`closed-by`) FROM ' . DB_TABLE_CANDIDATES . ' WHERE userid = UNHEX("' . $user_data['id'] . '") ORDER BY date DESC ' . $db_limit; $db_result = mysql_query($db_query, $db_connection); if (!$db_result) { - $error_message = "Failed to retrieve activity: MySQL error"; - $error_description = "Failed to retrieve candidates. MySQL error was: '" . mysql_error() . "'"; + $error_message = 'Failed to retrieve activity: MySQL error'; + $error_description = 'Failed to retrieve candidates. MySQL error was: "' . mysql_error() . '"'; break; } while ($candidate = mysql_fetch_assoc($db_result)) { - $retrieved_candidates[$candidate["id"]] = $candidate; # save data for further use + $retrieved_candidates[$candidate['id']] = $candidate; # save data for further use - $id = $candidate["HEX(libid)"]; + $id = $candidate['HEX(libid)']; $item = isset($retrieved_items[$id]) ? $retrieved_items[$id] : ($retrieved_items[$id] = $api->getItemById($id)); - $activity[] = array("header" => "$user proposed {$item["name"]} (v{$item["version"]}) for the stdlib", - "text" => $candidate["text"], - "image" => "images/activity/candidate.png", - "date" => $candidate["date"], - "link" => "candidates/" . $candidate["id"]); + $activity[] = array('header' => $user . ' proposed ' . $item['name'] . ' (v' . $item['version'] . ') for the stdlib', + 'text' => $candidate['text'], + 'image' => 'images/activity/candidate.png', + 'date' => $candidate['date'], + 'link' => 'candidates/' . $candidate['id']); - if ($candidate["closed"]) + if ($candidate['closed']) { - $user = $api->getUserById($candidate["HEX(`closed-by`)"]); - $accepted = $item["default"] ? "accepted" : "rejected"; - $activity[] = array("header" => "The stdlib candidate {$item["name"]} v{$item["version"]} has been $accepted by {$user["name"]}", - "text" => $candidate["closed-comment"], - "image" => "images/activity/candidate-$accepted.png", - "date" => $candidate["closed-date"], - "link" => "candidates/{$candidate["id"]}#closecomment"); + $user = $api->getUserById($candidate['HEX(`closed-by`)']); + $accepted = $item['default'] ? 'accepted' : 'rejected'; + $activity[] = array('header' => 'The stdlib candidate ' . $item['name'] . ' v' . $item['version'] . ' has been ' . $accepted . ' by ' . $user['name'] . '', + 'text' => $candidate['closed-comment'], + 'image' => 'images/activity/candidate-' . $accepted . '.png', + 'date' => $candidate['closed-date'], + 'link' => 'candidates/' . $candidate['id'] . '#closecomment'); } } - if (hasPrivilege($user_data["privileges"], PRIVILEGE_STDLIB)) + if (hasPrivilege($user_data['privileges'], PRIVILEGE_STDLIB)) { # get candidates closed by this user - $db_query = "SELECT *, HEX(libid), HEX(userid), HEX(`closed-by`) FROM $db_table_candidates WHERE `closed-by` = UNHEX('{$user_data["id"]}') ORDER BY date DESC $db_limit"; + $db_query = 'SELECT *, HEX(libid), HEX(userid), HEX(`closed-by`) FROM ' . DB_TABLE_CANDIDATES . ' WHERE `closed-by` = UNHEX("' . $user_data['id'] . '") ORDER BY date DESC ' . $db_limit; $db_result = mysql_query($db_query, $db_connection); if (!$db_result) { - $error_message = "Failed to retrieve activity: MySQL error"; - $error_description = "Could not read candidates closed by $user. MySQL error was: '" . mysql_error() . "'"; + $error_message = 'Failed to retrieve activity: MySQL error'; + $error_description = 'Could not read candidates closed by ' . $user . '. MySQL error was: "' . mysql_error() . '"'; break; } while ($candidate = mysql_fetch_assoc($db_result)) { #$candidates_closed[] = $candidate; # TODO - $retrieved_candidates[$candidate["id"]] = $candidate; + $retrieved_candidates[$candidate['id']] = $candidate; } } # get candidate comments - $db_query = "SELECT id, comment, date, vote FROM $db_table_candidate_comments WHERE user = UNHEX('{$user_data["id"]}') ORDER BY date DESC $db_limit"; + $db_query = 'SELECT id, comment, date, vote FROM ' . DB_TABLE_CANDIDATE_COMMENTS . ' WHERE user = UNHEX("' . $user_data['id'] . '") ORDER BY date DESC ' . $db_limit; $db_result = mysql_query($db_query, $db_connection); if (!$db_result) { - $error_message = "Failed to retrieve activity: MySQL error"; - $error_descrition = "Candidate comments could not be read. MySQL error was: '" . mysql_error() . "'"; + $error_message = 'Failed to retrieve activity: MySQL error'; + $error_descrition = 'Candidate comments could not be read. MySQL error was: "' . mysql_error() . '"'; break; } while ($comment = mysql_fetch_assoc($db_result)) { - $id = $comment["id"]; + $id = $comment['id']; $candidate = isset($retrieved_candidates[$id]) ? $retrieved_candidates[$id] @@ -175,20 +175,20 @@ { break; } - $item_id = $candidate["HEX(libid)"]; + $item_id = $candidate['HEX(libid)']; $item = isset($retrieved_items[$id]) ? $retrieved_items[$id] : ($retrieved_items[$id] = $api->getItemById($item_id)); - $activity[] = array("header" => "$user commented on {$item["name"]} v{$item["version"]} - Stdlib candidate", - "text" => $comment["comment"], - "image" => "images/activity/candidate-comment.png", - "date" => $comment["date"], - "link" => "candidates/$id"); + $activity[] = array('header' => $user . ' commented on ' . $item['name'] . ' v' . $item['version'] . ' - Stdlib candidate', + 'text' => $comment['comment'], + 'image' => 'images/activity/candidate-comment.png', + 'date' => $comment['date'], + 'link' => 'candidates/' . $id); } - $activity = sortArray($activity, array("date" => true)); - if ($activity_item_count != "all") + $activity = sortArray($activity, array('date' => true)); + if ($activity_item_count != 'all') { $activity = array_slice($activity, 0, $activity_item_count); } @@ -199,14 +199,14 @@ - +

"; + echo '' . $user . '\'s avatar'; echo $page_title; ?>

@@ -220,48 +220,48 @@ { foreach ($activity AS $item) { - $text = user_input_clean(strlen($item["text"]) <= $body_char_count - ? $item["text"] - : substr($item["text"], 0, $body_char_count) . "..."); + $text = user_input_clean(strlen($item['text']) <= $body_char_count + ? $item['text'] + : substr($item['text'], 0, $body_char_count) . '...'); - echo "
" - . "
" - . "" - . "\"activity" - . "" - . $item["header"] - . "
" - . "
$text
" - . "
{$item["date"]}
" - . "
"; + echo '
' + . '
' + . '' + . 'activity icon' + . '' + . $item['header'] + . '
' + . '
' . $text . '
' + . '
' . $item['date'] . '
' + . '
'; } } ?> getUser($user); } catch (HttpException $e) { - $error_message = "Failed to retrieve user: API error"; - $error_description = "User data could not be retrieved. API error was: '{$e->getMessage()}' (code: {$e->getCode()})"; + $error_message = 'Failed to retrieve user: API error'; + $error_description = 'User data could not be retrieved. API error was: "' . $e->getMessage() . '" (code: ' . $e->getCode() . ')'; break; } $page_title = $user; try { - $libs = $api->getItemList(0, "all", "lib", $user, NULL, NULL, "latest"); - $apps = $api->getItemList(0, "all", "app", $user, NULL, NULL, "latest"); + $libs = $api->getItemList(0, 'all', 'lib', $user, NULL, NULL, 'latest'); + $apps = $api->getItemList(0, 'all', 'app', $user, NULL, NULL, 'latest'); } catch (HttpException $e) { - $error_message = "Failed to retrieve uploaded items: API error"; - $error_description = "Uploaded items could not be retrieved. API error was: '{$e->getMessage()}' (code: {$e->getCode()})"; + $error_message = 'Failed to retrieve uploaded items: API error'; + $error_description = 'Uploaded items could not be retrieved. API error was: "' . $e->getMessage() . '" (code: ' . $e->getCode() . ')'; break; } - $libs = sortArray($libs, array("name" => false, "version" => true)); - $apps = sortArray($apps, array("name" => false, "version" => true)); + $libs = sortArray($libs, array('name' => false, 'version' => true)); + $apps = sortArray($apps, array('name' => false, 'version' => true)); $error = false; } ?> - +

"; + echo '' . $user . '\'s avatar'; echo $page_title; ?>

@@ -69,14 +69,14 @@ } else # output apps and lib uploaded { - foreach (array("Libraries" => $libs, "Applications" => $apps) AS $set_name => $set) + foreach (array('Libraries' => $libs, 'Applications' => $apps) AS $set_name => $set) { if ($item_count = count($set)) { - echo "

$set_name uploaded ($item_count)