tui: only perform database updates for modified messages
This commit is contained in:
@@ -47,16 +47,6 @@ func (m *Model) cloneMessage(message conversation.Message, selected bool) tea.Cm
|
||||
}
|
||||
}
|
||||
|
||||
func (m *Model) updateMessageContent(message *conversation.Message) tea.Cmd {
|
||||
return func() tea.Msg {
|
||||
err := m.App.UpdateMessageContent(message)
|
||||
if err != nil {
|
||||
return shared.WrapError(err)
|
||||
}
|
||||
return msgMessageUpdated(message)
|
||||
}
|
||||
}
|
||||
|
||||
func (m *Model) cycleSelectedRoot(conv *conversation.Conversation, dir model.MessageCycleDirection) tea.Cmd {
|
||||
if len(conv.RootMessages) < 2 {
|
||||
return nil
|
||||
|
||||
@@ -29,7 +29,7 @@ func (m *Model) setMessageContents(i int, content string) {
|
||||
if i >= len(m.App.Messages) {
|
||||
panic("i out of range")
|
||||
}
|
||||
m.App.Messages[i].Content = content
|
||||
m.App.SetMessageContents(i, content)
|
||||
m.messageCache[i] = m.renderMessage(i)
|
||||
}
|
||||
|
||||
@@ -91,19 +91,17 @@ func (m *Model) Update(msg tea.Msg) (shared.ViewModel, tea.Cmd) {
|
||||
case input:
|
||||
m.input.SetValue(contents)
|
||||
case selectedMessage:
|
||||
toEdit := m.App.Messages[m.selectedMessage]
|
||||
if toEdit.Content != contents {
|
||||
toEdit.Content = contents
|
||||
m.setMessage(m.selectedMessage, toEdit)
|
||||
if m.App.Messages[m.selectedMessage].Content != contents {
|
||||
m.setMessageContents(m.selectedMessage, contents)
|
||||
m.updateContent()
|
||||
if m.persistence && toEdit.ID > 0 {
|
||||
if m.persistence && m.App.Messages[m.selectedMessage].ID > 0 {
|
||||
// create clone of message with its new contents
|
||||
cmds = append(cmds, m.cloneMessage(toEdit, true))
|
||||
cmds = append(cmds, m.cloneMessage(m.App.Messages[m.selectedMessage], true))
|
||||
}
|
||||
}
|
||||
}
|
||||
case msgConversationMessagesLoaded:
|
||||
m.App.Messages = msg.messages
|
||||
m.App.SetMessages(msg.messages)
|
||||
m.selectedMessage = len(msg.messages) - 1
|
||||
m.rebuildMessageCache()
|
||||
m.updateContent()
|
||||
|
||||
Reference in New Issue
Block a user