[Jawelin]

Originally posted by Jawelin
FoA, thanx for such a complete answer...

1) Agree! Will check better but seems is still a point in which avatar.php is accessed: userCP when updating avatar. I'll let you know if interested.

[...]

1) I think was forgotten a hack point.
Infact, I understand that'll not affect performance purposes - the first goal of this
hack - but for a complete job, I think also the getavatarulr() function should be
modified, and become as follows:

// ###################### Start getavatarurl #######################
function getavatarurl($userid) {
  global $DB_site,$session;

  if ($avatarinfo=$DB_site->query_first("SELECT user.avatarid,user.avatarrevision,avatarpath,NOT ISNULL(avatardata) AS hascustom,customavatar.dateline
                                         FROM user
                                         LEFT JOIN avatar ON avatar.avatarid=user.avatarid
                                         LEFT JOIN customavatar ON customavatar.userid=user.userid
                                         WHERE user.userid='$userid'")) {
    if ($avatarinfo[avatarpath]!="") {
      return $avatarinfo[avatarpath];
    } else if ($avatarinfo['hascustom']) {
        $avatarpath=trim($GLOBALS['avatarpath']);
    	if ($GLOBALS['usefileavatar'] && $avatarpath>"") {
    		$avatarurl = "$avatarpath/avatar$userid"."_$avatarinfo[avatarrevision].gif";
    	} else {
    		$avatarurl = "avatar.php?userid=$userid&dateline=$avatarinfo[dateline]";
    	}
        return $avatarurl;
    } else {
      return '';
    }
  }
}

(I would put in bold main differences from the original... but ... )

Of course, I put my 'version' with the $avatarpath global variable. Your original, it should be replaced with the stored directory name...

What do you think ?
Thnx.