Wrong Datatype in bbcode_class file

[Cyricx]

Okay, so I'm trying to work the usergroup smilies hack to be more secure by replacing the smilies with text instead of the image if the smilie is in a particular category (102) and the user is not in the secondary usergroup (44).

I've tried the below, and I'm getting an error for "wrong data type in second argument"

Any ideas? :(

// ##### START HACK - Usergroup Smilies #####
if ($smilie['imagecategoryid'] == "102")
{
if (in_array("44", $this->registry->userinfo['membergroupids']))
{
if ($this->is_wysiwyg())
{
$replace = "<img src=\"$smilie[smiliepath]\" border=\"0\" alt=\"\" title=\"$smilie[title]\" smilieid=\"$smilie[smilieid]\" class=\"inlineimg\" />";
}
else
{
$replace = "<img src=\"$smilie[smiliepath]\" border=\"0\" alt=\"\" title=\"$smilie[title]\" class=\"inlineimg\" />";
}
}
else
{
$replace = "*censored_smilie*";
}
}
else
{
// ##### END HACK - Usergroup Smilies #####

// if you change this HTML tag, make sure you change the smilie remover in code/php/html tag handlers!
if ($this->is_wysiwyg())
{
$replace = "<img src=\"$smilie[smiliepath]\" border=\"0\" alt=\"\" title=\"$smilie[title]\" smilieid=\"$smilie[smilieid]\" class=\"inlineimg\" />";
}
else
{
$replace = "<img src=\"$smilie[smiliepath]\" border=\"0\" alt=\"\" title=\"$smilie[title]\" class=\"inlineimg\" />";
}

// ##### START HACK - Usergroup Smilies #####
}
// ##### END HACK - Usergroup Smilies #####
$sc["$find"] = $replace;


I'm trying to hardcode in the category and usergroup numbers first.

Thanks so much for any help :)

[Marco van Herwaarden]

You don't point us to the line where you are getting this error, so i am just guessing here. I think (not checked) that $this->registry->userinfo['membergroupids'] is not an array, but a comma seperated list. Also $this will only work inside classes.

[merk]

membergroupids is indeed just a comma seperated list. Use is_member_of :) (thats whats causing the message too, as Marco pointed out)

[Cyricx]

Sorry :)

It's in the class_bbcode.php file.

So $this should work. I'm adding this on line around 452.

So I tried a strpos


// ##### START HACK - Usergroup Smilies #####
if ($smilie['imagecategoryid'] == "102")
{
$groupids = $this->registry->userinfo['membergroupids'];
$allowedgroup = "44";
$pos = strpos($groupids, $allowedgroup);
if ($pos === false)
{
$replace = "*Censored Smilies*";
}
}
else
{
// ##### END HACK - Usergroup Smilies #####

// if you change this HTML tag, make sure you change the smilie remover in code/php/html tag handlers!
if ($this->is_wysiwyg())
{
$replace = "<img src=\"$smilie[smiliepath]\" border=\"0\" alt=\"\" title=\"$smilie[title]\" smilieid=\"$smilie[smilieid]\" class=\"inlineimg\" />";
}
else
{
$replace = "<img src=\"$smilie[smiliepath]\" border=\"0\" alt=\"\" title=\"$smilie[title]\" class=\"inlineimg\" />";
}

// ##### START HACK - Usergroup Smilies #####
}
// ##### END HACK - Usergroup Smilies #####


Now for some freak of nature reason, it does recognize the usergroup.. however it parses the smilie as the previous smilieid that is now in category 102.

The one I'm testing it with is smilie id 867, the smilie it shows is 866

LOL :)

Any clues where else I might need to look?

Thanks tons for help with the array problem :)

W00t! Got it working with this code


// ##### START HACK - Usergroup Smilies #####
if ($smilie['imagecategoryid'] == "102")
{
$groupids = $this->registry->userinfo['membergroupids'];
$allowedgroup = "150";
$pos = strpos($groupids, $allowedgroup);
if ($pos === false)
{
$replace = "*Censored Smilies*";
}
else
{
if ($this->is_wysiwyg())
{
$replace = "<img src=\"$smilie[smiliepath]\" border=\"0\" alt=\"\" title=\"$smilie[title]\" smilieid=\"$smilie[smilieid]\" class=\"inlineimg\" />";
}
else
{
$replace = "<img src=\"$smilie[smiliepath]\" border=\"0\" alt=\"\" title=\"$smilie[title]\" class=\"inlineimg\" />";
}
}
}
else
{
// ##### END HACK - Usergroup Smilies #####

// if you change this HTML tag, make sure you change the smilie remover in code/php/html tag handlers!
if ($this->is_wysiwyg())
{
$replace = "<img src=\"$smilie[smiliepath]\" border=\"0\" alt=\"\" title=\"$smilie[title]\" smilieid=\"$smilie[smilieid]\" class=\"inlineimg\" />";
}
else
{
$replace = "<img src=\"$smilie[smiliepath]\" border=\"0\" alt=\"\" title=\"$smilie[title]\" class=\"inlineimg\" />";
}

// ##### START HACK - Usergroup Smilies #####
}
// ##### END HACK - Usergroup Smilies #####


Thanks guys for helping to point me in the right direction :)

[merk]

You should use the function is_member_of.

[Cyricx]

Can I use is_member_of in a php file? I thought it was a template conditional only.