Page MenuHomephabricator

File move support
Needs ReviewPublic

Authored by kerberizer on Nov 1 2018, 9:18 AM.
This revision needs review, but there are no reviewers specified.

Details

Reviewers
None
Summary

[WiP] While it may happen rather infrequently, we need to support move operations. This patch focuses on move support from Phabricator to Wikipedia, which is more important. The reason is that in Wikipedia we care about the page histories, so when a file is moved in the Git repo, we'd like the respective page to be moved too, instead of being created anew with the new name, while the old gets deleted.

Test Plan

Once the code is finalized, it will be tested the usual way.

Diff Detail

Repository
rRSYNC Repository Synchronization
Branch
enh/movesupport
Lint
Lint OK
Unit
No Unit Test Coverage
Build Status
Buildable 5
Build 5: arc lint + arc unit

Event Timeline

kerberizer requested review of this revision.Nov 1 2018, 9:18 AM
kerberizer created this revision.
Harbormaster completed remote builds in B5: Diff 6.Nov 1 2018, 9:18 AM
kerberizer updated this revision to Diff 7.Nov 1 2018, 9:32 AM

reversed() is a generator

Harbormaster completed remote builds in B6: Diff 7.Nov 1 2018, 9:32 AM