Merge pull request #432 from Leptopoda/fix/list_cached_image

Fix/list cached image

packages/neon/neon/lib/neon.dart

@@ -91,9 +91,7 @@ part 'src/utils/validators.dart';
 part 'src/widgets/account_settings_tile.dart';
 part 'src/widgets/account_tile.dart';
 part 'src/widgets/app_implementation_icon.dart';
-part 'src/widgets/cached_api_image.dart';
 part 'src/widgets/cached_image.dart';
-part 'src/widgets/cached_url_image.dart';
 part 'src/widgets/dialog.dart';
 part 'src/widgets/exception.dart';
 part 'src/widgets/image_wrapper.dart';

packages/neon/neon/lib/src/widgets/cached_api_image.dart

@@ -1,32 +0,0 @@
-part of '../../neon.dart';
-
-typedef APIImageDownloader = Future<Uint8List> Function();
-
-class NeonCachedApiImage extends NeonCachedImage {
-  NeonCachedApiImage({
-    required final Account account,
-    required final String cacheKey,
-    required final APIImageDownloader download,
-    final String? etag,
-    super.size,
-    super.fit,
-    super.svgColor,
-    super.iconColor,
-    super.key,
-  }) : super(
-          getImageFile: () async {
-            final realKey = '${account.id}-$cacheKey';
-            final cacheFile = await _cacheManager.getFileFromCache(realKey);
-            if (cacheFile != null && cacheFile.validTill.isAfter(DateTime.now())) {
-              return cacheFile.file;
-            }
-
-            return _cacheManager.putFile(
-              realKey,
-              await download(),
-              maxAge: const Duration(days: 7),
-              eTag: etag,
-            );
-          },
-        );
-}

packages/neon/neon/lib/src/widgets/cached_image.dart

@@ -1,19 +1,50 @@
 part of '../../neon.dart';
 
-final _cacheManager = DefaultCacheManager();
+typedef CacheReviver = FutureOr<Uint8List?> Function(CacheManager cacheManager);
+typedef ImageDownloader = FutureOr<Uint8List> Function();
+typedef CacheWriter = Future<void> Function(CacheManager cacheManager, Uint8List image);
 
-abstract class NeonCachedImage extends StatefulWidget {
+class NeonCachedImage extends StatefulWidget {
   const NeonCachedImage({
-    required this.getImageFile,
+    required this.image,
+    required Key super.key,
     this.isSvgHint = false,
     this.size,
     this.fit,
     this.svgColor,
     this.iconColor,
-    super.key,
   });
 
-  final Future<File> Function() getImageFile;
+  NeonCachedImage.url({
+    required final String url,
+    final Key? key,
+    this.isSvgHint = false,
+    this.size,
+    this.fit,
+    this.svgColor,
+    this.iconColor,
+  })  : image = _getImageFromUrl(url),
+        super(key: key ?? Key(url));
+
+  NeonCachedImage.custom({
+    required final ImageDownloader getImage,
+    required final String cacheKey,
+    final CacheReviver? reviver,
+    final CacheWriter? writeCache,
+    this.isSvgHint = false,
+    this.size,
+    this.fit,
+    this.svgColor,
+    this.iconColor,
+  })  : image = _customImageGetter(
+          reviver,
+          getImage,
+          writeCache,
+          cacheKey,
+        ),
+        super(key: Key(cacheKey));
+
+  final Future<Uint8List> image;
   final bool isSvgHint;
 
   final Size? size;
@@ -22,20 +53,83 @@ abstract class NeonCachedImage extends StatefulWidget {
   final Color? svgColor;
   final Color? iconColor;
 
+  static Future<Uint8List> _getImageFromUrl(final String url) async {
+    final file = await _cacheManager.getSingleFile(url);
+    return file.readAsBytes();
+  }
+
+  static Future<Uint8List> _customImageGetter(
+    final CacheReviver? checkCache,
+    final ImageDownloader getImage,
+    final CacheWriter? writeCache,
+    final String cacheKey,
+  ) async {
+    final cached = await checkCache?.call(_cacheManager) ?? await _defaultCacheReviver(cacheKey);
+    if (cached != null) {
+      return cached;
+    }
+
+    final data = await getImage();
+
+    unawaited(writeCache?.call(_cacheManager, data) ?? _defaultCacheWriter(data, cacheKey));
+
+    return data;
+  }
+
+  static Future<Uint8List?> _defaultCacheReviver(final String cacheKey) async {
+    final cacheFile = await _cacheManager.getFileFromCache(cacheKey);
+    if (cacheFile != null && cacheFile.validTill.isAfter(DateTime.now())) {
+      return cacheFile.file.readAsBytes();
+    }
+
+    return null;
+  }
+
+  static Future<void> _defaultCacheWriter(
+    final Uint8List data,
+    final String cacheKey,
+  ) async {
+    await _cacheManager.putFile(
+      cacheKey,
+      data,
+      maxAge: const Duration(days: 7),
+    );
+  }
+
+  static final _cacheManager = DefaultCacheManager();
+
   @override
   State<NeonCachedImage> createState() => _NeonCachedImageState();
 }
 
 class _NeonCachedImageState extends State<NeonCachedImage> {
-  late Future<File> future = widget.getImageFile();
-
   @override
   Widget build(final BuildContext context) => Center(
-        child: FutureBuilder<File>(
+        child: FutureBuilder<Uint8List>(
-          future: future,
+          future: widget.image,
           builder: (final context, final fileSnapshot) {
-            if (fileSnapshot.hasData) {
+            if (!fileSnapshot.hasData) {
-              final content = fileSnapshot.requireData.readAsBytesSync();
+              return SizedBox(
+                width: widget.size?.width,
+                child: NeonLinearProgressIndicator(
+                  color: widget.iconColor,
+                ),
+              );
+            }
+
+            if (fileSnapshot.hasError) {
+              return NeonException(
+                fileSnapshot.error,
+                onRetry: () {
+                  setState(() {});
+                },
+                onlyIcon: true,
+                iconSize: widget.size?.shortestSide,
+                color: widget.iconColor ?? Theme.of(context).colorScheme.error,
+              );
+            }
+
+            final content = fileSnapshot.requireData;
 
             try {
               // TODO: Is this safe enough?
@@ -59,27 +153,6 @@ class _NeonCachedImageState extends State<NeonCachedImage> {
               fit: widget.fit,
               gaplessPlayback: true,
             );
-            }
-            if (fileSnapshot.hasError) {
-              return NeonException(
-                fileSnapshot.error,
-                onRetry: () {
-                  setState(() {
-                    // ignore: discarded_futures
-                    future = widget.getImageFile();
-                  });
-                },
-                onlyIcon: true,
-                iconSize: widget.size?.shortestSide,
-                color: widget.iconColor ?? Theme.of(context).colorScheme.error,
-              );
-            }
-            return SizedBox(
-              width: widget.size?.width,
-              child: NeonLinearProgressIndicator(
-                color: widget.iconColor,
-              ),
-            );
           },
         ),
       );

packages/neon/neon/lib/src/widgets/cached_url_image.dart

@@ -1,15 +0,0 @@
-part of '../../neon.dart';
-
-class NeonCachedUrlImage extends NeonCachedImage {
-  NeonCachedUrlImage({
-    required final String url,
-    super.size,
-    super.fit,
-    super.svgColor,
-    super.iconColor,
-    super.key,
-  }) : super(
-          getImageFile: () => _cacheManager.getSingleFile(url),
-          isSvgHint: Uri.parse(url).path.endsWith('.svg'),
-        );
-}

packages/neon/neon/lib/src/widgets/drawer.dart

@@ -187,7 +187,7 @@ class NeonDrawerHeader extends StatelessWidget {
               ],
               if (capabilities.requireData.capabilities.theming?.logo != null) ...[
                 Flexible(
-                  child: NeonCachedUrlImage(
+                  child: NeonCachedImage.url(
                     url: capabilities.requireData.capabilities.theming!.logo!,
                   ),
                 ),

packages/neon/neon/lib/src/widgets/user_avatar.dart

@@ -43,7 +43,7 @@ class _UserAvatarState extends State<NeonUserAvatar> {
   @override
   Widget build(final BuildContext context) => LayoutBuilder(
         builder: (final context, final constraints) {
-          final isDark = Theme.of(context).brightness == Brightness.dark;
+          final brightness = Theme.of(context).brightness;
           size = constraints.constrain(Size.square(widget.size)).shortestSide;
           final pixelSize = (size * MediaQuery.of(context).devicePixelRatio).toInt();
           return Stack(
@@ -53,11 +53,10 @@ class _UserAvatarState extends State<NeonUserAvatar> {
                 radius: size / 2,
                 backgroundColor: widget.backgroundColor,
                 child: ClipOval(
-                  child: NeonCachedApiImage(
+                  child: NeonCachedImage.custom(
-                    account: widget.account,
+                    cacheKey: '${widget.account.id}-avatar-${widget.username}-$brightness$pixelSize',
-                    cacheKey: 'avatar-${widget.username}-${isDark ? 'dark' : 'light'}$pixelSize',
+                    getImage: () async {
-                    download: () async {
+                      if (brightness == Brightness.dark) {
-                      if (isDark) {
                         return widget.account.client.core.getDarkAvatar(
                           userId: widget.username,
                           size: pixelSize,

packages/neon/neon_files/lib/widgets/file_preview.dart

@@ -21,28 +21,31 @@ class FilePreview extends StatelessWidget {
   final BorderRadius? borderRadius;
   final bool withBackground;
 
-  int get width => size.width.toInt();
-  int get height => size.height.toInt();
-
   @override
   Widget build(final BuildContext context) {
     final color = this.color ?? Theme.of(context).colorScheme.primary;
+
     return SizedBox.fromSize(
       size: size,
-      child: StreamBuilder<bool?>(
+      child: Builder(
-        stream: bloc.options.showPreviewsOption.stream,
+        builder: (final context) {
+          if (details.isDirectory) {
+            return Icon(
+              MdiIcons.folder,
+              color: color,
+              size: size.shortestSide,
+            );
+          }
+
+          return OptionBuilder<bool>(
+            option: bloc.options.showPreviewsOption,
             builder: (final context, final showPreviewsSnapshot) {
-          if ((showPreviewsSnapshot.data ?? false) && (details.hasPreview ?? false)) {
+              if (showPreviewsSnapshot && (details.hasPreview ?? false)) {
                 final account = Provider.of<AccountsBloc>(context, listen: false).activeAccount.value!;
-            final child = NeonCachedApiImage(
+                final child = FilePreviewImage(
                   account: account,
-              cacheKey: 'preview-${details.path.join('/')}-$width-$height',
+                  file: details,
-              etag: details.etag,
+                  size: size,
-              download: () async => account.client.core.getPreview(
-                file: details.path.join('/'),
-                x: width,
-                y: height,
-              ),
                 );
                 if (withBackground) {
                   return NeonImageWrapper(
@@ -54,21 +57,63 @@ class FilePreview extends StatelessWidget {
                 return child;
               }
 
-          if (details.isDirectory) {
-            return Icon(
-              MdiIcons.folder,
-              color: color,
-              size: size.shortestSide,
-            );
-          }
-
               return FileIcon(
                 details.name,
                 color: color,
                 size: size.shortestSide,
               );
             },
+          );
+        },
       ),
     );
   }
 }
+
+class FilePreviewImage extends NeonCachedImage {
+  factory FilePreviewImage({
+    required final Account account,
+    required final FileDetails file,
+    required final Size size,
+  }) {
+    final width = size.width.toInt();
+    final height = size.height.toInt();
+    final path = file.path.join('/');
+    final cacheKey = '${account.id}-preview-$path-$width-$height';
+
+    return FilePreviewImage._(
+      account: account,
+      file: file,
+      size: size,
+      cacheKey: cacheKey,
+      path: path,
+      width: width,
+      height: height,
+    );
+  }
+
+  FilePreviewImage._({
+    required final Account account,
+    required final FileDetails file,
+    required Size super.size,
+    required super.cacheKey,
+    required final String path,
+    required final int width,
+    required final int height,
+  }) : super.custom(
+          getImage: () async => account.client.core.getPreview(
+            file: path,
+            x: width,
+            y: height,
+          ),
+          writeCache: (final cacheManager, final data) async {
+            await cacheManager.putFile(
+              cacheKey,
+              data,
+              maxAge: const Duration(days: 7),
+              eTag: file.etag,
+            );
+          },
+          isSvgHint: file.mimeType?.contains('svg') ?? false,
+        );
+}

packages/neon/neon_news/lib/widgets/articles_view.dart

@@ -116,7 +116,7 @@ class _NewsArticlesViewState extends State<NewsArticlesView> {
               ),
             ),
             if (article.mediaThumbnail != null) ...[
-              NeonCachedUrlImage(
+              NeonCachedImage.url(
                 url: article.mediaThumbnail!,
                 size: const Size(100, 50),
                 fit: BoxFit.cover,

packages/neon/neon_news/lib/widgets/feed_icon.dart

@@ -21,7 +21,7 @@ class NewsFeedIcon extends StatelessWidget {
       size: Size.square(size),
       borderRadius: borderRadius,
       child: faviconLink != null && faviconLink.isNotEmpty
-          ? NeonCachedUrlImage(
+          ? NeonCachedImage.url(
               url: faviconLink,
               size: Size.square(size),
               iconColor: Theme.of(context).colorScheme.primary,

packages/neon/neon_notifications/lib/pages/main.dart

@@ -78,7 +78,7 @@ class _NotificationsMainPageState extends State<NotificationsMainPage> {
             )
           : SizedBox.fromSize(
               size: const Size.square(40),
-              child: NeonCachedUrlImage(
+              child: NeonCachedImage.url(
                 url: notification.icon!,
                 size: const Size.square(40),
                 svgColor: Theme.of(context).colorScheme.primary,