ember/fluffychat
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

deleted irrelevant constructs filtering function

Browse source
This commit is contained in:
ggurdin 2024-11-07 10:05:49 -05:00
parent 8ee4ea31c8
commit 1f5d66e203
No known key found for this signature in database
GPG key ID: A01CB41737CBB478
1 changed files with 4 additions and 28 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

32
lib/pangea/controllers/get_analytics_controller.dart
View file

@ -3,7 +3,6 @@ import 'dart:math';
import 'package:fluffychat/pangea/constants/class_default_values.dart';
import 'package:fluffychat/pangea/constants/local.key.dart';
import 'package:fluffychat/pangea/constants/match_rule_ids.dart';
import 'package:fluffychat/pangea/controllers/pangea_controller.dart';
import 'package:fluffychat/pangea/controllers/put_analytics_controller.dart';
import 'package:fluffychat/pangea/enum/construct_type_enum.dart';
@ -227,25 +226,20 @@ class GetAnalyticsController {
final List<ConstructAnalyticsEvent> constructEvents =
await allMyConstructs();
final List<OneConstructUse> unfilteredUses = [];
final List<OneConstructUse> uses = [];
for (final event in constructEvents) {
unfilteredUses.addAll(event.content.uses);
uses.addAll(event.content.uses);
}
// filter out any constructs that are not relevant to the user
final List<OneConstructUse> filteredUses = await filterConstructs(
unfilteredConstructs: unfilteredUses,
);
// if there isn't already a valid, local cache, cache the filtered uses
if (local == null) {
cacheConstructs(
constructType: constructType,
uses: filteredUses,
uses: uses,
);
}
return filteredUses;
return uses;
}
/// Get the last time the user updated their analytics for their current l2
@ -272,24 +266,6 @@ class GetAnalyticsController {
return await analyticsRoom.getAnalyticsEvents(userId: client.userID!) ?? [];
}
/// Filter out constructs that are not relevant to the user, specifically those from
/// rooms in which the user is a teacher and those that are interative translation span constructs
/// @ggurdin - is this still relevant now that we're not doing grammar constructs?
/// maybe it should actually be filtering all grammar uses, though this is maybe more efficiently done
/// in the fromJson of the model reading the event content, then maybe we can get rid of that enum entry entirely
Future<List<OneConstructUse>> filterConstructs({
required List<OneConstructUse> unfilteredConstructs,
}) async {
return unfilteredConstructs
.where(
(use) =>
use.lemma != "Try interactive translation" &&
use.lemma != "itStart" ||
use.lemma != MatchRuleIds.interactiveTranslation,
)
.toList();
}
/// Get the cached construct uses for the current user, if it exists
List<OneConstructUse>? getConstructsLocal({
ConstructTypeEnum? constructType,