Opened 2 years ago

Closed 21 months ago

Last modified 21 months ago

#15844 closed defect (bug) (fixed)

Prune dead code from when importers were in core.

Reported by: westi Owned by: westi
Priority: normal Milestone: 3.3
Component: Import Version: 3.1
Severity: normal Keywords: has-patch
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_ 2 years ago.

Download all attachments as: .zip

Change History (8)

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

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

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

duck_2 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.

comment:4   ryan2 years ago

  • Milestone changed from 3.2 to Future Release
  • Milestone changed from Future Release to 3.3
  • 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.

Good call on leaving validate_file().

Note: See TracTickets for help on using tickets.