Make WordPress Core

Opened 7 years ago

Closed 3 years ago

#17542 closed enhancement (invalid)

Remove upload file after deleting custom background

Reported by: denzel_chia Owned by:
Milestone: Priority: normal
Severity: normal Version:
Component: Customize Keywords: has-patch
Focuses: administration Cc:


I came across a comment on custom-background.php that says @todo: uploaded files are not removed here.

So, I thought I will come up with this codes for the upload file deletion if user remove background.

Attached custom-background.diff

I had tested on my localhost.

Hope this contribution is useful.


Attachments (2)

custom-background.diff (1.5 KB) - added by denzel_chia 7 years ago.
17542.diff (1.5 KB) - added by kawauso 7 years ago.
Delete on reset and check for attachment ID

Download all attachments as: .zip

Change History (8)

#1 @ocean90
7 years ago

You should use the current trunk, there is a revert in your patch, see 'context' => 'custom-background' .

#2 @kawauso
7 years ago

  • Keywords has-patch added

Not got a theme to test this with, but shouldn't we delete the attachment on reset as well?

Attaching a patch that does this and checks we actually have an attachment ID before trying to delete it.

7 years ago

Delete on reset and check for attachment ID

#3 @denzel_chia
7 years ago

Thanks everybody for the pointers.

#4 @nacin
7 years ago

Nice initial work on this, both of you.

Because backgrounds are now stored with context => custom-background, then we're actually remembering previous backgrounds. Keeping the old background around after settings changes doesn't seem like a problem. But I'll leave that for the UX.

#5 @nacin
4 years ago

  • Component changed from Administration to Appearance
  • Focuses administration added

#6 @celloexpressions
3 years ago

  • Milestone Awaiting Review deleted
  • Resolution set to invalid
  • Status changed from new to closed

4 years later, now that backgrounds are handled through the media library (and in the Customizer, see [30309], and [30459]), they definitely shouldn't be deleted when they're removed/changed from the background setting, since they can also be used elsewhere. So this ticket is no longer relevant.

Note: See TracTickets for help on using tickets.