Page MenuHomePhabricator

Support Custom Fields in List View
Needs ReviewPublic

Authored by avivey on Dec 7 2016, 12:54 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Mar 25, 6:19 AM
Unknown Object (File)
Sat, Mar 2, 7:26 AM
Unknown Object (File)
Feb 11 2024, 7:10 AM
Unknown Object (File)
Feb 7 2024, 4:46 AM
Unknown Object (File)
Feb 6 2024, 2:53 AM
Unknown Object (File)
Feb 3 2024, 1:21 PM
Unknown Object (File)
Jan 17 2024, 4:46 AM
Unknown Object (File)
Dec 28 2023, 1:20 PM

Details

Summary

Ref T418.
This adds values list, list.icon, list.label as described in T418#55775, and supports them in
Maniphest and in Differential.

Also modernized a bit the code that I actually went through wrt handles.

P2022 is an example of a custom Differential field that shows on the list.

Test Plan

pasted_file (651×664 px, 75 KB)

pasted_file (482×388 px, 28 KB)

Camera icon comes from "f1" field (Type text), and the orange USB is a boolean field.

Event Timeline

avivey retitled this revision from to Support Custom Fields in List View.
avivey updated this object.
avivey edited the test plan for this revision. (Show Details)
avivey added reviewers: epriestley, chad.

test plan seems dubious

avivey edited edge metadata.
  • Remove a blank line
  • Allow users to shoot themselves in foot
src/infrastructure/customfield/standard/PhabricatorStandardCustomFieldRemarkup.php
37–39

Oh, using this on the list page might have a few more queries...

avivey added inline comments.
src/infrastructure/customfield/field/PhabricatorCustomField.php
1313

getRequiredHandlePHIDsForListView

src/infrastructure/customfield/field/PhabricatorCustomFieldList.php
211–215

this can be one loop

src/infrastructure/customfield/standard/PhabricatorStandardCustomField.php
321

should be called getValueForListItem.

avivey marked 3 inline comments as done.
  • some minor updates

I believe enabling this will add ~100-200 queries to the page and it's basically a nonstarter until we have a fix for that.

This revision now requires changes to proceed.Jan 11 2017, 3:27 PM
avivey edited edge metadata.
  • use batch-loading system from D16351 to load fields.
  • update documentation.
  • hard-code fa- prefix for icons.

@epriestley: I think this one should actually be in Request Review state?

This revision now requires changes to proceed.May 4 2017, 8:30 PM

Do "Request Review" again? I think this got stuck in a questionable state during a brief bug with T11114, but kicking it back and forth should clear things.

avivey requested review of this revision.May 4 2017, 9:30 PM
avivey edited edge metadata.