Page MenuHomePhorge

Add Standard Custom Fields to Item List
ClosedPublic

Authored by avivey on Mar 1 2024, 16:26.
Tags
None
Referenced Files
F3299177: D25549.1743016443.diff
Tue, Mar 25, 19:14
F3289831: D25549.1742854190.diff
Sun, Mar 23, 22:09
F3289451: D25549.1742847584.diff
Sun, Mar 23, 20:19
F3283297: D25549.1742747385.diff
Sat, Mar 22, 16:29
F3269237: D25549.1742523728.diff
Thu, Mar 20, 02:22
F3249397: D25549.1742285954.diff
Mon, Mar 17, 08:19
F3248008: D25549.1742234841.diff
Sun, Mar 16, 18:07
F3246005: D25549.1742199433.diff
Sun, Mar 16, 08:17
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
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.