blockid #81 #83

-   feat #83 just add blockid when review new card.
-   fix #81 add multi blockids;

docs/docs/changelog.md

@@ -4,6 +4,12 @@ All notable changes to this project will be documented in this file. Dates are d
 
 Generated by [`auto-changelog`](https://github.com/CookPete/auto-changelog).
 
+
+## [1.12.5.12]
+
+-   feat #83 just add blockid when review new card.
+-   fix #81 add multi blockids;
+
 ## [1.12.5.11]
 
 -   feat #78 fsrs v5 setting support enable_short_term and enable_fuzz (by @ishiko732).

manifest.json

@@ -1,7 +1,7 @@
 {
     "id": "obsidian-spaced-repetition-recall",
     "name": "Spaced Repetition Recall",
-    "version": "1.12.5.11",
+    "version": "1.12.5.12",
     "minAppVersion": "1.2.8",
     "description": "Fight the forgetting curve by reviewing flashcards & entire notes.",
     "author": "Newdea",

package.json

@@ -1,6 +1,6 @@
 {
     "name": "obsidian-spaced-repetition",
-    "version": "1.12.5.11",
+    "version": "1.12.5.12",
     "description": "Fight the forgetting curve by reviewing flashcards & entire notes.",
     "main": "main.js",
     "scripts": {

src/Question.ts

@@ -252,9 +252,9 @@ export class Question {
                 blockId = this.questionText.obsidianBlockId = this.questionText.genBlockId;
             }
             if (blockId) {
-                this.questionText.original += "\n";
-                if (this.isCardCommentsOnSameLine(settings)) result += ` ${blockId}\n`;
+                if (!this.questionText.original.endsWith("```")) result += ` ${blockId}\n`;
                 else result += `\n${blockId}\n`;
+                this.questionText.original += "\n";
             }
 
             return result;

src/dataStore/itemTrans.ts

@@ -166,10 +166,10 @@ export class ItemTrans {
             let deckname = dtppath?.hasPath ? dtppath.path[0] : topicPath.path[0];
             deckname = Tags.isDefaultDackName(deckname) ? deckname : "#" + deckname;
             store.updateCardItems(trackedFile, cardinfo, count, deckname, false);
-            updateCardObjs(question.cards, cardinfo, scheduling);
+            const update = updateCardObjs(question.cards, cardinfo, scheduling);
 
             // update question
-            if (question.questionText.genBlockId) {
+            if (question.questionText.genBlockId && update) {
                 question.hasChanged = true;
             } else {
                 question.hasChanged = false;
@@ -182,14 +182,18 @@ function updateCardObjs(cards: Card[], cardinfo: CardInfo, scheduling: RegExpMat
     const schedInfoList: CardScheduleInfo[] =
         NoteCardScheduleParser.createInfoList_algo(scheduling);
     const carditemIds = cardinfo.itemIds;
+    let update = false;
     for (let i = 0; i < cards.length; i++) {
         const cardObj = cards[i];
         const hasScheduleInfo: boolean = i < schedInfoList.length;
         const schedule: CardScheduleInfo = schedInfoList[i];
-        cardObj.scheduleInfo =
-            hasScheduleInfo && !schedule.isDummyScheduleForNewCard() ? schedule : null;
+        const hassched = !schedule.isDummyScheduleForNewCard();
+        cardObj.scheduleInfo = hasScheduleInfo && hassched ? schedule : null;
         cardObj.Id = carditemIds[i];
+
+        if (hassched) update = true;
     }
+    return update;
 }
 
 export function itemToShedNote(item: RepetitionItem, note: TFile): SchedNote {

src/generateParser.ts

@@ -248,7 +248,7 @@ loose_line
     }
 
 annotation
-  = $("<!--SR:" (!"-->" .)+ "-->")
+  = $($("<!--SR:" (!"-->" .)+ "-->") / blockid)
     
 nonempty_text_till_newline
   = $non_newline+
@@ -271,6 +271,9 @@ nonemptyspace
 optional_whitespaces
   = whitespace_char*
 
+blockid
+=$([\\^][a-zA-Z0-9-]+)
+
 whitespace_char = ([ \\f\\t\\v\\u0020\\u00a0\\u1680\\u2000-\\u200a\\u2028\\u2029\\u202f\\u205f\\u3000\\ufeff])
 `;
 }