Revert "[mob][photos] Add option to make 'only them' filter work a little different only for testing, for feedback on which option is better"

This reverts commit ebae813ee37fde8132ea5e11312862176adf0cc2.
This commit is contained in:
ashilkn 2024-11-07 22:34:09 +05:30
parent 65e8ff4da8
commit 219ff7d2e9
3 changed files with 3 additions and 41 deletions

View File

@ -60,23 +60,6 @@ class _MLDebugSectionWidgetState extends State<MLDebugSectionWidget> {
final Logger logger = Logger("MLDebugSectionWidget");
return Column(
children: [
MenuItemWidget(
captionedTextWidget: const CaptionedTextWidget(
title:
"Exclude files of face IDs that are not in any cluster on applying 'only them' filter.",
),
trailingWidget: ToggleSwitchWidget(
value: () => localSettings
.excludeFilesOfFaceIDsThatAreNotInAnyClusterOnOnlyThemFilter,
onChanged: () async {
await localSettings
.setExcludeFilesOfFaceIDsThatAreNotInAnyClusterOnOnlyThemFilter(
!localSettings
.excludeFilesOfFaceIDsThatAreNotInAnyClusterOnOnlyThemFilter,
);
},
),
),
sectionOptionSpacing,
MenuItemWidget(
captionedTextWidget: FutureBuilder<IndexStatus>(

View File

@ -91,13 +91,10 @@ Future<List<EnteFile>> getFilteredFiles(
exceptClusters: selectedClusterIDs,
);
if (localSettings
.excludeFilesOfFaceIDsThatAreNotInAnyClusterOnOnlyThemFilter) {
final filesOfFaceIDsNotInAnyCluster =
await MLDataDB.instance.getAllFileIDsOfFaceIDsNotInAnyCluster();
final filesOfFaceIDsNotInAnyCluster =
await MLDataDB.instance.getAllFileIDsOfFaceIDsNotInAnyCluster();
fileIDsToAvoid.addAll(filesOfFaceIDsNotInAnyCluster);
}
fileIDsToAvoid.addAll(filesOfFaceIDsNotInAnyCluster);
final result =
intersectionOfSelectedFaceFiltersFileIDs.difference(fileIDsToAvoid);

View File

@ -17,8 +17,6 @@ class LocalSettings {
static const kRateUsPromptThreshold = 2;
static const shouldLoopVideoKey = "video.should_loop";
static const onGuestViewKey = "on_guest_view";
static const kExcludeFilesOfFaceIDsThatAreNotInAnyClusterOnOnlyThemFilter =
"excludeFilesOfFaceIDsThatAreNotInAnyClusterOnOnlyThemFilter";
final SharedPreferences _prefs;
@ -66,22 +64,6 @@ class LocalSettings {
bool get userEnabledMultiplePart =>
_prefs.getBool(kEnableMultiplePart) ?? false;
bool get excludeFilesOfFaceIDsThatAreNotInAnyClusterOnOnlyThemFilter =>
_prefs.getBool(
kExcludeFilesOfFaceIDsThatAreNotInAnyClusterOnOnlyThemFilter,
) ??
true;
Future<bool> setExcludeFilesOfFaceIDsThatAreNotInAnyClusterOnOnlyThemFilter(
bool value,
) async {
await _prefs.setBool(
kExcludeFilesOfFaceIDsThatAreNotInAnyClusterOnOnlyThemFilter,
value,
);
return value;
}
Future<bool> setUserEnabledMultiplePart(bool value) async {
await _prefs.setBool(kEnableMultiplePart, value);
return value;