Bug 350836 - change avatar doesn't work if you click ok
Summary: change avatar doesn't work if you click ok
Status: RESOLVED FIXED
Alias: None
Product: user-manager
Classification: Plasma
Component: kcontrol module (show other bugs)
Version: unspecified
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Jonathan Riddell
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-07-31 20:48 UTC by Jonathan Riddell
Modified: 2017-05-09 20:52 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In: 5.8.7


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Jonathan Riddell 2015-07-31 20:48:58 UTC
changing avatar doesn't get saved if you click ok
it does work if you click apply
this is because the kio::copyJob needs to finish before it does anything, but it doesn't get the finished signal if you click ok


Reproducible: Always
Comment 1 Elvis Angelaccio 2017-05-09 20:52:40 UTC
Git commit 826e41429917f6c1534e84e8b7821b8b53675910 by Elvis Angelaccio.
Committed on 08/05/2017 at 21:20.
Pushed by elvisangelaccio into branch 'Plasma/5.8'.

Make sure the new avatar is always saved

Summary:
The KIO job needs to be run synchronously, otherwise the KCM might be
deleted before the job finished. This happens when closing the KCM
module with the Ok button, or when closing the System Settings window
and clicking Apply.
FIXED-IN: 5.8.7

Test Plan: Change avatar from System Settings, then close System Settings and click Apply.

Reviewers: #plasma, jriddell

Subscribers: plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D5779

M  +2    -1    src/accountinfo.cpp

https://commits.kde.org/user-manager/826e41429917f6c1534e84e8b7821b8b53675910