diff --git a/kernel/model/history.go b/kernel/model/history.go index c50e297c8..5de549ff0 100644 --- a/kernel/model/history.go +++ b/kernel/model/history.go @@ -337,6 +337,8 @@ func FullTextSearchHistory(query, box, op string, typ, page int) (ret []string, stmt += " AND op = '" + op + "'" } stmt += " AND path LIKE '%/" + box + "/%' AND path LIKE '%.sy'" + } else if HistoryTypeDocID == typ { + stmt += " AND id = '" + query + "'" } else if HistoryTypeAsset == typ { stmt += " AND path LIKE '%/assets/%'" } @@ -387,6 +389,8 @@ func FullTextSearchHistoryItems(created, query, box, op string, typ int) (ret [] stmt += " AND op = '" + op + "'" } stmt += " AND path LIKE '%/" + box + "/%' AND path LIKE '%.sy'" + } else if HistoryTypeDocID == typ { + stmt += " AND id = '" + query + "'" } else if HistoryTypeAsset == typ { stmt += " AND path LIKE '%/assets/%'" } @@ -592,9 +596,10 @@ func ReindexHistory() (err error) { var validOps = []string{HistoryOpClean, HistoryOpUpdate, HistoryOpDelete, HistoryOpFormat, HistoryOpSync, HistoryOpReplace} const ( - HistoryTypeDocName = 0 - HistoryTypeDoc = 1 - HistoryTypeAsset = 2 + HistoryTypeDocName = 0 // Search docs by doc name + HistoryTypeDoc = 1 // Search docs by doc name and content + HistoryTypeAsset = 2 // Search assets + HistoryTypeDocID = 3 // Search docs by doc id ) func indexHistoryDir(name string, luteEngine *lute.Lute) { @@ -640,6 +645,7 @@ func indexHistoryDir(name string, luteEngine *lute.Lute) { p := strings.TrimPrefix(doc, util.HistoryDir) p = filepath.ToSlash(p[1:]) histories = append(histories, &sql.History{ + ID: tree.Root.ID, Type: HistoryTypeDoc, Op: op, Title: title, @@ -652,7 +658,12 @@ func indexHistoryDir(name string, luteEngine *lute.Lute) { for _, asset := range assets { p := strings.TrimPrefix(asset, util.HistoryDir) p = filepath.ToSlash(p[1:]) + _, id := util.LastID(asset) + if !ast.IsNodeIDPattern(id) { + id = "" + } histories = append(histories, &sql.History{ + ID: id, Type: HistoryTypeAsset, Op: op, Title: filepath.Base(asset), diff --git a/kernel/sql/database.go b/kernel/sql/database.go index bfdd81802..27e799260 100644 --- a/kernel/sql/database.go +++ b/kernel/sql/database.go @@ -237,7 +237,7 @@ func initHistoryDBConnection() { func initHistoryDBTables() { historyDB.Exec("DROP TABLE histories_fts_case_insensitive") - _, err := historyDB.Exec("CREATE VIRTUAL TABLE histories_fts_case_insensitive USING fts5(type UNINDEXED, op UNINDEXED, title, content, path UNINDEXED, created UNINDEXED, tokenize=\"siyuan case_insensitive\")") + _, err := historyDB.Exec("CREATE VIRTUAL TABLE histories_fts_case_insensitive USING fts5(id UNINDEXED, type UNINDEXED, op UNINDEXED, title, content, path UNINDEXED, created UNINDEXED, tokenize=\"siyuan case_insensitive\")") if nil != err { logging.LogFatalf(logging.ExitCodeReadOnlyDatabase, "create table [histories_fts_case_insensitive] failed: %s", err) } diff --git a/kernel/sql/history.go b/kernel/sql/history.go index f60cc646f..40af8f984 100644 --- a/kernel/sql/history.go +++ b/kernel/sql/history.go @@ -27,6 +27,7 @@ import ( ) type History struct { + ID string Type int Op string Title string @@ -87,7 +88,7 @@ func SelectHistoriesRawStmt(stmt string) (ret []*History) { func scanHistoryRows(rows *sql.Rows) (ret *History) { var history History - if err := rows.Scan(&history.Type, &history.Op, &history.Title, &history.Content, &history.Path, &history.Created); nil != err { + if err := rows.Scan(&history.ID, &history.Type, &history.Op, &history.Title, &history.Content, &history.Path, &history.Created); nil != err { logging.LogErrorf("query scan field failed: %s\n%s", err, logging.ShortStack()) return } @@ -112,8 +113,8 @@ func deleteHistoriesByPathPrefix(tx *sql.Tx, pathPrefix string, context map[stri } const ( - HistoriesFTSCaseInsensitiveInsert = "INSERT INTO histories_fts_case_insensitive (type, op, title, content, path, created) VALUES %s" - HistoriesPlaceholder = "(?, ?, ?, ?, ?, ?)" + HistoriesFTSCaseInsensitiveInsert = "INSERT INTO histories_fts_case_insensitive (id, type, op, title, content, path, created) VALUES %s" + HistoriesPlaceholder = "(?, ?, ?, ?, ?, ?, ?)" ) func insertHistories(tx *sql.Tx, histories []*History, context map[string]interface{}) (err error) { @@ -146,6 +147,7 @@ func insertHistories0(tx *sql.Tx, bulk []*History, context map[string]interface{ valueArgs := make([]interface{}, 0, len(bulk)*strings.Count(HistoriesPlaceholder, "?")) for _, b := range bulk { valueStrings = append(valueStrings, HistoriesPlaceholder) + valueArgs = append(valueArgs, b.ID) valueArgs = append(valueArgs, b.Type) valueArgs = append(valueArgs, b.Op) valueArgs = append(valueArgs, b.Title)