summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorLukas Fleischer <archlinux@cryptocrack.de>2013-08-27 11:31:49 +0200
committerLukas Fleischer <archlinux@cryptocrack.de>2013-08-27 11:35:26 +0200
commitc1c7f9b35090904a4c566de026c487a0eb33da49 (patch)
tree1e340a7d866495ccf34a81f9a1908b2c9bdad512
parentbf019a5b6c66ab8a53382a739f1adf7dd568db23 (diff)
downloadaurweb-c1c7f9b35090904a4c566de026c487a0eb33da49.tar.xz
Move package comment check to packages.php
Checking whether to add a comment is something that really does not belong to a function named display_package_details(). Signed-off-by: Lukas Fleischer <archlinux@cryptocrack.de>
-rw-r--r--web/html/packages.php6
-rw-r--r--web/lib/pkgfuncs.inc.php4
2 files changed, 6 insertions, 4 deletions
diff --git a/web/html/packages.php b/web/html/packages.php
index e03a099..a9f6263 100644
--- a/web/html/packages.php
+++ b/web/html/packages.php
@@ -99,6 +99,12 @@ if (check_token()) {
list($ret, $output) = pkg_change_category($pkgid, $atype);
}
+ if (isset($_REQUEST['comment'])) {
+ $uid = uid_from_sid($_COOKIE["AURSID"]);
+ add_package_comment($pkgid, $uid, $_REQUEST['comment']);
+ $ret = true;
+ }
+
if ($ret) {
/* Redirect back to package page on success. */
header('Location: ' . get_pkg_uri($pkgname));
diff --git a/web/lib/pkgfuncs.inc.php b/web/lib/pkgfuncs.inc.php
index 1350bf5..a4deee6 100644
--- a/web/lib/pkgfuncs.inc.php
+++ b/web/lib/pkgfuncs.inc.php
@@ -449,10 +449,6 @@ function display_package_details($id=0, $row, $SID="") {
# Actions Bar
if ($SID) {
include('actions_form.php');
- if (isset($_REQUEST['comment']) && check_token()) {
- $uid = uid_from_sid($SID);
- add_package_comment($id, $uid, $_REQUEST['comment']);
- }
include('pkg_comment_form.php');
}