codeninjasllc/discourse
1
0
Fork 0
mirror of https://github.com/codeninjasllc/discourse.git synced 2024-11-23 15:48:43 -05:00
Code Issues Projects Releases Packages Wiki Activity Actions

phpBB3 importer: fixes for internal links and guest users (#4350)

Browse source 
* phpBB3 importer: ignore query parameters in internal links

* phpBB3 importer: Don't fail if guest usernames differ only by case

* phpBB3 importer: Import username as name for guest users
This commit is contained in:
Gerhard Schlager 2016-07-27 22:57:33 +02:00 committed by Régis Hanol
parent fccc0c93c5
commit 37a78752e9
2 changed files with 3 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
script/import_scripts/phpbb3/importers/user_importer.rb
View file

@ -51,9 +51,9 @@ module ImportScripts::PhpBB3
{ {
id: username, id: username,
email: "anonymous_no_email_#{username}", email: "anonymous_no_email_#{SecureRandom.hex}",
username: username, username: username,
name: '', name: @settings.username_as_name ? username : '',
created_at: Time.zone.at(row[:first_post_time]), created_at: Time.zone.at(row[:first_post_time]),
active: true, active: true,
trust_level: TrustLevel[0], trust_level: TrustLevel[0],

2
script/import_scripts/phpbb3/support/text_processor.rb
View file

@ -137,7 +137,7 @@ module ImportScripts::PhpBB3
def create_internal_link_regexps(original_site_prefix) def create_internal_link_regexps(original_site_prefix)
host = original_site_prefix.gsub('.', '\.') host = original_site_prefix.gsub('.', '\.')
link_regex = "http(?:s)?://#{host}/viewtopic\\.php\\?(?:\\S*)(?:t=(\\d+)|p=(\\d+)(?:#p\\d+)?)" link_regex = "http(?:s)?://#{host}/viewtopic\\.php\\?(?:\\S*)(?:t=(\\d+)|p=(\\d+)(?:#p\\d+)?)(?:\\S*)"
@long_internal_link_regexp = Regexp.new(%Q|<!-- l --><a(?:.+)href="#{link_regex}"(?:.*)</a><!-- l -->|, Regexp::IGNORECASE) @long_internal_link_regexp = Regexp.new(%Q|<!-- l --><a(?:.+)href="#{link_regex}"(?:.*)</a><!-- l -->|, Regexp::IGNORECASE)
@short_internal_link_regexp = Regexp.new(link_regex, Regexp::IGNORECASE) @short_internal_link_regexp = Regexp.new(link_regex, Regexp::IGNORECASE)