diff --git a/pkg/api/transactions.go b/pkg/api/transactions.go index 10456ed9..7240641c 100644 --- a/pkg/api/transactions.go +++ b/pkg/api/transactions.go @@ -1447,7 +1447,7 @@ func (a *TransactionsApi) TransactionParseImportFileHandler(c *core.WebContext) return nil, errs.Or(err, errs.ErrOperationFailed) } - tagMap := a.transactionTags.GetTagNameMapByList(tags) + tagMap := a.transactionTags.GetVisibleTagNameMapByList(tags) parsedTransactions, _, _, _, _, _, err := dataImporter.ParseImportedData(c, user, fileData, utcOffset, accountMap, expenseCategoryMap, incomeCategoryMap, transferCategoryMap, tagMap) diff --git a/pkg/cli/user_data.go b/pkg/cli/user_data.go index 3de53d0f..2adaf3a1 100644 --- a/pkg/cli/user_data.go +++ b/pkg/cli/user_data.go @@ -957,7 +957,7 @@ func (l *UserDataCli) getUserEssentialDataForImport(c *core.CliContext, uid int6 return nil, nil, nil, nil, nil, err } - tagMap = l.tags.GetTagNameMapByList(tags) + tagMap = l.tags.GetVisibleTagNameMapByList(tags) return accountMap, expenseCategoryMap, incomeCategoryMap, transferCategoryMap, tagMap, nil } diff --git a/pkg/mcp/add_transaction_tool_handler.go b/pkg/mcp/add_transaction_tool_handler.go index 45111c0b..8024f37b 100644 --- a/pkg/mcp/add_transaction_tool_handler.go +++ b/pkg/mcp/add_transaction_tool_handler.go @@ -159,7 +159,7 @@ func (h *mcpAddTransactionToolHandler) Handle(c *core.WebContext, callToolReq *M return nil, nil, err } - tagMaps := services.GetTransactionTagService().GetTagNameMapByList(allTags) + tagMaps := services.GetTransactionTagService().GetVisibleTagNameMapByList(allTags) tagIds = make([]int64, 0, len(addTransactionRequest.Tags)) for _, tagName := range addTransactionRequest.Tags { diff --git a/pkg/services/transaction_tags.go b/pkg/services/transaction_tags.go index 2405f1e3..fd8e5b31 100644 --- a/pkg/services/transaction_tags.go +++ b/pkg/services/transaction_tags.go @@ -508,14 +508,20 @@ func (s *TransactionTagService) GetTagMapByList(tags []*models.TransactionTag) m return tagMap } -// GetTagNameMapByList returns a transaction tag map by a list -func (s *TransactionTagService) GetTagNameMapByList(tags []*models.TransactionTag) map[string]*models.TransactionTag { +// GetVisibleTagNameMapByList returns a visible transaction tag map by a list +func (s *TransactionTagService) GetVisibleTagNameMapByList(tags []*models.TransactionTag) map[string]*models.TransactionTag { tagMap := make(map[string]*models.TransactionTag) for i := 0; i < len(tags); i++ { tag := tags[i] + + if tag.Hidden { + continue + } + tagMap[tag.Name] = tag } + return tagMap }