PlotSquared icon indicating copy to clipboard operation
PlotSquared copied to clipboard

Vehicle Create Admin Permission

Open MeFisto94 opened this issue 4 years ago • 2 comments

PlotSquared Feature Suggestion: Adding a permission like plots.admin.vehicle.other to allow vehicles to be placed on other plots (and roads)

How should it be implemented? In the VehicleCreateEvent don't remove the vehicle, "if the permission is set". The problem is that the VehicleCreateEvent cannot return the player placing it, specifically because things like Dispensers/Droppers can also place Vehicles. This means we'd need to capture PlayerInteractEvents or PlayerBlockPlace Events and flag vehicles there (e.g. the vehicle has a unique id, this could be added into a hashmap which is then compared against in the VehicleCreateEvent).

Additional context

Checklist:

  • [X] I made sure there are no duplicates of my suggestion (Use search)
  • [X] I made sure my suggestion does not exist yet (Check the wiki)
  • [X] I made sure my suggestion is useful for the majority of PlotSquared users

I'm not 100% sure about the third check, it's mostly for "fun" (e.g. riding a minecart on users roller coasters), but it's not like having a negative impact on anyone etc

Edit: I think my analysis was wrong, shouldn't vehicle creates be covered by interact.other permissions already?

MeFisto94 avatar May 29 '20 20:05 MeFisto94

plots.admin.interact.road plots.admin.interact.unowned

Still not let users placing vehicles at road/unowned lands, so seems an unexisting node and a +1 to the suggestion :)

LordManzana avatar Jun 03 '20 08:06 LordManzana

Still an Issue/Direction of Solution?

PaBox avatar Feb 27 '22 20:02 PaBox