Homec4science

Fix Phriction document move on to existing document placeholder

Authored by Giedrius Dubinskas <gd@uber.com> on Oct 18 2016, 13:58.

Description

Fix Phriction document move on to existing document placeholder

Summary:
Looks like the logic was there already but some minor parts were missing.
Fixes T8082.

Test Plan:

  • Create document /w/foo
  • Delete document /w/foo
  • Create document /w/bar
  • Move document /w/bar for /w/foo

No error was displayed and document /w/bar was moved to /w/foo.

Reviewers: epriestley, #blessed_reviewers

Reviewed By: epriestley, #blessed_reviewers

Subscribers: Korvin

Maniphest Tasks: T8082

Differential Revision: https://secure.phabricator.com/D16713

Details

Committed
gd <autocommitter@example.com>Oct 18 2016, 13:58
Pushed
aubortJan 31 2017, 17:16
Parents
rPHdad17fb98a6b: Make "metamta.differential.inline-patches" imply a reasonable byte limit, not…
Branches
Unknown
Tags
Unknown

Event Timeline

gd <autocommitter@example.com> committed rPH4e831e786e7a: Fix Phriction document move on to existing document placeholder (authored by Giedrius Dubinskas <gd@uber.com>).Oct 18 2016, 13:58