Skip to:
Content

bbPress.org

Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#1892 closed defect (bug) (fixed)

Invision Importer Replies Broken

Reported by: xiosen's profile xiosen Owned by:
Milestone: 2.1.1 Priority: normal
Severity: normal Version: 2.1
Component: API - Importers Keywords: has-patch
Cc: stephen@…

Description

Just tried to use the importer to convert invision board data. Right now it fails to convert any of the replies (well actually it only converts the topic as a reply, thus duplicating the topic). This is due to it looking for a new topic in the posts table versus just the replies.

To fix change line 165 in Invision.php from:

		$this->field_map[] = array(
			'from_tablename' => 'posts', 'from_fieldname' => 'pid', 'from_expression' => 'WHERE posts.new_topic = 1',
			'to_type' => 'reply', 'to_fieldname' => '_bbp_post_id'
		);

to:

		$this->field_map[] = array(
			'from_tablename' => 'posts', 'from_fieldname' => 'pid', 'from_expression' => 'WHERE posts.new_topic = 0',
			'to_type' => 'reply', 'to_fieldname' => '_bbp_post_id'
		);

Attachments (2)

invision.diff (535 bytes) - added by netweb 11 years ago.
Invision.php (11.2 KB) - added by netweb 11 years ago.

Download all attachments as: .zip

Change History (5)

@netweb
11 years ago

@netweb
11 years ago

#1 @netweb
11 years ago

  • Cc stephen@… added
  • Keywords has-patch added
  • Milestone changed from Awaiting Review to 2.1.1

#2 @johnjamesjacoby
11 years ago

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

(In [4085]) Converter:

  • Fix issue with Invision converter causing replies not to be converted correctly.
  • Props netweb.
  • Fixes #1892 (2.2 branch)

#3 @johnjamesjacoby
11 years ago

(In [4086]) Converter:

  • Fix issue with Invision converter causing replies not to be converted correctly.
  • Props netweb.
  • Fixes #1892 (2.1 branch)
Note: See TracTickets for help on using tickets.