From 2a78767a4afd6fe500f0fcc261b6d23d7c7bef4a Mon Sep 17 00:00:00 2001 From: epriestley Date: Mon, 23 Sep 2013 14:33:00 -0700 Subject: [PATCH] Route lipsum writes through the new Maniphest transaction code Summary: Ref T2217. Ship these through the new stuff. Test Plan: See screenshot. Reviewers: btrahan Reviewed By: btrahan CC: aran Maniphest Tasks: T2217 Differential Revision: https://secure.phabricator.com/D7087 --- .../PhabricatorManiphestTaskTestDataGenerator.php | 14 +++++++++----- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/src/applications/maniphest/lipsum/PhabricatorManiphestTaskTestDataGenerator.php b/src/applications/maniphest/lipsum/PhabricatorManiphestTaskTestDataGenerator.php index 9e38f55b4f..f6f8f8b461 100644 --- a/src/applications/maniphest/lipsum/PhabricatorManiphestTaskTestDataGenerator.php +++ b/src/applications/maniphest/lipsum/PhabricatorManiphestTaskTestDataGenerator.php @@ -10,13 +10,14 @@ final class PhabricatorManiphestTaskTestDataGenerator $task = id(new ManiphestTask()) ->setSubPriority($this->generateTaskSubPriority()) ->setAuthorPHID($authorPHID) - ->setTitle($this->generateTitle()); + ->setTitle($this->generateTitle()) + ->setStatus(ManiphestTaskStatus::STATUS_OPEN); + $content_source = PhabricatorContentSource::newForSource( PhabricatorContentSource::SOURCE_UNKNOWN, array()); - $template = id(new ManiphestTransaction()) - ->setAuthorPHID($authorPHID) - ->setContentSource($content_source); + + $template = new ManiphestTransactionPro(); // Accumulate Transactions $changes = array(); $changes[ManiphestTransactionType::TYPE_TITLE] = @@ -42,8 +43,11 @@ final class PhabricatorManiphestTaskTestDataGenerator } // Apply Transactions - $editor = id(new ManiphestTransactionEditor()) + $editor = id(new ManiphestTransactionEditorPro()) ->setActor($author) + ->setContentSource($content_source) + ->setContinueOnNoEffect(true) + ->setContinueOnMissingFields(true) ->applyTransactions($task, $transactions); return $task->save(); }