Page MenuHomePhabricator

Correct file PHID extraction on image update in Pholio
ClosedPublic

Authored by epriestley on May 30 2017, 6:52 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Jan 26, 5:52 PM
Unknown Object (File)
Sun, Jan 26, 5:55 AM
Unknown Object (File)
Thu, Jan 23, 12:40 AM
Unknown Object (File)
Wed, Jan 1, 6:44 PM
Unknown Object (File)
Dec 27 2024, 12:54 PM
Unknown Object (File)
Dec 26 2024, 6:30 PM
Unknown Object (File)
Dec 19 2024, 8:59 AM
Unknown Object (File)
Dec 9 2024, 3:28 PM
Subscribers
None

Details

Summary

Ref T12776. This extraction of file PHIDs extracted "PholioImage" object PHIDs (PHID-PIMG-...), not "File" PHIDs (PHID-FILE-...). Instead, dig into the Pholio images and actually extract the file PHIDs. This method is now similar to the PholioImageFileTransaction method, which works already.

Test Plan
  • Create a mock.
  • Update one of the images.
  • In Files, view the "Attached" tab of the updated image.
  • Before patch: not attached to mock.
  • After patch: properly attached to mock.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable