[Pushed] Smilies https issue
#1
I noticed when you place an https link in the "Image Path" under Configuration > Smilies > Add New Smilie, it displays a broken image in the smilies list of the AdminCP. It displays fine in the smilies section of the editor when making a new post and works fine when you use the smilie in a post.

In admin\modules\configsmilies.php on line 729 (and a few other places) it's doing a substring to detect if the image is a fully qualified URL or a relative path.
if(my_strpos($smilie['image'], "p://") || substr($smilie['image'], 0, 1) == "/")

It would be nice if that could also take in to account https URLs. IcyBoards uses https for all static resources which is required for https on the forum to work without showing an error. I was able to resolve it myself by changing the line to the following.
if(my_strpos($smilie['image'], "p://") || my_strpos($smilie['image'], "ps://") || substr($smilie['image'], 0, 1) == "/")

Thanks.
RedHat Certified Systems Administrator
Reply
#2
Thanks, pushed and supplied with similar places: https://github.com/mybb/mybb/issues/2443
devilshakerz.com/pgp (DF3A 34D9 A627 42E5 BC6A 6750 1F2F B8AA 28FF E1BC) ▪ keybase.io/devilshakerz
Reply
#3
Wow that was fast. Thanks Smile
RedHat Certified Systems Administrator
Reply


Forum Jump:


Users browsing this thread: 1 Guest(s)