Make WordPress Core

Opened 13 years ago

Closed 12 years ago

#15737 closed defect (bug) (fixed)

Blogger Importer notices

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


I ran the Blogger Importer today on a blog with WP_DEBUG on and encountered too many notices, so I fixed them. Patch coming

Attachments (1)

blogger-importer.patch (2.5 KB) - added by joostdevalk 13 years ago.

Download all attachments as: .zip

Change History (4)

#1 @nacin
13 years ago

  • Milestone changed from Awaiting Review to

#2 @Workshopshed
12 years ago

Hi Joost,
I've got a working version of the importer based on Otto's OAuth fork. I also spotted the notices about unassigned variables.

I'm trying to understand part of your patch, I've changed get_user_options function to use get_users as follows.

function get_user_options($current) {
global $importer_users;
if ( ! isset( $importer_users ) )
        $importer_users = (array) get_users();  //Function: get_users_of_blog() Deprecated in version 3.1. Use get_users() instead.

        $options = '';

	foreach ( $importer_users as $user ) {
		$sel = ( $user->ID == $current ) ? " selected='selected'" : '';
		$options .= "<option value='$user->ID'$sel>$user->display_name</option>";
	return $options;

You've also change $user->ID to $user->data->ID, is that still relavent now that the function has been swapped out?

#3 @Otto42
12 years ago

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

With all the changes in 0.5, this patch should no longer be relevant. However, if there's any continuing notice issues, feel free to make a new ticket about it.

Note: See TracTickets for help on using tickets.