Merge pull request #457 from pangeachat/blue7/08

Added null checks, error handling, and used non-null version of l10n

lib/pages/chat/input_bar.dart

@@ -491,7 +491,12 @@ class InputBar extends StatelessWidget {
               textInputAction: textInputAction,
               autofocus: autofocus!,
               inputFormatters: [
-                LengthLimitingTextInputFormatter((maxPDUSize / 3).floor()),
+                //#Pangea
+                //LengthLimitingTextInputFormatter((maxPDUSize / 3).floor()),
+                //setting max character count to 1000
+                //after max, nothing else can be typed
+                LengthLimitingTextInputFormatter(1000),
+                //Pangea#
               ],
               onSubmitted: (text) {
                 // fix for library for now

lib/pangea/controllers/my_analytics_controller.dart

@@ -229,11 +229,8 @@ class MyAnalyticsController {
   /// top level analytics sending function. Gather recent messages and activity records,
   /// convert them into the correct formats, and send them to the analytics room
   Future<void> _updateAnalytics() async {
-    // if missing important info, don't send analytics
-    if (userL2 == null || _client.userID == null) {
-      debugger(when: kDebugMode);
-      return;
-    }
+    // if missing important info, don't send analytics. Could happen if user just signed up.
+    if (userL2 == null || _client.userID == null) return;
 
     // analytics room for the user and current target language
     final Room analyticsRoom = await _client.getMyAnalyticsRoom(userL2!);

lib/pangea/pages/p_user_age/p_user_age_view.dart

@@ -18,7 +18,13 @@ class PUserAgeView extends StatelessWidget {
       body: ListView(
         children: [
           Container(
-            margin: const EdgeInsets.only(top: 10),
+            decoration: BoxDecoration(
+              borderRadius: BorderRadius.circular(10),
+              color: Theme.of(context).colorScheme.onSecondaryContainer.withAlpha(50),
+            ),
+            child: Column(
+              children: [
+                Padding(
                   padding: const EdgeInsets.all(15),
                   child: Text(
                     L10n.of(context)!.yourBirthdayPlease,
@@ -26,23 +32,10 @@ class PUserAgeView extends StatelessWidget {
                     style: const TextStyle(
                       color: Colors.black,
                       fontSize: 14,
-                fontWeight: FontWeight.bold,
-              ),
+                      fontWeight: FontWeight.normal, 
                     ),
                   ),
-          const SizedBox(
-            height: 10,
                 ),
-          Container(
-            decoration: BoxDecoration(
-              borderRadius: BorderRadius.circular(10),
-              color: Theme.of(context)
-                  .colorScheme
-                  .onSecondaryContainer
-                  .withAlpha(50),
-            ),
-            child: Column(
-              children: [
                 ListTile(
                   title: Text(
                     L10n.of(context)!.certifyAge(13),
@@ -70,23 +63,16 @@ class PUserAgeView extends StatelessWidget {
               ],
             ),
           ),
-          const SizedBox(
-            height: 10,
-          ),
-          if (controller.error != null)
-            Padding(
-              padding: const EdgeInsets.all(12),
-              child: Text(
-                controller.error!,
-                style: const TextStyle(color: Colors.white),
-              ),
-            ),
+          const SizedBox(height: 20), 
           Hero(
             tag: 'loginButton',
             child: Padding(
-              padding: const EdgeInsets.all(12),
+              padding: const EdgeInsets.symmetric(horizontal: 12),
               child: ElevatedButton(
                 onPressed: controller.createUserInPangea,
+                style: ElevatedButton.styleFrom(
+                  minimumSize: const Size.fromHeight(50), 
+                ),
                 child: controller.loading
                     ? const LinearProgressIndicator()
                     : Text(L10n.of(context)!.getStarted),
@@ -95,7 +81,6 @@ class PUserAgeView extends StatelessWidget {
           ),
         ],
       ),
-      // ),
     );
   }
 }

lib/pangea/widgets/chat/message_toolbar.dart

@@ -193,6 +193,8 @@ class MessageToolbarState extends State<MessageToolbar> {
   late StreamSubscription<MessageMode> toolbarModeStream;
 
   void updateMode(MessageMode newMode) {
+    //Early exit from the function if the widget has been unmounted to prevent updates on an inactive widget.
+    if (!mounted) return;
     if (updatingMode) return;
     debugPrint("updating toolbar mode");
     final bool subscribed =

lib/utils/error_reporter.dart

@@ -11,13 +11,24 @@ class ErrorReporter {
   void onErrorCallback(Object error, [StackTrace? stackTrace]) async {
     Logs().e(message ?? 'Error caught', error, stackTrace);
     // #Pangea
+    // Attempt to retrieve the L10n instance using the current context
+    final L10n? l10n = L10n.of(context);
+
+    // Check if the L10n instance is null
+    if (l10n == null) {
+      // Log an error message saying that the localization object is null
+      Logs().e('Localization object is null, cannot show error message.');
+      // Exits early to prevent further execution
+      return;
+    }
     ScaffoldMessenger.of(context).showSnackBar(
       SnackBar(
         content: Text(
-          L10n.of(context)!.oopsSomethingWentWrong,
+          l10n.oopsSomethingWentWrong, // Use the non-null L10n instance to get the error message
         ),
       ),
     );
+  }
   // final text = '$error\n${stackTrace ?? ''}';
   // await showAdaptiveDialog(
   //   context: context,
@@ -64,4 +75,3 @@ class ErrorReporter {
   // );
   // Pangea#
 }
-}