Opened 13 years ago
Closed 13 years ago
#1523 closed task (blessed) (fixed)
Importers
Reported by: | GautamGupta | Owned by: | |
---|---|---|---|
Milestone: | 2.0 | Priority: | high |
Severity: | major | Version: | |
Component: | API - Installation/Upgrade | Keywords: | |
Cc: |
Description
We should include importers in the core. We could start with bbPress standalone. Attaching a version of a very prototype thing I've done. I'll likely complete it till tomorrow if someone doesn't beat me to it. :)
Attachments (5)
Change History (19)
#1
@
13 years ago
Added another patch. The only problem I'm facing is that $bbdb
isn't getting set. It is not becoming a db object and hence the import can not be carried on. Didn't dig deep into it.
#2
@
13 years ago
Try adding do_action( 'bbpress_init' );
to the functions that need the $bbdb global. You can see it done in BP if you need more help (http://buddypress.trac.wordpress.org/browser/trunk/bp-forums/bp-forums-functions.php#L268).
#3
@
13 years ago
The importer contains codes from BuddyPress and LiveJournal Importer. The only problem right now is that the recounts for topics & replies in forum are failing. I'd also like if someone could test out pingback import.
#5
in reply to:
↑ 4
;
follow-up:
↓ 7
@
13 years ago
Replying to Nightgunner5:
Tested, as requested
You actually need to submit the path with bb-config.php
. Added a safe-guard now. :)
#6
@
13 years ago
Ok, just noticed that we actually ask for the path of the install. Even though we've added a safe guard, we should change that message. :)
#7
in reply to:
↑ 5
@
13 years ago
Replying to GautamGupta:
Replying to Nightgunner5:
Tested, as requested
You actually need to submit the path with
bb-config.php
. Added a safe-guard now. :)
I did. It gives exactly the same error message with or without a trailing slash.
#8
@
13 years ago
The line 517
in bbpress.php
should be preventing from that to happen. Are you sure that you're using import.2.diff
?
#9
@
13 years ago
define( 'BB_PATH', trailingslashit( dirname( $this->bb_path ) ) );
will always take a directory above the one inputted.
#11
@
13 years ago
On line 144 in importers/bbpress.php
, we're checking for BuddyPress. I think we should do that in bbPress_Importer::autolocate_bb_path()
because the user might want to import a non-BuddyPress bbPress standalone install too.
#12
@
13 years ago
Attached a patch with improvements. I think it still doesn't address the freshness issue. That should be fixed by recount anyways - still, it doesn't.
Please take a look at @todos