Cazsius / Spice-of-Life-Carrot-Edition

Motivating people to explore new foods with the carrot, not the stick.
GNU Lesser General Public License v2.1
17 stars 43 forks source link

Can't use /foodlist clear on yourself if not opped #117

Closed jinkhya closed 3 years ago

jinkhya commented 3 years ago

Hello, I dunno if that's intended but you can't clear your foodlist if not opped. The message tells you you can't apply the command to other players even though I specified my own name. It'd be cool to be able to do it yourself without having to ask admins on a server when you've played a pack and want to restart. Thanks in advance. Jin

juliand665 commented 3 years ago

Hmm, does it work if you just don't specify a player at all? Just running /foodlist clear with no arguments.

jinkhya commented 3 years ago

It doesn't either, /foodlist clear gives the same error message "you do not have the permissions necessary to apply this command to other players"

juliand665 commented 3 years ago

Wait, I think I know what's going wrong… Is it possible you can clear other players' lists but not your own? I think I inverted a check lmao

https://github.com/Cazsius/Spice-of-Life-Carrot-Edition/blob/f4af6e9abc81882c2ca65fc77b24c040c96255ed/src/main/java/com/cazsius/solcarrot/command/FoodListCommand.java#L75-L76

juliand665 commented 3 years ago

Can you try if it's still wrong with this version? You'll have to put it on the server (the client shouldn't matter).

SoL-Carrot-1.16.5-1.10.1.jar.zip

jinkhya commented 3 years ago

That fixed it thank you ! :D

juliand665 commented 3 years ago

Glad to hear it! Will upload the update to CurseForge now.