In commit 3578e77ad4e9258495eed7e786b7dc3aebcf1b63 we implemented
listing of comments from the account details page , but this was
intended to only be available to TUs and Devs. As the comment says:
"display the comment list if they're a TU/dev"

The credential checking code, however, set this credential for all
users, contrary to the intention of the commit.

Signed-off-by: Eli Schwartz <eschwa...@archlinux.org>
---
 web/lib/credentials.inc.php | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/web/lib/credentials.inc.php b/web/lib/credentials.inc.php
index c125119..96c7233 100644
--- a/web/lib/credentials.inc.php
+++ b/web/lib/credentials.inc.php
@@ -49,7 +49,6 @@ function has_credential($credential, $approved_users=array()) 
{
        $atype = account_from_sid($_COOKIE['AURSID']);
 
        switch ($credential) {
-       case CRED_ACCOUNT_LIST_COMMENTS:
        case CRED_PKGBASE_FLAG:
        case CRED_PKGBASE_NOTIFY:
        case CRED_PKGBASE_VOTE:
@@ -60,6 +59,7 @@ function has_credential($credential, $approved_users=array()) 
{
        case CRED_ACCOUNT_CHANGE_TYPE:
        case CRED_ACCOUNT_EDIT:
        case CRED_ACCOUNT_LAST_LOGIN:
+       case CRED_ACCOUNT_LIST_COMMENTS:
        case CRED_ACCOUNT_SEARCH:
        case CRED_COMMENT_DELETE:
        case CRED_COMMENT_UNDELETE:
-- 
2.23.0

Reply via email to