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

Merge pull request #861 from pangeachat/stack-overflow-fix

Browse source 
don't call messagesSinceUpdate from clearMessagesSinceUpdate repeatedly
This commit is contained in:
ggurdin 2024-10-28 15:56:41 -04:00 • committed by GitHub
commit 80fc7a896b
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 8 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

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

@ -162,7 +162,8 @@ class GetAnalyticsController {
return formattedCache;
} catch (err) {
// if something goes wrong while trying to format the local data, clear it
_pangeaController.myAnalytics.clearMessagesSinceUpdate();
_pangeaController.myAnalytics
.clearMessagesSinceUpdate(clearDrafts: true);
return {};
}
} catch (exception, stackTrace) {

7
lib/pangea/controllers/my_analytics_controller.dart
View file

@ -243,7 +243,12 @@ class MyAnalyticsController extends BaseController<AnalyticsStream> {
}
/// Clears the local cache of recently sent constructs. Called before updating analytics
void clearMessagesSinceUpdate() {
void clearMessagesSinceUpdate({clearDrafts = false}) {
if (clearDrafts) {
_pangeaController.pStoreService.delete(PLocalKey.messagesSinceUpdate);
return;
}
final localCache = _pangeaController.analytics.messagesSinceUpdate;
final draftKeys = localCache.keys.where((key) => key.startsWith('draft'));
if (draftKeys.isEmpty) {