Page MenuHomePhabricator

Add modern Unit/Lint field support
ClosedPublic

Authored by epriestley on Feb 27 2014, 6:19 PM.
Tags
None
Referenced Files
F13322110: D8360.diff
Fri, Jun 14, 7:36 AM
F13287442: D8360.diff
Tue, Jun 4, 8:47 AM
F13274861: D8360.diff
Fri, May 31, 4:08 AM
F13271158: D8360.id19873.diff
Wed, May 29, 5:02 PM
F13260215: D8360.diff
Sun, May 26, 11:37 PM
F13259474: D8360.id.diff
Sun, May 26, 9:03 PM
F13258676: D8360.id19879.diff
Sun, May 26, 11:49 AM
F13256677: D8360.id19873.diff
Sat, May 25, 3:52 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.