Closed Bug 291220 Opened 15 years ago Closed 14 years ago

redo should use the returned event, not the original event

Categories

(Calendar :: Sunbird Only, defect)

defect
Not set

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: 14 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.