Context Navigation

← Previous Ticket
Next Ticket →

#21796 closed enhancement (fixed)

Don't show paths in wp_upload_dir()

Reported by:	nacin	Owned by:	ryan
Priority:	normal	Milestone:	3.5
Component:	Upload	Version:
Severity:	normal	Keywords:
Cc:		Focuses:

Description

wp_upload_dir() unnecessarily shows the full path to an upload directory in case it can't create it. We shouldn't even bother showing the path at all — an error of "Error uploading file. Is the uploads directory writeable by the server?" should be sufficient.

Change History (5)

#1 @nacin
14 years ago

In [21757]:

A more generic message for wp_upload_dir(). see #21796, for 3.4.

#2 @nacin
14 years ago

For 3.5, I'm debating between [21757] and a generic "Error uploading file. Is the uploads directory writeable by the server?" [21757] was designed to not break the string.

#3 @SergeyBiryukov
14 years ago

I'd prefer [21757]. With a generic message, new users would have to guess where the uploads directory is.

#4 @SergeyBiryukov
14 years ago

Milestone 3.4.2 → 3.5

#5 @ryan
14 years ago

Owner set to ryan
Resolution → fixed
Status new → closed

In [22276]:

A more generic message for wp_upload_dir(). Props nacin. fixes #21796

Note: See TracTickets for help on using tickets.

Download in other formats:

Make WordPress Core

Context Navigation

#21796 closed enhancement (fixed)

Don't show paths in wp_upload_dir()

Description

Change History (5)

#1 @nacin 14 years ago

#2 @nacin 14 years ago

#3 @SergeyBiryukov 14 years ago

#4 @SergeyBiryukov 14 years ago

#5 @ryan 14 years ago