Page MenuHomePhorge

Add Standard Custom Fields to Item List
ClosedPublic

Authored by avivey on Mar 1 2024, 16:26.
Tags
None
Referenced Files
F2330308: D25549.1722070195.diff
Fri, Jul 26, 08:49
Unknown Object (File)
Fri, Jul 26, 02:50
Unknown Object (File)
Wed, Jul 24, 03:51
Unknown Object (File)
Wed, Jul 24, 03:51
Unknown Object (File)
Sat, Jul 20, 18:38
Unknown Object (File)
Fri, Jul 19, 08:28
Unknown Object (File)
Fri, Jul 19, 07:53
Unknown Object (File)
Thu, Jul 18, 19:03
Tokens
"Love" token, awarded by aklapper."Mountain of Wealth" token, awarded by 20after4.

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
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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.