[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
 
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
 
Reply


Forum Jump:


Users browsing this thread: 1 Guest(s)