Fix underflow of learning count (#1444)
`counts.learning` includes interday learning cards, so it is not suitable to determine how many cards from the (intraday!) learning queue are already included in the learning count when updating it.
This commit is contained in:
parent
d665dbc9a7
commit
56d8402f89
@ -40,13 +40,14 @@ impl CardQueues {
|
||||
learning_count: self.counts.learning,
|
||||
learning_cutoff: self.current_learning_cutoff,
|
||||
};
|
||||
let last_ahead_cutoff = self.current_learn_ahead_cutoff();
|
||||
self.current_learning_cutoff = TimestampSecs::now();
|
||||
let ahead_cutoff = self.current_learn_ahead_cutoff();
|
||||
let new_ahead_cutoff = self.current_learn_ahead_cutoff();
|
||||
let new_learning_cards = self
|
||||
.intraday_learning
|
||||
.iter()
|
||||
.skip(self.counts.learning)
|
||||
.take_while(|e| e.due <= ahead_cutoff)
|
||||
.skip_while(|e| e.due <= last_ahead_cutoff)
|
||||
.take_while(|e| e.due <= new_ahead_cutoff)
|
||||
.count();
|
||||
self.counts.learning += new_learning_cards;
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user