[wp-trac] [WordPress Trac] #48720: Define all debugging constants for developers in wp-config.php
WordPress Trac
noreply at wordpress.org
Tue Nov 19 09:26:53 UTC 2019
#48720: Define all debugging constants for developers in wp-config.php
----------------------------+------------------------------
Reporter: davidnash | Owner: (none)
Type: enhancement | Status: new
Priority: normal | Milestone: Awaiting Review
Component: Bootstrap/Load | Version: 5.3
Severity: normal | Resolution:
Keywords: | Focuses:
----------------------------+------------------------------
Comment (by SergeyBiryukov):
Replying to [comment:7 psykro]:
> I just wanted to make sure you saw that my examples were added in the
comment block, not actually turning those constants on in the code. I
agree that simply turning them on would be a bad thing, hence adding them
to the comment block.
>
> However if you did, but your concern is that adding them there would
cause folks to inadvertently turn these things on without understanding
the consequences, I'll keep my mouth shut :-)
Yes, that's exactly my concern, users trying out these settings without a
proper consideration.
That's just my opinion though, happy to discuss this and explore more
ideas :) Just wanted to note that these settings can sometimes be more
nuanced than might seem at a glance.
For example, depending on PHP configuration, PHP warnings can be displayed
regardless of the `WP_DEBUG` value. Seeing a comment like this:
{{{
// define( 'WP_DEBUG_DISPLAY', false ); - prevent the errors from showing
in the HTML of pages
}}}
a user might want to try setting this to get rid of the warnings, without
realizing it doesn't have any effect if `WP_DEBUG` is disabled, and would
then be confused as to why it doesn't work as expected.
We could add some more comments to clarify this, but at that point, just
linking to the documentation would probably make more sense.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/48720#comment:9>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list