Implemented Change static call to dynamic call in XenForo_Template_Helper_Core::avatarHtml()

[Mike Tougeron]

The public static variable XenForo_Template_Helper_Core::$helperCallbacks can be used to override XenForo's pre-defined avatar url builder. In the case of IGN.com we already have an avatar system in place so by changing the value in that static variable I can make everywhere use IGN's avatar building code except for when avatarHtml() is called.

In library/XenForo/Template/Helper/Core.php the function XenForo_Template_Helper_Core::avatarHtml() on line 1444 calls:
$src = self::helperAvatarUrl($user, $size, $forceType, $canonical);
Can that please be changed to:
$src = call_user_func(self::$helperCallbacks['avatar'], $user, $size, $forceType, $canonical);

Thanks for your help,
Mike

 

[Kier]

This change has already been made for XenForo 1.1, along with a bunch of other changes to use callHelper()

 

[Mike Tougeron]

Rock on! Thanks.

 

[Mike Tougeron]

FYI, this change is not in v1.1.0-b1

 

[Kier]

Really? I'll check on that and see what happened in the morning.

 

[Kier]

Oops, looks like I missed that particular one while changing a bunch of others. It's done for the next release.

 

[Mike Tougeron]

Thanks.