Editing revision D80358 results in Unhandled Exception ("PhabricatorEdgeCycleException")
Categories
(Conduit :: Phabricator, defect, P3)
Tracking
(Not tracked)
People
(Reporter: jya, Unassigned)
Details
(Keywords: conduit-triaged, conduit-upstream)
STR (until I can land this stack)
- Open https://phabricator.services.mozilla.com/D80358
- Click edit revision
- In the title, change P6 into P4.
- Click on Save Changes
Unhandled Exception ("PhabricatorEdgeCycleException")
Graph cycle detected (type=5, cycle=PHID-DREV-ta3ked5ca2nehs7of6ec, PHID-DREV-toebspkms2ftmw5h4sw7, PHID-DREV-ta3ked5ca2nehs7of6ec).
is displayed.
changes aren't saved.
The summary on the revision was just Depends on D80357. The stack had been manually reordered via the Phabricator UI positioning D80357 as D80358's child rather than parent.
Editing the revision caused Phabricator to parse the summary and try to reattach D80357 as the parent, resulting in a dependency loop.
Looks like clearing the summary while updating the title was the fix.
We should file an issue upstream to make that error message clearer: it should state clearly that it's the result of a dependency loop highlighting the relevant revisions.
Description
•