Default avatar with Hovercards

You are viewing page #2

Show original post

inc/plugins/hovercards.php, line 388:

$users[] = $user; 

Add before:

if (!$user['avatar']) {
    $user['avatar'] = 'what you want';
} 

Btw, it's not overkill even for hundreds thousands of users. You could run a single query to set a default avatar, and install a plugin that takes care of new users when they register. Just my two cents.

This one doesn't seem to be working in my case.

Tried with ! and empty();

Do you have a board URL so I can look into it directly?

Do you have a board URL so I can look into it directly?

Shade (6 Aug 2019)

Solved. Seems like php was misbehaving and after reloading xampp on localhost it worked.

Look up for the AJAX request in the Network tab when you hover a username. Does the response include the avatar?

Look up for the AJAX request in the Network tab when you hover a username. Does the response include the avatar?

Shade (6 Aug 2019)

Yes and it includes repeated 52 calls for every user without avatar via xhr.

The call is unique and gathers all the users currently in the page you’re watching so it doesn’t have to query every time. Is the URL relative or absolute? I believe if it’s relative it won’t display.

The call is unique and gathers all the users currently in the page you’re watching so it doesn’t have to query every time. Is the URL relative or absolute? I believe if it’s relative it won’t display.

Shade (6 Aug 2019)

But it gets requeried on page refresh unfortunately. It's absolute path.

Yes, a browser caching was planned long ago, but never managed to add it. I will test the above code later.

Ah, you’re on localhost. This might explain why. Absolute urls are treated like remote avatars, and Google cannot load localhost urls of course. Try relative without the initial trailing slash and dot.

Ah, you’re on localhost. This might explain why. Absolute urls are treated like remote avatars, and Google cannot load localhost urls of course. Try relative without the initial trailing slash and dot.

Shade (6 Aug 2019)

Yeah localhost could be the thing. I guess you don't have to focus anymore on this as we achieved the main goal, hence its solved from my side.