Skip to content
This repository has been archived by the owner on Mar 14, 2024. It is now read-only.

Commit

Permalink
Revert "Hide incoming shared (#1584)"
Browse files Browse the repository at this point in the history
This reverts commit 3ed2cae, reversing
changes made to 2a24d2f.
  • Loading branch information
ashilkn authored and ua741 committed Dec 14, 2023
1 parent 78d82e3 commit 4e68907
Show file tree
Hide file tree
Showing 5 changed files with 13 additions and 70 deletions.
10 changes: 0 additions & 10 deletions lib/generated/l10n.dart

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

3 changes: 1 addition & 2 deletions lib/l10n/intl_en.arb
Original file line number Diff line number Diff line change
Expand Up @@ -1159,6 +1159,5 @@
"signOutFromOtherDevices": "Sign out from other devices",
"signOutOtherBody": "If you think someone might know your password, you can force all other devices using your account to sign out.",
"signOutOtherDevices": "Sign out other devices",
"doNotSignOut": "Do not sign out",
"archiveSharedAlbums": "Archive shared albums"
"doNotSignOut": "Do not sign out"
}
30 changes: 11 additions & 19 deletions lib/services/collections_service.dart
Original file line number Diff line number Diff line change
Expand Up @@ -147,7 +147,7 @@ class CollectionsService {
);
}
await _updateDB(updatedCollections);
await _prefs.setInt(_collectionsSyncTimeKey, maxUpdationTime);
_prefs.setInt(_collectionsSyncTimeKey, maxUpdationTime);
watch.logAndReset("till DB insertion ${updatedCollections.length}");
for (final collection in fetchedCollections) {
_cacheLocalPathAndCollection(collection);
Expand Down Expand Up @@ -222,24 +222,16 @@ class CollectionsService {
}

Set<int> archivedOrHiddenCollectionIds() {
final bool archiveIncomingCollections =
LocalSettings.instance.archiveSharedAlbums;
final int ownerID = _config.getUserID()!;
final Set<int> result = <int>{};
for (final collection in _collectionIDToCollections.values) {
if (collection.isHidden() ||
collection.isArchived() ||
collection.hasShareeArchived()) {
result.add(collection.id);
continue;
}
if (archiveIncomingCollections) {
if (collection.owner?.id != ownerID) {
result.add(collection.id);
}
}
}
return result;
return _collectionIDToCollections.values
.toList()
.where(
(element) =>
element.hasShareeArchived() ||
element.isHidden() ||
element.isArchived(),
)
.map((e) => e.id)
.toSet();
}

int getCollectionSyncTime(int collectionID) {
Expand Down
31 changes: 1 addition & 30 deletions lib/ui/settings/advanced_settings_screen.dart
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,6 @@ import "dart:async";

import 'package:flutter/material.dart';
import "package:photos/core/error-reporting/super_logging.dart";
import "package:photos/core/event_bus.dart";
import "package:photos/events/force_reload_home_gallery_event.dart";
import "package:photos/generated/l10n.dart";
import "package:photos/services/memories_service.dart";
import "package:photos/services/user_remote_flag_service.dart";
Expand Down Expand Up @@ -121,32 +119,6 @@ class _AdvancedSettingsScreenState extends State<AdvancedSettingsScreen> {
const SizedBox(
height: 24,
),
MenuItemWidget(
captionedTextWidget: CaptionedTextWidget(
title: S.of(context).archiveSharedAlbums,
),
menuItemColor: colorScheme.fillFaint,
singleBorderRadius: 8,
alignCaptionedTextToLeft: true,
trailingWidget: ToggleSwitchWidget(
value: () =>
LocalSettings.instance.archiveSharedAlbums,
onChanged: () async {
await LocalSettings.instance
.setArchiveSharedAlbums(
!LocalSettings.instance.archiveSharedAlbums,
);
Bus.instance.fire(
ForceReloadHomeGalleryEvent(
"Hide/show shared albums",
),
);
},
),
),
const SizedBox(
height: 24,
),
MenuItemWidget(
captionedTextWidget: CaptionedTextWidget(
title: S.of(context).manageDeviceStorage,
Expand All @@ -159,8 +131,7 @@ class _AdvancedSettingsScreenState extends State<AdvancedSettingsScreen> {
singleBorderRadius: 8,
alignCaptionedTextToLeft: true,
onTap: () async {
await routeToPage(
context, const AppStorageViewer());
routeToPage(context, const AppStorageViewer());
},
),
const SizedBox(
Expand Down
9 changes: 0 additions & 9 deletions lib/utils/local_settings.dart
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,6 @@ class LocalSettings {
static final LocalSettings instance = LocalSettings._privateConstructor();
static const kCollectionSortPref = "collection_sort_pref";
static const kPhotoGridSize = "photo_grid_size";
static const kArchiveSharedAlbums = "archive_shared_albums";
static const kRateUsShownCount = "rate_us_shown_count";
static const kRateUsPromptThreshold = 2;

Expand All @@ -39,18 +38,10 @@ class LocalSettings {
}
}

bool get archiveSharedAlbums {
return _prefs.getBool(kArchiveSharedAlbums) ?? false;
}

Future<void> setPhotoGridSize(int value) async {
await _prefs.setInt(kPhotoGridSize, value);
}

Future<void> setArchiveSharedAlbums(bool value) async {
await _prefs.setBool(kArchiveSharedAlbums, value);
}

int getRateUsShownCount() {
if (_prefs.containsKey(kRateUsShownCount)) {
return _prefs.getInt(kRateUsShownCount)!;
Expand Down

0 comments on commit 4e68907

Please sign in to comment.