Closed Bug 291220 Opened 20 years ago Closed 19 years ago

redo should use the returned event, not the original event

Categories

(Calendar :: Sunbird Only, defect)

defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: mvl, Assigned: jminta)

Details

Attachments

(1 file)

for example, for adding an item, redo and undo both use the original event. the server might set an ID or change something else. We should use that item for redo. See bug 258258 comment 6
QA Contact: gurganbl → sunbird
Attached patch use listenerSplinter Review
In bug 325477 I started getting the returned items for use in undo. I guess I just left out the redo case. :-/ This patch tidies this up.
Assignee: mvl → jminta
Status: NEW → ASSIGNED
Attachment #225917 - Flags: first-review?(dmose)
Comment on attachment 225917 [details] [diff] [review] use listener r=shaver (won't it be nice when |this| is in the scope chain?)
Attachment #225917 - Flags: first-review?(dmose) → first-review+
patch checked in.
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
verified with Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a1) Gecko/20060827 Calendar/0.3a2+
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: