Bug 350414

Summary:	Removed wallpaper is set for desktop after applying changes
Product:	[Plasma] plasmashell	Reporter:	soee <kontakt>
Component:	Image & Slideshow wallpaper plugins	Assignee:	Marco Martin <notmart>
Status:	RESOLVED FIXED
Severity:	normal	CC:	bshah, plasma-bugs-null
Priority:	NOR
Version First Reported In:	5.3.2
Target Milestone:	1.0
Platform:	Kubuntu
OS:	Linux
URL:	https://youtu.be/45wajPVGmvI
Latest Commit:	http://commits.kde.org/plasma-workspace/c3ea4099c2c8e3703a375bad121e9ed53dbff805	Version Fixed/Implemented In:
Sentry Crash Report:

Description soee 2015-07-20 14:33:27 UTC

If we add new wallpaper to existing list and marked it to be removed from this list right away, after applying changes removed wallpaper is set for desktop. Second click on Apply or Ok button sets valid wallpaper for our desktop.

Reproducible: Always

Steps to Reproduce:
1. Open desktop settings
2. Browse for new wallpaper and add it to the list
3. Mark newly added wallpaper to be removed from lost
4. Apply changes
5. Removed wallpaper is set for desktop
6. Second hit to Apply or Ok button will use valid/existing wallpaper

Actual Results:  
Removed wallpaper is set for desktop

Expected Results:  
Only wallpaper that exists on the list should be used for desktop

Comment 1 Marco Martin 2015-07-20 16:08:16 UTC

Git commit c3ea4099c2c8e3703a375bad121e9ed53dbff805 by Marco Martin.
Committed on 20/07/2015 at 16:06.
Pushed by mart into branch 'master'.

select a different wallpaper when the current one is deleted

M  +4    -1    wallpapers/image/imagepackage/contents/ui/WallpaperDelegate.qml

http://commits.kde.org/plasma-workspace/c3ea4099c2c8e3703a375bad121e9ed53dbff805