[SQL] BBcode breaks

[sockwater]

This [SQL] bbcode breaks when previewing or quick editing in full WYSIWYG mode.

To fix this, you can add an HTML comment to the beginning and end of the [sql] bbcode replacement text (to identify where it begins and ends), and then in a plugin on wysiwyg_parse_start (or wysiwyg_parse_complete (either one should work)), add something similar to this:

PHP Code:

$text = preg_replace('#<!-- {STARTSQLTAG} -->(.+)<!-- {ENDSQLTAG} -->#U', '[SQL]$1[/SQL]', $text); 


[Jase2]

This was reported before, although the staff didn't know how to fix it. I'm sure they will appreciate this.

[sockwater]

Actually my code is a little bit off, since I just wrote it from memory. I've looked back at my code, and I've found an even easier way to do this. Here is a full working example.

1. Create a custom SQL bbcode, fill in any dummy text in the replacement HTML box (it won't be used). This will cause the bbcode to show up on misc,php?do=bbcode

2. Create a plugin on bbcode_create: The trick is adding 'sql' to the list of unparsed tags (the last line).

PHP Code:

if (!function_exists ('handle_sql_bbcode_tag'))
{
    function handle_sql_bbcode_tag (&$theobj, &$value, &$option)
    {
        // format the HTML output for an SQL tag here
        return "<div style='border:1px solid red;padding:10px;'>$value</div>";
    }
}
$this->tag_list['no_option']['sql'] = array ();
$this->tag_list['no_option']['sql']['callback'] = 'handle_external';
$this->tag_list['no_option']['sql']['external_callback'] = 'handle_sql_bbcode_tag';
$this->unparsed_tags[] = 'sql'; 


[Jase2]

Any comment from staff?

[Marco van Herwaarden]

Staff will respond once we have information to share.

[Jase2]

Ok.

[Adrian Schneider]

Originally Posted by sockwater

This [SQL] bbcode breaks when previewing or quick editing in full WYSIWYG mode.

To fix this, you can add an HTML comment to the beginning and end of the [sql] bbcode replacement text (to identify where it begins and ends), and then in a plugin on wysiwyg_parse_start (or wysiwyg_parse_complete (either one should work)), add something similar to this:

PHP Code:

$text = preg_replace('#<!-- {STARTSQLTAG} -->(.+)<!-- {ENDSQLTAG} -->#U', '[SQL]$1[/SQL]', $text); 


Here's the original thread:
https://vborg.vbsupport.ru/showthread.php?t=168734

[sockwater]

Originally Posted by SirAdrian

Here's the original thread:
https://vborg.vbsupport.ru/showthread.php?t=168734

Thanks for the link. The instructions in post #3 will fix this bug.

[Jase2]

Originally Posted by Marco van Herwaarden

Staff will respond once we have information to share.

So have you decided on this or not?

[Paul M]

Originally Posted by Jase2

So have you decided on this or not?

Nope, so far no one has had time to look - its been this way for a number of years so there is no hurry.