Page MenuHomePhabricator

Add modern Unit/Lint field support
ClosedPublic

Authored by epriestley on Feb 27 2014, 6:19 PM.
Tags
None
Referenced Files
F14357476: D8360.diff
Fri, Dec 20, 5:18 AM
Unknown Object (File)
Tue, Dec 17, 11:13 AM
Unknown Object (File)
Mon, Dec 16, 7:04 PM
Unknown Object (File)
Fri, Dec 13, 3:10 AM
Unknown Object (File)
Wed, Dec 11, 2:06 AM
Unknown Object (File)
Wed, Dec 4, 4:35 PM
Unknown Object (File)
Wed, Dec 4, 12:41 PM
Unknown Object (File)
Tue, Dec 3, 8:38 PM
Subscribers

Details

Summary

Ref T2222. This is mostly copy/paste. No effect yet.

Test Plan

Looked at fields.

Diff Detail

Repository
rP Phabricator
Branch
dfield3
Lint
Lint Passed
Unit
Tests Passed

Event Timeline

btrahan added inline comments.
src/applications/differential/customfield/DifferentialUnitField.php
186–191

pht?

src/applications/differential/customfield/DifferentialLintField.php
37

These implementations need a cleanup pass in general (beyond missing pht(), this is bad too, e.g.), but I want to push through to running them as authoritative before mucking with them more than necessary.