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

Treat MERGE_INTO as a status transaction for mail tags

Browse source 
Summary: Fixes T6838. We use a special transaction type for merging, but don't handle it when figuring out mail tags.

Test Plan: Verified merge mail picks up the `maniphest-status` tag.

Reviewers: btrahan, chad

Reviewed By: chad

Subscribers: epriestley

Maniphest Tasks: T6838

Differential Revision: https://secure.phabricator.com/D11101
This commit is contained in:
epriestley 2014-12-31 08:37:53 -08:00
parent a84cd99893
commit 00192ec8ce
1 changed files with 1 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
src/applications/maniphest/storage/ManiphestTransaction.php
View file

@ -794,6 +794,7 @@ final class ManiphestTransaction
public function getMailTags() { public function getMailTags() {
$tags = array(); $tags = array();
switch ($this->getTransactionType()) { switch ($this->getTransactionType()) {
case self::TYPE_MERGED_INTO:
case self::TYPE_STATUS: case self::TYPE_STATUS:
$tags[] = self::MAILTAG_STATUS; $tags[] = self::MAILTAG_STATUS;
break; break;