migration without using old field name

removing old field from db, after bookmarks migration
This commit is contained in:
Alex S 2019-04-22 12:45:45 +07:00
parent 3c2ae80008
commit 229ce6abbc
2 changed files with 10 additions and 12 deletions

View File

@ -55,8 +55,6 @@ defmodule Pleroma.User do
field(:search_type, :integer, virtual: true) field(:search_type, :integer, virtual: true)
field(:tags, {:array, :string}, default: []) field(:tags, {:array, :string}, default: [])
field(:last_refreshed_at, :naive_datetime_usec) field(:last_refreshed_at, :naive_datetime_usec)
# TODO: add migration to delete `bookmarks` field from DB
field(:old_bookmarks, {:array, :string}, default: [], source: :bookmarks)
has_many(:bookmarks, Bookmark) has_many(:bookmarks, Bookmark)
has_many(:notifications, Notification) has_many(:notifications, Notification)
has_many(:registrations, Registration) has_many(:registrations, Registration)

View File

@ -6,24 +6,24 @@ defmodule Pleroma.Repo.Migrations.MigrateOldBookmarks do
alias Pleroma.User alias Pleroma.User
alias Pleroma.Repo alias Pleroma.Repo
def up do def change do
query = query =
from(u in User, from(u in User,
where: u.local == true, where: u.local == true,
where: fragment("array_length(?, 1)", u.old_bookmarks) > 0, where: fragment("array_length(bookmarks, 1)") > 0,
select: %{id: u.id, old_bookmarks: u.old_bookmarks} select: %{id: u.id, bookmarks: fragment("bookmarks")}
) )
Repo.stream(query) Repo.stream(query)
|> Enum.each(fn user -> |> Enum.each(fn %{id: user_id, bookmarks: bookmarks} ->
Enum.each(user.old_bookmarks, fn id -> Enum.each(bookmarks, fn ap_id ->
activity = Activity.get_create_by_object_ap_id(id) activity = Activity.get_create_by_object_ap_id(ap_id)
{:ok, _} = Bookmark.create(user.id, activity.id) {:ok, _} = Bookmark.create(user_id, activity.id)
end) end)
end) end)
end
def down do alter table(:users) do
execute("TRUNCATE TABLE bookmarks") remove(:bookmarks)
end
end end
end end