Context Navigation

← Previous Ticket
Next Ticket →

#11834 closed enhancement (fixed)

Use require() in wp-settings.php

Reported by:	scribu	Owned by:
Priority:	normal	Milestone:	3.0
Component:	General	Version:
Severity:	normal	Keywords:	has-patch
Cc:		Focuses:

Description

Currently, require() and require_once() are used seemingly at random throughout wp-settings.php

We should default to require(), for uniformity, and also because it's faster.

Attachments (1)

wp-settings.patch (1.5 KB ) - added by scottbasgaard 16 years ago.: Replaced 7 instances of require_once() with require() in wp-settings.php.

Download all attachments as: .zip

Change History (6)

#1 @secondv
17 years ago

The time difference between require_once() and require() is very small, you'd only see a significant difference if there were hundreds of require_once calls.

However, I'd agree that it does seem to be used at random throughout wp-settings. It looks like everything uses require() except the new multisite code & locale stuff.