Make WordPress Core

Opened 11 years ago

Closed 10 years ago

Last modified 10 years ago

#13960 closed defect (bug) (fixed)

LiveJournal Importer broken in 3.0

Reported by: grahams Owned by: filosofo
Milestone: 3.1 Priority: normal
Severity: normal Version: 3.0
Component: Import Keywords: has-patch
Focuses: Cc:


I tried to import my LJ (grahams.livejournal.com) into a clean WordPress 3.0 installation. After inputting my LJ username and password, as well as a protected post password, I clicked the "Connect to Livejournal and Import" button. After it churned for awhile I got the following error:

Import LiveJournal

Warning: include(/var/www/wp3/wp-admin/import/livejournal.php) [function.include]: failed to open stream: No such file or directory in /var/www/wp3/wp-admin/admin-ajax.php on line 1378

Warning: include() [function.include]: Failed opening '/var/www/wp3/wp-admin/import/livejournal.php' for inclusion (include_path='.:/usr/share/php:/usr/share/pear') in /var/www/wp3/wp-admin/admin-ajax.php on line 1378

Fatal error: Call to a member function step1() on a non-object in /var/www/wp3/wp-admin/admin-ajax.php on line 1379

Attachments (2)

remove-livejournal-import-vestige.13960.diff (741 bytes) - added by filosofo 11 years ago.
ljimporter.diff (1.0 KB) - added by nacin 10 years ago.

Download all attachments as: .zip

Change History (13)

#1 @filosofo
11 years ago

Here's part of what needs to happen to fix this, on the core WP side. The remainder needs to happen with the Livejournal import plugin.

#2 @nacin
10 years ago

  • Milestone changed from Awaiting Review to WordPress.org site

#3 @filosofo
10 years ago

  • Keywords has-patch added
  • Milestone changed from WordPress.org site to 3.1
  • Owner set to filosofo
  • Status changed from new to assigned

Patch still applies. Can we at least get rid of that vestige LJ importer code in admin-ajax, please?

Then switch it back to WordPress.org.

#4 @nacin
10 years ago

Diff for the importer attached. Will commit there but I don't have anything to easily test this on and I don't feel like releasing a borked 0.3 version.

10 years ago

#5 @RyanMurphy
10 years ago

patched importer works for me

#6 @westi
10 years ago

  • Cc westi added

Going to commit this.

Tagged v0.20 in prep for updating and then Tagging v0.30.

It may be worth renaming the AJAX action so it doesn't clash with the vestige.

#7 @westi
10 years ago

@nacin: I have to say I think this line in your patch might not be needed ;-)

include( ABSPATH . 'wp-admin/import/livejournal.php' );

#8 @nacin
10 years ago

Indeed, that's just a copy-paste vestige from admin-ajax.

#9 @westi
10 years ago


Updated the Importer Plugin.

Testing != fun when you get blocked for hitting the API too much

#10 @westi
10 years ago

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

(In [16293]) Chop this off as we don't need it anymore. Fixes #13960 props filosofo.

#11 @westi
10 years ago

New plugin version is live.
Worked for me with my new test LJ site.

Note: See TracTickets for help on using tickets.