10
not sure James, i've tried it on my site and it works fine (xoops 2.0.15 currently) but it also worked fine on 2.0.13.2 & 2.0.14..
here's the lines from m y userinfo.php
le="color: #000000"><?php $uid = intval($_GET['uid']); if ($uid <= 0) { redirect_header('index.php', 3, _US_SELECTNG); exit(); } if (!is_object($xoopsUser)) { redirect_header('index.php',3,_NOPERM); exit(); } $gperm_handler = & xoops_gethandler( 'groupperm' ); $groups = is_object($xoopsUser) ? $xoopsUser->getGroups() : XOOPS_GROUP_ANONYMOUS;
you can checkout at
http://72.29.82.174/~wwcncom/test2010/ if anonymous, it will say u have no permission, if logged on, it will let u view the profile.. :S
if having 'is_object' doesn't work for you, i dunno why, but if it works as you are using it without, then leave it as is being as it works :), but it is better to actually use is_object than not.. (i can't say why it's better, just that I was always told to use that instead of just !$xoopsUser), as it's the proper way of doing it. so i kept with doing it as a matter of practice..
maybe a more experienced coder can say why it's better to use is_object($xoopsUser) than just $xooopsUser??? i'd be curious to remember y too.. (i was told but i've forgotten the reasons..) i know 'is_object' will check if the variable in this case $xoopsUser is actually an object, but in this situation i can't actually remember why is_object($xoopsUser) is better than just $xoopsUser..
