PlayerHurt: fixed nil WeaponInstance for some cases (#171)

Author: markus-wa

datatables.go

@@ -5,6 +5,7 @@ import (
 	"strings"
 
 	"github.com/golang/geo/r3"
+	"github.com/markus-wa/go-unassert"
 
 	"github.com/markus-wa/demoinfocs-golang/common"
 	"github.com/markus-wa/demoinfocs-golang/events"
@@ -379,7 +380,17 @@ func (p *Parser) bindGrenadeProjectiles(entity *st.Entity) {
 	proj.EntityID = entityID
 	p.gameState.grenadeProjectiles[entityID] = proj
 
+	var wep common.EquipmentElement
 	entity.OnCreateFinished(func() {
+		// copy the weapon so it doesn't get overwritten by a new entity in Parser.weapons
+		wepCopy := *(getPlayerWeapon(proj.Thrower, wep))
+		proj.WeaponInstance = &wepCopy
+
+		unassert.NotNilf(proj.WeaponInstance, "couldn't find grenade instance for player")
+		if proj.WeaponInstance != nil {
+			unassert.NotNilf(proj.WeaponInstance.Owner, "getPlayerWeapon() returned weapon instance with Owner=nil")
+		}
+
 		p.gameEventHandler.addThrownGrenade(proj.Thrower, proj.WeaponInstance)
 
 		p.eventDispatcher.Dispatch(events.GrenadeProjectileThrow{
@@ -393,9 +404,7 @@ func (p *Parser) bindGrenadeProjectiles(entity *st.Entity) {
 
 	entity.FindPropertyI("m_nModelIndex").OnUpdate(func(val st.PropertyValue) {
 		proj.Weapon = p.grenadeModelIndices[val.IntVal]
-
-		equipment := common.NewEquipment(p.grenadeModelIndices[val.IntVal])
-		proj.WeaponInstance = &equipment
+		wep = p.grenadeModelIndices[val.IntVal]
 	})
 
 	// @micvbang: not quite sure what the difference between Thrower and Owner is.

game_events.go

@@ -117,7 +117,7 @@ func newGameEventHandler(parser *Parser) gameEventHandler {
 		"cs_round_start_beep":             nil,                                  // Round start beeps
 		"cs_win_panel_match":              geh.csWinPanelMatch,                  // Not sure, maybe match end event???
 		"cs_win_panel_round":              nil,                                  // Win panel, (==end of match?)
-		"decoy_detonate":                  geh.decoyDetonate,                    // Decoy exploded/expired
+		"decoy_detonate":                  delay(geh.decoyDetonate),             // Decoy exploded/expired. Delayed so deleteThrownGrenade() isn't called before player_hurt
 		"decoy_started":                   delay(geh.decoyStarted),              // Decoy started. Delayed because projectile entity is not yet created
 		"endmatch_cmm_start_reveal_items": nil,                                  // Drops
 		"entity_visible":                  nil,                                  // Dunno, only in locally recorded demo