TP-Docs
HTML5 Icon HTML5 Icon HTML5 Icon
TP on Social Media

Recent

Welcome to TinyPortal. Please login or sign up.

March 29, 2024, 12:27:35 PM

Login with username, password and session length
Members
Stats
  • Total Posts: 195,106
  • Total Topics: 21,213
  • Online today: 358
  • Online ever: 3,540 (September 03, 2022, 01:38:54 AM)
Users Online
  • Users: 0
  • Guests: 79
  • Total: 79

Avatar by URL not working in Article Comments

Started by lurkalot, February 15, 2018, 10:32:44 PM

Previous topic - Next topic

0 Members and 1 Guest are viewing this topic.

lurkalot

I have just noticed, the avatars don't show in article comments. Only when using the "Specify avatar by URL" option. (screenshot below).

tino

Quote from: lurkalot on February 15, 2018, 10:32:44 PM
I have just noticed, the avatars don't show in article comments. Only when using the "Specify avatar by URL" option. (screenshot below).

That works for me. Can you let me know the url you used?

Actually it only doesn't work when I have image_proxy_enabled and I visit the site whilst it is http, that causes everything to get in to a bit of a mess. Not sure if that is a valid set up though, as I wouldn't expect to enable image proxy and still have the site http.

lurkalot

Quote from: tino on February 16, 2018, 10:53:41 AM
Quote from: lurkalot on February 15, 2018, 10:32:44 PM
I have just noticed, the avatars don't show in article comments. Only when using the "Specify avatar by URL" option. (screenshot below).

That works for me. Can you let me know the url you used?

Actually it only doesn't work when I have image_proxy_enabled and I visit the site whilst it is http, that causes everything to get in to a bit of a mess. Not sure if that is a valid set up though, as I wouldn't expect to enable image proxy and still have the site http.

Must admit, I was using my local testbed.  Not sure I had image proxy switched on in SMF though, will have to check that.  The avatars were working everywhere else though as far as I can tell.  Will check this again in a while.

@rjen

I checked my testsite and I have the same problem:

For a user where the Avatar is a 'standard one' from the forum, it shows in the comments. I am using an uploaded avatar that is NOT displayed in the comments.

The link used in the comments is that: https://test.fjr-club.nl/avatars/ , it seems to link to the folder, not the actual uploaded image...

I also checked it in 1.3R and there it was working fine.
On the site: SSL with Image proxy fully enabled...
Deactivating the image proxy makes no difference: problem stays
Running Latest TP on SMF2.1 at: www.fjr-club.nl

tino

So it fails when linked to a image uploaded to the forum? Or it fails when specifing a image uploaded to a local ( to it ) url?

tino

As you both seem to have reported different errors.

lurkalot

Quote from: tino on February 16, 2018, 09:32:44 PM
As you both seem to have reported different errors.

The uploaded ones work, the stock smf ones work, but if you add one via url it doesn't show in comments, but shows everywhere else in the portal.

tino

Ok, well the error the @rjen reported I've managed to emulate.

tino

Line 736 of Sources/TPortal.php should be
$avatar = $avatar == '' ? ($row['ID_ATTACH'] > 0 ? '<img src="' . (empty($row['attachmentType']) ? $scripturl . '?action=tpmod;sa=tpattach;attach=' . $row['ID_ATTACH'] . ';type=avatar' : $modSettings['custom_avatar_url'] . '/' . $row['filename']) . '" alt="" class="avatar" border="0" />' : '') : (stristr($row['avatar'], 'https://') ? '<img src="' . $row['avatar'] . '"' . $avatar_width . $avatar_height . ' alt="" class="avatar" border="0" />' : '<img src="' . $modSettings['avatar_url'] . '/' . $smcFunc['htmlspecialchars']($row['avatar'], ENT_QUOTES) . '" alt="" class="avatar" border="0" />');


lurkalot

http://avatars.simplemachinesweb.com/smf/avatar_124590_1499841922.gif

You can see this image, yes?

If I add that url to my profile it won't show in the article comments, but does show in the article author info, and all other blocks which use an avatar. 

If I use the secure version https://avatars.simplemachinesweb.com/smf/avatar_124590_1499841922.gif

The avatar displays everywhere including the comments.

tino

Works via the proxy for me, can you see if the fix I posted fixes your issue also?

@rjen

Quote from: lurkalot on February 16, 2018, 09:59:43 PM
Quote from: tino on February 16, 2018, 09:32:44 PM
As you both seem to have reported different errors.

The uploaded ones work, the stock smf ones work, but if you add one via url it doesn't show in comments, but shows everywhere else in the portal.

Just tried that. URL avatar IS showing on my test site...
Running Latest TP on SMF2.1 at: www.fjr-club.nl

lurkalot

I'm not using proxy, it's just a local http test site.

tino

Quote from: lurkalot on February 16, 2018, 10:32:33 PM
I'm not using proxy, it's just a local http test site.

Can you try the fix a few posts back. It should sort it as both issues are linked

lurkalot

Sort it, as in remove them? I don't seem to have any avatars in comments now. Also the new post image has popped up. should I be seeing this as I was the one who posted?

tino

Quote from: lurkalot on February 16, 2018, 11:02:11 PM
Sort it, as in remove them? I don't seem to have any avatars in comments now. Also the new post image has popped up. should I be seeing this as I was the one who posted?

It didn't remove them for me. I guess you either didn't edit it correctly or I posted the wrong thing.

It's adding the $avatar == after the first = and before the ''

lurkalot

As far as I can see I've edited it correctly. Attached below.

@rjen

Quote from: tino on February 16, 2018, 10:41:25 PM
Quote from: lurkalot on February 16, 2018, 10:32:33 PM
I'm not using proxy, it's just a local http test site.

Can you try the fix a few posts back. It should sort it as both issues are linked

I tried your fix too, and the result is that in the first comment the Avatar shows, but in the second comment (same user) it does NOT...

Running Latest TP on SMF2.1 at: www.fjr-club.nl

tino

Try this code
if ($image_proxy_enabled && !empty($row['avatar']) && stripos($row['avatar'], 'http://') !== false)
      $avatar = '<img src="'. $boardurl . '/proxy.php?request=' . urlencode($row['avatar']) . '&hash=' . md5($row['avatar'] . $image_proxy_secret) .'" alt="&nbsp;" />';
else
      $avatar = $row['avatar'] == '' ? ($row['ID_ATTACH'] > 0 ? '<img src="' . (empty($row['attachmentType']) ? $scripturl . '?action=dlattach;attach=' . $row['ID_ATTACH'] . ';type=avatar' : $modSettings['custom_avatar_url'] . '/' . $row['filename']) . '" alt="&nbsp;"  />' : '') : (stristr($row['avatar'], 'https://') ? '<img src="' . $row['avatar'] . '" alt="&nbsp;" />' : stristr($row['avatar'], 'http://') ? '<img src="' . $row['avatar'] . '" alt="&nbsp;" />' : '<img src="' . $modSettings['avatar_url'] . '/' . $smcFunc['htmlspecialchars']($row['avatar'], ENT_QUOTES) . '" alt="&nbsp;" />');

@rjen

Quote from: tino on February 17, 2018, 10:32:56 AM
Try this code
if ($image_proxy_enabled && !empty($row['avatar']) && stripos($row['avatar'], 'http://') !== false)
      $avatar = '<img src="'. $boardurl . '/proxy.php?request=' . urlencode($row['avatar']) . '&hash=' . md5($row['avatar'] . $image_proxy_secret) .'" alt="&nbsp;" />';
else
      $avatar = $row['avatar'] == '' ? ($row['ID_ATTACH'] > 0 ? '<img src="' . (empty($row['attachmentType']) ? $scripturl . '?action=dlattach;attach=' . $row['ID_ATTACH'] . ';type=avatar' : $modSettings['custom_avatar_url'] . '/' . $row['filename']) . '" alt="&nbsp;"  />' : '') : (stristr($row['avatar'], 'https://') ? '<img src="' . $row['avatar'] . '" alt="&nbsp;" />' : stristr($row['avatar'], 'http://') ? '<img src="' . $row['avatar'] . '" alt="&nbsp;" />' : '<img src="' . $modSettings['avatar_url'] . '/' . $smcFunc['htmlspecialchars']($row['avatar'], ENT_QUOTES) . '" alt="&nbsp;" />');


That fixes the problem for me.
Running Latest TP on SMF2.1 at: www.fjr-club.nl

lurkalot

Quote from: @rjen on February 17, 2018, 11:31:19 AM

That fixes the problem for me.


And for me.  I've tried various combinations in the avatar setting and they appear to work nicely now.