Make WordPress Core

Opened 13 years ago

Closed 8 years ago

#9409 closed enhancement (worksforme)

Mandatory setting for variable uploads_use_yearmonth_folders

Reported by: hspaans Owned by:
Milestone: Priority: normal
Severity: normal Version:
Component: Upload Keywords:
Focuses: Cc:


The reasoning behing this patch is to make WordPress shared hosting friendly by using a shared codebase and mandatory settings.

This patch overrules database parameter "uploads_use_yearmonth_folders" and removes the interface option when the following constant is set in wp-config.php:


Attachments (1)

wordpress-2.7.1-hspaans-1.patch (1.3 KB) - added by hspaans 13 years ago.

Download all attachments as: .zip

Change History (7)

#1 @hspaans
13 years ago

  • Keywords has-patch added

#2 @Denis-de-Bernardy
13 years ago

  • Keywords mandatory upload structure removed
  • Milestone Unassigned deleted
  • Resolution set to invalid
  • Status changed from new to closed

why not use WPMU?

#3 @hspaans
13 years ago

WPMU doesn't work that well in a shared codebase environment and doesn't meet the requirements.

#4 @Denis-de-Bernardy
13 years ago

oh, in that case, use the WP_CONTENT_DIR and the WP_PLUGINS_DIR urls. should do the trick. you get one content dir for all sites and a shared plugins_dir, which is what I assume you want.

#5 @retlehs
8 years ago

  • Cc retlehs added
  • Keywords has-patch removed
  • Resolution invalid deleted
  • Status changed from closed to reopened
  • Version 2.7.1 deleted

I'd like to have this setting. Not because of the original ticket description, "...to make WordPress shared hosting friendly...", but because of having the ability to set both media settings for uploads via wp-config.php.

Right now we can define the uploads directory but can't define the organize uploads into folders setting.

#6 @nacin
8 years ago

  • Resolution set to worksforme
  • Status changed from reopened to closed

You could always use a filter via get_option() to actually enforce a particular choice. It's not perfect, but it works just fine.

I don't think this should be a constant — we ideally want to avoid constants for non-essential stuff, and this doesn't require a constant or filesystem-level enforcement.

Note: See TracTickets for help on using tickets.