Make WordPress Core

Opened 14 years ago

Closed 13 years ago

Last modified 13 years ago

#15844 closed defect (bug) (fixed)

Prune dead code from when importers were in core.

Reported by: westi's profile westi Owned by: westi's profile westi
Milestone: 3.3 Priority: normal
Severity: normal Version: 3.1
Component: Import Keywords: has-patch
Focuses: Cc:

Description

In wp-admin/import we have:

// Load all importers so that they can register.
$import_loc = 'wp-admin/import';
$import_root = ABSPATH.$import_loc;
$imports_dir = @ opendir($import_root);
if ($imports_dir) {
	while (($file = readdir($imports_dir)) !== false) {
		if ($file[0] == '.') {
			continue;
		} elseif (substr($file, -4) == '.php') {
			require_once($import_root . '/' . $file);
		}
	}
}
@closedir($imports_dir);

But we don't have that folder any more.

We should rip the code out so we don't have unnecessary suppressed notices.

Attachments (1)

15844.diff (1.8 KB) - added by duck_ 14 years ago.

Download all attachments as: .zip

Change History (8)

#1 @westi
14 years ago

  • Keywords 3.2-early added
  • Milestone changed from 3.1 to Future Release

Also need to take code out of admin.php too.

#2 @westi
14 years ago

  • Keywords 3.2-early removed
  • Milestone changed from Future Release to 3.2

@duck_
14 years ago

#3 @duck_
14 years ago

  • Keywords has-patch added

The validate_file call in admin.php doesn't look necessary now, so we might like to remove it as well.

#4 @ryan
14 years ago

  • Milestone changed from 3.2 to Future Release

#5 @SergeyBiryukov
13 years ago

  • Milestone changed from Future Release to 3.3

#6 @duck_
13 years ago

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

In [18640]:

Pull out old import remnant from when importers were shipped in core. Fixes #15844.

#7 @nacin
13 years ago

Good call on leaving validate_file().

Note: See TracTickets for help on using tickets.