HUGE ISSUE!

PVP Players can use the siege weapons to raid PVE players. just had this issue on the server!

I will certainly take a look into this

thank you i'll be awaiting update. also noticing this in the log sometimes

[WarMode] HERE
[02.16 01:03:43] Failed to call internal hook 'CanLootEntity' on plugin 'WarMode v1.3.5' [1627232611] (Object reference not set to an instance of an object)
[02.16 01:03:43] at ModeConfig Oxide.Plugins.WarModeMethods.ExtensionMethods.GetMode(BaseEntity entity) in /home/container/carbon/plugins/WarMode.cs:line 1463
[02.16 01:03:43] at object Oxide.Plugins.WarMode.CanLootHelper(BasePlayer looter, BaseEntity target, bool skipVendingCheck) in /home/container/carbon/plugins/WarMode.cs:line 2646
[02.16 01:03:43] at object Oxide.Plugins.WarMode.CanLootEntity(BasePlayer looter, StorageContainer container) in /home/container/carbon/plugins/WarMode.cs:line 2562
[02.16 01:03:43] at object Oxide.Plugins.WarMode.InternalCallHook(uint hook, object[] args) in WarMode.cs/Internal:line 65

@NobleServers which siege weapons are you seeing that can raid pve bases? I just tried ballistas, catapults and battering rams and none of those were able to damage a pve player's base (as a pvp player). At this point, I'm thinking maybe there is something setup wrong in your config?

hmmmm, do you have discord it would be easier to discuss there.

9 hours ago, Mr01sam said:

@NobleServers which siege weapons are you seeing that can raid pve bases? I just tried ballistas, catapults and battering rams and none of those were able to damage a pve player's base (as a pvp player). At this point, I'm thinking maybe there is something setup wrong in your config?

Okay I can confirm I found the issue, after some extensive testing. we found that if someone starts as PVP builds a base and ends up switching to PVE there base is  considered PVP raidable and vice versa. 

We need this plugin to update the owners building to whatever they switch too.

1 minute ago, NobleServers said:

Okay I am pretty sure I found the issue, after some extensive testing. we found that if someone starts as PVP builds a base and ends up switching to PVE there base is  considered PVP raidable and vice versa. 

We need this plugin to update the owners building to whatever they switch too.

It should already do that - however ALL the players that are authed on the TC need to be PVE in order for it to count as PVE. If there is even just one player authed on it that is PVP, then the whole base is considered PVP. This is because the PVP mode has first priority over the PVE mode, so in any situation where multiple players own something (a base) and it needs to decide a mode, it will pick PVP over PVE.

seems like it doesn't I tried with only 2 players both ways and both bases did not update to what they were.

You might have to show me a video or something - I tested the scenario this morning and it should be working.

I'm going to close this ticket for now because it doesnt seem to be the original issue with the siege weapons - but you can still respond here and I'll assist.

Changed Status from Pending to Not a Bug