From 1b912fbbe98ae9b0a78da2b448f6121944d9b9b1 Mon Sep 17 00:00:00 2001 From: Robin Jarry Date: Sat, 13 Apr 2024 21:15:56 +0200 Subject: compose: explicitly identify converted text/* parts When running :accept, an error is displayed on the review screen: text/calendar error: no command defined for mime/type When running :multipart text/xxx, its contents are not specified. They are regenerated every time the review screen is displayed. When running :accept, a text/calendar part is added with actual contents. Update the Part object to hold a boolean initialized when first being created. If body is nil, identify the part as "Converted" and update its contents every time the review screen is displayed. When body is not nil but contains text (e.g. when running :accept), identify the part as *not* converted and ignore the conversion step. Fixes: cbcabfafaab2 ("compose: allow writing multipart/alternative messages") Reported-by: Inwit Signed-off-by: Robin Jarry Tested-by: Inwit --- commands/compose/multipart.go | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) (limited to 'commands') diff --git a/commands/compose/multipart.go b/commands/compose/multipart.go index 13fe8ca3..a38e0a6e 100644 --- a/commands/compose/multipart.go +++ b/commands/compose/multipart.go @@ -1,7 +1,6 @@ package compose import ( - "bytes" "fmt" "git.sr.ht/~rjarry/aerc/app" @@ -52,7 +51,7 @@ func (m Multipart) Execute(args []string) error { map[string]string{"Charset": "UTF-8"}, // the actual content of the part will be rendered // every time the body of the email is updated - bytes.NewReader([]byte{}), + nil, ) if err != nil { return err -- cgit