search

EssentialsX / Essentials

The modern Essentials suite for Spigot and Paper.
https://essentialsx.net
GNU General Public License v3.0
1.96k stars 980 forks source link

Randomly enter words behind the "/weather" command #3572

Closed flandretw closed 4 years ago

flandretw commented 4 years ago

Information

Full output of /ess version:

<!-- [19:15:16 INFO]: Server Version: 1.15.2-R0.1-SNAPSHOT git-Paper-379 (MC: 1.15.2)
[19:15:16 INFO]: EssentialsX Version: 2.17.2.127
[19:15:16 INFO]: GroupManager Version: 0.3.0
[19:15:16 INFO]: PlaceholderAPI Version: 2.10.9
[19:15:16 INFO]: Vault Version: 1.7.2-b107
[19:15:16 INFO]: Citizens Version: 2.0.26-SNAPSHOT (build 1861)
[19:15:16 INFO]: EssentialsXChat Version: 2.17.2.127
[19:15:16 INFO]: EssentialsXProtect Version: 2.17.2.127
[19:15:16 INFO]: EssentialsXGeoIP Version: 2.17.2.127
[19:15:16 INFO]: EssentialsXAntiBuild Version: 2.17.2.127
[19:15:16 INFO]: EssentialsXSpawn Version: 2.17.2.127 -->

Server log: https://gist.github.com/gregman98/c9568109e81b33610b544e613996e589

EssentialsX config: https://gist.github.com/gregman98/d12f146768fe231aca09be8a20cb0020

Details

Description
This is just a small problem and does not constitute a serious impact. About "/weather" command type.

Steps to reproduce Enter a random word behind the "/weather" command and send the command. It will always output "set to sunny".

Expected behavior I think it should output the wrong usage.

Screenshots
https://i.imgur.com/ijODHJj.png https://i.imgur.com/VkeRvjH.png "晴天" means sunny in English.

(Sorry for my bad English.)

pop4959 commented 4 years ago

Thank you for your concern, but we will not be changing this right now since it has worked like this for ages, and there isn't really any particular reason to break it for all of the people who expect the current behavior (which is that the command always changes the weather to clear/sunny unless very deliberately set to "storm").