Skip to content

Commit

Permalink
Fix the migration of the No Notifications.
Browse files Browse the repository at this point in the history
When migrating the Notification Settings, the DisplayNotification was
taking precedence on what was set for the NotificationSettings.
Fixes #56
  • Loading branch information
Antoine Aflalo committed Dec 31, 2015
1 parent a613ec7 commit f6c0307
Showing 1 changed file with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -52,17 +52,17 @@ public SoundSwitchConfiguration()
public HotKeys RecordingHotKeys { get; set; }
public bool ChangeCommunications { get; set; }
public uint UpdateCheckInterval { get; set; }
public NotificationTypeEnum NotificationSettings { get; set; }

/*TODO: Remove in next VERSION (3.7.0)*/
[Obsolete("Use the NotificationSettings instead.")]
public bool DisplayNotifications
{
get { return NotificationSettings == NotificationTypeEnum.DefaultWindowsNotification; }
set {
NotificationSettings = value ? NotificationTypeEnum.DefaultWindowsNotification : NotificationTypeEnum.SoundNotification;
NotificationSettings = value ? NotificationTypeEnum.DefaultWindowsNotification : NotificationTypeEnum.NoNotification;
}
}
public NotificationTypeEnum NotificationSettings { get; set; }



Expand Down

0 comments on commit f6c0307

Please sign in to comment.