[ConnorMcLeod]

Old menu is perfectly fine.

Make sure :

- you use MENU_KEY_3 to register keys
- you use register_menucmd + register_menuid (or amxmisc stock register_menu() ).

Also, better to use a switch statement.

Code :

Spoiler

PHP Code:

#include < amxmodx >
#include < cstrike >
#include < fun >

#pragma semicolon 1

#define PLUGIN ""
#define VERSION "0.0.1"

const WINNING_TEAM_MENU_KEYS = MENU_KEY_1|MENU_KEY_2|MENU_KEY_3;

public plugin_init()
{
    register_menucmd(register_menuid("Winning Team"), WINNING_TEAM_MENU_KEYS, "Gifts");
}

public GiftsMenu(id)
{
    static const szMenu[] = "\rWinning Team :^n^n\w1. \yExtra cash 1000$^n\w2. \yExtra life + 25 HP^n\w3. \yAWP";

    show_menu(id, WINNING_TEAM_MENU_KEYS, szMenu);

    return PLUGIN_HANDLED;
}

public Gifts(id, key)
{
    switch( key )
    {
        case 0:
        {
            MomeyGift(id);
        }
        case 1:
        {
            HpGift(id);
        }
        case 2:
        {
            give_item(id, "weapon_awp");
        }
    }

    return PLUGIN_HANDLED;
} 


If you prefer to use new menu system, use a static menu, since your menu is not dynamic, so you only build it once and you are done, so in fine it is pretty similar in that case to old menu system.

Code :

Spoiler

PHP Code:

new g_iWinningTeamMenu;

public plugin_init()
{
    Create_WinningTeam_Menu();
}

Create_WinningTeam_Menu()
{
    g_iWinningTeamMenu = menu_create("Winning Team :", "Gifts");
    menu_additem(g_iWinningTeamMenu, "\yExtra cash 1000$");
    menu_additem(g_iWinningTeamMenu, "\yExtra life + 25 HP");
    menu_additem(g_iWinningTeamMenu, "\yAWP");
}

public GiftsMenu(id)
{
    menu_display(id, g_iWinningTeamMenu);

    return PLUGIN_HANDLED;
}

public Gifts(id, menu, item)
{
    switch( item )
    {
        case 0:
        {
            MomeyGift(id);
        }
        case 1:
        {
            HpGift(id);
        }
        case 2:
        {
            give_item(id, "weapon_awp");
        }
    }

    return PLUGIN_HANDLED;
}