EssentialsX

EssentialsX

Bukkit Plugins
2M Downloads

Can't set multiple homes in 1.17

akdukaan opened this issue ยท 1 comments

commented

Type of bug

Other unexpected behaviour

/ess version output

[00:54:10] [Server thread/INFO]: Server version: 1.17-R0.1-SNAPSHOT 3147-Spigot-9472b09-d7ef1e9 (MC: 1.17)
[00:54:10] [Server thread/INFO]: Brand version: CraftBukkit
[00:54:10] [Server thread/INFO]: EssentialsX version: 2.19.0-dev+158-56a1cd8
[00:54:10] [Server thread/INFO]: LuckPerms version: 5.3.47
[00:54:10] [Server thread/INFO]: PlaceholderAPI version: 2.10.9
[00:54:10] [Server thread/INFO]: Vault version: 1.7.3-b131
[00:54:10] [Server thread/INFO]: EssentialsXChat version: 2.19.0-dev+158-56a1cd8
[00:54:10] [Server thread/INFO]: EssentialsXSpawn version: 2.19.0-dev+158-56a1cd8
[00:54:10] [Server thread/INFO]: ChestShop version: 3.12 (build 307)
[00:54:10] [Server thread/INFO]: You are running a server with limited API functionality. EssentialsX will still work, but certain features may be disabled.
[00:54:10] [Server thread/INFO]: Fetching version information...
[00:54:10] [Craft Scheduler Thread - 938/INFO]: You're running an experimental build of EssentialsX that is 5 builds behind the latest dev build!
[00:54:10] [Craft Scheduler Thread - 938/INFO]: Feature Branch: mc/1.17.

Server startup log

https://gist.github.com/drbot7/dc6d8957827c9243ec98363eda85e274

EssentialsX config files

https://gist.github.com/drbot7/ee4c889fd47794e8efa60e5a41a88959

Error log (if applicable)

https://gist.github.com/drbot7/961ddde4217ac1e048d7586af492d41c

Bug description

Players are restricted to only one home.

Steps to reproduce

  1. Use the config file attached above (or at least the section about homes). Give yourself permissions for essentials.sethome, essentials.sethome.multiple, and essentilas.sethome.2, and make sure you don't have essentials.sethome.multiple.unlimited.
  2. Set your first home with /sethome.
  3. Try to set your second home with /sethome x.

Expected behaviour

The player should be told "Home set to current location" and their second home should be set to that location.

Actual behaviour

The player is told "Error: You cannot set more than 1 homes."

commented

Cannot fix this, bukkit was doing some weird trickery with yml keys in the past but our new config-parser is not as lax. You'll need to put quotes around the numbers in your keys to look something like this;

sethome-multiple:
  default: 1
  '1': 1
  '2': 2
  '3': 3
  '4': 4
  '5': 5
  '6': 6
  '7': 7
  '8': 8
  '9': 9
  '10': 10