Skip to content

Commit 16f969b

Browse files
feat: Added proper logging using logger package (#262)
Co-authored-by: Mario Behling <[email protected]>
1 parent 148fbf5 commit 16f969b

22 files changed

+96
-46
lines changed

lib/card_templates/card_template_selection_view.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -178,7 +178,7 @@ class CardTemplateSelectionView extends StatelessWidget {
178178
crossAxisAlignment: CrossAxisAlignment.center,
179179
mainAxisSize: MainAxisSize.min,
180180
children: [
181-
Container(
181+
SizedBox(
182182
height: 40,
183183
// color: Colors.red,
184184
child: Column(

lib/image_library/model/image_properties.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ class ImageProperties {
2727
}
2828
}
2929

30-
String get resolution => '${width} × ${height}';
30+
String get resolution => '$width × $height';
3131

3232
String get megapixels {
3333
final mp = (width * height) / 1000000;

lib/image_library/model/saved_image_model.dart

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
import 'dart:io';
22
import 'dart:typed_data';
3-
import 'package:flutter/material.dart';
3+
import '../../util/app_logger.dart';
44

55
class SavedImage {
66
final String id;
@@ -49,7 +49,7 @@ class SavedImage {
4949
}
5050
return null;
5151
} catch (e) {
52-
debugPrint('Error reading image file: $e');
52+
AppLogger.error('Error reading image file: $e');
5353
return null;
5454
}
5555
}

lib/image_library/provider/image_library_provider.dart

Lines changed: 20 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@ import 'package:magicepaperapp/image_library/model/saved_image_model.dart';
44
import 'dart:convert';
55
import 'dart:io';
66
import 'package:path_provider/path_provider.dart';
7+
import '../../util/app_logger.dart';
78

89
class ImageLibraryProvider extends ChangeNotifier {
910
List<SavedImage> _savedImages = [];
@@ -82,10 +83,10 @@ class ImageLibraryProvider extends ChangeNotifier {
8283
_searchQuery = '';
8384
_selectedSource = 'all';
8485

85-
debugPrint('All data cleared successfully');
86+
AppLogger.info('All data cleared successfully');
8687
notifyListeners();
8788
} catch (e) {
88-
debugPrint('Error clearing all data: $e');
89+
AppLogger.error('Error clearing all data: $e');
8990
rethrow;
9091
}
9192
}
@@ -107,30 +108,30 @@ class ImageLibraryProvider extends ChangeNotifier {
107108
if (await image.fileExists()) {
108109
_savedImages.add(image);
109110
} else {
110-
debugPrint('Image file not found: ${image.filePath}');
111+
AppLogger.warning('Image file not found: ${image.filePath}');
111112
}
112113
} catch (e) {
113-
debugPrint('Error parsing individual image metadata: $e');
114+
AppLogger.error('Error parsing individual image metadata: $e');
114115
}
115116
}
116117
} catch (e) {
117-
debugPrint('Error parsing JSON metadata file: $e');
118+
AppLogger.error('Error parsing JSON metadata file: $e');
118119
}
119120
}
120121
}
121122
if (_savedImages.isNotEmpty) {
122123
const encoder = JsonEncoder.withIndent(' ');
123124
final imageJsonList = _savedImages.map((img) => img.toJson()).toList();
124125
final prettyJson = encoder.convert(imageJsonList);
125-
debugPrint('Loaded image metadata (JSON):\n$prettyJson');
126+
AppLogger.debug('Loaded image metadata (JSON):\n$prettyJson');
126127
} else {
127-
debugPrint('No saved images to print.');
128+
AppLogger.debug('No saved images to print.');
128129
}
129130
await _cleanupOrphanedFiles();
130-
debugPrint('Loaded ${_savedImages.length} images successfully');
131+
AppLogger.info('Loaded ${_savedImages.length} images successfully');
131132
_isInitialized = true;
132133
} catch (e) {
133-
debugPrint('Error loading saved images: $e');
134+
AppLogger.error('Error loading saved images: $e');
134135
} finally {
135136
_isLoading = false;
136137
notifyListeners();
@@ -162,10 +163,11 @@ class ImageLibraryProvider extends ChangeNotifier {
162163
);
163164
_savedImages.add(savedImage);
164165
await _persistMetadata();
165-
debugPrint('Successfully saved image: $name (${imageData.length} bytes)');
166+
AppLogger.info(
167+
'Successfully saved image: $name (${imageData.length} bytes)');
166168
notifyListeners();
167169
} catch (e) {
168-
debugPrint('Error saving image: $e');
170+
AppLogger.error('Error saving image: $e');
169171
rethrow;
170172
}
171173
}
@@ -184,7 +186,7 @@ class ImageLibraryProvider extends ChangeNotifier {
184186
await _persistMetadata();
185187
notifyListeners();
186188
} catch (e) {
187-
debugPrint('Error deleting image: $e');
189+
AppLogger.error('Error deleting image: $e');
188190
rethrow;
189191
}
190192
}
@@ -206,7 +208,7 @@ class ImageLibraryProvider extends ChangeNotifier {
206208
await _persistMetadata();
207209
notifyListeners();
208210
} catch (e) {
209-
debugPrint('Error renaming image: $e');
211+
AppLogger.error('Error renaming image: $e');
210212
rethrow;
211213
}
212214
}
@@ -229,10 +231,10 @@ class ImageLibraryProvider extends ChangeNotifier {
229231
final jsonString = jsonEncode(imageJsonList);
230232
await _metadataFile!.writeAsString(jsonString);
231233
final fileSize = await _metadataFile!.length();
232-
debugPrint('Metadata file size: $fileSize bytes');
233-
debugPrint('Metadata saved to: ${_metadataFile!.path}');
234+
AppLogger.debug('Metadata file size: $fileSize bytes');
235+
AppLogger.debug('Metadata saved to: ${_metadataFile!.path}');
234236
} catch (e) {
235-
debugPrint('Error persisting metadata: $e');
237+
AppLogger.error('Error persisting metadata: $e');
236238
rethrow;
237239
}
238240
}
@@ -244,12 +246,12 @@ class ImageLibraryProvider extends ChangeNotifier {
244246
final validFilePaths = _savedImages.map((img) => img.filePath).toSet();
245247
for (final file in files) {
246248
if (file is File && !validFilePaths.contains(file.path)) {
247-
debugPrint('Deleting orphaned file: ${file.path}');
249+
AppLogger.debug('Deleting orphaned file: ${file.path}');
248250
await file.delete();
249251
}
250252
}
251253
} catch (e) {
252-
debugPrint('Error cleaning up orphaned files: $e');
254+
AppLogger.error('Error cleaning up orphaned files: $e');
253255
}
254256
}
255257
}

lib/image_library/services/image_operations_service.dart

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@ import 'package:image/image.dart' as img;
1212
import 'dart:typed_data';
1313
import 'package:magicepaperapp/l10n/app_localizations.dart';
1414
import 'package:magicepaperapp/provider/getitlocator.dart';
15+
import '../../util/app_logger.dart';
1516

1617
AppLocalizations appLocalizations = getIt.get<AppLocalizations>();
1718

@@ -233,7 +234,7 @@ class ImageOperationsService {
233234
imageFrame.dispose();
234235
return properties;
235236
} catch (e) {
236-
debugPrint('Error loading image properties: $e');
237+
AppLogger.error('Error loading image properties: $e');
237238
return null;
238239
}
239240
}

lib/image_library/services/image_save_handler.dart

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ import 'package:permission_handler/permission_handler.dart';
88
import 'package:magicepaperapp/image_library/provider/image_library_provider.dart';
99
import 'package:magicepaperapp/image_library/services/image_operations_service.dart';
1010
import 'package:magicepaperapp/image_library/widgets/dialogs/image_save_dialog.dart';
11+
import '../../util/app_logger.dart';
1112

1213
class ImageSaveHandler {
1314
final BuildContext context;
@@ -87,7 +88,7 @@ class ImageSaveHandler {
8788
}
8889
return _hasStoragePermission;
8990
} catch (e) {
90-
debugPrint('Error requesting storage permission: $e');
91+
AppLogger.error('Error requesting storage permission: $e');
9192
_hasStoragePermission = false;
9293
return false;
9394
}
@@ -100,7 +101,7 @@ class ImageSaveHandler {
100101
await requestStoragePermission();
101102
},
102103
onCancel: () {
103-
debugPrint('Storage permission dialog cancelled');
104+
AppLogger.debug('Storage permission dialog cancelled');
104105
},
105106
colorAccent: colorAccent,
106107
colorBlack: colorBlack,

lib/image_library/widgets/dialogs/image_properties_dialog.dart

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ import 'package:magicepaperapp/image_library/utils/date_utils.dart' as dt;
77
import 'package:magicepaperapp/image_library/utils/filter_utils.dart';
88
import 'package:magicepaperapp/image_library/utils/source_utils.dart';
99
import 'package:magicepaperapp/constants/color_constants.dart';
10+
import '../../../util/app_logger.dart';
1011

1112
class ImagePropertiesDialog extends StatefulWidget {
1213
final SavedImage image;
@@ -41,7 +42,7 @@ class _ImagePropertiesDialogState extends State<ImagePropertiesDialog> {
4142
_isLoadingProperties = false;
4243
});
4344
} catch (e) {
44-
debugPrint('Error in dialog loading image properties: $e');
45+
AppLogger.error('Error in dialog loading image properties: $e');
4546
setState(() {
4647
_isLoadingProperties = false;
4748
});

lib/ndef_screen/ndef_screen.dart

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@ import 'package:magicepaperapp/ndef_screen/widgets/nfc_read_card.dart';
1111
import 'package:magicepaperapp/ndef_screen/app_launcher_card.dart';
1212
import 'package:magicepaperapp/view/widget/common_scaffold_widget.dart';
1313
import 'dart:async';
14+
import '../util/app_logger.dart';
1415

1516
AppLocalizations appLocalizations = getIt.get<AppLocalizations>();
1617

@@ -106,7 +107,7 @@ class _NDEFScreenState extends State<NDEFScreen> with WidgetsBindingObserver {
106107
previousAvailability, _nfcController.availability);
107108
}
108109
} catch (e) {
109-
debugPrint('Error checking NFC availability: $e');
110+
AppLogger.error('Error checking NFC availability: $e');
110111
}
111112
}
112113

lib/ndef_screen/nfc_read_screen.dart

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ import 'package:magicepaperapp/ndef_screen/widgets/nfc_status_card.dart';
77
import 'package:magicepaperapp/ndef_screen/widgets/nfc_read_card.dart';
88
import 'package:magicepaperapp/view/widget/common_scaffold_widget.dart';
99
import 'dart:async';
10+
import '../util/app_logger.dart';
1011

1112
AppLocalizations appLocalizations = getIt.get<AppLocalizations>();
1213

@@ -84,7 +85,7 @@ class _NFCReadScreenState extends State<NFCReadScreen>
8485
);
8586
}
8687
} catch (e) {
87-
debugPrint('Error checking NFC availability: $e');
88+
AppLogger.error('Error checking NFC availability: $e');
8889
}
8990
}
9091

lib/ndef_screen/nfc_write_screen.dart

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@ import 'package:magicepaperapp/ndef_screen/widgets/nfc_status_card.dart';
1010
import 'package:magicepaperapp/ndef_screen/widgets/nfc_write_card.dart';
1111
import 'package:magicepaperapp/view/widget/common_scaffold_widget.dart';
1212
import 'dart:async';
13+
import '../util/app_logger.dart';
1314

1415
AppLocalizations appLocalizations = getIt.get<AppLocalizations>();
1516

@@ -107,7 +108,7 @@ class _NFCWriteScreenState extends State<NFCWriteScreen>
107108
);
108109
}
109110
} catch (e) {
110-
debugPrint('Error checking NFC availability: $e');
111+
AppLogger.error('Error checking NFC availability: $e');
111112
}
112113
}
113114

0 commit comments

Comments
 (0)