Closed Bug 901111 Opened 11 years ago Closed 11 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+
Status: NEW → RESOLVED
Closed: 11 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.

Attachment

General

Creator:
Created:
Updated:
Size: