Roundup Tracker - Issues

Issue 2117897

classification
exceptions.ValueError: second must be in 0..59
Type: crash Severity: normal
Components: Web interface, Command-line interface Versions: 1.5, 1.4, 1.3
process
Status: fixed fixed
:
: ThomasAH : ThomasAH, jkenn, malte
Priority: normal : patch

Created on 2008-09-18 13:19 by jkenn, last changed 2014-02-11 09:26 by ThomasAH.

Files
File name Uploaded Description Edit Remove
issue82.txt jkenn, 2008-09-18 13:19 Backtrace and issue+history
Messages
msg2587 Author: [hidden] (jkenn) Date: 2008-09-18 13:19
One of the issues in our issue tracker fails to display in the web interface because the exception in the summary.

The backtrace and issue history is listed in the appended text file.

This is using 1.4.6 with Python 2.4.
msg4183 Author: [hidden] (ThomasAH) Date: 2010-10-28 15:26
Maybe <Date 2008-09-11.14:02:59.998> was somehow rounded to :60?

I just got the same crash in roundup-admin and web interface.
In my case I got a journal entry with <Date 2010-10-28.14:34:60.000000>

My very ugly workaround was to change _utc_to_local() in roundup/date.py to:

    try:
        ...existing code...
    except ValueError:
        return (1970,1,1,12,0,0)
msg4202 Author: [hidden] (malte) Date: 2010-11-11 13:36
Same problem here with the current Ubuntu (maverick) version of roundup,
which they call 1.4.13-3.1 (so presumably based on 1.4.13) using Python
2.6.6.

I reported this as an issue with Ubuntu's roundup package here:
https://bugs.launchpad.net/ubuntu/+source/roundup/+bug/673677
I fixed the problem locally by capping the seconds field to the range
0..59 (see patch behind that link).

Meta-question: should I not bother submitting roundup bugs at Ubuntu's
launchpad site? The previous issue I reported there did not get any
reaction after more than a year:
https://bugs.launchpad.net/ubuntu/+source/roundup/+bug/469013
msg4987 Author: [hidden] (ThomasAH) Date: 2014-01-28 14:03
I have two test cases now in older tracker versions.
I'm planning to migrate them to roundup 1.5.x very soon, so I can see if
your patch is still needed (I assume that it is, the code is still
unchanged) and apply it to the Roundup repository (where I now have push
access).
msg4988 Author: [hidden] (ThomasAH) Date: 2014-02-10 15:48
Related or maybe the same problem:
issue2550802
roundup.date.Date()s formatted using serialise may get a seconds part of
60.000
msg4989 Author: [hidden] (ThomasAH) Date: 2014-02-10 15:49
It seems this tracker does not generate a link to issue 2550802 (now
trying with an additional space)
To be sure, here is the full URL:
http://issues.roundup-tracker.org/issue2550802
msg4990 Author: [hidden] (ThomasAH) Date: 2014-02-11 09:26
changeset:   4856:392a055fdc21
tag:         tip
user:        Thomas Arendsen Hein <thomas@intevation.de>
date:        Tue Feb 11 09:38:31 2014 +0100
files:       CHANGES.txt roundup/date.py
description:
issue2117897: Fixed two more rounding to 60.0 seconds in date.py

This caused exceptions in real-world data as reported in the issue by
Kenny Johansson, Malte Helmert and me. Change them to 59.999 as was done
in the fix for issue2550802.
History
Date User Action Args
2014-02-11 09:26:41ThomasAHsetstatus: open -> fixed
resolution: fixed
messages: + msg4990
2014-02-10 15:49:49ThomasAHsetmessages: + msg4989
2014-02-10 15:48:52ThomasAHsetmessages: + msg4988
2014-01-28 14:03:14ThomasAHsetkeywords: + patch
assignee: ThomasAH
messages: + msg4987
versions: + 1.5
2010-11-11 13:36:53maltesetnosy: + malte
messages: + msg4202
2010-10-28 15:26:41ThomasAHsetversions: + 1.3, 1.4
nosy: + ThomasAH
title: exceptions.ValueError: second must be in 0..59 -> exceptions.ValueError: second must be in 0..59
messages: + msg4183
components: + Web interface, Command-line interface, - None
type: crash
2008-09-18 13:19:16jkenncreate