Permalink
Browse files

fix an issue where the mailing module was looking up based on old fun…

…ction names
  • Loading branch information...
1 parent c082224 commit a6634db1f2dbcb46e4389a8d497dfbed19d3f57b @zegenie zegenie committed Dec 8, 2011
Showing with 6 additions and 4 deletions.
  1. +6 −4 modules/mailing/classes/TBGMailing.class.php
@@ -377,8 +377,9 @@ protected function _getIssueRelatedUsers(TBGIssue $issue)
}
elseif ($issue->getProject()->getLeader() instanceof TBGUser)
{
- if (!($issue->getProject()->getLeaderID() == $cu && !$ns) && !(!$this->getSetting(self::NOTIFY_ISSUE_PROJECT_ASSIGNED, $issue->getProject()->getLeaderID())))
- $uids[$issue->getProject()->getLeaderID()] = $issue->getProject()->getLeaderID();
+ $lid = $issue->getProject()->getLeader()->getID();
+ if (!($lid == $cu && !$ns) && !(!$this->getSetting(self::NOTIFY_ISSUE_PROJECT_ASSIGNED, $lid)))
+ $uids[$lid] = $lid;
}
// Same for QA
@@ -393,8 +394,9 @@ protected function _getIssueRelatedUsers(TBGIssue $issue)
}
elseif ($issue->getProject()->getQaResponsible() instanceof TBGUser)
{
- if (!($issue->getProject()->getQaResponsibleID() == $cu && !$ns) && !(!$this->getSetting(self::NOTIFY_ISSUE_PROJECT_ASSIGNED, $issue->getProject()->getQaResponsibleID())))
- $uids[$issue->getProject()->getQaResponsibleID()] = $issue->getProject()->getQaResponsibleID();
+ $qaid = $issue->getProject()->getQaResponsible()->getID();
+ if (!($qaid == $cu && !$ns) && !(!$this->getSetting(self::NOTIFY_ISSUE_PROJECT_ASSIGNED, $qaid)))
+ $uids[$qaid] = $qaid;
}
foreach ($issue->getProject()->getAssignedTeams() as $team_id => $assignments)

0 comments on commit a6634db

Please sign in to comment.