[Pushed] PHP8 warning for Get More Smilies
#1
If a post is created and there are more defined smilies than show in the smilies list, and Get More is clicked, it throws a warning
Undefined variable $pid in eval'd code line 2 called from /forum/misc.php line 817.

The default template misc_smilies_popup invokes a div with class="modal_{$pid}
I'm not sure why that class is named with a pid.

I cleared the warning by adding a line at 816
$pid = '';
Reply
#2
https://github.com/mybb/mybb/pull/4660
[MyBB 1.8 Czech translation] [MyBB 1.8 plugins]: Prune old PMs + optimize DB plugin --- Thank you/like system
Reply
#3
Hello,

is there a better way to show the close - "x" in foreground? 
Without _{$pid} we see only a little bit of it because it is in background ... 

picture ..01 with class="modal", picture ..02 with class="modal_{$pid}" 

       
PHP Version        8.2.22    (Test: 8.2.22)
MySQLi Version  5.7.42    (Test: 8.0.36)
Mybb Version      1.8.38    (Test: 1.8.38)
Reply
#4
@Laird could you share this on github too? The problem exists in principle, even here in the board
No support via PM, email, telephone, home visit, Discord, Telegram, WhatsApp and other useless tools
Reply
#5
Done, if I've understood correctly what you were wanting reported (let me know if not):

https://github.com/mybb/mybb/issues/4853
Reply
#6
you did understood, sorry, early morning, still in lazy writing mode

edit 2 hours later: now I get it, I had slipped in the topic
No support via PM, email, telephone, home visit, Discord, Telegram, WhatsApp and other useless tools
Reply


Forum Jump:


Users browsing this thread: 1 Guest(s)