revi-archive/phorge-phorge
1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-12-15 10:00:55 +01:00
Code Issues Releases Wiki Activity

(stable) Restore "Summary" and "Test Plan" to initial mail for non-draft configurations

Browse source 
Summary: See PHI210. Ref T2543. Currently, we don't set this flag if you have prototypes off and don't get any of the new draft stuff, so the mail drops some of the details it is supposed to have.

Test Plan: Disabled prototypes, created a revision, saw summary / test plan in the initial mail.

Reviewers: amckinley

Reviewed By: amckinley

Maniphest Tasks: T2543

Differential Revision: https://secure.phabricator.com/D18771
This commit is contained in:
epriestley 2017-11-14 12:06:24 -08:00
parent d2d5a1f283
commit c098aec669
1 changed files with 7 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
src/applications/differential/editor/DifferentialTransactionEditor.php
View file

@ -1612,6 +1612,13 @@ final class DifferentialTransactionEditor
$xactions[] = $xaction; $xactions[] = $xaction;
} }
} else {
// If this revision is being created into some state other than "Draft",
// this is the first broadcast and should include sections like "SUMMARY"
// and "TEST PLAN".
if ($this->getIsNewObject()) {
$this->firstBroadcast = true;
}
} }
return $xactions; return $xactions;