Hacktoberfest 2026 : les issues que les mainteneurs ont marquées pour octobre, ouvertes et accessibles aux débutants. Parcourir les issues Hacktoberfest

Make treshold the 1 day treshold for maxObjects configurable

Ouverte
  • Quelque chose ne va pas ?
#500 1 commentaire 3 réactions 0 personnes assignées Voir sur GitHub

Les mainteneurs répondent en général sous 1 jour

Personne n'a encore pris cette issue.

Évaluation

Difficulté
3/5
Temps estimé
1-2 jours
Accessibilité débutants
45/100
Type d'issue
Fonctionnalité
Clarté
Plutôt claire
Activité
À l'abandon
Stack technique
dart, flutter
Domaine
mobile-dev

Piste de recherche

Commencez par CacheObjectProvider.getObjectsOverCapacity et suivez la manière dont l'objet Config transmet actuellement maxObjects et stalePeriod au fournisseur. Ajoutez le paramètre optionnel Duration avec une valeur par défaut d'un jour, transmettez-le via le fournisseur, documentez-le avec les options de configuration associées et vérifiez que le cache respecte le seuil de rétention configuré.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

🏗 Enhancement Proposal

When setting maxNrOfCacheObjects to 100 the cache can still grow above that size. This is caused by logic in the CacheObjectProvider in the getObjectsOverCapacity function.

where: '${CacheObject.columnTouched} < ?',
whereArgs: [
        DateTime.now().subtract(const Duration(days: 1)).millisecondsSinceEpoch
      ],

From an this issue #349 I understand the reasoning behind this default value.

Pitch

For app which are not used very intense I think the default makes sense. However for apps which use a lot of images and are used intensively throughout the day this results in a very big local cache. I've got reports of several GB.

I would argue that the there should be a parameter to override this in the Config object. This would be in line with for example the maxObjects and stalePeriod parameters.

Concretely:

  • Add parameter to Config ObjectMinimumRetention (open for better naming)
  • Type Duration
  • Optional, default value 1 day
  • Pass and use this parameter to the CacheObjectProvider
  • Document

I'm happy to do a PR on this as you think this makes sense.

Platforms affected (mark all that apply)
  • 📱 iOS
  • 🤖 Android
Langage dominant
Dart
Étoiles
814
Forks
511
Merge moyen
28 min
PR mergées (30 j)
12

Préparer son environnement

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de Baseflow/flutter_cache_manager

Toutes les issues de Baseflow/flutter_cache_manager

Issues similaires

Plus d'issues Dart

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.