From e14a1ad1af19cc4cbdd480f09ca24d5c22f44a68 Mon Sep 17 00:00:00 2001 From: David Kelly Date: Mon, 6 Mar 2017 15:26:03 -0700 Subject: [PATCH] minor cleanup --- interface/resources/qml/hifi/Pal.qml | 20 ++++++++------------ 1 file changed, 8 insertions(+), 12 deletions(-) diff --git a/interface/resources/qml/hifi/Pal.qml b/interface/resources/qml/hifi/Pal.qml index 670a29c82d..9de7817d1b 100644 --- a/interface/resources/qml/hifi/Pal.qml +++ b/interface/resources/qml/hifi/Pal.qml @@ -61,13 +61,16 @@ Rectangle { property int sortIndicatorColumn: 1 property int sortIndicatorOrder: Qt.AscendingOrder } + function getSelectedSessionIDs() { + var sessionIDs = []; + table.selection.forEach(function (userIndex) { + sessionIDs.push(userModelData[userIndex].sessionId); + }); + return sessionIDs; + } function refreshWithFilter() { // We should just be able to set settings.filtered to filter.checked, but see #3249, so send to .js for saving. - var userIds = []; - table.selection.forEach(function (userIndex) { - userIds.push(userModelData[userIndex].sessionId); - }); - table.selection.clear(); + var userIds = getSelectedSessionIDs(); var params = {filter: filter.checked && {distance: settings.nearDistance}}; if (userIds.length > 0) { params.selected = [[userIds[0]], true, true]; @@ -620,13 +623,6 @@ Rectangle { console.log('Unrecognized message:', JSON.stringify(message)); } } - function getSelectedSessionIDs() { - var sessionIDs = []; - table.selection.forEach(function (userIndex) { - sessionIDs.push(userModelData[userIndex].sessionId); - }); - return sessionIDs; - } function sortModel() { var sortProperty = table.getColumn(table.sortIndicatorColumn).role; var before = (table.sortIndicatorOrder === Qt.AscendingOrder) ? -1 : 1;