WordPress.org

Make WordPress Core

Opened 14 months ago

Closed 8 months ago

Last modified 8 months ago

#23650 closed enhancement (fixed)

make get_space_allowed() filterable

Reported by: jkudish Owned by: nacin
Milestone: 3.7 Priority: normal
Severity: normal Version: 3.4
Component: Multisite Keywords: has-patch
Focuses: Cc:

Description

When the blog_upload_space option is set to 0, instead of returning 0, get_space_allowed() returns the default value which is currently hardcoded to 50 in core.

As discussed in #18119 and committed in [19314], the function should return 0 if the option is set to 0. However this was reverted in [19654] (see #19538). So I am not sure which is the desired behaviour.

The attached patch adds filters for the default value and for the returned value. This is useful for scenarios where you don't want it to fallback to 50 if the option is accidentally set to 0, and you want to set your own default.

I can adjust the patch if the consensus is that an option of 0 should in fact return 0.

Attachments (2)

23650.diff (547 bytes) - added by jkudish 14 months ago.
23650.2.diff (502 bytes) - added by nacin 8 months ago.

Download all attachments as: .zip

Change History (9)

jkudish14 months ago

comment:1 SergeyBiryukov14 months ago

  • Component changed from Upload to Multisite

comment:2 jeremyfelt8 months ago

  • Milestone changed from Awaiting Review to 3.7

Moving to 3.7 for discussion. Patch applies ok

comment:3 nacin8 months ago

I wonder if this actually needs there to be a filter for the default. That is basically just a fallback for when the blog_upload_space network-wide option doesn't exist, and it really really should exist. (It is also 100MB as of [20170], and this 50 should too become 100). There is so little logic in get_space_allowed() that a single filter at the bottom should be sufficient.

nacin8 months ago

comment:4 nacin8 months ago

I'm game for 23650.2.diff, any objections?

comment:5 jeremyfelt8 months ago

Looks good to me.

comment:6 nacin8 months ago

  • Owner set to nacin
  • Resolution set to fixed
  • Status changed from new to closed

In 25092:

Multisite: Add a filter to get_space_allowed() and increase its internal default value to the network default.

props jkudish for initial patch.
fixes #23650.

comment:7 nacin8 months ago

In 25111:

Multisite tests: Update default space allowed in get_space_allowed(). see #23650, [25092].

Note: See TracTickets for help on using tickets.