[wp-trac] [WordPress Trac] #34936: Reconcile wp-settings-cli.php with wp-settings.php

WordPress Trac noreply at wordpress.org
Thu Jun 2 21:32:27 UTC 2016


#34936: Reconcile wp-settings-cli.php with wp-settings.php
-------------------------------------------+------------------
 Reporter:  danielbachhuber                |       Owner:
     Type:  enhancement                    |      Status:  new
 Priority:  normal                         |   Milestone:  4.6
Component:  Bootstrap/Load                 |     Version:
 Severity:  normal                         |  Resolution:
 Keywords:  dev-feedback wp-cli has-patch  |     Focuses:
-------------------------------------------+------------------

Comment (by ocean90):

 Replying to [comment:20 jorbin]:
 > They are loaded before plugins are loaded, so we could change it to they
 can not be used by plugins.

 That makes probably more sense. Although, if WP-CLI can do it then a
 plugin in form of `auto_prepend_file` (see  [37207]) can do it too.

 > I chose `bypass` instead of `enable` as I felt that that action that a
 user of the filter is to bypass something rather than to unenable.  I
 don't feel super strong about this though so if you do, I'm open to
 changing it.

 I don't feel that strong too, but `bypass` is currently only used in
 inline docs and once in a function: `media_upload_html_bypass()`.

 > As noted in the link, it's to allow `--debug` to be used to set debug
 display settings.  Perhaps a better name and documentation would
 demonstrate that better.  Ideas?

 Okay. Maybe the name should include "error reporting" since that's what
 the function handles.

 ----

 For `WP_CACHE` I've found
 https://core.trac.wordpress.org/browser/trunk/src/wp-
 settings.php?rev=37626&marks=283-284#L281. That should probably be covered
 by the filter too, although it's unlikely that the function exists if
 `advanced-cache.php` isn't loaded.

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


More information about the wp-trac mailing list