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)

x86
Windows XP
defect
Not set
minor

Tracking

(Not tracked)

VERIFIED FIXED
Sunbird 0.5

People

(Reporter: damian.publicemail, Assigned: jminta)

Details

Attachments

(1 file)

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
Attached patch add tryToClose method — — Splinter Review
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 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 on attachment 248030 [details] [diff] [review]
add tryToClose method

r2=mvl
Attachment #248030 - Flags: second-review?(mvl) → second-review+
Whiteboard: [needs checkin]
Target Milestone: --- → Sunbird 0.5
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
Whiteboard: [needs checkin] → [needs checkin][litmus testcase wanted]
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]
Flags: in-litmus?
Whiteboard: [litmus testcase wanted]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: