Bug 16181 - Error trying to remove Last.FM
: Error trying to remove Last.FM
Status: RESOLVED FIXED
Product: MySqueezebox.com
Classification: Unclassified
Component: Web UI
: MySB
: PC Windows (legacy)
: P2 normal (vote)
: Hotfix
Assigned To: Andy Grundman
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2010-05-05 16:11 UTC by Chris Owens
Modified: 2011-03-16 04:45 UTC (History)
2 users (show)

See Also:
Category: ---


Attachments
replace "0000-00-00 00:00:00" date values with some valid date (613 bytes, patch)
2010-06-23 01:44 UTC, Michael Herger
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Chris Owens 2010-05-05 16:11:14 UTC
Oops!  An error has occurred.  We're working to fix the point, thank you for your patience.
Comment 1 Chris Owens 2010-05-05 17:33:21 UTC
I verified this also happens for me.
Comment 2 Julius Dauz 2010-06-22 17:19:19 UTC
This issue was recently reported by a T2 agent, but the error occured upon installation of the app.
Comment 3 Michael Herger 2010-06-23 01:44:53 UTC
Created attachment 6891 [details]
replace "0000-00-00 00:00:00" date values with some valid date

DateTime can't handle all zero date values
Comment 4 Michael Herger 2010-06-23 01:48:47 UTC
The problem here is that we can have null (or rather all zero) date values in last_connected. When this value is read from the DB, the code tries to convert it into some other format. But that module can't handle all zero values. Thus (after discussion with Brandon) I'd suggest we replace these cases with another far, far away date before converting it.

Brandon's comment: "assuming there's no silly syntax error in the paste, it looks pretty low risk to me"

Chris - if the case is critical enough I can roll this out while Andy is away.
Comment 5 Chris Owens 2010-06-24 09:20:00 UTC
Reducing priority
Comment 6 Andy Grundman 2010-07-12 05:49:52 UTC
I filled the null values in the DB with something, that should take care of this problem.