Players can still access the quarries

I just installed this on my server and I had someone test it out and they can access two quarries that I placed even though they are not shared.. I don't know if it matters, but I use the Prevent Looting plugin along with TruePVE on my server.

This is my config:
 

{
  "General Settings": {
    "Wipe Data on new save": true
  },
  "Commands": {
    "Quarry Manager Command": "qm"
  },
  "Chat Settings": {
    "Chat prefix": "QuarryLocker >>",
    "Chat prefix colour": "#FF6A13",
    "Chat icon": 0,
    "Quarry Share Messages": {
      "Print a message if team share is enabled": true,
      "Print a message if team share is disabled": true,
      "Print a message if a player is added to the share list": true,
      "Print a message if a player is removed to the share list": true,
      "Print a message if a player has activated a static quarry": true
    },
    "RCON/Server Console Messages": {
      "Message the console if someone enables team share": false,
      "Message the console if someone disables team share": false,
      "Message the console if someone is added to the share list": false,
      "Message the console if a player is removed to the share list": false,
      "Message the console on static quarry lock activated": false
    }
  },
  "Share Settings": {
    "Players can toggle other players quarries": false,
    "Lock quarry containers": false,
    "Team Sharing": {
      "Allow team to toggle the quarry": true,
      "Allow team to open the quarries containers": true
    },
    "Individual Player Sharing": {
      "Allow added players to toggle the quarry": true,
      "Allow added players to open the quarries containers": true
    },
    "Static Quarry Sharing": {
      "Enable static quarry lock": false,
      "Static Quarry Sharing": {
        "If the player leaves the dome unlock the static quarry": false
      }
    }
  }
}

I will look into it soon

Changed Status from Pending to Closed

Changed Fixed In to 2.0.2

I just tested this update and the player was able to access the fuel. Now that I type this, I realize should have tested the part where the resources are deposited, but I guess I figured if they can access the fuel, then it's not working as intended anyway.

You have "Lock quarry containers" set to false it should be true have you tested it out with it being true?

7 hours ago, Enforcer said:

You have "Lock quarry containers" set to false it should be true have you tested it out with it being true?

I will test it again. I didn't think that the plugin would be turned off by default. I wouldn't even think there was a setting to do that.

I did another test this morning. I can't get the /qm a command to work. When I type /qm a playername it has no reaction at all. If I type /qm a and nothing, it tells me I have to specify the player name.

Also, this is in my console. I think it appeared when the other player accessed (successfully) the quarry:
 

Failed to call hook 'QuarryShareCommand' on plugin 'QuarryManager v2.0.2' (NullReferenceException: Object reference not set to an instance of an object)
at Oxide.Plugins.QuarryManager.AddOrRemoveUser (BasePlayer player, System.Boolean isAdding, System.Boolean isRemoving, System.String[] args) [0x00011] in <1649046b9e3b4cb6a44d0fd993e3c5f4>:0
at Oxide.Plugins.QuarryManager.QuarryShareCommand (BasePlayer bPlayer, System.String command, System.String[] args) [0x004da] in <1649046b9e3b4cb6a44d0fd993e3c5f4>:0
at Oxide.Plugins.QuarryManager.DirectCallHook (System.String name, System.Object& ret, System.Object[] args) [0x0034e] in <1649046b9e3b4cb6a44d0fd993e3c5f4>:0
at Oxide.Plugins.CSharpPlugin.InvokeMethod (Oxide.Core.Plugins.HookMethod method, System.Object[] args) [0x00079] in <60c318df79ed41688ea59335e48d61ad>:0
at Oxide.Core.Plugins.CSPlugin.OnCallHook (System.String name, System.Object[] args) [0x000d8] in <50629aa0e75d4126b345d8d9d64da28d>:0
at Oxide.Core.Plugins.Plugin.CallHook (System.String hook, System.Object[] args) [0x00060] in <50629aa0e75d4126b345d8d9d64da28d>:0

On 4/2/2022 at 11:03 PM, BetterDeadThanZed said:

I did another test this morning. I can't get the /qm a command to work. When I type /qm a playername it has no reaction at all. If I type /qm a and nothing, it tells me I have to specify the player name.

 

Also, this is in my console. I think it appeared when the other player accessed (successfully) the quarry:
 

Failed to call hook 'QuarryShareCommand' on plugin 'QuarryManager v2.0.2' (NullReferenceException: Object reference not set to an instance of an object)
at Oxide.Plugins.QuarryManager.AddOrRemoveUser (BasePlayer player, System.Boolean isAdding, System.Boolean isRemoving, System.String[] args) [0x00011] in <1649046b9e3b4cb6a44d0fd993e3c5f4>:0
at Oxide.Plugins.QuarryManager.QuarryShareCommand (BasePlayer bPlayer, System.String command, System.String[] args) [0x004da] in <1649046b9e3b4cb6a44d0fd993e3c5f4>:0
at Oxide.Plugins.QuarryManager.DirectCallHook (System.String name, System.Object& ret, System.Object[] args) [0x0034e] in <1649046b9e3b4cb6a44d0fd993e3c5f4>:0
at Oxide.Plugins.CSharpPlugin.InvokeMethod (Oxide.Core.Plugins.HookMethod method, System.Object[] args) [0x00079] in <60c318df79ed41688ea59335e48d61ad>:0
at Oxide.Core.Plugins.CSPlugin.OnCallHook (System.String name, System.Object[] args) [0x000d8] in <50629aa0e75d4126b345d8d9d64da28d>:0
at Oxide.Core.Plugins.Plugin.CallHook (System.String hook, System.Object[] args) [0x00060] in <50629aa0e75d4126b345d8d9d64da28d>:0

 

It is case sensitive currently I will fix that soon. If a player has spaces in their name you would have to add " at the start and end of their name.

On 4/2/2022 at 10:15 PM, BetterDeadThanZed said:

I will test it again. I didn't think that the plugin would be turned off by default. I wouldn't even think there was a setting to do that.

There will be more options added to the plugin in later updates.

- Add clan support <- 2.0.3 (NEXT)
- Add friend support <- 2.0.3 (NEXT)
- Lock fuel tank, hopper/dispenser and quarry toggeling seperatly <- 2.0.4
- Quarry HP, repairs etc... <- 2.0.5
- Economics/RP support <- 2.0.5
- Discord support <- 2.0.6
- Proper help text if a player has a certain permission <- 2.0.7
- Quarry fuel consumption <- 2.0.7
- Static quarry team share support <- 2.0.9
- Static quarry friend share support <- 2.0.9
- Static quarry clan share support <- 2.0.9
- Static quarry added player support  <- 2.0.9
- More static quarry configuration options (like time until unlock after x amount of time, dome size etc.) <- 2.0.10
- Add chat icon <- 2.0.11
- Add UI <- 2.0.11

21 hours ago, Enforcer said:

It is case sensitive currently I will fix that soon. If a player has spaces in their name you would have to add " at the start and end of their name.

I don't think you understand. It still doesn't work. I had a quarry that I gave no permissions to and the player was able to access the fuel and the output and that error that I posted came up when he accessed them.

It should be fixed now. Was busy with school and some other stuff.

Changed Fixed In from 2.0.2 to 2.0.3

I tested this again tonight with the update and it works, mostly. I tried to add a player to the authorized list and I received no response from the command in-game. Then I looked in the console and found that this plugin is conflicting with PreventLooting. Giving my team access worked fine, though. 

This is the error:

 

04/06 19:03:31 | Failed to call hook 'QuarryShareCommand' on plugin 'QuarryManager v2.0.3' (NullReferenceException: Object reference not set to an instance of an object)
at Oxide.Plugins.QuarryManager.AddOrRemoveUser (BasePlayer player, System.Boolean isAdding, System.Boolean isRemoving, System.String[] args) [0x00011] in <313ac249abe14a0bad85c62023d5416b>:0
at Oxide.Plugins.QuarryManager.QuarryShareCommand (BasePlayer bPlayer, System.String command, System.String[] args) [0x004da] in <313ac249abe14a0bad85c62023d5416b>:0
at Oxide.Plugins.QuarryManager.DirectCallHook (System.String name, System.Object& ret, System.Object[] args) [0x0034e] in <313ac249abe14a0bad85c62023d5416b>:0
at Oxide.Plugins.CSharpPlugin.InvokeMethod (Oxide.Core.Plugins.HookMethod method, System.Object[] args) [0x00079] in <60c318df79ed41688ea59335e48d61ad>:0
at Oxide.Core.Plugins.CSPlugin.OnCallHook (System.String name, System.Object[] args) [0x000d8] in <50629aa0e75d4126b345d8d9d64da28d>:0
at Oxide.Core.Plugins.Plugin.CallHook (System.String hook, System.Object[] args) [0x00060] in <50629aa0e75d4126b345d8d9d64da28d>:0
04/06 19:03:44 | Calling hook CanLootEntity resulted in a conflict between the following plugins: PreventLooting - False (Boolean), QuarryManager (True (Boolean))