search

ArVdC / TimeManager

Bukkit plugin for time management and display
https://dev.bukkit.org/projects/mc-timemanager/files
Eclipse Public License 1.0
7 stars 3 forks source link

cmds.yml quotes bug #28

Closed sorandno closed 2 months ago

sorandno commented 3 months ago

cmdsList: '1': cmds: '1': /shigenreset cmdsRefTime: UTC+9 plholderRefWorld: world time: '11:59' date: '2024-06-01' repeatFreq: month

Command is not executed.

time: It seems that it does not work if the time specification is set to 10:00 or more.

repeatFreq: May only be month.

ArVdC commented 3 months ago

I tried to reproduce the error, without success. Perhaps there is another element that could be the cause of the malfunction ?

version: 1.9.0 useCmds: 'true' cmdsList: '1': cmds: '1': /say Hello world ! cmdsRefTime: UTC+9 plholderRefWorld: World01 time: '22:40' date: '2024-06-05' repeatFreq: month

-> [22:40:00 INFO]: [Server] Hello world !

sorandno commented 3 months ago

time: 22:40 In my environment, if it is 10:00 or more, it will be converted between single quotes. And if it's after 10:00, it may change to a different time. time: '10:00'

Isn't it because it's sandwiched between single quotes?

sorandno commented 3 months ago

It may not work if it is enclosed in single quotes.

time: '20:00'
repeatFreq: day

This is working fine.

sorandno commented 3 months ago 
time: '11:59'
repeatFreq: month

This is not working.
ArVdC commented 3 months ago

You're right, something is wrong with the time values ​​but I haven't found what yet. Single quotes allow values ​​containing : to be read correctly. I therefore think it is important to use them.

sorandno commented 3 months ago

There are times when using single quotes doesn't work properly in my case. I believe there is a bug because there are times when it works fine.

sorandno commented 3 months ago

I don't know if it's a bug that only exists in the month setting, but I hope they find the cause.

sorandno commented 3 months ago

If it doesn't work, it may have been converted to a different time.

ArVdC commented 2 months ago

Please try this patch, it will probably fix this issue : v1.9.1-b

sorandno commented 2 months ago

thank you i will try!

sorandno commented 2 months ago

It worked normally when double quotes were added. '09:38'→'9:38'

If you don't put double quotes, it was converted to another time. 09:39→04:00

Use with double quotes. Thank you for your response.

sorandno commented 2 months ago

Ah... it's a single quote.

ArVdC commented 2 months ago

Does the 'month' frequency still pose any problem?

sorandno commented 2 months ago

The one at '9:38' above worked with the month setting, so I don't think there is a problem.