OreCruncher / DynamicSurroundingsFabric

Dynamic Surroundings mod for Minecraft
MIT License
63 stars 25 forks source link

/execute run [...] does not work with any of the ds commands #18

Closed y2k04 closed 1 year ago

y2k04 commented 2 years ago

Attempting to run /execute run [ dsbiome | dsdump | dsreload | dsscript | dstod ] results in a Incorrect argument for command error at the end of the ds command.

Screenshot 2022-01-27 083321

y2k04 commented 2 years ago

This also means that the output of the command cannot be stored using the /execute store command.

HarvelsX commented 2 years ago

These are client commands, they cannot be called from the server side, which makes /execute

y2k04 commented 2 years ago

These are client commands, they cannot be called from the server side, which makes /execute

But surely it could be adapted to be allowed on server side and require the client to have the mod as well? It would be useful for making a clock above the hotbar. I did make a data pack that used math to convert ticks to hours and minutes, but the AM/PM gets stuffed because of a programming error where it wouldn't fix itself until a certain tick in the day cycle. The problem was caused by sleeping. I found that the less than and greater than operators were a bit wack when it came to calculating those numbers. Easy fix, but annoying when you spend 8 hours writing, debugging, and improving the script.