Updated XOOPS Function:
This update addresses an issue if $usereal = 1 and Var $user->getVar( 'name' ) is empty. I have corrected this with the following Function.

I think I understand, but could you explain further?

John. i was reading your code and i dont understand some of the things you are coding.

can i suggest this code? (based on your code)


untested code though.
(sry about my 8 space tabs..)

Hi,

The problem is an error checking one.

There is no checking to see if a person realname (i.e.) 'name' var 'is_null' or 'empty'. The result being that if it is an 'empty' value is returned for 'name' and breaks the function (Basically you cannot see anything)

What I have done is to check that 'name' value is used else return 'uname' instead.

I also did a check to see that 'fct' is not 'empty' if it is not then edit the player in admin (as 'fct' would only be valid is if the administrator was in the admin side), but with some better hindsight, maybe a switch could be used within the function to better use here.

ps: I just remembered I wrote that for my personal version of XOOPS and would need rewriting for 2.17 instead. I will do that shortly.

Yes that would work, but I would prefer it if the admin check (or switch used) was actually (corrected of course) was left in for flexability reasons. i.e. I would like to use this function to direct the administrator to edit a users profile in the admin area (actually now I think about it we could use a switch to allow for displaying or editing both user and admin side).

Also the XoopsUser::getUnameFromId() should be updated to reflect those changes also.