827849 - [Calendar] Hour long events show up as day long events.

Status: RESOLVED WORKSFORME

(Firefox OS Graveyard :: Gaia::Calendar, defect, P2)

Description

[Faramarz Rashed]

Attachment: screen-shot

Showed this to James Lal on the 1/7/2013 build.  The details of the event are correct, but the visuals take up the whole day making it look like an day long event -- screenshot attached.

Comment 1

[Faramarz Rashed]

James: work on this opportunistically and after your blockers are done. This may be specific to my calendar events.

Comment 2

[Jason Smith [:jsmith]]

If we are showing wrong event lengths in a calendar view that's imported, then this blocks.

Faramarz - Does James have access to your calendar to test this?

Comment 3

[Al Tsai [:atsai]]

Attachment: 20130108070203

Comment 4

[Al Tsai [:atsai]]

not quite sure what we need from QA's input.
It seems fine on BUILD ID 20130108070203.
Please reference the attachment

Comment 5

[Jason Smith [:jsmith]]

QAWanted back on. It's likely a timezone issue with the imported calendar so you'll need to work Faramarz directly to test this.

Comment 6

[Jason Smith [:jsmith]]

(In reply to Jason Smith [:jsmith] from comment #5)
> QAWanted back on. It's likely a timezone issue with the imported calendar so
> you'll need to work Faramarz directly to test this.

We're going to need a build though with that OOM crash fixed though. It's preventing testing this right now.

Comment 7

[Jason Smith [:jsmith]]

Blocked by the OOM crash in bug 826872.

Comment 8

[James Lal [:lightsofapollo]]

On latest master we should be good, I can flash your phone with latest gaia so we can unblock testing.

Comment 9

[James Lal [:lightsofapollo]]

We need to reproduce this somehow, I can't see it on my own.

This _may_ be related to timezone issues or a case where we incorrectly cache timezone information. I will try to debug this further but it would be a major help to some up with a STR.

Comment 10

[Jason Smith [:jsmith]]

Faramarz - Come find James & I when you get in tomorrow.

The OOM crash you were probably hitting when you showed this to me is probably ixed by bug 826872. We should be able to test this again.

I want to see if we can consistently reproduce this behavior on every import.

Comment 11

[James Lal [:lightsofapollo]]

I may have fixed this problem in the recurring time patch. When I optimized parts of the ical layer in the timezone bug I later realized I had incorrectly handled durations when subtracting dates. It's possible that is related.

Lets very that the issue exits on an older build. Dietrich's calendar has an event that easily reproduces this problem so I will ask him again for his credentials on a build from three days ago.

Comment 12

[James Lal [:lightsofapollo]]

I cannot reproduce this at this point. Anyone who had experienced this issue should try again on latest gaia (after f10ca860f261f1072092c7638eafdb03384c3101).