[waylaidwanderer]

I'm trying to set flags for both admins/non-admins if they've donated to the gaming group I'm writing the plugin for. This allows them to open a VIP-only menu where they can access VIP-only commands.

However, it seems to be overwriting the flags that admins already have - meaning that while the admin-player can access the menu, they lose their admin privileges. Quote from the admin:

As for the admin issue it seems to happen randomly. On occasion both will work and other times only the perks work and the admin wont.

Here's the relevant code:

SetFlags(client)
{
    if (g_VipRank[client] == 2)
    {
        // check if user is admin already
        new AdminId:id = GetUserAdmin(client);
        if (id == INVALID_ADMIN_ID)
        {
            new AdminId:admin = CreateAdmin("VIP2");
            SetAdminFlag(admin, Admin_Reservation, true);
            SetUserAdmin(client, admin);
        }
        else
        {
            SetAdminFlag(id, Admin_Reservation, true);
        }
    }
    else if (g_VipRank[client] == 1)
    {
        // check if user is admin already
        new AdminId:id = GetUserAdmin(client);
        if (id == INVALID_ADMIN_ID)
        {
            new AdminId:admin = CreateAdmin("VIP1");
            SetAdminFlag(admin, Admin_Custom1, true);
            SetUserAdmin(client, admin);
        }
        else
        {
            SetAdminFlag(id, Admin_Custom1, true);
        }
    }
}

Any ideas?

[asherkin]

Where is that code being called from?

[waylaidwanderer]

"OnClientAuthorized" calls "CheckUser", which checks the database for the SteamID, and if not present, checks if the user is a member of the Steam group, then calls "SetFlags".

Now that you ask though... Should I change it to "OnClientPostAdminCheck" instead?

[DJ Data]

Originally Posted by waylaidwanderer "OnClientAuthorized" calls "CheckUser", which checks the database for the SteamID, and if not present, checks if the user is a member of the Steam group, then calls "SetFlags". Now that you ask though... Should I change it to "OnClientPostAdminCheck" instead?

Totally off topic, but arent you that famous SteamBot coder?

[waylaidwanderer]

Originally Posted by DJ Data Totally off topic, but arent you that famous SteamBot coder?

I may be one of them. I'm the owner and developer of http://scrapbank.me.