Page MenuHomePhabricator

Add modern Unit/Lint field support
ClosedPublic

Authored by epriestley on Feb 27 2014, 6:19 PM.
Tags
None
Referenced Files
F13362958: D8360.id19879.diff
Wed, Jun 26, 1:18 AM
F13358418: D8360.diff
Tue, Jun 25, 7:39 AM
F13355236: D8360.diff
Mon, Jun 24, 10:13 AM
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
May 26 2024, 11:37 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.