[AppletConfiguration] Don't call saveConfig() twice
ClosedPublic

Authored by broulik on Sep 6 2016, 9:51 AM.

Details

Summary

root.saveConfig() already calls saveConfig() on the current page, no need to explicitly do that in the apply button.

Test Plan

No longer saves twice. Still saves once

Diff Detail

Repository
R119 Plasma Desktop
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
broulik updated this revision to Diff 6470.Sep 6 2016, 9:51 AM
broulik retitled this revision from to [AppletConfiguration] Don't call saveConfig() twice.
broulik updated this object.
broulik edited the test plan for this revision. (Show Details)
broulik added a reviewer: Plasma.
broulik set the repository for this revision to R119 Plasma Desktop.
Restricted Application added a project: Plasma. · View Herald TranscriptSep 6 2016, 9:51 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
mart accepted this revision.Sep 6 2016, 1:33 PM
mart added a reviewer: mart.
This revision is now accepted and ready to land.Sep 6 2016, 1:33 PM
This revision was automatically updated to reflect the committed changes.