Page MenuHomePhabricator

Fix an issue where Phriction moves to new locations would fail with a "content required" error
ClosedPublic

Authored by epriestley on Jun 2 2017, 11:14 PM.
Tags
None
Referenced Files
F19831682: D18069.id43459.diff
Mar 10 2026, 8:18 PM
F19822111: D18069.id43460.diff
Mar 7 2026, 2:53 AM
F19822109: D18069.id43460.diff
Mar 7 2026, 2:52 AM
F19772180: D18069.diff
Feb 20 2026, 2:46 PM
F19686623: D18069.id.diff
Feb 9 2026, 1:12 PM
F19637890: D18069.diff
Feb 5 2026, 3:26 PM
F19303256: D18069.id.diff
Dec 23 2025, 9:58 PM
F18836770: D18069.diff
Oct 27 2025, 3:36 AM
Subscribers
None

Details

Summary

Ref T12793. I'd like to understand exactly when we broke this, but this seems to be a minimal fix that shouldn't do anything surprising.

When you move document /a/ to /a/b/ and that path doesn't exist yet, the Content transaction currently fails because there's "no content". The content gets added later by the "move" transaction but this is implicit.

To make this work, just ignore the "missing field" error. This is a little roundabout but unlikely to break anything in weird ways.

Test Plan
  • Moved document /a/b/ to /a/b/c/.
  • Before patch: error about missing content.
  • After patch: move worked properly.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable