Page MenuHomePhorge

Add Standard Custom Fields to Item List
ClosedPublic

Authored by avivey on Mar 1 2024, 16:26.

Details

Summary

Allow "Standard" (config-based") custom fields to be displayed in search-results.

Depends on D25548. Ref T15750.

Test Plan

Set maniphest.custom-field-definitions to the value of P32, and start playing with custom values on tasks.

Diff Detail

Repository
rP Phorge
Branch
custom-list-2
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 1090
Build 1090: arc lint + arc unit

Event Timeline

avivey held this revision as a draft.
avivey edited the test plan for this revision. (Show Details)
avivey published this revision for review.Mar 1 2024, 16:48

Here's what it looks like with all the bells:

image.png (578×743 px, 63 KB)

If you're stylistically aligned, I'm sorry for this.

I'll test this locally, I can't see any reason we shouldn't merge it.

For some reasons I cannot cleanly apply the patch anymore.

I get:

error: while searching for:
    return $indexes;
  }

  public function renderPropertyViewValue(array $handles) {
    $value = $this->getFieldValue();

    if (!strlen($value)) {

error: patch failed: src/infrastructure/customfield/standard/PhabricatorStandardCustomFieldLink.php:18

Uh! I like this.

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

I see that the addIcon() also supports a addIcon($value, $label) and it's nice.

Maybe useful to somehow expose that label.

This revision is now accepted and ready to land.May 18 2024, 08:35
src/infrastructure/customfield/standard/PhabricatorStandardCustomField.php
376

I think we can do that at a later date, by letting list.icon be either a simple string (icon name) or an object. I'll put a note in T15750 and in the code.