[Burridge]

I'm pretty sure you don't need the || newkeys == KEY_FIRE in your code if you using the Holding define. That is probably where you error lies.

Код:

if(Holding(KEY_FIRE))

Just try leaving the line like that and see if it works.

Also isn't Holding defined in all caps?

Код:

// HOLDING(keys)
#define HOLDING(%0) \
	((newkeys & (%0)) == (%0))

Unless you defined it as lowercase.

Also why are you formatting a string when it doesn't even require any formatting.

Код:

.....
new string[256];
.....
format(string, sizeof(string), "Auto-Dispatcher: The fire that was started has been successfully put out. Continue with normal duties.");
.....
if(IsACop(i) || IsAGov(i) || IsMedic(i) || IsSASD(i))
{
    SendClientMessage(i, COLOR_DEPTRADIO, string);
}

You could just delete the string variable and the format line and have the string get send out through SendClientMessage.

Код:

if(IsACop(i) || IsAGov(i) || IsMedic(i) || IsSASD(i))
{
    SendClientMessage(i, COLOR_DEPTRADIO, "Auto-Dispatcher: The fire that was started has been successfully put out. Continue with normal duties.");
}