CommandHelper

CommandHelper

46.5k Downloads

An error found whilst attempting to queue a script and a command

LadyCailinBot opened this issue ยท 5 comments

commented

CMDHELPER-2741 - Reported by dddeeefff

Someone suggested to me a while ago to create an alias for /reloadaliases so that I could save all my import/export data to disk before it's destroyed, then reload it during main.ms. I did this, and for ages it's worked fin until I recently updated to 1.6.1. I'm not sure what CH version I was on before but now I'm on 1565-b841bda

I'm not sure yet what triggers the error exactly, as I think I've been able to run my command without issue since it began. I first noticed that it failed after I omitted an opening ' in a function, so it may be tied to the script

Here is the error:
http://paste.thezomg.com/10223/15968513/

Here is the config.txt command:
http://paste.thezomg.com/10224/59908137/

Here is the pre-reloadaliases script:
http://paste.thezomg.com/10225/73161158/

This issue is stopping me implementing a script because I don't want to have to do a restart every time I want to load a script - I've also found that after this bug occured, a /restart command that kicks everyone from the server with a custom message and then does /stop is also failing. There's no error message in console. Presumably other things are broken that I haven't figured out yet

Thanks very much

commented

Comment by kookster

  1. Install the following main.ms script:
    http://paste.thezomg.com/10608/76087385/
  2. Start the server.
  3. run "/reloadaliases"
  4. You will see the stacktrace that happens in the server log, here is the complete server log:
    http://paste.thezomg.com/10605/60866371/

Commandhelper Version: 1608
Spigot Version: 1031

This was also tested on Bukkit 2850, you can see its log here:
http://paste.thezomg.com/10607/08697613/

commented

Comment by LadyCailin

Fixed in latest build.

commented

Comment by dddeeefff

Looks like the issue only occurs after the first time a script does not compile correctly (due to my errors). Thereafter the script will not work, even after reloading the scripts with my /forcereloadaliases alias

commented

Comment by Hekta

I get the same error.

I think this comes from /reloadaliases, queued operations work perfectly after restarting a server, but if I do a /reloadaliases, after this, all queued operations will fail, and it becomes impossible to use queued operations.

I get this errors since the 1564 version.

commented

Comment by PseudoKnight

Having queues silently fail after this error is terrible. :(

It might be an extension of issue 2730, or rather the result of the attempted fix for that.
http://youtrack.sk89q.com/issue/CMDHELPER-2730
http://build.sk89q.com/job/CommandHelper/1564/changes

Before build 1564 it'd just freeze up the server for a long while on recompile. (example: 30 seconds) I'd actually prefer a temporary freeze over this. I could even work around that issue. Now it breaks queues altogether and you just have to avoid recompiling ever.