diff options
Diffstat (limited to 'web')
-rw-r--r-- | web/html/packages.php | 54 | ||||
-rw-r--r-- | web/lang/en/pkgfuncs_po.inc | 2 | ||||
-rw-r--r-- | web/lang/en/search_po.inc | 6 | ||||
-rw-r--r-- | web/lib/aurjson.class.php | 2 | ||||
-rw-r--r-- | web/lib/pkgfuncs.inc | 41 |
5 files changed, 65 insertions, 40 deletions
diff --git a/web/html/packages.php b/web/html/packages.php index 85ab6b75..9deff6af 100644 --- a/web/html/packages.php +++ b/web/html/packages.php @@ -509,44 +509,36 @@ if (isset($_REQUEST["do_Flag"])) { # There currently shouldn't be multiple requests here, but the format in which # it's sent requires this while (list($pid, $v) = each($ids)) { - $q = "INSERT INTO CommentNotify (PkgID, UserID) VALUES (".$pid.', '.$uid.')'; - db_query($q, $dbh); - print '<p>'; - print __("You have been added to the comment notification list."); - print '<br /></p>'; - pkgdetails_link($pid); + $q = "SELECT Name FROM Packages WHERE ID = " . $pid; + $pkgname = mysql_result(db_query($q, $dbh), 0); + + $q = "SELECT * FROM CommentNotify WHERE UserID = ".$uid; + $q.= " AND PkgID = ".$pid; + + if (!mysql_num_rows(db_query($q, $dbh))) { + $q = "INSERT INTO CommentNotify (PkgID, UserID) VALUES (".$pid.', '.$uid.')'; + db_query($q, $dbh); + print '<p>'; + print __("You have been added to the comment notification list for %s.", + array("<b>" . $pkgname . "</b>")); + print '<br /></p>'; + } else { + $q = "DELETE FROM CommentNotify WHERE PkgID = ".$pid; + $q.= " AND UserID = ".$uid; + db_query($q, $dbh); + print '<p>'; + print __("You have been removed from the comment notification list for %s.", + array("<b>" . $pkgname . "</b>")); + print '<br /></p>'; + } } + pkgdetails_link($pid); } else { print '<p>'; print __("Couldn't add to notification list."); print '<br /></p>'; } } -} elseif (isset($_REQUEST["do_UnNotify"])) { - if (!$atype) { - print __("You must be logged in before you can cancel notification on comments."); - print "<br />\n"; - } else { - if (!empty($ids)) { - $dbh = db_connect(); - $uid = uid_from_sid($_COOKIE["AURSID"]); - # There currently shouldn't be multiple requests here, but the format in which - # it's sent requires this - while (list($pid, $v) = each($ids)) { - $q = "DELETE FROM CommentNotify WHERE PkgID = ".$pid; - $q.= " AND UserID = ".$uid; - db_query($q, $dbh); - print '<p>'; - print __("You have been removed from the comment notification list."); - print '<br /></p>'; - pkgdetails_link($pid); - } - } else { - print '<p>'; - print __("Couldn't remove from notification list."); - print '<br /></p>'; - } - } } else { # do_More/do_Less/do_Search/do_MyPackages - just do a search # diff --git a/web/lang/en/pkgfuncs_po.inc b/web/lang/en/pkgfuncs_po.inc index 7dde5fbc..2f6a62e8 100644 --- a/web/lang/en/pkgfuncs_po.inc +++ b/web/lang/en/pkgfuncs_po.inc @@ -157,4 +157,6 @@ $_t["en"]["The above files have been verified (by %h%s%h) and are safe to use."] $_t["en"]["This package has been flagged out of date."] = "This package has been flagged out of date."; +$_t["en"]["Toggle Notify"] = "Toggle Notify"; + ?> diff --git a/web/lang/en/search_po.inc b/web/lang/en/search_po.inc index 8feb57bd..601782c0 100644 --- a/web/lang/en/search_po.inc +++ b/web/lang/en/search_po.inc @@ -39,7 +39,7 @@ $_t["en"]["Error trying to retrieve package details."] = "Error trying to retrie $_t["en"]["The selected packages have been adopted."] = "The selected packages have been adopted."; -$_t["en"]["You have been removed from the comment notification list."] = "You have been removed from the comment notification list."; +$_t["en"]["You have been removed from the comment notification list for %s."] = "You have been removed from the comment notification listfor %s."; $_t["en"]["Your votes have been cast for the selected packages."] = "Your votes have been cast for the selected packages."; @@ -47,7 +47,7 @@ $_t["en"]["You must be logged in before you can cancel notification on comments. $_t["en"]["You must be logged in before you can adopt packages."] = "You must be logged in before you can adopt packages."; -$_t["en"]["You have been added to the comment notification list."] = "You have been added to the comment notification list."; +$_t["en"]["You have been added to the comment notification list for %s."] = "You have been added to the comment notification list for %s."; $_t["en"]["You did not select any packages to disown."] = "You did not select any packages to disown."; @@ -75,4 +75,4 @@ $_t["en"]["The selected packages have been unflagged safe."] = "The selected pac $_t["en"]["Couldn't unflag package safe."] = "Couldn't unflag package safe."; -?>
\ No newline at end of file +?> diff --git a/web/lib/aurjson.class.php b/web/lib/aurjson.class.php index e6e40ffb..2df8dd31 100644 --- a/web/lib/aurjson.class.php +++ b/web/lib/aurjson.class.php @@ -108,7 +108,7 @@ class AurJSON { private function info($pqdata) { $base_query = "SELECT ID,Name,Version,Description,URL,URLPath,License,NumVotes,OutOfDate FROM Packages WHERE "; - if is_numeric($pqdata) { + if ( is_numeric($pqdata) ) { // just using sprintf to coerce the pqd to an int // should handle sql injection issues, since sprintf will // bork if not an int, or convert the string to a number 0 diff --git a/web/lib/pkgfuncs.inc b/web/lib/pkgfuncs.inc index eaeeca8b..f560ef91 100644 --- a/web/lib/pkgfuncs.inc +++ b/web/lib/pkgfuncs.inc @@ -265,6 +265,26 @@ function pkgvotes_from_sid($sid="") { return $pkgs; } +# array of package ids that you're being notified for +# *yoink* +# +function pkgnotify_from_sid($sid="") { + $pkgs = array(); + if (!$sid) {return $pkgs;} + $dbh = db_connect(); + $q = "SELECT PkgID "; + $q.= "FROM CommentNotify, Users, Sessions "; + $q.= "WHERE Users.ID = Sessions.UsersID "; + $q.= "AND Users.ID = CommentNotify.UserID "; + $q.= "AND Sessions.SessionID = '".mysql_real_escape_string($sid)."'"; + $result = db_query($q, $dbh); + if ($result) { + while ($row = mysql_fetch_row($result)) { + $pkgs[$row[0]] = 1; + } + } + return $pkgs; +} # display package details # @@ -475,7 +495,7 @@ function package_details($id=0, $SID="") { echo "<input type='submit' class='button' name='do_Notify'"; echo " value='".__("Notify")."' title='".__("New Comment Notification")."'>"; } else { - echo "<input type='submit' class='button' name='do_UnNotify'"; + echo "<input type='submit' class='button' name='do_Notify'"; echo " value='".__("UnNotify")."' title='".__("No New Comment Notification")."'>"; } @@ -636,6 +656,7 @@ function pkg_search_page($SID="") { $myuid = uid_from_sid($SID); $acct = account_from_sid($SID); $my_votes = pkgvotes_from_sid($SID); + $my_notify = pkgnotify_from_sid($SID); } # The search form @@ -964,11 +985,9 @@ function pkg_search_page($SID="") { print "<input type='submit' class='button' name='do_Delete'"; print " value='".__("Delete Packages")."'></td>\n"; print " <td align='center'>"; - print "<input type='submit' class='button' name='do_Vote'"; - print " value='".__("Vote")."'></td>\n"; + print "<input type='submit' class='button' name='do_Notify'"; + print " value='".__("Toggle Notify")."'></td>\n"; print " <td align='center'>"; - print "<input type='submit' class='button' name='do_UnVote'"; - print " value='".__("Un-Vote")."'></td>\n"; print "</tr>\n"; print "</table>\n"; print " </td>\n"; @@ -1023,6 +1042,10 @@ function pkg_search_page($SID="") { print " <th style='border-bottom: #666 1px solid; vertical-align:"; print " bottom'><span class='f2'>".__("Voted")."</span></th>\n"; } + if ($SID) { + print " <th style='border-bottom: #666 1px solid; vertical-align:"; + print " bottom'><span class='f2'>".__("Notify")."</span></th>\n"; + } print " <th style='border-bottom: #666 1px solid; vertical-align:"; print " bottom'><span class='f2'>".__("Description")."</a>"; print "</span></th>\n"; @@ -1091,6 +1114,14 @@ function pkg_search_page($SID="") { print " </span></td>\n"; } } + if ($SID) { + print " <td class='".$c."'><span class='f5'><span class='blue'>"; + if (isset($my_notify[$row["ID"]])) { + print " ".__("Yes")."</span></td>\n"; + } else { + print " </span></td>\n"; + } + } print " <td class='".$c."'><span class='f4'><span class='blue'>"; print $row["Description"]."</span></span></td>\n"; print " <td class='".$c."'><span class='f5'><span class='blue'>"; |