2
Perhaps something's wrong in this piece of the code
(myself's not too good in coding)
Quote:
if ($thisUser->getVar('user_viewemail') == 1) {
$xoopsTpl->assign('user_email', $thisUser->getVar('email', 'E'));
} else {
if (is_object($xoopsUser)) {
// All admins will be allowed to see emails, even those that are not allowed to edit users (I think it's ok like this)
if ($xoopsUserIsAdmin || ($xoopsUser->getVar("uid") == $thisUser->getVar("uid"))) {
$xoopsTpl->assign('user_email', $thisUser->getVar('email', 'E'));
} else {
$xoopsTpl->assign('user_email', ' ');
}
}
}
Of course, I could leave of it all only
Quote:
$xoopsTpl->assign('user_email', ' ');
or just drop it and the emails would not be displayed at all
(majority of users wish not to show emails).
But I think it's a really bad and vulgar way out
