Merge branch 'main' into 218-improve-interactive-translator

ggurdin · web-flow · commit 27c641e8e4e2 · 2024-05-29T12:44:22.000-04:00
diff --git a/lib/pangea/controllers/my_analytics_controller.dart b/lib/pangea/controllers/my_analytics_controller.dart
@@ -3,6 +3,7 @@ import 'dart:developer';
 
 import 'package:fluffychat/pangea/controllers/pangea_controller.dart';
 import 'package:fluffychat/pangea/enum/construct_type_enum.dart';
+import 'package:fluffychat/pangea/matrix_event_wrappers/construct_analytics_event.dart';
 import 'package:fluffychat/pangea/models/student_analytics_summary_model.dart';
 import 'package:fluffychat/pangea/utils/error_handler.dart';
 import 'package:flutter/foundation.dart';
@@ -111,4 +112,48 @@ class MyAnalyticsController {
       ErrorHandler.logError(e: err, s: s);
     }
   }
+
+  // used to aggregate ConstructEvents, from multiple senders (students) with the same lemma
+  List<AggregateConstructUses> aggregateConstructData(
+    List<ConstructEvent> constructs,
+  ) {
+    final Map<String, List<ConstructEvent>> lemmasToConstructs = {};
+    for (final construct in constructs) {
+      lemmasToConstructs[construct.content.lemma] ??= [];
+      lemmasToConstructs[construct.content.lemma]!.add(construct);
+    }
+
+    final List<AggregateConstructUses> aggregatedConstructs = [];
+    for (final lemmaToConstructs in lemmasToConstructs.entries) {
+      final List<ConstructEvent> lemmaConstructs = lemmaToConstructs.value;
+      final AggregateConstructUses aggregatedData = AggregateConstructUses(
+        constructs: lemmaConstructs,
+      );
+      aggregatedConstructs.add(aggregatedData);
+    }
+    return aggregatedConstructs;
+  }
+}
+
+class AggregateConstructUses {
+  final List<ConstructEvent> _constructs;
+
+  AggregateConstructUses({required List<ConstructEvent> constructs})
+      : _constructs = constructs;
+
+  String get lemma {
+    assert(
+      _constructs.isNotEmpty &&
+          _constructs.every(
+            (construct) =>
+                construct.content.lemma == _constructs.first.content.lemma,
+          ),
+    );
+    return _constructs.first.content.lemma;
+  }
+
+  List<OneConstructUse> get uses => _constructs
+      .map((construct) => construct.content.uses)
+      .expand((element) => element)
+      .toList();
 }
diff --git a/lib/pangea/pages/analytics/analytics_list_tile.dart b/lib/pangea/pages/analytics/analytics_list_tile.dart
@@ -104,7 +104,6 @@ class AnalyticsListTileState extends State<AnalyticsListTile> {
                   )
                 : null,
             selected: widget.selected,
-            enabled: widget.enabled,
             onTap: () {
               (room?.isSpace ?? false) && widget.allowNavigateOnSelect
                   ? context.go(
diff --git a/lib/pangea/pages/analytics/base_analytics.dart b/lib/pangea/pages/analytics/base_analytics.dart
@@ -5,6 +5,7 @@ import 'package:fluffychat/pangea/extensions/client_extension.dart';
 import 'package:fluffychat/pangea/pages/analytics/base_analytics_view.dart';
 import 'package:fluffychat/pangea/pages/analytics/student_analytics/student_analytics.dart';
 import 'package:flutter/material.dart';
+import 'package:future_loading_dialog/future_loading_dialog.dart';
 import 'package:matrix/matrix.dart';
 
 import '../../../widgets/matrix.dart';
@@ -101,18 +102,40 @@ class BaseAnalyticsController extends State<BaseAnalyticsPage> {
     }
   }
 
-  void toggleSelection(AnalyticsSelected selectedParam) {
+  Future<void> toggleSelection(AnalyticsSelected selectedParam) async {
+    final bool joinSelectedRoom =
+        selectedParam.type == AnalyticsEntryType.room &&
+            !enableSelection(
+              selectedParam,
+            );
+
+    if (joinSelectedRoom) {
+      await showFutureLoadingDialog(
+        context: context,
+        future: () async {
+          final waitForRoom = Matrix.of(context).client.waitForRoomInSync(
+                selectedParam.id,
+                join: true,
+              );
+          await Matrix.of(context).client.joinRoom(selectedParam.id);
+          await waitForRoom;
+        },
+      );
+    }
+
     setState(() {
       debugPrint("selectedParam.id is ${selectedParam.id}");
       currentLemma = null;
       selected = isSelected(selectedParam.id) ? null : selectedParam;
     });
+
     pangeaController.analytics.setConstructs(
       constructType: ConstructType.grammar,
       defaultSelected: widget.defaultSelected,
       selected: selected,
       removeIT: true,
     );
+
     Future.delayed(Duration.zero, () => setState(() {}));
   }
 
diff --git a/lib/pangea/pages/analytics/base_analytics_view.dart b/lib/pangea/pages/analytics/base_analytics_view.dart
@@ -145,6 +145,7 @@ class BaseAnalyticsView extends StatelessWidget {
                                     ) *
                                     72,
                                 child: TabBarView(
+                                  physics: const NeverScrollableScrollPhysics(),
                                   children: [
                                     Column(
                                       crossAxisAlignment:
diff --git a/lib/pangea/pages/analytics/class_list/class_list.dart b/lib/pangea/pages/analytics/class_list/class_list.dart
@@ -1,11 +1,10 @@
 import 'dart:async';
 
+import 'package:fluffychat/pangea/enum/time_span.dart';
+import 'package:fluffychat/pangea/pages/analytics/class_list/class_list_view.dart';
 import 'package:flutter/material.dart';
-
 import 'package:matrix/matrix.dart';
 
-import 'package:fluffychat/pangea/enum/time_span.dart';
-import 'package:fluffychat/pangea/pages/analytics/class_list/class_list_view.dart';
 import '../../../../widgets/matrix.dart';
 import '../../../constants/pangea_event_types.dart';
 import '../../../controllers/pangea_controller.dart';
@@ -42,7 +41,11 @@ class AnalyticsClassListController extends State<AnalyticsClassList> {
     if (!(refreshTimer[newState.room.id]?.isActive ?? false)) {
       refreshTimer[newState.room.id] = Timer(
         const Duration(seconds: 3),
-        () => updateClassAnalytics(context, newState.room),
+        () {
+          if (newState.room.isSpace) {
+            updateClassAnalytics(context, newState.room);
+          }
+        },
       );
     }
   }
diff --git a/lib/pangea/pages/analytics/construct_list.dart b/lib/pangea/pages/analytics/construct_list.dart
@@ -3,9 +3,9 @@ import 'dart:async';
 import 'package:collection/collection.dart';
 import 'package:fluffychat/config/app_config.dart';
 import 'package:fluffychat/pangea/constants/pangea_event_types.dart';
+import 'package:fluffychat/pangea/controllers/my_analytics_controller.dart';
 import 'package:fluffychat/pangea/controllers/pangea_controller.dart';
 import 'package:fluffychat/pangea/enum/construct_type_enum.dart';
-import 'package:fluffychat/pangea/matrix_event_wrappers/construct_analytics_event.dart';
 import 'package:fluffychat/pangea/matrix_event_wrappers/pangea_message_event.dart';
 import 'package:fluffychat/pangea/matrix_event_wrappers/pangea_representation_event.dart';
 import 'package:fluffychat/pangea/models/constructs_analytics_model.dart';
@@ -169,7 +169,7 @@ class ConstructListViewState extends State<ConstructListView> {
 
   int get lemmaIndex =>
       constructs?.indexWhere(
-        (element) => element.content.lemma == widget.controller.currentLemma,
+        (element) => element.lemma == widget.controller.currentLemma,
       ) ??
       -1;
 
@@ -217,7 +217,7 @@ class ConstructListViewState extends State<ConstructListView> {
 
     setState(() => fetchingUses = true);
     try {
-      final List<OneConstructUse> uses = currentConstruct!.content.uses;
+      final List<OneConstructUse> uses = currentConstruct!.uses;
       _msgEvents.clear();
 
       for (final OneConstructUse use in uses) {
@@ -236,16 +236,24 @@ class ConstructListViewState extends State<ConstructListView> {
       ErrorHandler.logError(
         e: err,
         s: s,
-        m: "Failed to fetch uses for current construct ${currentConstruct?.content.lemma}",
+        m: "Failed to fetch uses for current construct ${currentConstruct?.lemma}",
       );
     }
   }
 
-  List<ConstructEvent>? get constructs =>
-      widget.pangeaController.analytics.constructs;
-
-  ConstructEvent? get currentConstruct => constructs?.firstWhereOrNull(
-        (element) => element.content.lemma == widget.controller.currentLemma,
+  List<AggregateConstructUses>? get constructs =>
+      widget.pangeaController.analytics.constructs != null
+          ? widget.pangeaController.myAnalytics
+              .aggregateConstructData(
+                widget.pangeaController.analytics.constructs!,
+              )
+              .sorted(
+                (a, b) => b.uses.length.compareTo(a.uses.length),
+              )
+          : null;
+
+  AggregateConstructUses? get currentConstruct => constructs?.firstWhereOrNull(
+        (element) => element.lemma == widget.controller.currentLemma,
       );
 
   // given the current lemma and list of message events, return a list of
@@ -280,6 +288,13 @@ class ConstructListViewState extends State<ConstructListView> {
     return allMsgErrorSteps;
   }
 
+  Future<void> showConstructMessagesDialog() async {
+    await showDialog<ConstructMessagesDialog>(
+      context: context,
+      builder: (c) => ConstructMessagesDialog(controller: this),
+    );
+  }
+
   @override
   Widget build(BuildContext context) {
     if (!widget.init || fetchingUses) {
@@ -294,58 +309,92 @@ class ConstructListViewState extends State<ConstructListView> {
       );
     }
 
-    final msgEventMatches = getMessageEventMatches();
+    return Expanded(
+      child: ListView.builder(
+        itemCount: constructs!.length,
+        itemBuilder: (context, index) {
+          return ListTile(
+            title: Text(
+              constructs![index].lemma,
+            ),
+            subtitle: Text(
+              '${L10n.of(context)!.total} ${constructs![index].uses.length}',
+            ),
+            onTap: () async {
+              final String lemma = constructs![index].lemma;
+              widget.controller.setCurrentLemma(lemma);
+              fetchUses().then((_) => showConstructMessagesDialog());
+            },
+          );
+        },
+      ),
+    );
+  }
+}
 
-    return widget.controller.currentLemma == null
-        ? Expanded(
-            child: ListView.builder(
-              itemCount: constructs!.length,
-              itemBuilder: (context, index) {
-                return ListTile(
-                  title: Text(
-                    constructs![index].content.lemma,
-                  ),
-                  subtitle: Text(
-                    '${L10n.of(context)!.total} ${constructs![index].content.uses.length}',
-                  ),
-                  onTap: () {
-                    final String lemma = constructs![index].content.lemma;
-                    widget.controller.setCurrentLemma(lemma);
-                    fetchUses();
-                  },
-                );
-              },
+class ConstructMessagesDialog extends StatelessWidget {
+  final ConstructListViewState controller;
+  const ConstructMessagesDialog({
+    super.key,
+    required this.controller,
+  });
+
+  @override
+  Widget build(BuildContext context) {
+    if (controller.widget.controller.currentLemma == null) {
+      return const AlertDialog(content: CircularProgressIndicator.adaptive());
+    }
+
+    final msgEventMatches = controller.getMessageEventMatches();
+
+    return AlertDialog(
+      title: Center(child: Text(controller.widget.controller.currentLemma!)),
+      content: Column(
+        mainAxisSize: MainAxisSize.min,
+        crossAxisAlignment: CrossAxisAlignment.start,
+        children: [
+          if (controller.constructs![controller.lemmaIndex].uses.length >
+              controller._msgEvents.length)
+            Center(
+              child: Padding(
+                padding: const EdgeInsets.all(8.0),
+                child: Text(L10n.of(context)!.roomDataMissing),
+              ),
             ),
-          )
-        : Expanded(
+          SingleChildScrollView(
             child: Column(
-              crossAxisAlignment: CrossAxisAlignment.start,
               children: [
-                if (constructs![lemmaIndex].content.uses.length >
-                    _msgEvents.length)
-                  Center(
-                    child: Padding(
-                      padding: const EdgeInsets.all(8.0),
-                      child: Text(L10n.of(context)!.roomDataMissing),
-                    ),
-                  ),
-                Expanded(
-                  child: ListView.separated(
-                    separatorBuilder: (context, index) =>
+                ...msgEventMatches.mapIndexed(
+                  (index, event) => Column(
+                    children: [
+                      ConstructMessage(
+                        msgEvent: event.msgEvent,
+                        lemma: controller.widget.controller.currentLemma!,
+                        errorMessage: event.lemmaMatch,
+                      ),
+                      if (index < msgEventMatches.length - 1)
                         const Divider(height: 1),
-                    itemCount: msgEventMatches.length,
-                    itemBuilder: (context, index) {
-                      return ConstructMessage(
-                        msgEvent: msgEventMatches[index].msgEvent,
-                        lemma: widget.controller.currentLemma!,
-                        errorMessage: msgEventMatches[index].lemmaMatch,
-                      );
-                    },
+                    ],
                   ),
                 ),
               ],
             ),
-          );
+          ),
+        ],
+      ),
+      actions: [
+        TextButton(
+          onPressed: () => Navigator.of(context, rootNavigator: false).pop(),
+          child: Text(
+            L10n.of(context)!.close.toUpperCase(),
+            style: TextStyle(
+              color:
+                  Theme.of(context).textTheme.bodyMedium?.color?.withAlpha(150),
+            ),
+          ),
+        ),
+      ],
+    );
   }
 }
 

Original file line number	Diff line number	Diff line change
`@@ -104,7 +104,6 @@ class AnalyticsListTileState extends State<AnalyticsListTile> {`
`104`	`104`	`)`
`105`	`105`	`: null,`
`106`	`106`	`selected: widget.selected,`
`107`		`- enabled: widget.enabled,`
`108`	`107`	`onTap: () {`
`109`	`108`	`(room?.isSpace ?? false) && widget.allowNavigateOnSelect`
`110`	`109`	`? context.go(`