Hashed Storage: `AfterRenameService` was not renaming Pages or Uploads folder on legacy storage
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?
Does this MR meet the acceptance criteria?
-
Changelog entry added, if necessary -
Documentation created/updated via this MR -
Documentation reviewed by technical writer or follow-up review issue created -
Tests added for this feature/bug -
Tested in all supported browsers -
Conforms to the code review guidelines -
Conforms to the merge request performance guidelines -
Conforms to the style guides -
Conforms to the database guides -
Link to e2e tests MR added if this MR has Requires e2e tests label. See the Test Planning Process. -
Security reports checked/validated by reviewer
Closes #56636 (closed)
Edited by Gabriel Mazetto