Bug 350414 - Removed wallpaper is set for desktop after applying changes
Summary: Removed wallpaper is set for desktop after applying changes
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Image & Slideshow wallpaper plugins (other bugs)
Version First Reported In: 5.3.2
Platform: Kubuntu Linux
: NOR normal
Target Milestone: 1.0
Assignee: Marco Martin
URL: https://youtu.be/45wajPVGmvI
Keywords:
Depends on:
Blocks:
 
Reported: 2015-07-20 14:33 UTC by soee
Modified: 2015-07-20 16:08 UTC (History)
2 users (show)

See Also:
Latest Commit: http://commits.kde.org/plasma-workspace/c3ea4099c2c8e3703a375bad121e9ed53dbff805
Version Fixed/Implemented In:
Sentry Crash Report:

Attachments
Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
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