[Doc-Holiday]

VIP Menu
Version 0.0.4

Description:

This plugin is made to give the VIP a menu to choose a pistol other then the USP. It also gives them x amount of ammo for the pistol. For use on AS maps. (i.e. as_oilrig)

Tested in CZ.
For cz bot support you need the rage module found here
and the cz bot addon found here

If you compile and use this without rage you will get an error in console saying could not hook entity czbot.. it works with out it.. but wont support cz bots.

Cvars:

• vip_enabled - Turns the plugin on and off. (default: 1)
• vip_ammo - Amount of ammo to give for the pistol.(default: 200)

Credits:

• Apollyon - For testing.

Change log:

0.0.1
Initial Release
0.0.2
Removed menu_item_getinfo
Made the menu look better (capitals then convert to lower case before the give)
Global Menu

0.0.3

Added bot support (Gets random weapon from the weapons list)
Cleaned up some code and added comments

0.0.4

Fixed runtime error for index out of bounds

[ConnorMcLeod]

You usage of menus is incorrect :

Either you make the menu global (recommended), either you have to destroy it in callback.

Also, you don't need to store that info and to retrieve it to get the "key" pressed, item just do the same and is already an integer.
This way you don't need menu_item_getinfo in callback, you directly use item value.

Also, you could check if map is a vip map.
What about supercede the buy menu instead of forcing the menu to open ?

What about seeing further ?
http://forums.alliedmods.net/showthread.php?p=233877
http://forums.alliedmods.net/showthread.php?p=847019

[Doc-Holiday]

Hopefully thats what your talking about making it global? Im not really sure what you mean by its not global.. The g_VipMenu was a global variable.

also removed the stuff you asked.

[fysiks]

Your menu will continue to grow longer and longer. For a menu to be "global" you need to build the menu once then only display it when you need to show the menu.

[Doc-Holiday]

Originally Posted by fysiks Your menu will continue to grow longer and longer. For a menu to be "global" you need to build the menu once then only display it when you need to show the menu.

Ah.. So thats why i kept getting multiple pages?

[ConnorMcLeod]

PHP Code:

    new szNum[3]; 


This param is not used anymore since you use iItem in callback and that it is an optional param.
You can do

PHP Code:

menu_additem( g_vipmenu, g_WeaponDesc[i]) 


PHP Code:

        menu_setprop( g_vipmenu , MPROP_EXIT , MEXIT_NEVER ); 


This should also move to plugin_init.

PHP Code:

    format(VipWep, sizeof(VipWep), "weapon_%s", g_WeaponDesc[iItem]); 


Use formatex there, you don't use VipWe in format args.

Also enum is not needed :

PHP Code:

/*enum _:PISTOLS
{
    usp,
    glock,
    deagle,
    p228,
    elite,
    fiveseven
}*/

new const g_WeaponDesc[/* PISTOLS */] [] = 


[Doc-Holiday]

There we go.

[ConnorMcLeod]

By security, it would be better to inverse those 2 checks :

PHP Code:

    if(cs_get_user_vip(id) && is_user_alive(id)) 


You need to increase VipWep size by 1.



You COULD do tolower( VipWep[7] )


I think that's all

[r4ndomz]

Interesting idea. Maybe let them choose and ak47 too?

[Doc-Holiday]

Originally Posted by r4ndomz Interesting idea. Maybe let them choose and ak47 too?

Giving the Ak takes a way from the game play of the as style maps though?

How ever eventually i want to create a text file that reads to add them to menus.