WordPress.org

Make WordPress Core

Opened 17 months ago

Last modified 8 months ago

#22648 new defect (bug)

No error reporting for failed upload of background image in theme customizer

Reported by: WPsites Owned by:
Milestone: Future Release Priority: normal
Severity: normal Version: 3.4
Component: Appearance Keywords: needs-patch
Focuses: Cc:

Description

I tried uploading a background image using the theme cusomizer in 3.5 RC2 but nothing happened. After a few more tries I worked out that there was a problem uploading the file but WordPress didn't inform me of this, which it should.

It was only once I'd looked at the response from async-upload.php that I realised there even was a problem.

filename: "Screen Shot 2012-11-28 at 17.50.55.png"
message: "Unable to create directory wp-content/uploads/2012/11. Is its parent directory writable by the server?"
success: false

Attachments (1)

error.png (15.9 KB) - added by ocean90 17 months ago.

Download all attachments as: .zip

Change History (10)

comment:2 WPsites17 months ago

Yes looks like it is related to that. I searched through Trac before posting this but couldn't find anything!

ocean9017 months ago

comment:3 follow-up: ocean9017 months ago

https://core.trac.wordpress.org/raw-attachment/ticket/22648/error.png

You don't get this message?

Yes, it's sad that there isn't currently an informative message.

comment:4 in reply to: ↑ 3 koopersmith17 months ago

Replying to ocean90:

You don't get this message?

Yes, it's sad that there isn't currently an informative message.

This ticket is referring to the customizer, not media (but yes, you're correct).

While this is not so great, it is also not a regression.

comment:5 nacin17 months ago

  • Keywords needs-patch added
  • Milestone changed from Awaiting Review to Future Release
  • Version changed from trunk to 3.4

Since we don't have any error handling in the customizer at all yet, we need to punt on this.

Separately, informative messages are now fixed in trunk, for media. Woo.

comment:6 helen15 months ago

#23288 was marked as a duplicate.

comment:7 unsalkorkmaz15 months ago

  • Cc unsalkorkmaz@… added

comment:8 tw21138 months ago

This is going to be true on any file upload using customizer, not just background images. Would be awesome get this bumped up and taken care of to help users avoid confusion, especially if we're trying to encourage developers to use this instead of options pages.

comment:9 SergeyBiryukov8 months ago

  • Component changed from Media to Appearance
Note: See TracTickets for help on using tickets.