Erroneous Time Recalculation #4


  • New
  • Defect
Open
Assigned to briman0094
  • _ForgeUser3962498 created this issue Dec 6, 2011

    What steps will reproduce the problem?
    1. Set my config (attached for reference).
    2. Started server with only this plugin enabled.
    3. Plugin sets the time (nextrun) to incorrect datetime.

    What is the expected output? What do you see instead?
    • Expected: not have my configuration edited OR for the plugin to correctly calculate the nextrun based off of the configuration
    • Instead: Configuration was edited by the plugin; incorrect values were written.

    What version of the product are you using?
    • Version 1.5 (29 Oct 2011)

    Do you have an error log of what happened?
    http://pastebin.com/M2VTHLYX

    Please provide any additional information below.
    • Appears to be an issue where the plugin calculates the next run time based on the current time instead of the time in the configuration.

  • _ForgeUser3962498 added the tags New Defect Dec 6, 2011
  • _ForgeUser3962498 added an attachment config.yml Dec 6, 2011

    config.yml

  • briman0094 posted a comment Dec 7, 2011

    Your nextrun was set to before the server started, so it tried to recalculate it to the next time it should run. This actually pointed out a bug to me, because it should increase the hour to 0 and the day to 7 instead of the hour to 24 (which creates a glitch in Java) and leaving the day at 6. Set the nextrun to a time after the server starts and I'll fix this bug when I get a chance.

  • _ForgeUser3962498 posted a comment Dec 8, 2011

    @ArcanoxDragon: Go

    Wouldn't, for example, setting the nextrun to 00:00:00 12-10-2011 mean that the hourly backup would never occur until December 10th, 2011 at midnight? This is what my problem is. I cannot force my server to do hourly backups because if for some reason it is down when the nextrun happens, this breaks the time calculation and causes it to revert to the current time.

    Thanks for looking into this, as I can't be the only person experiencing this problem.

  • briman0094 posted a comment Dec 16, 2011

    @cembry90: Go

    It would work in theory, but I forgot to make the units carry over. So it would create an invalid time (24:00:00) with your setup. A fix will be coming soon which will also be the official update to Bukkit 1.0.1-R1.

  • _ForgeUser3962498 posted a comment Dec 20, 2011

    @ArcanoxDragon: Go

    Cool, I really appreciate your plugin as it's very nice to have hourly saves in case of power failure and daily shutdowns to reduce overall lag. :)

    Would you by chance be inclined to use GitHub to share your source, as I'm interested in possibly modifying this and using it for my server.

    Thank you.


To post a comment, please login or register a new account.