[Huntereb]

Originally Posted by TheHardstyleBro Well how can I use it? If it's for admins only

Just install it and people won't be able to use the glitch anymore, nothing to configure.

[Dr. McKay]

Nice job!

You forgot to fill in your public cvar in your OP, so I did so for you.

Also, I'd recommend documenting the anticolorabuseflag override, which defaults to ADMFLAG_GENERIC, in the OP.

Finally, instead of this:

PHP Code:

g_bEnabled = (StringToInt(newValue) == 0 ? false : true); 


You could just use this:

PHP Code:

g_bEnabled = bool:StringToInt(newValue); 


This will convert the integer into a bool, since all cells are the same in Pawn anyway, just tagged differently. If you were unable to tag it for any reason, you should do this:

PHP Code:

g_bEnabled = (StringToInt(newValue) == 1); 


The (StringToInt(newValue) == 1) returns a boolean result, no need to use a conditional.

Either way, good job. I'd love to see this implemented directly into the processor.

[11530]

Originally Posted by Dr. McKay Nice job! You forgot to fill in your public cvar in your OP, so I did so for you.

Yea for some reason whenever I edit the public cvar myself it doesn't take effect.

Originally Posted by Dr. McKay Also, I'd recommend documenting the anticolorabuseflag override, which defaults to ADMFLAG_GENERIC, in the OP.

Done

Originally Posted by Dr. McKay Finally, instead of this: PHP Code: g_bEnabled = (StringToInt(newValue) == 0 ? false : true);  You could just use this: PHP Code: g_bEnabled = bool:StringToInt(newValue);  This will convert the integer into a bool, since all cells are the same in Pawn anyway, just tagged differently. If you were unable to tag it for any reason, you should do this: PHP Code: g_bEnabled = (StringToInt(newValue) == 1);  The (StringToInt(newValue) == 1) returns a boolean result, no need to use a conditional. Either way, good job. I'd love to see this implemented directly into the processor.

I've never been a great fan of casting, even in an almost typeless language. Your last expression isn't quite the same as mine however. Mine will set the variable to false if the convar is 0, and true for everything else. Yours sets the variable to true if the convar is 1 and false for everything else. Granted, I could use:

PHP Code:

g_bEnabled = (StringToInt(newValue) != 0); 


But I may as well stick to mine since they'd be synonymous and I'm a lazy critter.

[Dr. McKay]

Originally Posted by 11530 Yea for some reason whenever I edit the public cvar myself it doesn't take effect.

If you change the public cvar, it requires an unapproval and reapproval, so that's why that happens.

[11530]

Originally Posted by Dr. McKay If you change the public cvar, it requires an unapproval and reapproval, so that's why that happens.

Ah interesting, even from blank?

[Dr. McKay]

Originally Posted by 11530 Ah interesting, even from blank?

Yup.

[vodka00]

It seems that people are still able to use colors even when I have this plugin running. Help please.

[Horsedick]

Originally Posted by vodka00 It seems that people are still able to use colors even when I have this plugin running. Help please.

explain how? You didn't give much information with this.

[vodka00]

Originally Posted by Horsedick explain how? You didn't give much information with this.

Well, they are not telling me. I just assume they are using same method this plugin is trying to block. Unless there is a different method for Eventscripts.

[Horsedick]

Originally Posted by vodka00 Well, they are not telling me. I just assume they are using same method this plugin is trying to block. Unless there is a different method for Eventscripts.

lol who is they? Post some SteamID's so rest of can be on the look out and perhaps ask them or view the logs from our end. Look at your logs they can't skirt the logs it will show up in those.