From fecde0f6d2c503ed8f860417ae30a66f1dc58e75 Mon Sep 17 00:00:00 2001 From: hunterbridges Date: Tue, 2 Aug 2011 13:58:52 -0500 Subject: [PATCH] Fixed auxiliary field submit bug on new task Reviewers: epriestley Test Plan: Configure auxiliary fields, create new task --- .../taskedit/ManiphestTaskEditController.php | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/src/applications/maniphest/controller/taskedit/ManiphestTaskEditController.php b/src/applications/maniphest/controller/taskedit/ManiphestTaskEditController.php index cda792d58e..5d7e8d78aa 100644 --- a/src/applications/maniphest/controller/taskedit/ManiphestTaskEditController.php +++ b/src/applications/maniphest/controller/taskedit/ManiphestTaskEditController.php @@ -164,14 +164,6 @@ class ManiphestTaskEditController extends ManiphestController { $changes[ManiphestTransactionType::TYPE_PROJECTS] = $new_proj_arr; } - // TODO: Capture auxiliary field changes in a transaction - foreach ($aux_fields as $aux_field) { - $task->setAuxiliaryAttribute( - $aux_field->getAuxiliaryKey(), - $aux_field->getValueForStorage() - ); - } - if ($files) { $file_map = mpull($files, 'getPHID'); $file_map = array_fill_keys($file_map, true); @@ -196,6 +188,14 @@ class ManiphestTaskEditController extends ManiphestController { $editor->applyTransactions($task, $transactions); } + // TODO: Capture auxiliary field changes in a transaction + foreach ($aux_fields as $aux_field) { + $task->setAuxiliaryAttribute( + $aux_field->getAuxiliaryKey(), + $aux_field->getValueForStorage() + ); + } + $redirect_uri = '/T'.$task->getID(); if ($workflow) {