Check usergroup of thread starters in threadbit

[LifesGreatestGift]

create new plugin

Hook Location: threadbit_display
Title: usergroup check for threadstarter in threadbit
Plugin PHP Code:

PHP Code:

$current_thread = $thread['threadid'];

$query = $vbulletin->db->query_first_slave("
    SELECT user.usergroupid, user.membergroupids
    FROM " . TABLE_PREFIX . "thread AS thread
    LEFT JOIN " . TABLE_PREFIX . "user AS user ON (user.userid = thread.firstpostid)
    WHERE thread.threadid = " . $current_thread . "
    ");
    
$primary_group = $query['usergroupid'];
$secondary_groups = $query['membergroupids'];

if(!empty($secondary_groups)) {
    $groups = $primary_group . "," . $secondary_groups;
} else {
    $groups = $primary_group;
}    
$groups_all = explode(",", $groups);

vB_Template::preRegister('threadbit',array('group' => $groups_all)); 


then in template threadbit

You could do

<vb:if condition="in_array(6, $group)">This thread starter is in admin group</vb:if>

The 6 being the group you want to check, and this example 6 is the Administrator group.

This checks the primary and additional groups of a thread starter.

[addamroy]

Thanks!

Like Lynne said about the additional queries, I really only want to do this one or two sections of the forum. Is there anyway to have the plug-in only run that query for specific forums?

[LifesGreatestGift]

PHP Code:

if (in_array($thread['forumid'], array(1,2,3)))

{

    $current_thread = $thread['threadid'];

    $query = $vbulletin->db->query_first_slave("
        SELECT user.usergroupid, user.membergroupids
        FROM " . TABLE_PREFIX . "thread AS thread
        LEFT JOIN " . TABLE_PREFIX . "user AS user ON (user.userid = thread.firstpostid)
        WHERE thread.threadid = " . $current_thread . "
        ");
        
    $primary_group = $query['usergroupid'];
    $secondary_groups = $query['membergroupids'];

    if(!empty($secondary_groups)) {
        $groups = $primary_group . "," . $secondary_groups;
    } else {
        $groups = $primary_group;
    }    
    $groups_all = explode(",", $groups);

    vB_Template::preRegister('threadbit',array('group' => $groups_all));

} 


change 1,2,3 to forums you want it to run in.

[addamroy]

When I go to save the threadbit template after inserting your if conditional, I get the following error message:

"Warning: Invalid argument supplied for foreach() in [path]/includes/functions.php on line 3568

Forum Message
The following error occurred when attempting to evaluate this template:
%1$s
This is likely caused by a malformed conditional statement. It is highly recommended that you fix this error before continuing, but you may continue as-is if you wish."

[LifesGreatestGift]

Originally Posted by kh99

Kudos to LifesGreatestGift for writing the plugin. But I think there may be one small problem: on this line:

    LEFT JOIN " . TABLE_PREFIX . "user AS user ON (user.userid = thread.firstpostid)

I think you want thread.postuserid instead of thread.firstpostid.

As is, if you do a var_dump on $groups it shows the proper primary and secondary groups of the threadstarter.
The code works, I just messed up the "Specific forums" conditional.

if (in_array($thread['forumid'], array(1,2,3)))

please review other mods and vB code for proper usage. But other than that, the code is sound, and taken directly from forumdisplay.php

--------------- Added [DATE]1361811270[/DATE] at [TIME]1361811270[/TIME] ---------------

Originally Posted by addamroy

When I go to save the threadbit template after inserting your if conditional, I get the following error message:

"Warning: Invalid argument supplied for foreach() in [path]/includes/functions.php on line 3568

Forum Message
The following error occurred when attempting to evaluate this template:
%1$s
This is likely caused by a malformed conditional statement. It is highly recommended that you fix this error before continuing, but you may continue as-is if you wish."

you must be doing something wrong because the code is tested as working 100% on vanilla vbulletin 4.2 pl3

--------------- Added [DATE]1361811317[/DATE] at [TIME]1361811317[/TIME] ---------------

i didnt even mess up the conditional it all works 100%

--------------- Added [DATE]1361811597[/DATE] at [TIME]1361811597[/TIME] ---------------

It was throwing an error, try this code

PHP Code:

if (in_array($thread['forumid'], array(3,4,5))) 

{ 

    $current_thread = $thread['threadid']; 

    $query = $vbulletin->db->query_first_slave(" 
        SELECT user.usergroupid, user.membergroupids 
        FROM " . TABLE_PREFIX . "thread AS thread 
        LEFT JOIN " . TABLE_PREFIX . "user AS user ON (user.userid = thread.firstpostid) 
        WHERE thread.threadid = " . $current_thread . " 
        "); 
         
    $primary_group = $query['usergroupid']; 
    $secondary_groups = $query['membergroupids']; 

    if(!empty($secondary_groups)) { 
        $groups = $primary_group . "," . $secondary_groups; 
    } else { 
        $groups = $primary_group; 
    }     
    $groups_all = explode(",", $groups); 

} else { 

$groups_all = array(); 

} 
    vB_Template::preRegister('threadbit',array('group' => $groups_all)); 


[LifesGreatestGift]

change 3,4,5 to forums you want it ran in.

--------------- Added [DATE]1361812033[/DATE] at [TIME]1361812033[/TIME] ---------------

PHP Code:

if (in_array($thread['forumid'], array(3,4,5))) 

{ 

    $current_thread = $thread['threadid']; 

    $query = $vbulletin->db->query_first_slave(" 
        SELECT user.usergroupid, user.membergroupids 
        FROM " . TABLE_PREFIX . "thread AS thread 
        LEFT JOIN " . TABLE_PREFIX . "user AS user ON (user.userid = thread.firstpostid) 
        WHERE thread.threadid = " . $current_thread . " 
        "); 
         
    $primary_group = $query['usergroupid']; 
    $secondary_groups = $query['membergroupids']; 

    if(!empty($secondary_groups)) { 
        $groups = $primary_group . "," . $secondary_groups; 
    } else { 
        $groups = $primary_group; 
    }     
    $groups_all = explode(",", $groups); 

} else { 

$groups_all = array(); 

} 
    vB_Template::preRegister('threadbit',array('group' => $groups_all)); 


This is the proper working code. The reason the first conditional threw an error was because when you visit a forum not listed in the 3,4,5 the template threadbit was still looking for the $group array and it wasn't defined. New code puts null array in place so there isn't an error.

[addamroy]

Still getting the same error.

I'm using vb 4.1.12 if that makes any difference (since you said you tested on 4.2)? I don't believe I have any mods doing anything in the threaddisplay hook or threadbit template at all so I can't think of what might be conflicting

[LifesGreatestGift]

post me the conditional you are using exactly as is. or just post the contents of threadbit template in code tags.

[mokujin]

Originally Posted by kh99

Isn't there some vb code somewhere that's depending on the last poster information being there?

I dont know

[LifesGreatestGift]

lol, ridiculous.

--------------- Added [DATE]1361827948[/DATE] at [TIME]1361827948[/TIME] ---------------

then when you upgrade its something else to remember to change.

--------------- Added [DATE]1361828625[/DATE] at [TIME]1361828625[/TIME] ---------------

also, my code accounts not only a users primary usergroup, but additional usergroups as well.