Sabnzbd: Calculation error when pausing for "13 hours"

[ali-l]

Gets converted to 4320 mins instead of 780 mins.

[thezoggy]

just tired this in latest version, 13h works. 13hours works. 13 hours doesnt. the space breaks it.. but 3 hours is fine.. so bad regex parsing?

[ali-l]

This is a bug with the DateJS dependency, which may or may not have been fixed. I'll have a look there, and maybe update the file if it's been fixed.

EDIT:
In the latest version of DateJS, "13 hours" comes out to 660 minutes. Still broken, but in a different way.

[Safihre]

Too bad it can't be fixed. There's more errors in that library that we already try to catch. Maybe I can add this case.

[thezoggy]

Just replace hours with hours...
To strip the space out

[ali-l]

Seems like it's also broken for anything between 13 and 48 minutes. But replacing the space fixes that too

[ali-l]

How does #918 look?

[Safihre]

Thanks! Will check and merge when I'm back from my holiday :)

[thezoggy]

i just tried this again.. and 13 hours is working fine for me in latest firefox.. tried a bunch of variances but could not cause the issue again...

[ali-l]

Weird...

13 hours seems fine for me too now, on the latest Chrome. Now I see it breaking at/above 17 hours and 17 minutes.

[thezoggy]

can see that 17 hours is fine, now its 18 hours that doesnt work.. aka take the day number + 1. looks like its more of a parsing bug.. and looking at the way were doing the logic.. it may actually be a bug on glitter's end..

[Safihre]

This is quite the weird bug.. But you fixed it well!