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

Make Maniphest emails align title numbers better and fix a transaction

Browse source 
aggregation bug.

Summary:
Transactions always aggregated even when separated by large time
differences becuase of some clown typoing; fix that. Also make the subject
lines for tasks align better in the inbox.

Test Plan:
I'm feeling lucky.

Differential Revision: 29
Reviewed By: tomo
Reviewers: tomo
This commit is contained in:
epriestley 2011-02-09 20:12:20 -08:00
parent 193dbf16b4
commit e8e185ceb3
2 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/applications/maniphest/editor/transaction/ManiphestTransactionEditor.php
View file

@ -154,7 +154,7 @@ class ManiphestTransactionEditor {
id(new PhabricatorMetaMTAMail()) id(new PhabricatorMetaMTAMail())
->setSubject( ->setSubject(
'[Maniphest] '.$action.': T'.$task->getID().' '.$task->getTitle()) '[Maniphest] T'.$task->getID().' '.$action.': '.$task->getTitle())
->setFrom($transaction->getAuthorPHID()) ->setFrom($transaction->getAuthorPHID())
->addTos($email_to) ->addTos($email_to)
->addCCs($email_cc) ->addCCs($email_cc)

2
src/applications/maniphest/storage/transaction/ManiphestTransaction.php
View file

@ -66,7 +66,7 @@ class ManiphestTransaction extends ManiphestDAO {
return false; return false;
} }
$ttime = $target->getDateCreated(); $ttime = $target->getDateCreated();
$stime = $target->getDateCreated(); $stime = $this->getDateCreated();
if (abs($stime - $ttime) > 60) { if (abs($stime - $ttime) > 60) {
return false; return false;
} }