Page MenuHomePhabricator

Consolidate handling of special properties for newly uploaded files
ClosedPublic

Authored by epriestley on Aug 11 2014, 3:41 PM.
Tags
None
Referenced Files
F15395862: D10216.id24579.diff
Sun, Mar 16, 10:14 AM
F15352689: D10216.id24586.diff
Mon, Mar 10, 5:47 PM
Unknown Object (File)
Feb 7 2025, 10:24 PM
Unknown Object (File)
Feb 7 2025, 10:24 PM
Unknown Object (File)
Feb 7 2025, 10:24 PM
Unknown Object (File)
Feb 7 2025, 10:24 PM
Unknown Object (File)
Feb 2 2025, 10:21 AM
Unknown Object (File)
Jan 26 2025, 8:18 PM
Subscribers

Details

Summary

Fixes T5849. When a new file is created, we might have to actually write the data to a storage engine, or we might be able to just point at data which is already there.

Currently, these two paths handle $params with different code and mild behavioral differences. Instead, have them call the same code so they get the same behavior.

Test Plan
  • Uploaded the same file multiple times to home page.
  • Uploaded the same file multiple times as profile picture.
  • Generated files via Diffusion.
  • All the files got the expected properties, whether they were reusing data or not.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley retitled this revision from to Consolidate handling of special properties for newly uploaded files.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added reviewers: 20after4, btrahan.
20after4 edited edge metadata.

This looks good, and a lot cleaner.

This revision is now accepted and ready to land.Aug 11 2014, 4:29 PM
epriestley updated this revision to Diff 24586.

Closed by commit rPeb9dcd6fbaf5 (authored by @epriestley).