diff options
| author | Craig Jennings <c@cjennings.net> | 2026-05-05 05:10:52 -0500 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-05-05 05:10:52 -0500 |
| commit | 8bbb6d30529abd5350884e822b308da917966e73 (patch) | |
| tree | f3c9cb845a2e4780b11ea1bf94417214ba21f836 /org-drill.el | |
| parent | 656faefc0e030ea476f9937116b727331cec607f (diff) | |
| download | org-drill-8bbb6d30529abd5350884e822b308da917966e73.tar.gz org-drill-8bbb6d30529abd5350884e822b308da917966e73.zip | |
fix: drop dead translate_number entry from card-type alist (upstream #43)
The card-type alist mapped translate_number to a function that no
longer exists in the file. Cards with DRILL_CARD_TYPE: translate_number
crashed with void-function during drill instead of being skipped.
Reporter (issue #43, 2021) said they had old decks using the
documented translate_number type and were getting the crash on
restore. The function was apparently removed at some point without
clearing the alist entry.
Removed the alist entry so entry-f's no-presentation-fn branch fires
and returns skip after messaging the user. Legacy decks now degrade
gracefully instead of crashing the session.
Tests in tests/test-org-drill-translate-number-regression.el lock the
behavior in (entry-f returns skip on translate_number, alist no
longer carries the entry).
Diffstat (limited to 'org-drill.el')
| -rw-r--r-- | org-drill.el | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/org-drill.el b/org-drill.el index 599406f..f98bbfe 100644 --- a/org-drill.el +++ b/org-drill.el @@ -294,8 +294,7 @@ This is a buffer-local variable.") ("decline_noun" org-drill-present-noun-declension org-drill-show-answer-noun-declension) - ("spanish_verb" org-drill-present-spanish-verb) - ("translate_number" org-drill-present-translate-number)) + ("spanish_verb" org-drill-present-spanish-verb)) "Alist associating card types with presentation functions. Each entry in the alist takes the form: |