kanbanworkflowstimelinescrumrubyroadmapproject-planningproject-managementopenprojectangularissue-trackerifcgantt-chartganttbug-trackerboardsbcf
You can not select more than 25 topics
Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
69 lines
2.5 KiB
69 lines
2.5 KiB
class AddSlugToWikiPages < ActiveRecord::Migration[4.2]
|
|
def up
|
|
add_column :wiki_pages, :slug, :string
|
|
add_index :wiki_pages, [:wiki_id, :slug], name: 'wiki_pages_wiki_id_slug', unique: true
|
|
|
|
migrate_titles
|
|
|
|
# Disallow null values from now on
|
|
# Adding this above is impossible due to columns having NULL values before `migrate_titles`
|
|
change_column_null :wiki_pages, :slug, false
|
|
end
|
|
|
|
def down
|
|
remove_column :wiki_pages, :slug
|
|
rollback_titles
|
|
end
|
|
|
|
##
|
|
#
|
|
# Restore the old title wherever possible
|
|
# This tries to remove the slug usages without guaranteeing that links
|
|
# will be valid afterwards.
|
|
def rollback_titles
|
|
ActiveRecord::Base.transaction do
|
|
WikiPage.select(:id, :wiki_id, :title).find_each do |page|
|
|
# Undo the spaced title
|
|
old_title = page.title.gsub(' ', '_')
|
|
|
|
page.update_columns(title: old_title)
|
|
WikiRedirect.where(wiki_id: page.wiki_id, title: old_title).delete_all
|
|
end
|
|
end
|
|
end
|
|
|
|
##
|
|
# OpenProject < 6.0.0 processed wiki titles in a specific fashion (#titleize)
|
|
# replacing spaces with underscores and removing some characters.
|
|
# With OpenProject 6.0.0, titles were allowed to contain any chars.
|
|
# This causes issues since a title `foo bar` was saved as `foo_bar`, but displayed with spaces again
|
|
# and in turn, many page links actualy use [[foo bar]] instead of the actual title.
|
|
# Spaces were no longer matching the existing pages.
|
|
#
|
|
# Additionally, not all characters can be properly detected in the `parse_wiki_links` steps
|
|
# and this makes it impossible to link to some pages.
|
|
#
|
|
# To remove this issue, we introduce wiki slugs to create proper permalinks that can be linked
|
|
# to, and only fall back to finding by titles for the old pages.
|
|
def migrate_titles
|
|
# Create a redirect for all old titles
|
|
ActiveRecord::Base.transaction do
|
|
WikiPage.select(:id, :wiki_id, :title, :parent_id).find_each do |page|
|
|
# Restore the spaces in the old title (`pretty_title` in wiki.rb)
|
|
old_title = page.title
|
|
pretty_title = page.title.gsub('_', ' ')
|
|
|
|
# Save the title with spaces restored
|
|
# And generate the url slug
|
|
page.title = pretty_title
|
|
page.save!
|
|
|
|
# Generate a redirect from the old title to the slug
|
|
if page.title != page.slug
|
|
WikiRedirect.where(title: old_title, wiki_id: page.wiki_id).delete_all
|
|
WikiRedirect.create(title: old_title, wiki_id: page.wiki_id, redirects_to: page.slug)
|
|
end
|
|
end
|
|
end
|
|
end
|
|
end
|
|
|