Differential D21475

[Notifications] Check proper group for show popups in dnd mode setting
ClosedPublic
Actions

Authored by broulik on May 29 2019, 7:46 AM.

Details

Reviewers
bugseforuns
apol
Group Reviewers
Plasma
Commits
R120:a24166f72fbd: [Notifications] Check proper group for show popups in dnd mode setting
Summary

It checked the wrong one resulting in the setting not being written.

BUG: 407534
FIXED-IN: 5.16.0

Test Plan

Can now disable show popups in dnd mode for Spectacle which has them enabled by default.
Verified that it writes the key into plasmanotifyrc

Diff Detail

Repository
R120 Plasma Workspace
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
broulik created this revision.May 29 2019, 7:46 AM
Restricted Application added a project: Plasma. · View Herald TranscriptMay 29 2019, 7:46 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
broulik requested review of this revision.May 29 2019, 7:46 AM
apol accepted this revision.May 29 2019, 10:47 AM
This revision is now accepted and ready to land.May 29 2019, 10:47 AM
Closed by commit R120:a24166f72fbd: [Notifications] Check proper group for show popups in dnd mode setting (authored by broulik). · Explain WhyMay 29 2019, 11:49 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 58833

View Options

libnotificationmanager/settings.cpp

Loading...