diff --git a/mobile/lib/ui/viewer/file_details/face_widget.dart b/mobile/lib/ui/viewer/file_details/face_widget.dart index 06784105e8..0b55f30628 100644 --- a/mobile/lib/ui/viewer/file_details/face_widget.dart +++ b/mobile/lib/ui/viewer/file_details/face_widget.dart @@ -5,6 +5,7 @@ import "dart:typed_data"; import "package:flutter/cupertino.dart"; import "package:flutter/foundation.dart" show kDebugMode; import "package:flutter/material.dart"; +import "package:logging/logging.dart"; import "package:photos/db/ml/db.dart"; import "package:photos/generated/l10n.dart"; import "package:photos/models/base/id.dart"; @@ -50,6 +51,8 @@ class FaceWidget extends StatefulWidget { class _FaceWidgetState extends State { bool isJustRemoved = false; + final _logger = Logger("FaceWidget"); + @override Widget build(BuildContext context) { final bool givenFaces = widget.faceCrops != null; @@ -266,7 +269,9 @@ class _FaceWidgetState extends State { ); } if (snapshot.hasError) { - log('Error getting face: ${snapshot.error}'); + _logger.severe( + 'Error getting face: ${snapshot.error} ${snapshot.stackTrace}', + ); } return const ClipRRect( borderRadius: BorderRadius.all(Radius.elliptical(16, 12)), @@ -296,7 +301,9 @@ class _FaceWidgetState extends State { isJustRemoved = !isJustRemoved; }); } catch (e, s) { - log("removing face/file from cluster from file info widget failed: $e, \n $s"); + _logger.severe( + "removing face/file from cluster from file info widget failed: $e, \n $s", + ); } } } diff --git a/mobile/lib/ui/viewer/people/people_page.dart b/mobile/lib/ui/viewer/people/people_page.dart index 73c2a2cfa5..044dd280b7 100644 --- a/mobile/lib/ui/viewer/people/people_page.dart +++ b/mobile/lib/ui/viewer/people/people_page.dart @@ -1,5 +1,4 @@ import "dart:async"; -import "dart:developer"; import 'package:flutter/material.dart'; import "package:logging/logging.dart"; @@ -102,7 +101,6 @@ class _PeoplePageState extends State { } Future> loadPersonFiles() async { - log("loadPersonFiles"); final result = await SearchService.instance .getClusterFilesForPersonID(_person.remoteID); if (result.isEmpty) { @@ -243,7 +241,7 @@ class _PeoplePageState extends State { ], ); } else if (snapshot.hasError) { - log("Error: ${snapshot.error} ${snapshot.stackTrace}}"); + _logger.severe("Error: ${snapshot.error} ${snapshot.stackTrace}}"); //Need to show an error on the UI here return const SizedBox.shrink(); } else { diff --git a/mobile/lib/ui/viewer/people/save_or_edit_person.dart b/mobile/lib/ui/viewer/people/save_or_edit_person.dart index a3563a6b4c..18c651c8bd 100644 --- a/mobile/lib/ui/viewer/people/save_or_edit_person.dart +++ b/mobile/lib/ui/viewer/people/save_or_edit_person.dart @@ -1,5 +1,4 @@ import 'dart:async'; -import "dart:developer" as dev; import "dart:io"; import "dart:math" show max; @@ -444,7 +443,7 @@ class _SaveOrEditPersonState extends State { stream: _getPersonsWithRecentFileStream(), builder: (context, snapshot) { if (snapshot.hasError) { - dev.log("Error: ${snapshot.error} ${snapshot.stackTrace}}"); + _logger.severe("Error in _getPersonItems: ${snapshot.error} ${snapshot.stackTrace}}"); if (kDebugMode) { return Column( mainAxisSize: MainAxisSize.min, diff --git a/mobile/lib/ui/viewer/search/result/person_face_widget.dart b/mobile/lib/ui/viewer/search/result/person_face_widget.dart index e5a01cac1f..4b0c7760a7 100644 --- a/mobile/lib/ui/viewer/search/result/person_face_widget.dart +++ b/mobile/lib/ui/viewer/search/result/person_face_widget.dart @@ -76,7 +76,7 @@ class _PersonFaceWidgetState extends State { } else { if (snapshot.hasError) { _logger.severe( - "Error getting cover face for person: ${snapshot.error}", + "Error getting cover face for person: ${snapshot.error} ${snapshot.stackTrace}}", ); } return widget.thumbnailFallback