Page MenuHomePhabricator
Differential D12456

Require TokenizerControl to always have a datasource
ClosedPublic
Actions

Authored by epriestley on Apr 18 2015, 3:43 PM.
Tags
None
Referenced Files
F13283050: D12456.diff
Sun, Jun 2, 12:45 PM
F13268166: D12456.diff
Wed, May 29, 4:43 AM
F13253668: D12456.diff
Sat, May 25, 2:50 AM
F13251451: D12456.id29899.diff
Fri, May 24, 11:31 PM
F13247322: D12456.diff
Thu, May 23, 6:12 PM
F13241585: D12456.id29952.diff
Wed, May 22, 10:33 PM
F13232409: D12456.diff
Tue, May 21, 1:15 AM
F13211170: D12456.diff
Fri, May 17, 5:32 AM
View All Files
Subscribers
epriestley

Details

Reviewers
btrahan
Maniphest Tasks
T4100: Support query parameterization in typeaheads
Commits
Restricted Diffusion Commit
rPa11dab59b0d5: Require TokenizerControl to always have a datasource
Summary

Ref T4100. I can simplify the logic a bit here by moving some rendering into the datasources, but a few TokenizerControls currently don't have datasources.

Require datasources and always provide datasources.

Test Plan
  • Used previously-datasourceless controls (e.g., "Add Reviewers").
  • Used normal controls.
  • Manually verified that no other controls are missing datasources.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 29899.Apr 18 2015, 3:43 PM
epriestley retitled this revision from to Require TokenizerControl to always have a datasource.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
epriestley added a task: T4100: Support query parameterization in typeaheads.
Herald added a subscriber: epriestley. · View Herald TranscriptApr 18 2015, 3:43 PM
btrahan accepted this revision.Apr 20 2015, 4:23 PM
btrahan edited edge metadata.
This revision is now accepted and ready to land.Apr 20 2015, 4:23 PM
Closed by commit rPa11dab59b0d5: Require TokenizerControl to always have a datasource (authored by epriestley, committed by epriestley). · Explain WhyApr 20 2015, 5:14 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
differential/
view/
12 lines
diffusion/
controller/
12 lines
maniphest/
controller/
17 lines
view/
form/
control/
42 lines

Diff 29952

View Options

src/applications/differential/view/DifferentialAddCommentView.php

Loading...
View Options

src/applications/diffusion/controller/DiffusionCommitController.php

Loading...
View Options

src/applications/maniphest/controller/ManiphestTaskDetailController.php

Loading...
View Options

src/view/form/control/AphrontFormTokenizerControl.php

Loading...
Phabricator · Built by Phacility