Skip to content

Hashed Storage: `AfterRenameService` was not renaming Pages or Uploads folder on legacy storage

Gabriel Mazetto requested to merge 56636-hashed-storage-afterrenameservice into master

What does this MR do?

Since after the refactored code in 4b9c17f1, we were not correctly renaming Pages nor Uploads shared folders, because the class was receiving the wrong path_before (it was receiving the value of new path as path_before):

def rename_transferred_documents
  if rename_uploads?
    Gitlab::UploadsTransfer
      .new
      .rename_project(path_before, project_path, namespace_full_path)
  end

  Gitlab::PagesTransfer
    .new
    .rename_project(path_before, project_path, namespace_full_path)
end

What are the relevant issue numbers?

#56636 (closed)

Does this MR meet the acceptance criteria?

Closes #56636 (closed)

Edited by Gabriel Mazetto

Merge request reports