xan/awips2
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Omaha #3267 fixed collaboration NPE possibility

Browse source 
Change-Id: I9bc68be7d9106396a0c369e3181e15baf25ca260

Former-commit-id: a23e57f3d1a6f4213216a7e038b85b17b6628faa
This commit is contained in:
Brian Clements 2014-06-12 15:05:46 -05:00
parent d3417d8e45
commit 91eb7d40c2
1 changed files with 7 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
cave/com.raytheon.uf.viz.collaboration.ui/src/com/raytheon/uf/viz/collaboration/ui/UsersTreeLabelProvider.java
View file

@ -63,6 +63,7 @@ import com.raytheon.uf.viz.collaboration.ui.data.SessionGroupContainer;
* Feb 13, 2014 2751 bclement made AbstractUsersLabelProvider generic
* Feb 17, 2014 2751 bclement added block image logic to userLabelProvider
* Mar 06, 2014 2848 bclement get venueName directly from session
* Jun 12, 2014 3267 bclement fixed missing null-check for outdated UI info
*
* </pre>
*
@ -122,7 +123,12 @@ public class UsersTreeLabelProvider extends ColumnLabelProvider {
if (!account.isSameUser(user)) {
ContactsManager cm = conn.getContactsManager();
RosterEntry entry = cm.getRosterEntry(user);
if (ContactsManager.isBlocked(entry)) {
/*
* if entry is null here, it means that the UI has an outdated
* roster entry, this will get resolved on the next refresh
* which is likely already starting to happen right now
*/
if (entry != null && ContactsManager.isBlocked(entry)) {
rval = true;
}
}