[wp-trac] [WordPress Trac] #32075: Only set WP_MAX_MEMORY_LIMIT by default when its greater than memory_limit

[WordPress Trac]

#32075: Only set WP_MAX_MEMORY_LIMIT by default when its greater than memory_limit
---------------------------------------------+----------------------
 Reporter:  danielbachhuber                  |       Owner:  ocean90
     Type:  defect (bug)                     |      Status:  closed
 Priority:  normal                           |   Milestone:  4.6
Component:  Bootstrap/Load                   |     Version:  3.2
 Severity:  normal                           |  Resolution:  fixed
 Keywords:  has-patch has-unit-tests commit  |     Focuses:
---------------------------------------------+----------------------
Changes (by ocean90):

 * status:  accepted => closed
 * resolution:   => fixed


Comment:

 In [changeset:"38015"]:
 {{{
 #!CommitTicketReference repository="" revision="38015"
 Bootstrap: Enhance core's memory limit handling.

 * Don't lower memory limit if the current limit is greater than
 `WP_MAX_MEMORY_LIMIT`.
 * Set `WP_MEMORY_LIMIT` and `WP_MAX_MEMORY_LIMIT` to current limit if the
 `memory_limit` setting can't be changed at runtime.
 * Use `wp_convert_hr_to_bytes()` when parsing the value of the
 `memory_limit` setting because it can be a shorthand or an integer value.
 * Introduce `wp_raise_memory_limit( $context )` to raise the PHP memory
 limit for memory intensive processes. This DRYs up some logic and includes
 the existing `admin_memory_limit` and `image_memory_limit` filters. The
 function can also be used for custom contexts, the
 `{$context}_memory_limit` filter allows to customize the limit.
 * Introduce `wp_is_ini_value_changeable( $setting )` to determine whether
 a PHP ini value is changeable at runtime.
 * Remove a `function_exists( 'memory_get_usage' )` check. Since PHP 5.2.1
 support for memory limit is always enabled.

 Related commits: [38011-38013]

 Props jrf, A5hleyRich, swissspidy, ocean90.
 Fixes #32075.
 }}}

--
Ticket URL: <https://core.trac.wordpress.org/ticket/32075#comment:36>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform