Fix wallpaper destination path
ClosedPublic

Authored by trmdi on Jan 8 2020, 3:46 PM.

Details

Summary
  • Correct the destination path when downloading images.
  • Switch from FileCopyJob to CopyJob to allow showing file overwrite dialog.

BUG: 379469

Test Plan

Drop files from Firefox to the desktop to set the wallpaper successfully.

Diff Detail

Repository
R120 Plasma Workspace
Branch
fix-wallpaper
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 20807
Build 20825: arc lint + arc unit
trmdi created this revision.Jan 8 2020, 3:46 PM
Restricted Application added a project: Plasma. · View Herald TranscriptJan 8 2020, 3:46 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
trmdi requested review of this revision.Jan 8 2020, 3:46 PM
trmdi updated this revision to Diff 73115.Jan 9 2020, 6:11 AM
  • Improve code
mart added a subscriber: mart.Jan 10 2020, 12:20 PM
mart added inline comments.
wallpapers/image/image.cpp
554

this is for first run when ~/.local/share/wallpapers isn't there yet?

555

I would prefer it to ask before owerwriting

trmdi added inline comments.Jan 10 2020, 12:40 PM
wallpapers/image/image.cpp
554

This is to check if the directory is available before copying, otherwise FileCopyJob would fail.

mart accepted this revision.Jan 10 2020, 12:49 PM
This revision is now accepted and ready to land.Jan 10 2020, 12:49 PM
trmdi added inline comments.Jan 10 2020, 1:57 PM
wallpapers/image/image.cpp
555

Do I need to modify this or just land it now?

trmdi updated this revision to Diff 73217.Jan 10 2020, 5:16 PM
  • Switch from FileCopyJob to CopyJob to allow showing file overwrite dialog
trmdi edited the summary of this revision. (Show Details)Jan 10 2020, 5:18 PM
trmdi edited the summary of this revision. (Show Details)
trmdi added a comment.Jan 10 2020, 5:21 PM

@mart
Are you ok with this now ?

trmdi marked 4 inline comments as done.Jan 10 2020, 5:23 PM
trmdi added a comment.Jan 15 2020, 4:36 PM

Ping. Could I land this now?

ngraham accepted this revision.Jan 15 2020, 5:02 PM

Yep, go ahead.

This revision was automatically updated to reflect the committed changes.