Browse Source

:bug: Opening notebook repeatedly causes reference count to increase https://github.com/siyuan-note/siyuan/issues/11378

Daniel 1 year ago
parent
commit
fc1e9154b3
4 changed files with 14 additions and 30 deletions
  1. 1 1
      kernel/model/index.go
  2. 11 0
      kernel/sql/block_ref.go
  3. 2 18
      kernel/sql/queue.go
  4. 0 11
      kernel/sql/upsert.go

+ 1 - 1
kernel/model/index.go

@@ -280,7 +280,7 @@ func IndexRefs() {
 			}
 
 			util.IncBootProgress(bootProgressPart, "Indexing ref "+defTree.ID)
-			sql.InsertRefsTreeQueue(defTree)
+			sql.UpdateRefsTreeQueue(defTree)
 			if 1 < i && 0 == i%64 {
 				util.PushStatusBar(fmt.Sprintf(Conf.Language(55), i))
 			}

+ 11 - 0
kernel/sql/block_ref.go

@@ -57,3 +57,14 @@ func deleteRefs(tx *sql.Tx, tree *parse.Tree) (err error) {
 	}
 	return
 }
+
+func insertRefs(tx *sql.Tx, tree *parse.Tree) (err error) {
+	refs, fileAnnotationRefs := refsFromTree(tree)
+	if err = insertBlockRefs(tx, refs); nil != err {
+		return
+	}
+	if err = insertFileAnnotationRefs(tx, fileAnnotationRefs); nil != err {
+		return
+	}
+	return err
+}

+ 2 - 18
kernel/sql/queue.go

@@ -40,9 +40,9 @@ var (
 
 type dbQueueOperation struct {
 	inQueueTime                   time.Time
-	action                        string      // upsert/delete/delete_id/rename/rename_sub_tree/delete_box/delete_box_refs/insert_refs/index/delete_ids/update_block_content/delete_assets
+	action                        string      // upsert/delete/delete_id/rename/rename_sub_tree/delete_box/delete_box_refs/index/delete_ids/update_block_content/delete_assets
 	indexTree                     *parse.Tree // index
-	upsertTree                    *parse.Tree // upsert/insert_refs/update_refs/delete_refs
+	upsertTree                    *parse.Tree // upsert/update_refs/delete_refs
 	removeTreeBox, removeTreePath string      // delete
 	removeTreeID                  string      // delete_id
 	removeTreeIDs                 []string    // delete_ids
@@ -185,8 +185,6 @@ func execOp(op *dbQueueOperation, tx *sql.Tx, context map[string]interface{}) (e
 		err = deleteByBoxTx(tx, op.box)
 	case "delete_box_refs":
 		err = deleteRefsByBoxTx(tx, op.box)
-	case "insert_refs":
-		err = insertRefs(tx, op.upsertTree)
 	case "update_refs":
 		err = upsertRefs(tx, op.upsertTree)
 	case "delete_refs":
@@ -273,20 +271,6 @@ func UpdateRefsTreeQueue(tree *parse.Tree) {
 	operationQueue = append(operationQueue, newOp)
 }
 
-func InsertRefsTreeQueue(tree *parse.Tree) {
-	dbQueueLock.Lock()
-	defer dbQueueLock.Unlock()
-
-	newOp := &dbQueueOperation{upsertTree: tree, inQueueTime: time.Now(), action: "insert_refs"}
-	for i, op := range operationQueue {
-		if "insert_refs" == op.action && op.upsertTree.ID == tree.ID {
-			operationQueue[i] = newOp
-			return
-		}
-	}
-	operationQueue = append(operationQueue, newOp)
-}
-
 func DeleteBoxRefsQueue(boxID string) {
 	dbQueueLock.Lock()
 	defer dbQueueLock.Unlock()

+ 0 - 11
kernel/sql/upsert.go

@@ -389,17 +389,6 @@ func insertFileAnnotationRefs0(tx *sql.Tx, bulk []*FileAnnotationRef) (err error
 	return
 }
 
-func insertRefs(tx *sql.Tx, tree *parse.Tree) (err error) {
-	refs, fileAnnotationRefs := refsFromTree(tree)
-	if err = insertBlockRefs(tx, refs); nil != err {
-		return
-	}
-	if err = insertFileAnnotationRefs(tx, fileAnnotationRefs); nil != err {
-		return
-	}
-	return err
-}
-
 func indexTree(tx *sql.Tx, tree *parse.Tree, context map[string]interface{}) (err error) {
 	blocks, spans, assets, attributes := fromTree(tree.Root, tree)
 	refs, fileAnnotationRefs := refsFromTree(tree)