1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2025-02-26 21:49:08 +01:00
phorge-phorge/src
epriestley 17f092307c Fix an issue where Phriction moves to new locations would fail with a "content required" error
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.

Reviewers: chad, amckinley

Reviewed By: amckinley

Maniphest Tasks: T12793

Differential Revision: https://secure.phabricator.com/D18069
2017-06-02 16:27:24 -07:00
..
__tests__ Use PhutilClassMapQuery instead of PhutilSymbolLoader 2015-08-14 07:49:01 +10:00
aphront Improve the behavior of PhabricatorFileEditField for Macros 2017-05-08 16:23:12 -07:00
applications Fix an issue where Phriction moves to new locations would fail with a "content required" error 2017-06-02 16:27:24 -07:00
docs Diffusion import documentation update 2017-05-26 08:18:40 -07:00
extensions Add src/extensions/ to Phabricator 2013-08-14 15:38:06 -07:00
infrastructure Group query results under the "ANCESTOR" operator unconditionally 2017-05-24 13:29:25 -07:00
view Clean up some PHUI/PHUIX button behaviors 2017-06-02 08:37:52 -07:00
__phutil_library_init__.php
__phutil_library_map__.php Add PHUIXButtonView and a UIExample 2017-05-30 18:01:16 -07:00