PHP Code:
public logevent_round_end(id)
=>
PHP Code:
public logevent_round_end( )
Even though it shouldn't do any harm, it's always better if it's as it should be.
Remove is_user_alive check from the loop, it's useless as you're already looping only through alive players (flag "a")
Remove cs_get_user_team check and add flag "e" to get_players, then add an extra parameter "CT":
PHP Code:
get_players( players, num, "ae", "CT" )
For the second part;
Code:
public fw_player_killed(victim, attacker, shouldgib)
{
new HasFlag = _:( get_user_flags( attacker ) & ADMIN_LEVEL_H ), _cacheCvar = get_pcvar_num( g_killsjp )
if(!is_user_connected(victim) || !is_user_connected(attacker) || attacker == victim || get_playersnum() < 4)
return HAM_IGNORED
if(get_user_team(attacker) == 1)
{
if( HasFlag )
{
g_jbpacks[attacker] += _cacheCvar * 2
}
else
{
g_jbpacks[attacker] += _cacheCvar
}
if(get_pdata_int(victim, 75) == HIT_HEAD)
{
if( HasFlag )
{
g_jbpacks[attacker] += _cacheCvar * 2
}
else
{
g_jbpacks[attacker] += _cacheCvar
}
}
}
return HAM_IGNORED
}
You could also change this:
Code:
if(HasFlag)
{
g_jbpacks[attacker] += _cacheCvar * 2
}
else
{
g_jbpacks[attacker] += _cacheCvar
}
to this:
Code:
g_jbpacks[ attacker ] += HasFlag ? _cacheCvar * 2 : _cacheCvar
Same for the second if/else.