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

reload twice create error after config-change of TimeSpeed setting #8641

Closed
3 tasks done
CloudeLecaw opened this issue Apr 15, 2024 · 1 comment
Closed
3 tasks done
Labels
bug report Reports of bugs/inconsistencies

Comments

@CloudeLecaw
Copy link

CloudeLecaw commented Apr 15, 2024

Description of Issue

Well i didn't really made much changes in config. just tried to enable a world in the TimeSpeed settings.

  • i set one value in TimeSpeed Settings (a world) from false to true and did / cmi reload -> an error appeared in console.

  • i did agggain /cmi reload -> seems to be finde

  • a thrid time /cmi reload -> the error appeared aggain.

  • Now i set this setting back from false to true and did /cmi reload -> all fine

  • /cmi reload aggain -> all fine

  • /cmi reload aggain -> all fine

i redid the whole thing twice with same result. So it looks like there is an error if you wanne enable varrious timespeed for a world if it was disabled before and try to reload cmi.

Version Information

image

Errors

>....
[15:35:38 WARN]: Unexpected exception while parsing console command "cmi reload"
org.bukkit.command.CommandException: Unhandled exception executing command 'cmi' in plugin CMI v9.7.0.6
	at org.bukkit.command.PluginCommand.execute(PluginCommand.java:47) ~[paper-api-1.20.4-R0.1-SNAPSHOT.jar:?]
	at org.bukkit.command.SimpleCommandMap.dispatch(SimpleCommandMap.java:155) ~[paper-api-1.20.4-R0.1-SNAPSHOT.jar:?]
	at org.bukkit.craftbukkit.v1_20_R3.CraftServer.dispatchCommand(CraftServer.java:999) ~[paper-1.20.4.jar:git-Paper-484]
	at org.bukkit.craftbukkit.v1_20_R3.CraftServer.dispatchServerCommand(CraftServer.java:984) ~[paper-1.20.4.jar:git-Paper-484]
	at net.minecraft.server.dedicated.DedicatedServer.handleConsoleInputs(DedicatedServer.java:501) ~[paper-1.20.4.jar:git-Paper-484]
	at net.minecraft.server.dedicated.DedicatedServer.tickChildren(DedicatedServer.java:448) ~[paper-1.20.4.jar:git-Paper-484]
	at net.minecraft.server.MinecraftServer.tickServer(MinecraftServer.java:1525) ~[paper-1.20.4.jar:git-Paper-484]
	at net.minecraft.server.MinecraftServer.runServer(MinecraftServer.java:1226) ~[paper-1.20.4.jar:git-Paper-484]
	at net.minecraft.server.MinecraftServer.lambda$spin$0(MinecraftServer.java:319) ~[paper-1.20.4.jar:git-Paper-484]
	at java.lang.Thread.run(Thread.java:833) ~[?:?]
Caused by: java.util.ConcurrentModificationException
	at java.util.HashMap$HashIterator.nextNode(HashMap.java:1597) ~[?:?]
	at java.util.HashMap$EntryIterator.next(HashMap.java:1630) ~[?:?]
	at java.util.HashMap$EntryIterator.next(HashMap.java:1628) ~[?:?]
	at com.Zrips.CMI.utils.TimeManager.loadConfig(TimeManager.java:90) ~[CMI_9.7.0.6.jar:?]
	at com.Zrips.CMI.Config.load(Config.java:1682) ~[CMI_9.7.0.6.jar:?]
	at com.Zrips.CMI.Config.reload(Config.java:1734) ~[CMI_9.7.0.6.jar:?]
	at com.Zrips.CMI.commands.list.reload.perform(reload.java:21) ~[CMI_9.7.0.6.jar:?]
	at com.Zrips.CMI.commands.CommandsHandler.onCommand(CommandsHandler.java:401) ~[CMI_9.7.0.6.jar:?]
	at org.bukkit.command.PluginCommand.execute(PluginCommand.java:45) ~[paper-api-1.20.4-R0.1-SNAPSHOT.jar:?]
	... 9 more
>....

Relevant Config Sections

Before: - value of Farmwelt is false

  TimeSpeed:
    # Time is defined in seconds. Vanilla 24 hour ingame duration is 1200 seconds of real time
    world:
      Enabled: true
      # Default value: 600 Starts from tick: 0 Ends at tick: 12000
      day: 600
      # Default value: 90 Starts from tick: 12000 Ends at tick: 13800
      sunset: 90
      # Default value: 420 Starts from tick: 13800 Ends at tick: 22200
      night: 420
      # Default value: 90 Starts from tick: 22200 Ends at tick: 24000
      sunrise: 90
    # Time is defined in seconds. Vanilla 24 hour ingame duration is 1200 seconds of real time
    Farmwelt:
      Enabled: false
      # Default value: 600 Starts from tick: 0 Ends at tick: 12000
      day: 1200
      # Default value: 90 Starts from tick: 12000 Ends at tick: 13800
      sunset: 60
      # Default value: 420 Starts from tick: 13800 Ends at tick: 22200
      night: 480
      # Default value: 90 Starts from tick: 22200 Ends at tick: 24000
      sunrise: 60
    # Time is defined in seconds. Vanilla 24 hour ingame duration is 1200 seconds of real time

after: - value of Farmwelt is set true

  TimeSpeed:
    # Time is defined in seconds. Vanilla 24 hour ingame duration is 1200 seconds of real time
    world:
      Enabled: true
      # Default value: 600 Starts from tick: 0 Ends at tick: 12000
      day: 600
      # Default value: 90 Starts from tick: 12000 Ends at tick: 13800
      sunset: 90
      # Default value: 420 Starts from tick: 13800 Ends at tick: 22200
      night: 420
      # Default value: 90 Starts from tick: 22200 Ends at tick: 24000
      sunrise: 90
    # Time is defined in seconds. Vanilla 24 hour ingame duration is 1200 seconds of real time
    Farmwelt:
      Enabled: true
      # Default value: 600 Starts from tick: 0 Ends at tick: 12000
      day: 1200
      # Default value: 90 Starts from tick: 12000 Ends at tick: 13800
      sunset: 60
      # Default value: 420 Starts from tick: 13800 Ends at tick: 22200
      night: 480
      # Default value: 90 Starts from tick: 22200 Ends at tick: 24000
      sunrise: 60
    # Time is defined in seconds. Vanilla 24 hour ingame duration is 1200 seconds of real time

Agreements

  • My server is supported by CMI.
  • My version of CMI at the time of this report is up to date.
  • I have searched the github and asked around before making this report.
@CloudeLecaw CloudeLecaw added the bug report Reports of bugs/inconsistencies label Apr 15, 2024
@Zrips
Copy link
Owner

Zrips commented Apr 30, 2024

Should be fixed with next update

@Zrips Zrips closed this as completed Apr 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug report Reports of bugs/inconsistencies
Projects
None yet
Development

No branches or pull requests

2 participants