All submissions to this site are governed by the Second Life Viewer Contribution Agreement. By submitting patches and other information using this site, you acknowledge that you have read, understood, and agreed to those terms.

Review Board 1.6.11

Welcome to the Second Life Viewer Code Review tool.
See the documentation on our wiki for how to use this site.

STORM-1268 Viewer update resets some viewer settings

Review Request #443 - Created Aug. 19, 2011 and submitted

Vadim ProductEngine Reviewers
viewer
STORM-1268 richard.linden
None viewer-development
Bug:
Settings for ignorable dialogs were reset during deferred auto-upgrade.

Reason:
In case of deferred upgrade (i.e. when you select "Later...") the defaults for notifications settings are not loaded,
so when the viewer exits after launching the updater, it incorrectly re-saves notifications settings.

Fix:
Initialize settings earlier, so that viewer picks them up in update mode.
See acceptance criteria in the JIRA ticket.
Ship it!
Posted (Aug. 19, 2011, 10:14 a.m.)
good catch!