History

return a legit selectedItemsList js object as long as the list exist, only void if the list name is unknown

samcake committed 8 years ago

Commit 7a9d77f1 parent ace8e151 file · +9 −12Browse files

Showing the changes to interface/src/scripting/SelectionScriptingInterface.cpp (1 of 1 file). Show all

Unified · Split

interface/src/scripting/SelectionScriptingInterface.cpp+9−12
@@ -247,35 +247,32 @@ QVariantMap SelectionScriptingInterface::getSelectedItemsList(const QString& lis · 246 unmodified lines
247247 QVariantMap list;
248248 auto currentList = _selectedItemsListMap.find(listName);
249249 if (currentList != _selectedItemsListMap.end()) {
250− if ((*currentList).getContainsData()) {
250+ QList<QVariant> avatarIDs;
251+ QList<QVariant> entityIDs;
252+ QList<QVariant> overlayIDs;
251253
254+ if ((*currentList).getContainsData()) {
252255 if (!(*currentList).getAvatarIDs().empty()) {
253− QList<QVariant> avatarIDs;
254256 for (auto j : (*currentList).getAvatarIDs()) {
255257 avatarIDs.push_back((QUuid)j);
256258 }
257− list["avatars"] = (avatarIDs);
258259 }
259260 if (!(*currentList).getEntityIDs().empty()) {
260− QList<QVariant> entityIDs;
261261 for (auto j : (*currentList).getEntityIDs()) {
262262 entityIDs.push_back((QUuid)j );
263263 }
264− list["entities"] = (entityIDs);
265264 }
266265 if (!(*currentList).getOverlayIDs().empty()) {
267− QList<QVariant> overlayIDs;
268266 for (auto j : (*currentList).getOverlayIDs()) {
269267 overlayIDs.push_back((QUuid)j);
270268 }
271− list["overlays"] = (overlayIDs);
272269 }
273−
274− return list;
275− }
276− else {
277− return list;
278270 }
271+ list["avatars"] = (avatarIDs);
272+ list["entities"] = (entityIDs);
273+ list["overlays"] = (overlayIDs);
274+
275+ return list;
279276 }
280277 else {
281278 return list;