[medow]

PHP Code:

#include <amxmodx>
#include <amxmisc>
#include <fun>

#define PLUGIN "menu" 
#define VERSION "1.0" 
#define AUTHOR "r0ck" 

new bool:g_bFirstOptionAllowed

public plugin_init(){
    register_plugin(PLUGIN, VERSION, AUTHOR)

    register_logevent("logevent_round_start", 2, "1=Round_Start")

    register_menucmd(register_menuid(PLUGIN), 1023, "action_my_menu")
} 

public logevent_round_start(){
    new players[32], bnum
    get_players(players, bnum, "ae", "CT" )
    g_bFirstOptionAllowed = bnum >= 7 ? true : false
    new iPlayers[32], pNum
    get_players(iPlayers, pNum, "ae", "TERRORIST")
    for(new p = 0; p < pNum; p++){
        show_my_menu(iPlayers[p])
    }
}


public show_my_menu(id){

    // Variables for the menu
    new MenuKeys, MenuBody[512], szTemp[64]

    /*
    *     My Menu Title
    *
    *     1. My first option
    *     2. My second option
    *
    *     0. Exit
    */
    add(MenuBody, 511, "\rMy Menu Title\w^n^n")
    
    if (g_bFirstOptionAllowed){
        format(szTemp, 63, "\r1. \wMy first option\w^n")
        MenuKeys |= MENU_KEY_1
    }
    else{
        format(szTemp, 63, "\d1. \wMy first option\w^n")
    }
    add(MenuBody, 511, szTemp)

    add(MenuBody, 511, "\r2. \wMy second option^n^n")
    add(MenuBody, 511, "\r0. \wExit")
    MenuKeys |= MENU_KEY_2 | MENU_KEY_0

    // Display the menu for 30 secs
    show_menu(id, MenuKeys, MenuBody, 30, PLUGIN)

    return PLUGIN_HANDLED
}

public action_my_menu(id, key){
    new iSelected = key + 1

    switch(iSelected){
        case 1: client_cmd(id,"say /lol")
        case 2: set_user_health(id, 10000)
    }
    return PLUGIN_HANDLED
} 


I want the first option to work only if the number of CTs is higher than 7

PHP Code:

 new players[32], bnum
    get_players(players, bnum, "ae", "CT" )
    g_bFirstOptionAllowed = bnum >= 7 ? true : false
    new iPlayers[32], pNum
    get_players(iPlayers, pNum, "ae", "TERRORIST")
    for(new p = 0; p < pNum; p++){
        show_my_menu(iPlayers[p]) 


What's wrong ? Thanks in advance.

[Unkolix]

PHP Code:

new iCTCount;
/***/
public logevent_round_start()
{
    iCTCount = 0;
    new players[32], player, pnum; 
    get_players(players, pnum, "a");
    for ( new i; i < pnum; i++ )
    {
        switch( cs_get_user_team( players[i] ) )
        {
            case CS_TEAM_CT: iCTCount++;
        }
    }
    if ( iCTCount >= 7 )
    {
        show_my_menu(iPlayers[p])
    }
} 


[medow]

I want just the first option of the menu to be available only if there are more than 7 CTS, not the whole menu.

PHP Code:

*/ 
    add(MenuBody, 511, "\rMy Menu Title\w^n^n") 
     
    if (g_bFirstOptionAllowed){ 
        format(szTemp, 63, "\r1. \wMy first option\w^n") 
        MenuKeys |= MENU_KEY_1 
    } 
    else{ 
        format(szTemp, 63, "\d1. \wMy first option\w^n") 
    } 


And the menu is for terrorists, not CTS.

[ConnorMcLeod]

Your code is ok, what is wrong with your plugin at runtime ?

Try : register_menuid("My Menu Title"), because this string must be contained in the menu string

Or try to change :

#define PLUGIN "menu"

with

#define PLUGIN "Menu"

[Leon M.]

What do you mean by

Originally Posted by ConnorMcLeod because this string must be contained in the menu string

Is it not defined by

PHP Code:

show_menu(id, MenuKeys, MenuBody, 30, PLUGIN) 


because it works just fine for me.

EDIT:
@medow what's your issue? The menu doesn't appear?
Note: Probably you need to loop through all players and get the team with cs_get_user_team although get_players should work properly on roundstart. It may leads to failures later since get_players works like get_user_team, Connor will know what I'm talking about

PHP Code:

public logevent_round_start(){
    new iPlayers[32], pNum
    get_players(iPlayers, pNum, "ae", "CT")
    g_bFirstOptionAllowed = pNum >= 8 ? true : false

    get_players(iPlayers, pNum, "ae", "TERRORIST")
    for(new p = 0; p < pNum; p++){
        show_my_menu(iPlayers[p])
    }
} 


[ConnorMcLeod]

/* Gets unique id of menu. Outside set to 1 allows
* to catch menus outside a plugin where register_menuid is called. */
native register_menuid(const menu[], outside=0 );

/* Displays menu. Keys have bit values (key 1 is (1<<0), key 5 is (1<<4) etc.). */
native show_menu(index,keys,const menu[], time = -1, const title[] = "");

String you pass in register_menuid() MUST be a part of menu string you pass in show_menu(), and is better when the string you choose is unique (prevent bugs such as bugs with restmenu when you register "Buy" string and that "Buy" is contained in a lot of default cs menus.

That's why in his case it is better to use :

register_menuid("My Menu Title")

Because i don't know if register_menuid is case sensitive, but if it is, public function action_my_menu won't be called on player key hit.


It is a fact and it is not negaciable

[Leon M.]

Alright thanks for the explanation. Then I got lucky since I call my menues like the plugin

Gonna change it for the future, thanks again

[ConnorMcLeod]

Things are not magic, if you think a bit, it is the only way that amxx can make the link between show_menu thing, and callback ;)

Also, that is the reason why sometimes, people translate a plugin and menu is not working anymore because register_menuid string doesn't match any menu anymore