Update extra_logging.lua

* Update clear_items function to accept player name

* Update inventory_clear.lua

* Update exp_scenario/module/control/inventory_clear.lua

Co-authored-by: Cooldude2606 <25043174+Cooldude2606@users.noreply.github.com>

---------

Co-authored-by: Cooldude2606 <25043174+Cooldude2606@users.noreply.github.com>

exp_scenario/module/control/extra_logging.lua

@@ -37,8 +37,11 @@ local function on_pre_player_died(event)
     local player = assert(game.get_player(event.player_index))
     local cause = event.cause
     if cause then
-        local by_player = event.cause.player
-        add_log_line("[DEATH]", player.name, "died because of", by_player and by_player.name or event.cause.name)
+        if cause.type == "character" then
+            add_log_line("[DEATH]", player.name, "died because of", (cause.player and cause.player.name) or cause.name)
+        else
+            add_log_line("[DEATH]", player.name, "died because of", cause.name)
+        end
     else
         add_log_line("[DEATH]", player.name, "died because of unknown reason")
     end
@@ -52,9 +55,9 @@ local function on_research_finished(event)
 
     local inf_research_level = config_res.inf_res[config_res.mod_set][event.research.name]
     if inf_research_level and event.research.level >= inf_research_level then
-        add_log_line_locale{ "", "[RES] ", event.research.prototype.localised_name, " at level ", event.research.level - 1, "has been researched\n" }
+        add_log_line_locale{ "", "[RES]", event.research.prototype.localised_name, " at level ", event.research.level - 1, " has been researched\n" }
     else
-        add_log_line_locale{ "", "[RES] ", event.research.prototype.localised_name, "has been researched\n" }
+        add_log_line_locale{ "", "[RES]", event.research.prototype.localised_name, " has been researched\n" }
     end
 end
 
@@ -67,7 +70,7 @@ end
 --- @param event EventData.on_player_left_game
 local function on_player_left_game(event)
     local player = assert(game.get_player(event.player_index))
-    add_log_line("[LEAVE]", game.players[event.player_index].name, config.disconnect_reason[event.reason])
+    add_log_line("[LEAVE]", player.name, config.disconnect_reason[event.reason])
 end
 
 local e = defines.events

exp_scenario/module/control/inventory_clear.lua

@@ -1,5 +1,8 @@
 --[[-- Control - Inventory Clear
 Will move players items to spawn when they are banned or kicked, option to clear on leave
+
+on_player_banned - player_name
+on_player_kicked - player_index
 ]]
 
 local ExpUtil = require("modules/exp_util")
@@ -7,6 +10,7 @@ local events = require("modules.exp_legacy.config.inventory_clear")
 
 --- @param event { player_index: number }
 local function clear_items(event)
+    if not event.player_index then return end
     local player = assert(game.get_player(event.player_index))
     local inventory = assert(player.get_main_inventory())
     ExpUtil.transfer_inventory_to_surface{