[Cyril Tuto]

Hello,
Thank you for this plugin!
That should be removed or added to only the players with the flag "o" can be colored smoke? Thank you!

[ahmedamin]

Thank you m8

[RedSword]

Hi there,

I'm unapproving your plugin, as past errors (like TnTSCS ones, as well as DoD:S still being announced as compatible) weren't corrected.

Red

[HSFighter]

Originally Posted by RedSword Hi there, I'm unapproving your plugin, as past errors (like TnTSCS ones, as well as DoD:S still being announced as compatible) weren't corrected. Red

Hi RedSword,

• Test on 10 CS:S Server's!
• No errors until jet!
• Announce for DoD:S removet.

Please Aprove again.

[RedSword]

When creating an entity, you should ensure that it is well created (!= -1) before using functions on it. L112

Also, please, please, please don't do that :

public bool:IsClientOK(client)
{
	if (client != 0)
	{
		// Get all clients on the server
		for (new i = 1; i <= MaxClients; i++)
		{
			// Check if player ok
			if (IsClientConnected(i) && IsClientInGame(i) && !IsFakeClient(client))
			{
				// Check if client is a player on the server
				if (i == client) return true;			
			}
		}	
	}	
	return false;
}

Use something simpler like :

public bool:IsValidClient(client)
{
	if ( !( 1 <= client <= MaxClients ) || !IsClientInGame(client) || IsFakeClient(client) )
	{
		return false
	}
	return true
}

You really don't need to loop through all players to validate if a player is present.

That might not be; all pass more conciously latter.

[RedSword]

Hi again,

There is still something that should be adressed before approving :

If near round end, a player throw and detonate a smoke, than a round restart and another player get to have the same index for his smoke grenade, his smoke will be killed earlier.

To prevent that, use EntIndexToEntRef when calling the timer and retrieve the entity index with EntRefToEntIndex and make sure it is != -1.

Also, the rest below doesn't prevent approval, but it should be considered :

• If you're unlucky, a player could throw a smoke grenade, go spectate, than it would detonate and a random input would be given as color. This can't crash, but I don't think it is a wanted behaviour (gray, possibly random). To prevent that simply handle team 1, or make it so you remember last team.
  
• Don't implement OnMapStart if you're not going to use it. You're wasting ~9 lines.
  
• L123 to 127. You don't need to do that. "smokegrenade_detonate" is already hooked; and it only exist in CS:S/GO. Currently, you're only locking yourself to CS:S. You also don't have to make sure your plugin doesn't produce errors on other unannounced mods. If you really feel repulsion towards CS:GO players, then you can let it as it is lol.
• You better let it free to all mods (I mean by this, last point; but still write this plugin as CSS only) and if you're lucky someone will come report it working under CS:GO.

Red

[minimoney1]

Originally Posted by RedSword Hi again, There is still something that should be adressed before approving : If near round end, a player throw and detonate a smoke, than a round restart and another player get to have the same index for his smoke grenade, his smoke will be killed earlier. To prevent that, use EntIndexToEntRef when calling the timer and retrieve the entity index with EntRefToEntIndex and make sure it is != -1.

Yeah, I actually fixed this locally when I was using it but forgot to post it here.
I had players reporting that their weapons just randomly disappeared and got this fixed with the help of TheTwistedPanda.

[Pialx]

Could-you confirm the bug with weapons when sometimes the primary weapon disappears was fixed?
And what about the logs flooded in sm_grenadesmokecolor_mode "2" ?

I want to use this funny plugin!

[Bacardi]

*this should prevent players weapons from disappear.

...

who want do fixes, try replace timer

PHP Code:

            CreateTimer(20.0, Delete, iEntity, TIMER_FLAG_NO_MAPCHANGE);
        } 


with this.

PHP Code:

            Format(sBuffer, sizeof(sBuffer), "OnUser1 !self:kill::20.0:-1");
            SetVariantString(sBuffer);
            AcceptEntityInput(iEntity, "AddOutput");
            AcceptEntityInput(iEntity, "FireUser1");
        } 


This timer will be entity own and will terminated automatically if round restart and restarting all entities.

[thetwistedpanda]

Bacardi - That won't correctly resolve the issue, if at all. The issue is CreateTimer is being passed an entity index, which is always subject to change (much like a client index). The correct way to resolve that bug is as RedSword suggested above. Although it looks like they overlooked a memory leak with Checktime only being killed once regardless of how many smoke grenades are passed.