Help with plugin to strip table tags

[Azum]

Looking to strip table tags from posts on a particular style. I haven't put the if statement in yet for the styleid as the main code doesn't work.

Here is what I've been trying, with variations, but it never seems to work (created the phrase below):

hook location: postbit_display_start

$this->post['pagetext_html'] = preg_replace('/\[TABLE=.*?\](.*?)\[\/TABLE\]/si', construct_phrase($vbphrase['hidetable_message'], $session['sessionurl_q']), $this->post['pagetext_html'])

Guessing my regex is off...

[kh99]

Well, post['pagetext_html'] probably contains html instead of bbcode. Maybe what you want to do is use hook bbcode_create to change $this->tag_list, and change the function that gets called to parse the table tag (check the current styleid if you only want it to change for one style).

[Azum]

Originally Posted by kh99

Well, post['pagetext_html'] probably contains html instead of bbcode. Maybe what you want to do is use hook bbcode_create to change $this->tag_list, and change the function that gets called to parse the table tag (check the current styleid if you only want it to change for one style).

Well the code I borrowed this from will successfully remove .[img] tags:

$this->post['pagetext_html'] = preg_replace('/\[img\](.*)\[\/img\]/im', construct_phrase($vbphrase['hideimg_message'], $session['sessionurl_q']), $this->post['pagetext_html']);

with the same hook location, so I figured I was on the right track and the regex is just off. I will try it your way, but I'm not entirely sure I understand everything you said... maybe you could break it down into simpler terms for me.

[kh99]

The posts are parsed from bbcode into html, and then the result is saved in the postparsed table so that it doesn't have to be done every time (it may be a feature that can be turned off, I'm not sure offhand). And the pagetext_html field is the parsed version. But I think for some reason IMG tags aren't preparsed, which is probably why the code you copied works for img tags.

I'm not sure offhand what [TABLE]...[/TABLE] ends up as in html. If there's any way to do your replacement with html tags, maybe you can do that. Otherwise, what I was talking about above would be done something like this:

hook location bbcode_create:

    
    $this->tag_list['option']['table']['callback'] = 'handle_external';
    $this->tag_list['option']['table']['external_callback'] = 'my_handle_table';
    $this->tag_list['no_option']['table']['callback'] = 'handle_external';
    $this->tag_list['no_option']['table']['external_callback'] = 'my_handle_table';
    if (!function_exists('my_handle_table'))
    {
        function my_handle_table($parser, $data, $option)
       {
           global $styleid;

           if ($styleid != X)
              return $parser->parseTableTag($data, $option);
           
           global $vbphrase, $session;
           return construct_phrase($vbphrase['hideimg_message'], $session['sessionurl_q']);
       } 
    }

where of course you'd change X to the styleid that you want.

I haven't tested the above at all, so I'm not certain about it. One thing is that this will do the replacement even for 'alt' bbcode parsers, like if it's being parsed to plain text for a preview or something. You may or may not want that, I don't know.

Sorry if this isn't broken down enough - feel free to ask more questions if it doesn't work like you want.

[Azum]

Thanks, haven't had time to play with this but will give it a go later.