[0]failed to use camelCase properly, (
serverCmdCloak is correct usage)
- no isObject checks anywhere, if the target client's player doesn't exist, it spews a console error out. this is a scripters nightmare, and jetz would've killed you if he had seen this code
[0]poor spacing on the package, packaged wrong function: your message will display every time the player spawns, instead of only when you first enter the game - oh hey
onClientEnterGame- $pref::server::name returns everything but the host's name, so in "Johnny's a douche" server, you'll get a beautiful "Welcome to a douche!" message
[0]you forgot to use camelCase on loving everything agjahsfjkagwfjkawfhawkf
- checks like this are totally useless: if(%cl.isAdmin || %cl.isSuperAdmin)
just do if(%cl.isAdmin) BTW you actually got camelCase right there
- %targetname = findclientbyname(%target).name; could be better used as this: %v = findclientbyname(%target).getPlayerName(); where %v stands for victim
- I don't honestly believe this will work findclientbyname(%target).player.addhealth(-%power);
- $pref::server::adminxver = 0.1; you have no current use for this, and why the hell is it a $pref that's stored when the server shuts down?
- another thin-
forget NO you did NOT JUST DO THAT
findclientbyname(%cl.name).setcontrolobject(findclientbyname(%targetname));findclientbyname(%cl.name)NO NO NO NO NO NO NO NO NO NO NO NO NO NOOOOOOOOOOOOOOOOOOOOOOOOOOOO
O
i'm ognna go die- you just defined a client by looking for the clients name
look, i'm not trying to call you stupid, but that line right there makes you look
really loving stupidfix it, learn from your mistake, and
never let me catch you doing it again!forget
replace EVERYTHING HERE
findclientbyname(%cl.name) with this
%cl%CL IS ALREADY DEFINED. YOU DO NOT NEED TO FIND THE CLIENT AGAIN - That's the magic of serverCmd's!
You have a LOT OF WORK TO DO, GET CRACK'N
dude you need a mentor to run code past seriously