[echo_cs]

Plugin has not errors But He Does not work

Includes:

PHP Code:

#include <amxmodx>
#include <chatcolor>
#include <fun> 


Defines:

PHP Code:

#define PLUGIN "Gift Menu"
#define VERSION "1.0"
#define AUTHOR "echo_cs" 


Plugin Main:

PHP Code:

public plugin_init() {
    register_plugin(PLUGIN, VERSION, AUTHOR)
    register_cvar("gift_menu", "1")
    register_clcmd("say /gift", "Gift_Menu")
} 


The Script:

PHP Code:

public Gift_Menu(id)
{
    if(!get_cvar_num("gift_menu") )
        return PLUGIN_CONTINUE
    
    new menu = menu_create("\rSend Gift To:", "menu_handler");
    
    new players[32], pnum, tempid;
    
    new szName[32], szTempid[10];
    
    get_players(players, pnum);
    
    for( new i; i<pnum; i++ )
    {
        tempid = players[i];
        
        get_user_name(tempid, szName, charsmax(szName));
        num_to_str(tempid, szTempid, charsmax(szTempid));
        
        menu_additem(menu, szName, szTempid, 0);
        
    }
    
    menu_display(id, menu, 0);
    return PLUGIN_CONTINUE
}

public menu_handler(id, menu, item)
{
    new rand = random_num(1,5)
    
    if( item == MENU_EXIT )
    {
        menu_destroy(menu);
        return PLUGIN_HANDLED;
    }
    
    
    new data[6], szName[64], Iname[32]
    
    new access, callback;
    menu_item_getinfo(menu, item, access, data,charsmax(data), szName,charsmax(szName), callback);

    new tempid = str_to_num(data);
    {
    get_user_name(tempid, szName, 31)
    get_user_name(id, Iname, 31)
    
    switch(rand) 
        {
        case 1:set_user_health(tempid,get_user_health(tempid) + 50)
        case 2:set_user_noclip(tempid,1)
        case 3:set_user_footsteps(tempid, 0)
        case 4:set_user_frags(tempid, get_user_frags(tempid) + 10)
        case 5:set_user_gravity(tempid, 0.5)
        }
        
    client_print_color(0, DontChange,"%s Send a Little Gift To %s", Iname, szName);
    }
menu_destroy(menu);
return PLUGIN_HANDLED
} 


Help me and thanks

[kotinha]

You have an extra pair of {} in menu_handler() next to new tempid = str_to_num(data); and client_print_color(0, DontChange,"%s Send a Little Gift To %s", Iname, szName);

Also, you don't have to retrieve the name of tempid in menu_handler(), because you already have it in szName when you use menu_get_iteminfo().