Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#11493 closed bug (no change required)

Notification preflet: unable to save settings

Reported by: Giova84 Owned by: pulkomandy
Priority: normal Milestone: R1
Component: Preferences/Notifications Version: R1/Development
Keywords: Cc:
Blocked By: Blocking:
Has a Patch: no Platform: x86

Description

I was attempting to change settings for Notifications (I'd wish to change the Window width), once i click on "Apply" button, an alert window told me "An error occurred saving the preferences. It's possible you are running out of disk space." Obviously I'm not running out of disk space :-)

hrev48327 screenshot attached.

Attachments (1)

Notification_Error.png (36.5 KB ) - added by Giova84 5 years ago.

Download all attachments as: .zip

Change History (9)

by Giova84, 5 years ago

Attachment: Notification_Error.png added

comment:1 by pulkomandy, 5 years ago

Hi, If you updated from an older version of the nightlies, you need to delete the config/settings/system/notifications folder. The folder was replaced with a single settings file now, but the preferences panel is not able to migrate from the old format.

comment:2 by Giova84, 5 years ago

Ok, deleting the "notification" folder did solved the situation. Thank you! So this ticket can be closed as invalid!

comment:3 by Giova84, 5 years ago

Ops, if the preferences panel is not able to migrate from the old format, maybe this ticket should be stay open.

comment:4 by pulkomandy, 5 years ago

Resolution: no change required
Status: newclosed

It was decided to do it this way because migrating from alpha4 to beta1 is not supported, and we don't want to have support for the old format in the notification preflet (it removes a lot of code to use the new format).

comment:5 by tangobravo, 5 years ago

So it currently uses the defaults if you have the old-style settings folder? Perhaps just checking for the folder and deleting it when apply settings is a good compromise?

comment:6 by X512, 5 years ago

Silently delete settings probably is not a good solution, but better than became unavalible to save settings. Program should show alert that setting have invalid format or obsolete after being started. User sould have ability to keep or delete old settings file.

comment:7 by pulkomandy, 5 years ago

We usually do that, but in this case there is no upgrade path from alpha4 so it is useless. We don't want to clutter the code with handling of the old settings. Problem will appear only for users that update an install done before october to a current one, then try to change notification preferences.

There weren't much settings in the Notifications preferences anyway, the important things (ability to block apps, notification history) were never saved anyway. So it's easy to reconfigure the whole thing.

comment:8 by Giova84, 5 years ago

After deleting the config/settings/system/notifications folder, at the next reboot I've found that notifications were disabled (so I enabled them again from the notification preflet). Shouldn't notifications be enabled by default? I mean that notifications could be optionally disabled. IIRC on Haiku Alpha 4, notifications were enabled by default.

Note: See TracTickets for help on using tickets.