[Psyk0tik]

Why are you assigning an empty global Handle to another global Handle? You're not closing your global Handle properly. All you're doing is assigning empty garbage to it.

Also, you're creating an event without firing it. According to the API, you must close the Handle to the event by either firing or canceling it.

https://sm.alliedmods.net/new-api/events/CreateEvent

[axelnieves2012]

Originally Posted by Crasher_3637 Why are you assigning an empty global Handle to another global Handle? You're not closing your global Handle properly. All you're doing is assigning empty garbage to it. Also, you're creating an event without firing it. According to the API, you must close the Handle to the event by either firing or canceling it. https://sm.alliedmods.net/new-api/events/CreateEvent

Will take a look, I've done this plugin the past year, I don't even remember what it is for lol

[axelnieves2012]

Hello everybody. Thanks a lot for your reports. I've updated this plugin to new syntax, fixed bugs and removed unnecessary content. I will test it for some days and if everything is ok, will post it

[axelnieves2012]

New version has been released. Check above link

[Angerfist2188]

Hello everyone! How fix this error?

L 10/03/2020 - 09:02:15: [SM] Exception reported: Client 4 is not connected
L 10/03/2020 - 09:02:15: [SM] Blaming: l4d_kills_manager.smx
L 10/03/2020 - 09:02:15: [SM] Call stack trace:
L 10/03/2020 - 09:02:15: [SM] [0] GetClientName
L 10/03/2020 - 09:02:15: [SM] [1] Line 338, C:\Users\Desktop\sm10\sourcemod-1.11.0-git6625-windows\addons\sourcemod\scripting\l4d_kills_ manager.sp::show_statisctics
L 10/03/2020 - 09:02:15: [SM] [2] Line 412, C:\Users\Desktop\sm10\sourcemod-1.11.0-git6625-windows\addons\sourcemod\scripting\l4d_kills_ manager.sp::event_player_death

[Zheldorg]

This version is only for Left 4 Dead 2, since I don't have the opportunity to test the work in l4d1, the support for it was cut out.
Removed unused and sorted the code, fixed the "[SM] [0] GetClientName" error in line 338 and made some changes, the plugin no longer uses fake events to store damage data, the ArrayList object is used instead; OnEntityCreated is not used for making the witch's damage hook, instead the hook for the "witch_spawn" event is used; the plugin also uses variables to store ConVars at startup and monitors changes in ConVars during execution, rather than constantly receiving them through the GetConVar function; cVar l4d_killsmgr_stats_enable no longer exists, instead l4d2_killsmgr_stats_show_type 0 to disable statistics output or l4d2_killsmgr_stats_show_type ... {1, 2, 3} as originally described; l4d_killsmgr_include_bosses_l4d2 is now just l4d2_killsmgr_include_bosses.
P.S. The color of the statistic text has been changed to be less fussy. It is still easy to understand that an infected has been killed and to see if someone has done damage without overlapping other chat notifications.

[axelnieves2012]

Originally Posted by Zheldorg This version is only for Left 4 Dead 2, since I don't have the opportunity to test the work in l4d1, the support for it was cut out. Removed unused and sorted the code, fixed the "[SM] [0] GetClientName" error in line 338 and made some changes, the plugin no longer uses fake events to store damage data, the ArrayList object is used instead; OnEntityCreated is not used for making the witch's damage hook, instead the hook for the "witch_spawn" event is used; the plugin also uses variables to store ConVars at startup and monitors changes in ConVars during execution, rather than constantly receiving them through the GetConVar function; cVar l4d_killsmgr_stats_enable no longer exists, instead l4d2_killsmgr_stats_show_type 0 to disable statistics output or l4d2_killsmgr_stats_show_type ... {1, 2, 3} as originally described; l4d_killsmgr_include_bosses_l4d2 is now just l4d2_killsmgr_include_bosses. P.S. The color of the statistic text has been changed to be less fussy. It is still easy to understand that an infected has been killed and to see if someone has done damage without overlapping other chat notifications.

Thanks for your work!! Congrats

[Beatles]

Just as an idea, I think it would be better, than the death message of the game itself, instead of showing the murderer who did the most damage, it should show the 2 participants as murderers, something like this:

Player1, Player2 killed Hunter

Being Player1 the one who caused the most damage, and Player2 the one who managed to take the last kill shot, and as he is after Player1, that marks him as an assist.

[Proaxel]

@Zheldorg It works great but there's one problem, when used with Custom Player Stats, it does properly give points to the person that did the most damage. However, if the person that did the most damage also killed the target with a headshot, the stats plugin never detects this, it always acts as if the target was killed normally even if it was a headshot. Can this be fixed?

Edit: I think I may have fixed it myself. I edited this part in player_death_event:

PHP Code:

            int new_killer = sortDamagers(victim);
            int org_killer = GetClientOfUserId(event.GetInt("attacker"));
            if (IsValidClientInGame(new_killer))
            {
                char weaponname[32], attackerName[64];
                int PropArray[2];
                GetClientName(new_killer, attackerName, sizeof(attackerName));
                g_hBaseEventsPropArr[victim][new_killer].GetString(0, weaponname, sizeof(weaponname));
                g_hBaseEventsPropArr[victim][new_killer].GetArray(1, PropArray, 2);
                
                //Proaxel: attempted headshot fix. Force headshot to true if the killing blow was a headshot and the person that got the killing blow is the same person that did the most damage, 
                //if not then do it however the original author was doing it
                bool topDmgHeadshot = false;
                if(event.GetBool("headshot") && IsValidClientInGame(org_killer) && org_killer == new_killer)
                    topDmgHeadshot = true;
                
                //Modifying death event...
                event.SetInt("attacker", GetClientUserId(new_killer));
                event.SetString("attackername", attackerName);
                event.SetInt("attackerentid", 0);
                event.SetString("weapon", weaponname);
                //Proaxel: Determine Headshot from earlier
                if(topDmgHeadshot)
                    event.SetBool("headshot", true);
                else
                    event.SetBool("headshot", (PropArray[c_mHitgroup] == 1)?true:false);
                event.SetBool("attackerisbot", IsFakeClient(new_killer));
                event.SetBool("abort", false);
                event.SetInt("type", PropArray[c_mDamagetype]);
} 


With this change the stats plugin correctly sees headshots now.

[CosmicD]

in l4d1 I notice that one user can do 6000 damage and another one 3000 (in expert 8000hp tanks), but the specials are mostly registering OK (total hp). I know that you can crown a 1117hp witch, but somehow the plugin only reports 1007..