[possible Bug]: ChangeAwareCache not capturing initialized data

Aperta
#669 2 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
3/5
Tempo stimato
1-2 giorni
Idoneità per principianti
35/100
Tipo di issue
Bug
Chiarezza
Abbastanza chiara
Stato di attività
Ferma
Stack tecnologico
csharp
Ambito
data

Direzione di ricerca

Start with src/DynamicData/Cache/ChangeAwareCache.cs at the linked line 54, then compare its initialization and CaptureChanges() behavior with the ChangeAwareList code mentioned in the issue. Check whether an initialized dictionary should produce an initial ChangeSet and whether existing tests cover that behavior. Done means the expected behavior is clear and covered by a regression test; the issue does not identify a test file.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

bug
Describe the bug 🐞

looking through the code because of something unrelated the following took me by surprise:

https://github.com/reactivemarbles/DynamicData/blob/main/src/DynamicData/Cache/ChangeAwareCache.cs#L54

initializing the ChangeAwareCache with a Dictionary and subsequently calling CaptureChanges() did not produce an initial ChangeSet!

I don't know if that might have an effect on the whole library as for example the ChangeAwareList does have code for the initial ChangeSet

Step to reproduce

just code review

Reproduction repository

https://github.com/reactivemarbles/DynamicData

Expected behavior

to send an initial ChangeSet

Screenshots 🖼️

No response

IDE

No response

Operating system

No response

Version

No response

Device

No response

DynamicData Version

No response

Additional information ℹ️

No response

Lingua principale
C#
Stelle
1.9k
Fork
194
Merge medio
12m
PR unite (30g)
1

Preparare l'ambiente

Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di reactivemarbles/DynamicData

Tutte le issue di reactivemarbles/DynamicData

Issue simili

Altre issue su C#

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.