Page MenuHomePhabricator

Use CustomField, not AuxiliaryField, to power RevisionView
ClosedPublic

Authored by epriestley on Feb 27 2014, 6:27 PM.
Tags
None
Referenced Files
F14953942: D8361.diff
Sun, Feb 16, 12:27 AM
Unknown Object (File)
Sat, Feb 15, 9:35 AM
Unknown Object (File)
Tue, Feb 11, 1:47 PM
Unknown Object (File)
Sun, Feb 9, 11:41 AM
Unknown Object (File)
Sun, Feb 9, 11:41 AM
Unknown Object (File)
Sun, Feb 9, 11:41 AM
Unknown Object (File)
Sun, Feb 9, 11:41 AM
Unknown Object (File)
Wed, Feb 5, 9:53 AM
Subscribers

Details

Summary

Ref T2222. This will probably have some rough edges for a bit (e.g., weird cases I didn't remember or think of), but there's no change to the underlying data and we can easily revert if things get too messy.

Test Plan

Looked at a variety of revisions and saw sensible output.

Diff Detail

Repository
rP Phabricator
Branch
dfield1
Lint
Lint Passed
Unit
Tests Passed

Event Timeline

epriestley updated this revision to Unknown Object (????).Feb 27 2014, 6:30 PM
  • Fix a spelling issue.
  • Remove an unused property.
conf/default.conf.php
790

As far as I know, only Facebook uses this. The modern event-based version has been available for something like 18 months.