#21115 closed defect (bug) (fixed)

Remove ancient wp-settings.php global unsetting code

Reported by: nacin Owned by: nacin
Priority: normal Milestone: 3.5
Component: Unit Tests Version:
Severity: normal Keywords:
Cc:

Description

We have this:

// Ensure these global variables do not exist so they do not interfere with WordPress.
unset( $wp_filter, $cache_lastcommentmodified );

It used to include more variables, but they were removed when a few of us took an axe to wp-settings.php back in 3.0.

I suggest we remove both. $cache_lastcommentmodified can easily become a static variable within that function (I mean, we're dealing with pre-1.5 code here) and a lot of things would be broken if people overrode WordPress globals prior to startup, not just $wp_filter.

Attachments (1)

21115.diff (2.3 KB) - added by nacin 10 months ago.

Download all attachments as: .zip

Change History (3)

Side effect: Removing the unset() for $wp_filter can allow the test suite to add some filters prior to startup, rather than a more explicit re-organization proposed in #20731.

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

In [21186]:

Remove old code unsetting a few variables in wp-settings.php. fixes #21115.

nacin10 months ago

Note: See TracTickets for help on using tickets.