aerc: commands: remove compose commands from message viewer v1 APPLIED

Robin Jarry: 1
 commands: remove compose commands from message viewer

 1 files changed, 1 insertions(+), 1 deletions(-)

Tested-by: Maarten Aertsen <maarten@nlnetlabs.nl>

[PATCH aerc] commands: remove compose commands from message viewer Export this patch

All compose commands were made available in the message viewer context
by mistake. Remove them.

This fixes a crash when the :send command is invoked while viewing
a message:

Fixes: 159fb38daf53 ("commands: refactor registration")
Fixes: https://lists.sr.ht/~rjarry/aerc-devel/%3CCZ01NTGWNUBV.14A9POG9ITJ6F%40posteo.net%3E
Reported-by: Sertonix <sertonix@posteo.net>
Signed-off-by: Robin Jarry <robin@jarry.cc>
---
 commands/commands.go | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/commands/commands.go b/commands/commands.go
index b617bade2cb7..951d5fdceeed 100644
--- a/commands/commands.go
+++ b/commands/commands.go
@@ -46,7 +46,7 @@ func CurrentContext() CommandContext {
 	case *app.Composer:
 		context |= COMPOSE
 	case *app.MessageViewer:
-		context |= COMPOSE | MESSAGE | MESSAGE_VIEWER
+		context |= MESSAGE | MESSAGE_VIEWER
 	case *app.Terminal:
 		context |= TERMINAL
 	}
-- 
2.43.0

Robin Jarry <robin@jarry.cc>

1 year, 1 month ago

Maarten Aertsen, Feb 22, 2024 at 21:15:

Robin Jarry <robin@jarry.cc>

1 year, 1 month ago

Applied. Thanks!

builds.sr.ht <builds@sr.ht>

1 year, 1 month ago

aerc/patches: SUCCESS in 1m53s

[commands: remove compose commands from message viewer][0] from [Robin Jarry][1]

[0]: https://lists.sr.ht/~rjarry/aerc-devel/patches/49303
[1]: mailto:robin@jarry.cc

✓ #1146234 SUCCESS aerc/patches/openbsd.yml     https://builds.sr.ht/~rjarry/job/1146234
✓ #1146233 SUCCESS aerc/patches/alpine-edge.yml https://builds.sr.ht/~rjarry/job/1146233