Closed Bug 901111 Opened 9 years ago Closed 9 years ago

When editing an event with invalid data, despite the warning, the data is still saved.

Categories

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

x86
macOS
defect
Not set
normal

Tracking

(blocking-b2g:koi+)

VERIFIED FIXED
blocking-b2g koi+

People

(Reporter: gghosh, Assigned: gghosh)

Details

Attachments

(1 file, 2 obsolete files)

The data is not actually saved but cached and shows the user the invalid data. But if you refresh, the data is correct.
Attached patch Event only commits if valid (obsolete) — Splinter Review
Attachment #785364 - Flags: review?(jlal)
Attached patch Event only commits if valid (obsolete) — Splinter Review
Attachment #785364 - Attachment is obsolete: true
Attachment #785364 - Flags: review?(jlal)
Attachment #785574 - Flags: review?(jlal)
Comment on attachment 785574 [details] [diff] [review]
Event only commits if valid

some minor style needs should be fine after.. Please flag me for review (after rebasing).
Attachment #785574 - Flags: review?(jlal) → review-
Attachment #785574 - Attachment is obsolete: true
Attachment #786155 - Flags: review?(jlal)
Comment on attachment 786155 [details] [diff] [review]
Event only commits if valid

I would like to give this a quick manual check but r+ on the code side this looks really nice. I will land this after I give it a quick check.
Attachment #786155 - Flags: review?(jlal) → review+
https://github.com/mozilla-b2g/gaia/commit/08dfcb5d5b8ea395ab56043b2700bbdcf0841ac4
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
blocking-b2g: --- → koi?
fixed
blocking-b2g: koi? → koi+
Verified Fix: User cannot save (or does not appear to save) an edit to an event that has an end date before the start date. Tested Offline, Google, Yahoo, and CalDav Calendars.

Environmental Variables
Device: Buri v1.2 COM RIL
Build ID: 20131103004003
Gecko: http://hg.mozilla.org/releases/mozilla-b2g26_v1_2/rev/eec4da1b27eb
Gaia: cb981e2f47bc644b4d178d54378c3676c946facf
Platform Version: 26.0
RIL Version: 01.01.00.019.276 
Firmware Version: US_20131015
Status: RESOLVED → VERIFIED
Assignee: nobody → gghosh
You need to log in before you can comment on or make changes to this bug.