Drupal 6: Can't add/revoke user role programatically (Triggers/Actions module) - drupal

Question

I'm trying to use Triggers and a custom action to give special role to users (on add/edit) given a particular profile field. So here's my code:
function mymodule_action_info() {
return array(
'mymodule_user_appropriate_role' => array(
'description' => t('Assign user special role'),
'type' => 'user',
'configurable' => FALSE,
'hooks' => array(
'user' => array('insert', 'update'),
),
),
);
}
and then
function mymodule_user_appropriate_role(&$object, $context = array()) {
// get the uid from the object
if( isset( $object->uid ) ){
$thisUID = $object->uid;
}else if( isset( $context['uid'] ) ){
$thisUID = $context['uid'];
}else{
global $user;
$thisUID = $user->uid;
}
// make sure we have a user record
if( $thisUID ){
// load user object
$thisUser = user_load( $thisUID );
// get user profile object
profile_load_profile( $thisUser );
if( $thisUser->profile_special_field == "value1" ){
// FIRST APPROACH
db_query( 'INSERT INTO {users_roles} (uid, rid) VALUES (%d, %d)', $thisUser->uid, 5 ); // 5 is the custom role ID
// SECOND APPROACH
$thisUserRoles = $thisUser->roles;
if( !isset( $thisUserRoles[5] ) ){
$thisUserRoles[5] = "RID 5 Rule Name";
}
user_save( $thisUser, array( 'roles' => $thisUserRoles ) );
// THIRD APPROACH
$allUserRoles = user_roles();
user_save( $thisUser, array( 'roles' => array( array_search( 'RID 5 Rule Name', $allUserRoles ) => 1 ) ) );
}
}
}
But none of these 3 approaches worked. I'm sure the action is called and entering the if( $thisUser->profile_special_field == "value1" ) statement
I'm struggling with this since yesterday, so any help is most welcome...

Answer 1

I ended up with hook_user. Here is the code for reference:
function mymodule_helper_user($op, &$edit, &$account, $category = NULL){
// DEFINE OPERATIONS LIST TO ACT ON
$opList = array( 'insert', 'update', 'after_update', 'submit' );
if( in_array( $op, $opList ) ){
// REVOKE ALL CUSTOM ROLES, HERE RID 5 AND 6
db_query( 'DELETE FROM {users_roles} WHERE rid IN (5,6) AND uid = %d', $account->uid );
if( $account->profile_custom_field == 'value1' ){
// GIVE CUSTOM RID 5
db_query( 'INSERT IGNORE INTO {users_roles} (uid, rid) VALUES (%d, %d)', $account->uid, 5 );
}else if( $account->profile_custom_field == 'value2' ){
// GIVE CUSTOM RID 6
db_query( 'INSERT IGNORE INTO {users_roles} (uid, rid) VALUES (%d, %d)', $account->uid, 6 );
}
}
}
For the operation list, doing it on "update" didn't work for me, as opposed to "after_update", that's why I added the four suspected ops until I test and keep only what's needed
Hope it helps