Stuck on php code for the postbit? lol

[HMBeaty]

Ok, I'm a little confused as to why this isn't working. The code works, however, when I try adding in the option so only certain usergroups can('t) view the content, it doesn't work, nothing shows up.

Here's the current code I'm using:
Hook location: postbit_display_start

PHP Code:

$noadsense = explode(',', $vbulletin->options['usml_google_adsense_usergroups']);

if ($this->registry->options['usml_google_adsense_enable'] && is_member_of($vbulletin->userinfo, $noadsense))
{
    if ($this->registry->options['usml_google_adsense_firstpost'] && $post['isfirstshown'])
    {
        $template_hook['postbit_end'] .= '
        <li class="firstpost_advert_container">
            <div class="firstpost_advert">
                '.$this->registry->options[usml_google_adsense_firstpostadcode].'
            </div>
        </li>';
    }
} 


Obviously it's something silly I'm looking over (again ), but what is the problem that

PHP Code:

is_member_of($vbulletin->userinfo, $noadsense) 


doesn't work as it should?

[Boofo]

Try:

is_member_of($this->userinfo, $noadsense)

AND

$noadsense = explode(',', $this->registry->options['usml_google_adsense_usergroups']);

[HMBeaty]

Originally Posted by Boofo

Try:

is_member_of($this->userinfo, $noadsense)

AND

$noadsense = explode(',', $this->registry->options['usml_google_adsense_usergroups']);

Still nothing lol

[kh99]

Maybe try $this->registry->userinfo.

The list of usergroups in the option is the groups that *will* see the ad, right?

[HMBeaty]

Originally Posted by kh99

Maybe try $this->registry->userinfo.

The list of usergroups in the option is the groups that *will* see the ad, right?

That didn't work either lol, and yes (for now anyway).

[Boofo]

Try using your old code and making $vbulletin global. Did you try using the postbit_display_complete hook instead?

[HMBeaty]

Originally Posted by Boofo

Try using your old code and making $vbulletin global. Did you try using the postbit_display_complete hook instead?

Ok, did that and now it works (still with postbit_display_start). Thanks guys!

[Boofo]

I was going to suggest making $vbulletin global the first time around but I usually try avoiding doing that in the postbit hooks if at all possible.

[HMBeaty]

Originally Posted by Boofo

but I usually try avoiding doing that in the postbit hooks if at all possible.

Why is that? Or is that just personal preference?

[Boofo]

They took out the global $vbulletin for the postbit a while back and I just got used to using to not having to make it global, I guess.