0ct0pu5/ente
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Refactor: Use fn as getter for sortAsc value

Browse source 
Signed-off-by: Neeraj Gupta <254676+ua741@users.noreply.github.com>
This commit is contained in:
Neeraj Gupta 2023-05-26 10:04:32 +05:30
parent 4560f99574
commit 0dc5279f8b
2 changed files with 15 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
lib/ui/viewer/gallery/collection_page.dart
View file

@ -74,7 +74,7 @@ class CollectionPage extends StatelessWidget {
selectedFiles: _selectedFiles, selectedFiles: _selectedFiles,
initialFiles: initialFiles, initialFiles: initialFiles,
albumName: c.collection.name, albumName: c.collection.name,
sortOrderAsc: c.collection.pubMagicMetadata.asc ?? false, sortAsyncFn: () => c.collection.pubMagicMetadata.asc ?? false,
); );
return Scaffold( return Scaffold(
appBar: PreferredSize( appBar: PreferredSize(

20
lib/ui/viewer/gallery/gallery.dart
View file

@ -25,6 +25,8 @@ typedef GalleryLoader = Future<FileLoadResult> Function(
bool? asc, bool? asc,
}); });
typedef SortAscFn = bool Function();
class Gallery extends StatefulWidget { class Gallery extends StatefulWidget {
final GalleryLoader asyncLoader; final GalleryLoader asyncLoader;
final List<File>? initialFiles; final List<File>? initialFiles;
@ -42,7 +44,9 @@ class Gallery extends StatefulWidget {
final Widget loadingWidget; final Widget loadingWidget;
final bool disableScroll; final bool disableScroll;
final bool limitSelectionToOne; final bool limitSelectionToOne;
final bool sortOrderAsc;
// add a Function variable to get sort value in bool
final SortAscFn? sortAsyncFn;
const Gallery({ const Gallery({
required this.asyncLoader, required this.asyncLoader,
@ -61,7 +65,7 @@ class Gallery extends StatefulWidget {
this.loadingWidget = const EnteLoadingWidget(), this.loadingWidget = const EnteLoadingWidget(),
this.disableScroll = false, this.disableScroll = false,
this.limitSelectionToOne = false, this.limitSelectionToOne = false,
this.sortOrderAsc = false, this.sortAsyncFn,
Key? key, Key? key,
}) : super(key: key); }) : super(key: key);
@ -84,6 +88,7 @@ class _GalleryState extends State<Gallery> {
StreamSubscription<TabDoubleTapEvent>? _tabDoubleTapEvent; StreamSubscription<TabDoubleTapEvent>? _tabDoubleTapEvent;
final _forceReloadEventSubscriptions = <StreamSubscription<Event>>[]; final _forceReloadEventSubscriptions = <StreamSubscription<Event>>[];
late String _logTag; late String _logTag;
bool _sortOrderAsc = false;
@override @override
void initState() { void initState() {
@ -91,6 +96,7 @@ class _GalleryState extends State<Gallery> {
"Gallery_${widget.tagPrefix}${kDebugMode ? "_" + widget.albumName! : ""}"; "Gallery_${widget.tagPrefix}${kDebugMode ? "_" + widget.albumName! : ""}";
_logger = Logger(_logTag); _logger = Logger(_logTag);
_logger.finest("init Gallery"); _logger.finest("init Gallery");
_sortOrderAsc = widget.sortAsyncFn != null ? widget.sortAsyncFn!() : false;
_itemScroller = ItemScrollController(); _itemScroller = ItemScrollController();
if (widget.reloadEvent != null) { if (widget.reloadEvent != null) {
_reloadEventSubscription = widget.reloadEvent!.listen((event) async { _reloadEventSubscription = widget.reloadEvent!.listen((event) async {
@ -122,13 +128,15 @@ class _GalleryState extends State<Gallery> {
_forceReloadEventSubscriptions.add( _forceReloadEventSubscriptions.add(
event.listen((event) async { event.listen((event) async {
_logger.finest("Force refresh all files on ${event.reason}"); _logger.finest("Force refresh all files on ${event.reason}");
_sortOrderAsc =
widget.sortAsyncFn != null ? widget.sortAsyncFn!() : false;
final result = await _loadFiles(); final result = await _loadFiles();
_setFilesAndReload(result.files); _setFilesAndReload(result.files);
}), }),
); );
} }
} }
if (widget.initialFiles != null && !widget.sortOrderAsc) { if (widget.initialFiles != null && !_sortOrderAsc) {
_onFilesLoaded(widget.initialFiles!); _onFilesLoaded(widget.initialFiles!);
} }
_loadFiles(limit: kInitialLoadLimit).then((result) async { _loadFiles(limit: kInitialLoadLimit).then((result) async {
@ -156,7 +164,7 @@ class _GalleryState extends State<Gallery> {
galleryLoadStartTime, galleryLoadStartTime,
galleryLoadEndTime, galleryLoadEndTime,
limit: limit, limit: limit,
asc: widget.sortOrderAsc, asc: _sortOrderAsc,
); );
final endTime = DateTime.now().microsecondsSinceEpoch; final endTime = DateTime.now().microsecondsSinceEpoch;
final duration = Duration(microseconds: endTime - startTime); final duration = Duration(microseconds: endTime - startTime);
@ -216,7 +224,7 @@ class _GalleryState extends State<Gallery> {
disableScroll: widget.disableScroll, disableScroll: widget.disableScroll,
emptyState: widget.emptyState, emptyState: widget.emptyState,
asyncLoader: widget.asyncLoader, asyncLoader: widget.asyncLoader,
sortOrderAsc: widget.sortOrderAsc, sortOrderAsc: _sortOrderAsc,
removalEventTypes: widget.removalEventTypes, removalEventTypes: widget.removalEventTypes,
tagPrefix: widget.tagPrefix, tagPrefix: widget.tagPrefix,
scrollBottomSafeArea: widget.scrollBottomSafeArea, scrollBottomSafeArea: widget.scrollBottomSafeArea,
@ -248,7 +256,7 @@ class _GalleryState extends State<Gallery> {
if (dailyFiles.isNotEmpty) { if (dailyFiles.isNotEmpty) {
collatedFiles.add(dailyFiles); collatedFiles.add(dailyFiles);
} }
if (widget.sortOrderAsc) { if (_sortOrderAsc) {
collatedFiles collatedFiles
.sort((a, b) => a[0].creationTime!.compareTo(b[0].creationTime!)); .sort((a, b) => a[0].creationTime!.compareTo(b[0].creationTime!));
} else { } else {