How do I stop parsing of BB code?

[kau]

Right now on my current version I hacked it so function handle_bbcode_img($bbcode, $dobbimagecode) does not work unless you are a certain user group, to stop trolls.

Now with 3.5 this is gone. I can't update until I find where this code is now.

What file is the parsing of the IMG tag in now???????

[akanevsky]

class_bbcode

[merk]

You should be able to use a plugin to achieve what you want.

[kau]

How would I use a plugin to do that? I'm not very technical.

I looked throughout class_bbcode and I can't see a clear cut function that does it. Can someone point me to which function in that file handles the parsing of the IMG tag or explain how to setup a plugin to do what I need.

[akanevsky]

File class_bbcode.php, around line 1714:

PHP Code:

    /**
    * Handles an [img] tag.
    *
    * @param    string    The text to search for an image in.
    * @param    string    Whether to parse matching images into pictures or just links.
    *
    * @return    string    HTML representation of the tag.
    */
    function handle_bbcode_img($bbcode, $do_imgcode, $has_img_code = false) 


[kau]

Great! Thanks very much.

[merk]

Wait. Thats not the best method. (modifying code when you dont have to is baaad )

plugin bbcode_parse_start,

PHP Code:

if(is_member_of($this->registry->userinfo, X)) { 
$dobbimagecode = true; 
}
else
{
$dobbimagecode = false;
} 


[akanevsky]

I didn't say that it's a best method. I just pointed at a location. As for your method, which is definitely better, the most error-proof solution would be simply:

PHP Code:

if(!(is_member_of($this->registry->userinfo, X) OR is_member_of($this->registry->userinfo, Y)))
{
    $dobbimagecode = false;
} 


Whereas the "true" assignment in your code might cause unexpected behavior in some cases when imgcode should not be parsed.

The easiest to use modify code would be:

PHP Code:

$usergroupids_no_img = 'X,Y,Z';
$usergroupids_no_img = explode(',', $usergroupids_no_img);

foreach ($usergroupids_no_img as $usergroupid)
{
    if(!is_member_of($this->registry->userinfo, $usergroupid))
    {
        $dobbimagecode = false;
        break;
    }
} 


The reason there is a break statement is to stop checking for usergroups once a restricted one has been found - in other words, for optimization.

[merk]

It is unnecessary to use multiple is_member_of calls. You can pass an array of usergroupids.

[akanevsky]

Really? I didn't know. Thanks for pointing that out It's not really an "array", though. Rather an expanded list of parameters.

if(!is_member_of($this->registry->userinfo, X, Y, Z)) 
{ 
    $dobbimagecode = false; 
}