summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorLukas Fleischer <lfleischer@archlinux.org>2015-10-21 22:29:40 +0200
committerLukas Fleischer <lfleischer@archlinux.org>2015-10-21 22:32:29 +0200
commit9c70e10aeb9b07981345e5af86350140753a7707 (patch)
tree2204aa3db1ec2de953a80c013ade424135b53ecf
parent16765d553233e50b326456393108729b1f3828bf (diff)
downloadaur-9c70e10aeb9b07981345e5af86350140753a7707.tar.gz
aur-9c70e10aeb9b07981345e5af86350140753a7707.tar.xz
Check comment length in the backend
Signed-off-by: Lukas Fleischer <lfleischer@archlinux.org>
-rw-r--r--web/html/pkgbase.php7
-rw-r--r--web/lib/pkgbasefuncs.inc.php4
2 files changed, 5 insertions, 6 deletions
diff --git a/web/html/pkgbase.php b/web/html/pkgbase.php
index b98dc648..cbbf3cc4 100644
--- a/web/html/pkgbase.php
+++ b/web/html/pkgbase.php
@@ -49,12 +49,7 @@ $output = "";
$fragment = "";
if (check_token()) {
if (current_action("do_Flag")) {
- if (strlen($_POST['comments']) >= 3) {
- list($ret, $output) = pkgbase_flag($ids, $_POST['comments']);
- } else {
- $output = __("The selected packages have not been flagged, please enter a comment.");
- $ret = false;
- }
+ list($ret, $output) = pkgbase_flag($ids, $_POST['comments']);
} elseif (current_action("do_UnFlag")) {
list($ret, $output) = pkgbase_unflag($ids);
} elseif (current_action("do_Adopt")) {
diff --git a/web/lib/pkgbasefuncs.inc.php b/web/lib/pkgbasefuncs.inc.php
index aad9d14d..afccc7de 100644
--- a/web/lib/pkgbasefuncs.inc.php
+++ b/web/lib/pkgbasefuncs.inc.php
@@ -330,6 +330,10 @@ function pkgbase_flag($base_ids, $comment) {
return array(false, __("You did not select any packages to flag."));
}
+ if (strlen($comment) < 3) {
+ return array(false, __("The selected packages have not been flagged, please enter a comment."));
+ }
+
$uid = uid_from_sid($_COOKIE['AURSID']);
$dbh = DB::connect();