Page MenuHomePhabricator

Fix file PHID extraction in Owners and Differential
ClosedPublic

Authored by epriestley on Nov 16 2015, 6:02 PM.
Tags
None
Referenced Files
F14038657: D14493.diff
Mon, Nov 11, 1:05 AM
F14029995: D14493.id35077.diff
Fri, Nov 8, 10:51 PM
F14029994: D14493.id.diff
Fri, Nov 8, 10:51 PM
F14024851: D14493.diff
Thu, Nov 7, 12:48 PM
F14001475: D14493.diff
Fri, Oct 25, 8:25 AM
F13976492: D14493.id.diff
Oct 18 2024, 2:31 PM
F13974900: D14493.id35063.diff
Oct 18 2024, 7:52 AM
F13973600: D14493.id35077.diff
Oct 18 2024, 1:39 AM
Subscribers

Details

Summary

Ref T9787. To fix this, I want to change how file PHIDs are extracted slightly: specifically, I'm going to extract them later in the editing process.

Before doing this, clean up a couple of bad implementations:

  • Owners extracts its description as a file PHID. This is an error.
    • Extract the description as a remarkup block instead.
    • Add an edge table so stuff like file attachment works properly.
  • Differential has a no-op extract method. This is presumably just a copy/paste issue from long ago.
Test Plan
  • Edited a revision in Differential.
  • Dropped a file into the description of an Owners package.
    • Before change: this did not attach the file.
    • After change: the file now attaches properly and shows up as "Attached" in the file details.

Diff Detail

Repository
rP Phabricator
Branch
eengine14
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 8873
Build 10380: Run Core Tests
Build 10379: arc lint + arc unit

Event Timeline

epriestley retitled this revision from to Fix file PHID extraction in Owners and Differential.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
This revision is now accepted and ready to land.Nov 16 2015, 7:20 PM
This revision was automatically updated to reflect the committed changes.