call async version of teachers in inviteSpaceTeachersToAnalyticsRoom … (#1082)

* call async version of teachers in inviteSpaceTeachersToAnalyticsRoom to ensure teachers aren't double invited to analytics rooms

* log breadcrumb on error getting message offset

* better error logging for activity fromJSON method

lib/pangea/extensions/pangea_room_extension/room_analytics_extension.dart

@@ -100,7 +100,7 @@ extension AnalyticsRoomExtension on Room {
     }
 
     final List<User> participants = analyticsRoom.getParticipants();
-    final List<User> uninvitedTeachers = teachersLocal
+    final List<User> uninvitedTeachers = (await teachers)
         .where((teacher) => !participants.contains(teacher))
         .toList();
 

lib/pangea/models/analytics/construct_list_model.dart

@@ -6,6 +6,7 @@ import 'package:fluffychat/pangea/models/analytics/construct_use_model.dart';
 import 'package:fluffychat/pangea/models/analytics/constructs_model.dart';
 import 'package:fluffychat/pangea/models/practice_activities.dart/practice_activity_model.dart';
 import 'package:fluffychat/pangea/utils/error_handler.dart';
+import 'package:sentry_flutter/sentry_flutter.dart';
 
 /// A wrapper around a list of [OneConstructUse]s, used to simplify
 /// the process of filtering / sorting / displaying the events.
@@ -145,14 +146,16 @@ class ConstructListModel {
       level = levelCalculation.floor();
     } else {
       level = 0;
-      ErrorHandler.logError(
+      Sentry.addBreadcrumb(
-        e: "Calculated level in Nan or Infinity",
+        Breadcrumb(
-        data: {
+          data: {
-          "totalXP": totalXP,
+            "totalXP": totalXP,
-          "prevXP": prevXP,
+            "prevXP": prevXP,
-          "level": levelCalculation,
+            "level": levelCalculation,
-        },
+          },
+        ),
       );
+      ErrorHandler.logError(e: "Calculated level in Nan or Infinity");
     }
   }
 

lib/pangea/models/practice_activities.dart/message_activity_request.dart

@@ -2,6 +2,7 @@ import 'package:collection/collection.dart';
 import 'package:fluffychat/pangea/enum/activity_type_enum.dart';
 import 'package:fluffychat/pangea/models/pangea_token_model.dart';
 import 'package:fluffychat/pangea/models/practice_activities.dart/practice_activity_model.dart';
+import 'package:sentry_flutter/sentry_flutter.dart';
 
 // includes feedback text and the bad activity model
 class ActivityQualityFeedback {
@@ -118,6 +119,16 @@ class MessageActivityResponse {
   });
 
   factory MessageActivityResponse.fromJson(Map<String, dynamic> json) {
+    if (!json.containsKey('activity')) {
+      Sentry.addBreadcrumb(Breadcrumb(data: {"json": json}));
+      throw Exception('Activity not found in message activity response');
+    }
+
+    if (json['activity'] is! Map<String, dynamic>) {
+      Sentry.addBreadcrumb(Breadcrumb(data: {"json": json}));
+      throw Exception('Activity is not a map in message activity response');
+    }
+
     return MessageActivityResponse(
       activity: PracticeActivityModel.fromJson(
         json['activity'] as Map<String, dynamic>,

lib/pangea/widgets/chat/message_selection_overlay.dart

@@ -26,6 +26,7 @@ import 'package:flutter/foundation.dart';
 import 'package:flutter/material.dart';
 import 'package:flutter/scheduler.dart';
 import 'package:matrix/matrix.dart';
+import 'package:sentry_flutter/sentry_flutter.dart';
 
 class MessageSelectionOverlay extends StatefulWidget {
   final ChatController chatController;
@@ -485,8 +486,8 @@ class MessageOverlayController extends State<MessageSelectionOverlay>
 
     try {
       return _messageRenderBox?.localToGlobal(Offset.zero);
-    } catch (e, s) {
+    } catch (e) {
-      ErrorHandler.logError(e: "Error getting message offset: $e", s: s);
+      Sentry.addBreadcrumb(Breadcrumb(message: "Error getting message offset"));
       return null;
     }
   }