works on 1.20, but every time a stone block or dirt block is mined, it says "object refrence is not set to an instance of an object" in the server console. Doesn't cause any issues other than console bloat

Does anyone know if this works on 1.20?

And where is that csv file located?

28.5.2024 16:29:35 [Error] Object reference not set to an instance of an object.
28.5.2024 16:29:43 [Error] Exception: Index and count must refer to a location within the string. (Parameter 'count')
at System.String.Remove(Int32 startIndex, Int32 count)
at grieflogger.griefloggerModSystem.OnEntityDeath(Entity entity, DamageSource damageSource) in C:\Users\jgord\source\repos\grief_logger\grief_logger\grief_loggerModSystem.cs:line 532
at Vintagestory.API.Common.Entities.Entity.Die(EnumDespawnReason reason, DamageSource damageSourceForDeath) in VintagestoryApi\Common\Entity\Entity.cs:line 1706
at Vintagestory.API.Common.EntityPlayer.Die(EnumDespawnReason reason, DamageSource damageSourceForDeath) in VintagestoryApi\Common\Entity\EntityPlayer.cs:line 1177
at Vintagestory.GameContent.EntityBehaviorHealth.OnEntityReceiveDamage(DamageSource damageSource, Single& damage) in VSEssentials\Entity\Behavior\BehaviorHealth.cs:line 169
at Vintagestory.API.Common.Entities.Entity.ReceiveDamage(DamageSource damageSource, Single damage) in VintagestoryApi\Common\Entity\Entity.cs:line 813
at Vintagestory.GameContent.AiTaskMeleeAttack.attackTarget() in VSEssentials\Entity\AI\Task\TasksImpl\AiTaskMeleeAttack.cs:line 173
at Vintagestory.GameContent.AiTaskMeleeAttack.ContinueExecute(Single dt) in VSEssentials\Entity\AI\Task\TasksImpl\AiTaskMeleeAttack.cs:line 146
at Vintagestory.GameContent.AiTaskManager.OnGameTick(Single dt) in VSEssentials\Entity\AI\Task\AiTaskManager.cs:line 174
at Vintagestory.GameContent.EntityBehaviorTaskAI.OnGameTick(Single deltaTime) in VSEssentials\Entity\AI\Task\BehaviorTaskAI.cs:line 114
at Vintagestory.API.Common.Entities.Entity.OnGameTick(Single dt) in VintagestoryApi\Common\Entity\Entity.cs:line 874
at Vintagestory.API.Common.EntityAgent.OnGameTick(Single dt) in VintagestoryApi\Common\Entity\EntityAgent.cs:line 540
at Vintagestory.Server.ServerSystemEntitySimulation.TickEntities(Single dt) in VintagestoryLib\Server\Systems\World\EntitySimulation.cs:line 330
at Vintagestory.Server.ServerSystemEntitySimulation.OnServerTick(Single dt) in VintagestoryLib\Server\Systems\World\EntitySimulation.cs:line 169
at Vintagestory.Server.ServerMain.Process() in VintagestoryLib\Server\ServerMain.cs:line 886

GRIEFLOG (claim error): Object reference not set to an instance of an object.
shown in console without further infos

Also the grief logger makes players jump to the info chat tab every time someone is griefing, which is kind of annoying.
It should log without forcing people to another chat tab.

@MadGnome

1. I think I fixed a null bug in DidUseBlock, but I wasn't able to reproduce it.  Time will tell.
2. You can now filter messages by role.  This can be set in the config file.  Currently all roles are included, so you can just eliminate the ones you don't want.
3. If the block is in a claimed area, it doesn't log or report the grief.  If there's some special logic that's needed in these cases, fine, but I assume if you're allowed to work in the claimed area, you're not griefing.
4. You can now set the chat channel in the config.  Acceptable values are: AllChatGroups, ConsoleGroup, DamageLogChatGroup, GeneralChatGroup, InfoLogChatGroup, ServerInfoChatGroup

You might need to delete your old modconfig file to get this new version working.

Getting quite a few error messages in my logs:
server-main, server-event
9.4.2024 07:59:26 [Error] Mod exception during DidUseBlock
9.4.2024 07:59:26 [Error] Exception: Object reference not set to an instance of an object.
at grieflogger.griefloggerModSystem.getLogString(String locString, IServerPlayer byPlayer, BlockSelection blockSel, List`1 oldData) in C:\Users\jgord\source\repos\grief_logger\grief_logger\grief_loggerModSystem.cs:line 172
at grieflogger.griefloggerModSystem.possibleGriefEvent(IServerPlayer byPlayer, BlockSelection blockSel) in C:\Users\jgord\source\repos\grief_logger\grief_logger\grief_loggerModSystem.cs:line 214
at Vintagestory.Server.ServerEventManager.TriggerDidUseBlock(IServerPlayer player, BlockSelection blockSel) in VintagestoryLib\Server\ServerEventManager.cs:line 553

Some feedback from testing this out on a server currently, we seem to avg around 6-15 players online.  We are currently running it on VS 1.19.7, server has 100+ mods on it.

Haven't noticed any performance issues yet, it's handy getting the grief notifications, our players have been poking and prodding it to see how it works.

Couple of things.

Is there anyway to filter the messages to just the admins, or by role?

It logs griefs when a small team is sharing the same claimed space, even though they have the permissions from claims to mine those blocks.  Could a feature be added to not report griefing, or disable that chat message, when players have permissions to mine blocks in a claim?

Would there be a way to get the messages in the main chat, since discord bots don't recognize other chat channels.  If the server does a reset the logs are lost, but if it's in the main chat to discord it's recorded there forever for an Admin to find it.

I appreciate the work you've put into this, so far it's working very well and we haven't encountered any errors while testing.  (I'm in the main VS Discord under the same name)

This is great! Let's colaborate if you'd like to. I'd like to help get animal tracking into this. I sent you a discord request.

Currently this is only tested on 1.19.5, but I expect that it proably works for 1.19.* and maybe 1.18.*.  If you test it and find it does or doesn't work, please let me know.

Has this been tested for 1.19.3?

I think it will work in 1.19.4, but I haven't tested it.  I marked it as compatible, but let me know if you have issues.

work in 1.19.4 ?

Now it requires installation on the server only.  The client doesn't need it.

I think I have a solution to the lag issues that other mod ran into.  I'm storing the placement data on a per chunk basis, which makes it much quicker to query and save.  That new version is live.  I've included the source in it, if anyone wants to take a look.

i guess ill wait. 
check out https://mods.vintagestory.at/blocklog i think all its code wasnt turned into a dll file. it might help, with progression. RogueRaiden, yah the one i posted. we stopped using it, cause of the lag it would cause xD having something like https://www.spigotmc.org/resources/logblock.67333/ for VS would be amazing xD

This could be an absolutely amazing addition to servers!
The last mod that was doing this was fantastic for a while but it ended up causing world saves to take several seconds, 5-10+ seconds at times with lots of activity (20+ player activity after several hours, problem would compound). Made it too much of a issue to use.
Looking forward to more updates!