This would be a really useful feature.

◀ Merged tasks: T5887.

◀ Merged tasks: T5950.

Without this, the excel export feature is unfortunately of no use for us here, since we need the information in custom fields for reporting purposes.
Specifically this concerns estimation and due date related information.

Could anybody give me a short code-snippet which shows how to get the value of a custom-field of a task? So I'm able to implement my own export-format in the meanwhile.

maniphest.info from conduit will give you a bunch of json including

"auxiliary"          : {
  "std:maniphest:mycorp:agile-buzzword" : "mycorp:story"
},

See the conduit docs for a little more info on how to use the api to build your own thing (which you will then have to support).

In T5391#78484, @cburroughs wrote:

maniphest.info from conduit will give you a bunch of json including

"auxiliary"          : {
  "std:maniphest:mycorp:agile-buzzword" : "mycorp:story"
},

See the conduit docs for a little more info on how to use the api to build your own thing (which you will then have to support).

Sorry for this misunderstanding. I meant the PHP API to get the stored value of a custom-field of a task, assuming I've already the task object fetched. To be more precisely I'm talking about how to use the class [ManiphestCustomField](https://secure.phabricator.com/book/phabdev/class/ManiphestCustomField/).

Problem solved:

$custom_fields = PhabricatorCustomField::getObjectFields(
        $task,
        PhabricatorCustomField::ROLE_VIEW);
$custom_fields
        ->setViewer($user)
        ->readFieldsFromStorage($task);

$custom_fields->getFields()['std:maniphest:company:field']->getValueForStorage();

is T5954 really a fully blocking Task of this feature? Forgive my ignorance as I haven't worked on or even looked at the codebase, but it seems that adding all task fields as additional columns to an Excel download would be relatively trivial?

If you don't want them they are easily deleted afterwards.

The API shows them as under an "auxiliary" key. Which would make it rather simple for us to do it from there, but it leaves us having to create a small App alongside Phabricator, specifically for Project managers to produce queries to download CSV/Excel from. Which isn't as tidy an implementation.

is T5954 really a fully blocking Task of this feature?

Yes.

When we could build a feature today, but know we would have to undo that implementation after planned work tomorrow, we wait until tomorrow.

You're welcome to do the work today locally by forking, if you believe the value of having those columns between now and the time we complete T5954 exceeds the cost of the work in your situation.

In T5391#78626, @witrin wrote:

Problem solved:

$custom_fields = PhabricatorCustomField::getObjectFields(
        $task,
        PhabricatorCustomField::ROLE_VIEW);
$custom_fields
        ->setViewer($user)
        ->readFieldsFromStorage($task);

$custom_fields->getFields()['std:maniphest:company:field']->getValueForStorage();

Where u put that code (Filename) ?

I was almost a week using Phabricator i'm a totally newbie

@camilorojas I think I just added the following file

<?php

final class ManiphestExcelCustomFormat extends ManiphestExcelFormat {

  public function getName() {
    return pht('Custom');
  }

  public function getFileName() {
    return 'maniphest_tasks_'.date('Ymd');
  }

  /**
   * @phutil-external-symbol class PHPExcel
   * @phutil-external-symbol class PHPExcel_IOFactory
   * @phutil-external-symbol class PHPExcel_Style_NumberFormat
   * @phutil-external-symbol class PHPExcel_Cell_DataType
   */
  public function buildWorkbook(
    PHPExcel $workbook,
    array $tasks,
    array $handles,
    PhabricatorUser $user) {

    $sheet = $workbook->setActiveSheetIndex(0);
    $sheet->setTitle(pht('Tasks'));

    $widths = array(
      null,
      15,
      null,
      10,
      15,
      15,
      60,
      30,
      20,
      100,
    );

    foreach ($widths as $col => $width) {
      if ($width !== null) {
        $sheet->getColumnDimension($this->col($col))->setWidth($width);
      }
    }

    $status_map = ManiphestTaskStatus::getTaskStatusMap();
    $pri_map = ManiphestTaskPriority::getTaskPriorityMap();

    $date_format = null;

    $rows = array();
    $rows[] = array(
      pht('ID'),
      pht('Owner'),
      pht('Status'),
      pht('Priority'),
      pht('Date Created'),
      pht('Date Updated'),
      pht('Title'),
      pht('Projects'),
      pht('URI'),
      pht('Description'),
      pht('Custom Field'),
    );

    $is_date = array(
      false,
      false,
      false,
      false,
      true,
      true,
      false,
      false,
      false,
      false,
      false,
    );

    $header_format = array(
      'font'  => array(
        'bold' => true,
      ),
    );

    foreach ($tasks as $task) {
      $task_owner = null;
      if ($task->getOwnerPHID()) {
        $task_owner = $handles[$task->getOwnerPHID()]->getName();
      }

      $custom_fields = PhabricatorCustomField::getObjectFields(
        $task,
        PhabricatorCustomField::ROLE_VIEW);
      $custom_fields
        ->setViewer($user)
        ->readFieldsFromStorage($task);
      $custom_fields = $custom_fields->getFields();

      $projects = array();
      foreach ($task->getProjectPHIDs() as $phid) {
        $projects[] = $handles[$phid]->getName();
      }
      $projects = implode(', ', $projects);

      $rows[] = array(
        'T'.$task->getID(),
        $task_owner,
        idx($status_map, $task->getStatus(), '?'),
        idx($pri_map, $task->getPriority(), '?'),
        $this->computeExcelDate($task->getDateCreated()),
        $this->computeExcelDate($task->getDateModified()),
        $task->getTitle(),
        $projects,
        PhabricatorEnv::getProductionURI('/T'.$task->getID()),
        phutil_utf8_shorten($task->getDescription(), 512),
        $custom_fields['std:maniphest:vendor:field']->getValueForStorage(),
      );
    }

    foreach ($rows as $row => $cols) {
      foreach ($cols as $col => $spec) {
        $cell_name = $this->col($col).($row + 1);
        $cell = $sheet
          ->setCellValue($cell_name, $spec, $return_cell = true);

        if ($row == 0) {
          $sheet->getStyle($cell_name)->applyFromArray($header_format);
        }

        if ($is_date[$col]) {
          $code = PHPExcel_Style_NumberFormat::FORMAT_DATE_YYYYMMDD2;
          $sheet
            ->getStyle($cell_name)
            ->getNumberFormat()
            ->setFormatCode($code);
        } else {
          $cell->setDataType(PHPExcel_Cell_DataType::TYPE_STRING);
        }
      }
    }
  }

  private function col($n) {
    return chr(ord('A') + $n);
  }

}

Don't forget to update your class map:

$ cd webroot/phabricator
$ ../arcanist/bin/arc liberate src

In T5391#167083, @witrin wrote:

@camilorojas I think I just added the following file

src/applications/maniphest/export/ManiphestExcelCustomFormat.php

<?php

final class ManiphestExcelCustomFormat extends ManiphestExcelFormat {

  public function getName() {
    return pht('Custom');
  }

  public function getFileName() {
    return 'maniphest_tasks_'.date('Ymd');
  }

  /**
   * @phutil-external-symbol class PHPExcel
   * @phutil-external-symbol class PHPExcel_IOFactory
   * @phutil-external-symbol class PHPExcel_Style_NumberFormat
   * @phutil-external-symbol class PHPExcel_Cell_DataType
   */
  public function buildWorkbook(
    PHPExcel $workbook,
    array $tasks,
    array $handles,
    PhabricatorUser $user) {

    $sheet = $workbook->setActiveSheetIndex(0);
    $sheet->setTitle(pht('Tasks'));

    $widths = array(
      null,
      15,
      null,
      10,
      15,
      15,
      60,
      30,
      20,
      100,
    );

    foreach ($widths as $col => $width) {
      if ($width !== null) {
        $sheet->getColumnDimension($this->col($col))->setWidth($width);
      }
    }

    $status_map = ManiphestTaskStatus::getTaskStatusMap();
    $pri_map = ManiphestTaskPriority::getTaskPriorityMap();

    $date_format = null;

    $rows = array();
    $rows[] = array(
      pht('ID'),
      pht('Owner'),
      pht('Status'),
      pht('Priority'),
      pht('Date Created'),
      pht('Date Updated'),
      pht('Title'),
      pht('Projects'),
      pht('URI'),
      pht('Description'),
      pht('Custom Field'),
    );

    $is_date = array(
      false,
      false,
      false,
      false,
      true,
      true,
      false,
      false,
      false,
      false,
      false,
    );

    $header_format = array(
      'font'  => array(
        'bold' => true,
      ),
    );

    foreach ($tasks as $task) {
      $task_owner = null;
      if ($task->getOwnerPHID()) {
        $task_owner = $handles[$task->getOwnerPHID()]->getName();
      }

      $custom_fields = PhabricatorCustomField::getObjectFields(
        $task,
        PhabricatorCustomField::ROLE_VIEW);
      $custom_fields
        ->setViewer($user)
        ->readFieldsFromStorage($task);
      $custom_fields = $custom_fields->getFields();

      $projects = array();
      foreach ($task->getProjectPHIDs() as $phid) {
        $projects[] = $handles[$phid]->getName();
      }
      $projects = implode(', ', $projects);

      $rows[] = array(
        'T'.$task->getID(),
        $task_owner,
        idx($status_map, $task->getStatus(), '?'),
        idx($pri_map, $task->getPriority(), '?'),
        $this->computeExcelDate($task->getDateCreated()),
        $this->computeExcelDate($task->getDateModified()),
        $task->getTitle(),
        $projects,
        PhabricatorEnv::getProductionURI('/T'.$task->getID()),
        phutil_utf8_shorten($task->getDescription(), 512),
        $custom_fields['std:maniphest:vendor:field']->getValueForStorage(),
      );
    }

    foreach ($rows as $row => $cols) {
      foreach ($cols as $col => $spec) {
        $cell_name = $this->col($col).($row + 1);
        $cell = $sheet
          ->setCellValue($cell_name, $spec, $return_cell = true);

        if ($row == 0) {
          $sheet->getStyle($cell_name)->applyFromArray($header_format);
        }

        if ($is_date[$col]) {
          $code = PHPExcel_Style_NumberFormat::FORMAT_DATE_YYYYMMDD2;
          $sheet
            ->getStyle($cell_name)
            ->getNumberFormat()
            ->setFormatCode($code);
        } else {
          $cell->setDataType(PHPExcel_Cell_DataType::TYPE_STRING);
        }
      }
    }
  }

  private function col($n) {
    return chr(ord('A') + $n);
  }

}

Don't forget to update your class map:

$ cd webroot/phabricator
$ ../arcanist/bin/arc liberate src

Thanks a lot, do you know why i'm getting a

>>> UNRECOVERABLE FATAL ERROR <<<
 "Call to a member function getValueForStorage() on a non-object"

on this line of code

$custom_fields['std:maniphest:vendor:field']->getValueForStorage(),

@camilorojas That's because you haven't defined the custom field std:maniphest:vendor:field. Checkout the docs to see how to do that!

In T5391#167263, @witrin wrote:

@camilorojas That's because you haven't defined the custom field std:maniphest:vendor:field. Checkout the docs to see how to do that!

i'm so dumb, i get it now, thanks guys.

it's working good.

@camilorojas You're welcome! PS: I like your avatar ... ^^

@witrin your implementation is really useful. Just an addition for custom fields of storage type PhabricatorStandardCustomFieldSelect (dropdowns). The getValueForStorage() function only returns the index of the field's value.. I found this useful in getting the human readable format of the field's value

$rootCause = $custom_fields->getFields()['std:maniphest:fast:root-cause']->getProxy()->renderPropertyViewValue($handles);

The $handles array is needed in the call but not used in the function so can also be null.

@witrin Thanks for that contribution! I really need this.
I tried to set this up but I got the followeing error:

>>> UNRECOVERABLE FATAL ERROR <<<

Call to undefined function phutil_utf8_shorten()

/opt/bitnami/apps/phabricator/htdocs/src/applications/maniphest/export/ManiphestExcelCustomFormat.php:117


┻━┻ ︵ ¯\_(ツ)_/¯ ︵ ┻━┻

Is there any fix?
+ How can I get all the custom fields? just adding more of this lines:

$custom_fields['std:maniphest:vendor:field']->getValueForStorage(),
$custom_fields['std:maniphest:vendor:field2']->getValueForStorage(),
$custom_fields['std:maniphest:vendor:field3']->getValueForStorage(),

Thanks!

See D10391

If it helps anyone, I've been using this extension to allow exporting tasks with custom fields as well as the workboard column. Please read the comment at the top of it for full details:

https://github.com/neandrake/phab-utils/blob/master/extensions/ManiphestExcelDefaultIncludeCustomFieldsFormat.php

Why the export does not contains the task author?

We need something like this too because the exporter doesn't include points at the moment.

Seems like the ideal solution would be to have the user pick all the fields they want with some checkboxes.

In T5391#234838, @epriestley wrote:

Resolved by D18959 + D18960. See also T13049.

Not all custom field types have export support today, but the infrastructure is in place and additional support is generally straightforward.

This enhancement caused custom select fields from getting exported. As you mentioned, the infrastructure does exist but a minor enhancement is required for this. Please review this pull request to support the feature.
https://github.com/phacility/phabricator/pull/845