Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

1.16.5 issue with bal after the bal top fix #3

Open
genfar1980 opened this issue Jul 22, 2021 · 7 comments
Open

1.16.5 issue with bal after the bal top fix #3

genfar1980 opened this issue Jul 22, 2021 · 7 comments

Comments

@genfar1980
Copy link

The latest version for 1.16.5 has a issue with using /bal command and other things like signs.

Here is a crashlog that was generated.
crash-2021-07-22_15.20.36-server.txt

@Martmists-GH
Copy link
Member

Does the issue persist in the 1.17 build?

@BushTrocity
Copy link

Yes and in single player. Balance works fine for me. If i try to buy, sale or do anything that interacts with the database will crash.

@genfar1980
Copy link
Author

For me going to 1.17 is not a option, most of the mods I use have not updated to that version, also then you got 1.18 comming which will again mess up exsisting generation of worlds as depth and height of the worlds will get changed. Also most of the mods in my server I would want them. I personaly am not in any way at the moment good enough to just take the source code and make fixxes I personalnly been trying to figure if there is a json way to just disable to use of balance top, as I went back to that version because at least outside of that one crash the rest of the mod is usable.

@Gunpowder-MC Gunpowder-MC deleted a comment from dandin87 Aug 2, 2021
@Martmists-GH
Copy link
Member

Removed the above comment because it doesn't actually solve the issue, it's just a temporary workaround.

@genfar1980
Copy link
Author

Anyhow while this is being investigated or whatnot I need to really know what the permission line is for balance top putting currency.balance.top false into luckperms does not prevent players from using balance top. Really need to disable the top command. Also luckperms does not seem to even notice commands ran so couldnt use verbose to get the permission listing.

@Martmists-GH
Copy link
Member

Martmists-GH commented Aug 5, 2021

Don't use false, just don't add currency.balance.top at all. Permissions with patbox' API don't use key-value pairs, only permission nodes being present or not.

However, if that IS correct, open an issue on the luckperms repository.

@genfar1980
Copy link
Author

its not really a luckperms issue its more a fabric issue at least thats what I read. More so also player roles accualy worked where luckperms did not because player roles does its stuff differnt. But yea outside the point I will just use the player roles as a temp fix as now no one on my server can use the top command for balance bal or money veriations of the command.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants