Don't set modified if content matches

imdone · Aug 6, 2023 · 086ac14 · 086ac14
1 parent 8e02187
commit 086ac14
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 3 deletions.
diff --git a/lib/file.js b/lib/file.js
@@ -467,6 +467,8 @@ File.prototype.extractCheckboxTasks = function (config, pos, content) {
     this.content = newContent
 
     this.setModified(true)
+    console.log('extractCheckboxTasks setModified true for file:', this.path)
+
   }
 }
 
@@ -689,10 +691,12 @@ File.prototype.modifyTask = function (task, config, noEmit) {
       text,
       ...description,
       ...linesAfter,
-    ]
+    ].join(lineEnd)
 
-    this.setContent(newContent.join(lineEnd))
+    if (this.getContent().trim() === newContent.trim()) return
+    this.setContent(newContent)
     this.setModified(true)
+    console.log('modifyTask setModified true for file:', this.path)
     if (!noEmit) {
       this.emit('task.modified', task)
       this.emit('file.modified', this)
@@ -752,6 +756,7 @@ File.prototype.modifyCodeOrHashTask = function (re, task, config, noEmit) {
         this.modifyDescription(task, config)
       }
       this.setModified(true)
+      console.log('modifyCodeOrHashTask setModified true for file:', this.path)
       if (!noEmit) {
         this.emit('task.modified', task)
         this.emit('file.modified', self)
@@ -786,6 +791,7 @@ File.prototype.modifyTaskFromHtml = function (task, html) {
 
   this.setContent(beforeDescContent + descContent + afterDescContent)
   this.setModified(true)
+  console.log('modifyTaskFromHtml setModified true for file:', this.path)
 }
 
 File.prototype.modifyTaskFromContent = function (task, content, config) {

diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "imdone-core",
-  "version": "1.32.2",
+  "version": "1.32.3",
   "description": "imdone-core",
   "main": "index.js",
   "scripts": {