Topic: shoutbox @ fix not working on my site

[steve51184]

here's the problem:
http://simpleportal.net/index.php?topic=4168.msg26018#msg26018
or
http://simpleportal.net/index.php?topic=4126.msg25859#msg25859

here's the fix:
http://simpleportal.net/index.php?topic=4126.msg26219#msg26219

and here's my problem with the fix not working for me:
http://simpleportal.net/index.php?topic=4168.msg28682#msg28682


also before you all tell me to post in those topics etc:

please start a new topic in English Support board. Topics in Bugs board does not appear on our support topics list.

 

[[SiNaN]]

Well, the problem is, I'm unable to reproduce the problem you are having. Any chance I can debug it myself? I will need a temporary FTP access only.

[steve51184]

could i not give you all the files you need but in .tar/.zip archive or something as i don't have ftp installed on my server

(i'm an scp man not and ftp man )

[[SiNaN]]

Okay, all files in Sources and Themes directories + index.php would be enough.

[steve51184]

Okay, all files in Sources and Themes directories + index.php would be enough.

pm sent

[[SiNaN]]

Okay. I could reproduce it but will check it later. Need to do my homework first.

[steve51184]

ok thank you

[MultiformeIngegno]

any news on this..?

[steve51184]

Okay. I could reproduce it but will check it later. Need to do my homework first.

any news on this?

[steve51184]

come on guys i've been waiting over a month for a fix now... i understand that your busy but i need help with this!

[[SiNaN]]

I'm sorry. This is one of few issues I can't offer a solution without examining the code.

I tried it again and now can't reproduce it even with your files. I'll let Nath check this one as well.

Also, are you sure that you correctly replaced your Subs.php file with this one and it didn't work?

[steve51184]

I'm sorry. This is one of few issues I can't offer a solution without examining the code.

I tried it again and now can't reproduce it even with your files. I'll let Nath check this one as well.

Also, are you sure that you correctly replaced your Subs.php file with this one and it didn't work?

VERY sure... i install ALL smf mods by hand and have done for years so i know how to make edits

also is there not just an easy was to disable email addresses being posted in the shoutbox?

[[SiNaN]]

I'm saying that because the original Subs.php file you attached had the first edit applied but not the second edit.

This issue is actually fixed with SMF 2.0 RC3. But to prevent e-mail address being parsed in shoutbox, this should be enough:

Code: (Find) [Select]

'text' => parse_bbc($row['body'], true, '', $bbc),

Code: (Replace) [Select]

'text' => parse_bbc(str_replace('@', '[at]', $row['body']), true, '', $bbc),

[steve51184]

I'm saying that because the original Subs.php file you attached had the first edit applied but not the second edit.

This issue is actually fixed with SMF 2.0 RC3. But to prevent e-mail address being parsed in shoutbox, this should be enough:

Code: (Find) [Select]

'text' => parse_bbc($row['body'], true, '', $bbc),

Code: (Replace) [Select]

'text' => parse_bbc(str_replace('@', '[at]', $row['body']), true, '', $bbc),

is that in subs.php as i don't want to disable them for the whole forum

[[SiNaN]]

Opps, sorry. It's in Subs-Portal.php. It will only affect shouts in the shoutbox.

[steve51184]

perfect thank you

[[SiNaN]]

Regarding this issue, to quote myself from Beta Development board:

Okay, I've found the issue. It was related to how SMF handles $disabled variable and how PHP handled static variables. Here is the fix:

Code: (Find) [Select]

$default_disabled = $disabled;

Code: (Replace) [Select]

$default_disabled = isset($disabled) ? $disabled : array();
When all tags are enabled, $disabed variable is not set by SMF, thus $default_disabed is not set until it reaches to shout parsing. At that point, $default_disabled is set according to shoutbox settings. I expected $disabled to throw out an error if it wasn't set, but looks like PHP initializes static variables as NULL, so it didn't cause an error, which made it quite difficult to find, especially if you don't allow all BBC tags.

Thanks goes to AngelinaBelle for detailed report on the issue and allowing me to investigate the issue on her forum.

[MultiformeIngegno]

Awesome news! Have you submitted it to mantis?

[[SiNaN]]

It's already fixed in our latest dev package.

[MultiformeIngegno]

Yes yes.. I was only wondering if it was an smf or sp bug.. if is the first you can add it to the mantis tracker at sm.org