Anki/rslib/src/scheduler/bury_and_suspend.rs
RumovZ 173a5bfed5
Improve temporary table handling (#1976)
* Use all_cards_for_search() helper

* Add CardTableGuard

* Add for_each_card_in_search() helper

* Add all_cards_for_search_in_order() helper

* Add all_cards_for_ids() helper

* Return siblings for bury instead of only searching

* Remove redundant clear_searched_cards_table() calls

* Add with_searched_cards_table()

* Remove false comment

* Add NoteTableGuard

* Add with_ids_in_searched_notes_table() helper

* Make some last routines use table helpers
2022-07-22 17:51:26 +10:00

197 lines
6.5 KiB
Rust

// Copyright: Ankitects Pty Ltd and contributors
// License: GNU AGPL, version 3 or later; http://www.gnu.org/licenses/agpl.html
use super::timing::SchedTimingToday;
use crate::{
card::CardQueue,
config::SchedulerVersion,
pb::{
bury_or_suspend_cards_request::Mode as BuryOrSuspendMode,
unbury_deck_request::Mode as UnburyDeckMode,
},
prelude::*,
search::{JoinSearches, SearchNode, StateKind},
};
impl Card {
/// True if card was buried/suspended prior to the call.
pub(crate) fn restore_queue_after_bury_or_suspend(&mut self) -> bool {
if !matches!(
self.queue,
CardQueue::Suspended | CardQueue::SchedBuried | CardQueue::UserBuried
) {
false
} else {
self.restore_queue_from_type();
true
}
}
}
impl Collection {
pub(crate) fn unbury_if_day_rolled_over(&mut self, timing: SchedTimingToday) -> Result<()> {
let last_unburied = self.get_last_unburied_day();
let today = timing.days_elapsed;
if last_unburied < today || (today + 7) < last_unburied {
self.unbury_on_day_rollover(today)?;
}
Ok(())
}
/// Unbury cards from the previous day.
/// Done automatically, and does not mark the cards as modified.
pub(crate) fn unbury_on_day_rollover(&mut self, today: u32) -> Result<()> {
self.for_each_card_in_search(StateKind::Buried, |col, mut card| {
card.restore_queue_after_bury_or_suspend();
col.storage.update_card(&card)
})?;
self.set_last_unburied_day(today)
}
/// Unsuspend/unbury cards. Marks the cards as modified.
fn unsuspend_or_unbury_searched_cards(&mut self, cards: Vec<Card>) -> Result<()> {
let usn = self.usn()?;
for original in cards {
let mut card = original.clone();
if card.restore_queue_after_bury_or_suspend() {
self.update_card_inner(&mut card, original, usn)?;
}
}
Ok(())
}
pub fn unbury_or_unsuspend_cards(&mut self, cids: &[CardId]) -> Result<OpOutput<()>> {
self.transact(Op::UnburyUnsuspend, |col| {
let cards = col.all_cards_for_ids(cids, false)?;
col.unsuspend_or_unbury_searched_cards(cards)
})
}
pub fn unbury_deck(&mut self, deck_id: DeckId, mode: UnburyDeckMode) -> Result<OpOutput<()>> {
let state = match mode {
UnburyDeckMode::All => StateKind::Buried,
UnburyDeckMode::UserOnly => StateKind::UserBuried,
UnburyDeckMode::SchedOnly => StateKind::SchedBuried,
};
self.transact(Op::UnburyUnsuspend, |col| {
let cards =
col.all_cards_for_search(SearchNode::DeckIdWithChildren(deck_id).and(state))?;
col.unsuspend_or_unbury_searched_cards(cards)
})
}
/// Bury/suspend cards in search table, and clear it.
/// Marks the cards as modified.
fn bury_or_suspend_cards_inner(
&mut self,
cards: Vec<Card>,
mode: BuryOrSuspendMode,
) -> Result<usize> {
let mut count = 0;
let usn = self.usn()?;
let sched = self.scheduler_version();
let desired_queue = match mode {
BuryOrSuspendMode::Suspend => CardQueue::Suspended,
BuryOrSuspendMode::BurySched => CardQueue::SchedBuried,
BuryOrSuspendMode::BuryUser => {
if sched == SchedulerVersion::V1 {
// v1 scheduler only had one bury type
CardQueue::SchedBuried
} else {
CardQueue::UserBuried
}
}
};
for original in cards {
let mut card = original.clone();
if card.queue != desired_queue {
// do not bury suspended cards as that would unsuspend them
if card.queue != CardQueue::Suspended {
if sched == SchedulerVersion::V1 {
card.remove_from_filtered_deck_restoring_queue(sched);
card.remove_from_learning();
}
card.queue = desired_queue;
count += 1;
self.update_card_inner(&mut card, original, usn)?;
}
}
}
Ok(count)
}
pub fn bury_or_suspend_cards(
&mut self,
cids: &[CardId],
mode: BuryOrSuspendMode,
) -> Result<OpOutput<usize>> {
let op = match mode {
BuryOrSuspendMode::Suspend => Op::Suspend,
BuryOrSuspendMode::BurySched | BuryOrSuspendMode::BuryUser => Op::Bury,
};
self.transact(op, |col| {
let cards = col.all_cards_for_ids(cids, false)?;
col.bury_or_suspend_cards_inner(cards, mode)
})
}
pub(crate) fn bury_siblings(
&mut self,
cid: CardId,
nid: NoteId,
include_new: bool,
include_reviews: bool,
include_day_learn: bool,
) -> Result<usize> {
let cards = self.storage.all_siblings_for_bury(
cid,
nid,
include_new,
include_reviews,
include_day_learn,
)?;
self.bury_or_suspend_cards_inner(cards, BuryOrSuspendMode::BurySched)
}
}
#[cfg(test)]
mod test {
use crate::{
card::{Card, CardQueue},
collection::{open_test_collection, Collection},
search::{SortMode, StateKind},
};
#[test]
fn unbury() {
let mut col = open_test_collection();
let mut card = Card {
queue: CardQueue::UserBuried,
..Default::default()
};
col.add_card(&mut card).unwrap();
let assert_count = |col: &mut Collection, cnt| {
assert_eq!(
col.search_cards(StateKind::Buried, SortMode::NoOrder)
.unwrap()
.len(),
cnt
);
};
assert_count(&mut col, 1);
// day 0, last unburied 0, so no change
let timing = col.timing_today().unwrap();
col.unbury_if_day_rolled_over(timing).unwrap();
assert_count(&mut col, 1);
// move creation time back and it should succeed
let mut stamp = col.storage.creation_stamp().unwrap();
stamp.0 -= 86_400;
col.set_creation_stamp(stamp).unwrap();
let timing = col.timing_today().unwrap();
col.unbury_if_day_rolled_over(timing).unwrap();
assert_count(&mut col, 0);
}
}