WordPress.org
Get WordPress

Make WordPress Core

Opened 6 years ago

Closed 6 years ago

Last modified 5 years ago

#51680 closed enhancement (fixed)

Add WP_MEMORY_LIMIT to WP Debug Data class

Reported by: nre Owned by: SergeyBiryukov
Priority: normal Milestone: 5.7
Component: Site Health Version: 5.2
Severity: normal Keywords: good-first-bug has-patch has-dev-note
Cc: Focuses: administration

Description

Please add the constant WP_MEMORY_LIMIT to class WP_Debug_Data.

When I check the WP report in site health report (wp-admin/site-health.php?tab=debug), I see a list of all WordPress constants, amongst others there is WP_MAX_MEMORY_LIMIT. Both constants are part of the documentation (​https://wordpress.org/support/article/editing-wp-config-php/#increasing-memory-allocated-to-php)

What is missing there is the constant WP_MEMORY_LIMIT.
Apparently because it's not registered in Class WP_Debug_Data in wp-admin/includes/class-wp-debug-data.php.

/editing-wp-config-php/#increasing-memory-allocated-to-php).

Attachments (1)

51680.diff​ (576 bytes ) - added by sanketchodavadiya 6 years ago.
Created patch.

Download all attachments as: .zip

Change History (10)

#1 @SergeyBiryukov
6 years ago

  • Component Administration → Site Health
  • Focuses administration added

This ticket was mentioned in ​Slack in #core by noisysocks. ​View the logs.
6 years ago

#3 @talldanwp
6 years ago

  • Keywords needs-patch good-first-bug added
  • Milestone Awaiting Review → Future Release

Hi @nre. The ticket was discussed during a triage session in slack:
​https://wordpress.slack.com/archives/C02RQBWTW/p1604381419364900

We agreed this sounds like a good thing to do!

@sanketchodavadiya
6 years ago

Created patch.

#4 @sanketchodavadiya
6 years ago

  • Keywords has-patch added; needs-patch removed
  • Version → trunk

#5 @hellofromTonya
6 years ago

  • Version trunk → 5.2

#6 @SergeyBiryukov
6 years ago

  • Milestone Future Release → 5.7
  • Owner set to SergeyBiryukov
  • Status new → reviewing

#7 @SergeyBiryukov
6 years ago

  • Resolution → fixed
  • Status reviewing → closed

In 49908:

Site Health: Add WP_MEMORY_LIMIT constant to debug info.

This complements the WP_MAX_MEMORY_LIMIT constant previously displayed there.

Props sanketchodavadiya, nre, talldanwp, noisysocks.
Fixes #51680.

#8 @audrasjb
6 years ago

  • Keywords needs-dev-note added

It would be nice to mention this change in the Miscellaneous Changes dev note.

Note: See TracTickets for help on using tickets.