revi-archive/phorge-phorge
1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-11-22 14:52:41 +01:00
Code Issues Releases Wiki Activity

Check is viewer is a participant before showing count

Browse source 
Summary: In Conpherence ProfileMenuItem we show an unread count if you're a participant, but all message count if you're not. Just remove that.

Test Plan: Log out of room in Conpherence, leave messages on second account, check menu item on both accounts.

Reviewers: epriestley

Reviewed By: epriestley

Subscribers: Korvin

Differential Revision: https://secure.phabricator.com/D17664
This commit is contained in:
Chad Little 2017-04-12 13:21:47 -07:00
parent 75303567b3
commit 6bf595b951
1 changed files with 7 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
src/applications/search/menuitem/PhabricatorConpherenceProfileMenuItem.php
View file

@ -102,8 +102,13 @@ final class PhabricatorConpherenceProfileMenuItem
return array();
}
$data = $room->getDisplayData($viewer);
$unread_count = $data['unread_count'];
$participants = $room->getParticipants();
$viewer_phid = $viewer->getPHID();
$unread_count = null;
if (isset($participants[$viewer_phid])) {
$data = $room->getDisplayData($viewer);
$unread_count = $data['unread_count'];
}
$count = null;
if ($unread_count) {