mirror of
https://we.phorge.it/source/phorge.git
synced 2024-11-20 13:52:40 +01:00
Implicitly CC inline commenters in Pholio
Summary: Fixes T2722. Both the "comment" (default) and "inline comment" (needs to be specialized) actions should imply CC. Test Plan: Made an inline on a mock, got implicitly cc'd. Reviewers: chad Reviewed By: chad CC: aran Maniphest Tasks: T2722 Differential Revision: https://secure.phabricator.com/D5312
This commit is contained in:
parent
b41b1b43db
commit
a858c14791
1 changed files with 12 additions and 0 deletions
|
@ -158,4 +158,16 @@ final class PholioMockEditor extends PhabricatorApplicationTransactionEditor {
|
|||
}
|
||||
|
||||
|
||||
protected function shouldImplyCC(
|
||||
PhabricatorLiskDAO $object,
|
||||
PhabricatorApplicationTransaction $xaction) {
|
||||
|
||||
switch ($xaction->getTransactionType()) {
|
||||
case PholioTransactionType::TYPE_INLINE:
|
||||
return true;
|
||||
}
|
||||
|
||||
return parent::shouldImplyCC($object, $xaction);
|
||||
}
|
||||
|
||||
}
|
||||
|
|
Loading…
Reference in a new issue