Closed
Bug 362876
Opened 18 years ago
Closed 18 years ago
after updating restart is needed but data in editing event is not saved
Categories
(Calendar :: General, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
Sunbird 0.5
People
(Reporter: damian.publicemail, Assigned: jminta)
Details
Attachments
(1 file)
2.44 KB,
patch
|
mattwillis
:
first-review+
mvl
:
second-review+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; pl; rv:1.8.1) Gecko/20061010 Firefox/2.0 Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a1) Gecko/20061205 Calendar/0.4a1 After each software upadate restart is required, however if there is any event/task opened data is lost Reproducible: Always Steps to Reproduce: 1. launch sunbird 2. press "check for updates" from help menu 3. open event and start editing 4. second dialog has finished updating and application needs restart 5. let application be restarted Actual Results: edited event is closed without saving data Expected Results: there should be alert that there is opened event which should be closed before restarting
Assignee | ||
Comment 1•18 years ago
|
||
The updater will iterate through all open windows and look for a 'tryToClose' method. If one exists, it'll run the method and only quit if it returns true. This patch implements such a method in the item-dialog.
Assignee: nobody → jminta
Status: NEW → ASSIGNED
Attachment #248030 -
Flags: second-review?(mvl)
Attachment #248030 -
Flags: first-review?(lilmatt)
Comment 2•18 years ago
|
||
Comment on attachment 248030 [details] [diff] [review] add tryToClose method >+ return promptService.confirm(window, Trailing spaces on this line ^^ r=lilmatt with that, but I'd love it if we could get beltzner to massage the text.
Attachment #248030 -
Flags: first-review?(lilmatt) → first-review+
Comment 3•18 years ago
|
||
Comment on attachment 248030 [details] [diff] [review] add tryToClose method r2=mvl
Attachment #248030 -
Flags: second-review?(mvl) → second-review+
Updated•18 years ago
|
Whiteboard: [needs checkin]
Target Milestone: --- → Sunbird 0.5
Assignee | ||
Comment 4•18 years ago
|
||
patch checked in. Feel free to file a follow-up if anyone can come up with better verbage.
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Reporter | ||
Updated•18 years ago
|
Whiteboard: [needs checkin] → [needs checkin][litmus testcase wanted]
Reporter | ||
Comment 5•18 years ago
|
||
verified with Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a2pre) Gecko/20061225 Calendar/0.4a1
Status: RESOLVED → VERIFIED
Whiteboard: [needs checkin][litmus testcase wanted] → [litmus testcase wanted]
You need to log in
before you can comment on or make changes to this bug.
Description
•