Convert Secondary UserGroup to Primary UserGroup

[s.blair]

I've tried this hack, but when I only select an Additional Usergroup that only has 6 people in it, it instead emails all 1600 users.

Running v3.5.4 here with the hacks as listed by Marco and tylneyt.

I had to leave these lines below commented out as otherwise they would cause the crash smdani was seeing above.

$condition .= ' AND (' . implode(' OR ', $u_condition) . ')';
unset($u_condition);

Any help??

Thanks.
Scott

[Warlock40]

Originally Posted by tylneyt

there needs to be a line added for 3.5.4, then it works just fine:

REPLACE by:

PHP Code:

// Start hack MarcoH64 
    if (is_array($user['usergroupid']) OR is_array($user['membergroupids'])) 
    { // for emails
// start addition, by tylneyt
// this line is necessary to create the variable in this scope
//
        $u_condition = array();
// end addition 
        if (is_array($user['usergroupid'])) 
        { 
            foreach($user['usergroupid'] AS $id) 
            { 
                $u_condition[] = "usergroupid = $id"; 
            } 
        } 
        if (is_array($user['membergroupids'])) 
        { 
            foreach($user['membergroupids'] AS $id) 
            { 
                $u_condition[] = "find_in_set($id, membergroupids)"; 
            } 
        } 
        // End hack MarcoH64 


hope that helps, should actually be a standard feature of next vB if you ask me!

Thanks for the update. I'm going to wait for 3.6 stable before I tackle this one again. I still love the idea and agree it should be a standard option in VB.

[makaiguy]

Now that 3.6.0 is out, does anybody have any experience with this hack in 3.6.0?

[johnmont]

I had to do this for 3.5.4 -- its also apparently good practice to add intval to the condition....

REPLACE by:

PHP Code:

// Start hack MarcoH64 
if (is_array($user['usergroupid']) OR is_array($user['membergroupids'])) 
    { // for emails 
        if (is_array($user['usergroupid'])) 
        { 
            foreach($user['usergroupid'] AS $id) 
            { 
                $u_condition[] = "{$prefix}usergroupid = " . intval($id);
            } 
        } 
        if (is_array($user['membergroupids'])) 
        { 
            foreach($user['membergroupids'] AS $id) 
            { 
                $u_condition[] = "FIND_IN_SET('" . intval($id) . "', {$prefix}membergroupids)";
            } 
        } 
// End hack MarcoH64