WordPress.org

Make WordPress Core

Opened 16 years ago

Closed 16 years ago

Last modified 15 years ago

#1427 closed defect (bug) (fixed)

Import files should always use require_once()

Reported by: chuyskywalker Owned by: markjaquith
Milestone: Priority: low
Severity: normal Version: 1.5.1.1
Component: Administration Keywords: import require plugin bg|has-patch bg|commit
Focuses: Cc:

Description

Some of the import files use require, while others use require_once. They should all be changed to require_once because if a plugin loads one of the files (as mine does one) then it conflicts and issues errors.

A diff (run agains the wp-admin folder) is attached.

Attachments (1)

import-require.diff (3.8 KB) - added by chuyskywalker 16 years ago.
diff against wp-admin to fix up all the require's to require_once's

Download all attachments as: .zip

Change History (5)

@chuyskywalker
16 years ago

diff against wp-admin to fix up all the require's to require_once's

#1 @matt
16 years ago

  • Milestone set to 1.5.2

#2 @markjaquith
16 years ago

  • Keywords bg|has-patch bg|commit added
  • Owner changed from anonymous to markjaquith
  • Status changed from new to assigned

Patch still works on 1.6 SVN... passing up the ladder

#3 @matt
16 years ago

  • Resolution set to fixed
  • Status changed from assigned to closed

(In [2687]) Update includes in import files, fixes #1427

#4 @(none)
15 years ago

  • Milestone 2.0 deleted

Milestone 2.0 deleted

Note: See TracTickets for help on using tickets.