aboutsummaryrefslogtreecommitdiffstats
path: root/commands
diff options
context:
space:
mode:
authorBen Fiedler <git@services.bfiedler.ch>2020-04-23 03:56:28 +0200
committerReto Brunner <reto@labrat.space>2020-04-23 21:02:45 +0200
commit7c89143c709e58f22a4939886ccb15bd639eec17 (patch)
tree4714984f29b0f42014436401665d9430ec288d31 /commands
parent280d8e7e1cce40f11c6d5f368f9060486556f96b (diff)
downloadaerc-7c89143c709e58f22a4939886ccb15bd639eec17.tar.gz
Use aerc.PushError where appropriate
Forgot an unused import, to save you the hassle here is v2.
Diffstat (limited to 'commands')
-rw-r--r--commands/account/mkdir.go5
-rw-r--r--commands/compose/send.go3
-rw-r--r--commands/exec.go3
-rw-r--r--commands/msg/archive.go5
-rw-r--r--commands/msg/copy.go4
-rw-r--r--commands/msg/delete.go5
-rw-r--r--commands/msg/modify-labels.go4
-rw-r--r--commands/msg/move.go4
-rw-r--r--commands/msg/pipe.go3
-rw-r--r--commands/msg/read.go8
-rw-r--r--commands/term.go9
11 files changed, 14 insertions, 39 deletions
diff --git a/commands/account/mkdir.go b/commands/account/mkdir.go
index bb7e38a9..9d16063c 100644
--- a/commands/account/mkdir.go
+++ b/commands/account/mkdir.go
@@ -5,8 +5,6 @@ import (
"strings"
"time"
- "github.com/gdamore/tcell"
-
"git.sr.ht/~sircmpwn/aerc/widgets"
"git.sr.ht/~sircmpwn/aerc/worker/types"
)
@@ -42,8 +40,7 @@ func (MakeDir) Execute(aerc *widgets.Aerc, args []string) error {
aerc.PushStatus("Directory created.", 10*time.Second)
acct.Directories().Select(name)
case *types.Error:
- aerc.PushStatus(" "+msg.Error.Error(), 10*time.Second).
- Color(tcell.ColorDefault, tcell.ColorRed)
+ aerc.PushError(" "+msg.Error.Error())
}
})
return nil
diff --git a/commands/compose/send.go b/commands/compose/send.go
index c8f7cc58..f726a602 100644
--- a/commands/compose/send.go
+++ b/commands/compose/send.go
@@ -246,8 +246,7 @@ func (Send) Execute(aerc *widgets.Aerc, args []string) error {
r.Close()
composer.Close()
case *types.Error:
- aerc.PushStatus(" "+msg.Error.Error(), 10*time.Second).
- Color(tcell.ColorDefault, tcell.ColorRed)
+ aerc.PushError(" "+msg.Error.Error())
r.Close()
composer.Close()
}
diff --git a/commands/exec.go b/commands/exec.go
index f615b71e..e90652de 100644
--- a/commands/exec.go
+++ b/commands/exec.go
@@ -33,8 +33,7 @@ func (ExecCmd) Execute(aerc *widgets.Aerc, args []string) error {
go func() {
err := cmd.Run()
if err != nil {
- aerc.PushStatus(" "+err.Error(), 10*time.Second).
- Color(tcell.ColorDefault, tcell.ColorRed)
+ aerc.PushError(" "+err.Error())
} else {
color := tcell.ColorDefault
if cmd.ProcessState.ExitCode() != 0 {
diff --git a/commands/msg/archive.go b/commands/msg/archive.go
index 783ab049..b7a14e7b 100644
--- a/commands/msg/archive.go
+++ b/commands/msg/archive.go
@@ -7,8 +7,6 @@ import (
"sync"
"time"
- "github.com/gdamore/tcell"
-
"git.sr.ht/~sircmpwn/aerc/commands"
"git.sr.ht/~sircmpwn/aerc/models"
"git.sr.ht/~sircmpwn/aerc/widgets"
@@ -88,8 +86,7 @@ func (Archive) Execute(aerc *widgets.Aerc, args []string) error {
case *types.Done:
wg.Done()
case *types.Error:
- aerc.PushStatus(" "+msg.Error.Error(), 10*time.Second).
- Color(tcell.ColorDefault, tcell.ColorRed)
+ aerc.PushError(" "+msg.Error.Error())
success = false
wg.Done()
}
diff --git a/commands/msg/copy.go b/commands/msg/copy.go
index 30022f18..c2f79c6f 100644
--- a/commands/msg/copy.go
+++ b/commands/msg/copy.go
@@ -6,7 +6,6 @@ import (
"time"
"git.sr.ht/~sircmpwn/getopt"
- "github.com/gdamore/tcell"
"git.sr.ht/~sircmpwn/aerc/commands"
"git.sr.ht/~sircmpwn/aerc/widgets"
@@ -61,8 +60,7 @@ func (Copy) Execute(aerc *widgets.Aerc, args []string) error {
case *types.Done:
aerc.PushStatus("Messages copied.", 10*time.Second)
case *types.Error:
- aerc.PushStatus(" "+msg.Error.Error(), 10*time.Second).
- Color(tcell.ColorDefault, tcell.ColorRed)
+ aerc.PushError(" "+msg.Error.Error())
}
})
return nil
diff --git a/commands/msg/delete.go b/commands/msg/delete.go
index fb0d1f81..39a9cc5c 100644
--- a/commands/msg/delete.go
+++ b/commands/msg/delete.go
@@ -4,8 +4,6 @@ import (
"errors"
"time"
- "github.com/gdamore/tcell"
-
"git.sr.ht/~sircmpwn/aerc/lib"
"git.sr.ht/~sircmpwn/aerc/models"
"git.sr.ht/~sircmpwn/aerc/widgets"
@@ -49,8 +47,7 @@ func (Delete) Execute(aerc *widgets.Aerc, args []string) error {
case *types.Done:
aerc.PushStatus("Messages deleted.", 10*time.Second)
case *types.Error:
- aerc.PushStatus(" "+msg.Error.Error(), 10*time.Second).
- Color(tcell.ColorDefault, tcell.ColorRed)
+ aerc.PushError(" "+msg.Error.Error())
}
})
diff --git a/commands/msg/modify-labels.go b/commands/msg/modify-labels.go
index a53292e4..6fcd6edf 100644
--- a/commands/msg/modify-labels.go
+++ b/commands/msg/modify-labels.go
@@ -7,7 +7,6 @@ import (
"git.sr.ht/~sircmpwn/aerc/commands"
"git.sr.ht/~sircmpwn/aerc/widgets"
"git.sr.ht/~sircmpwn/aerc/worker/types"
- "github.com/gdamore/tcell"
)
type ModifyLabels struct{}
@@ -59,8 +58,7 @@ func (ModifyLabels) Execute(aerc *widgets.Aerc, args []string) error {
case *types.Done:
aerc.PushStatus("labels updated", 10*time.Second)
case *types.Error:
- aerc.PushStatus(" "+msg.Error.Error(), 10*time.Second).
- Color(tcell.ColorDefault, tcell.ColorRed)
+ aerc.PushError(" "+msg.Error.Error())
}
})
return nil
diff --git a/commands/msg/move.go b/commands/msg/move.go
index 37402ef1..182817e0 100644
--- a/commands/msg/move.go
+++ b/commands/msg/move.go
@@ -6,7 +6,6 @@ import (
"time"
"git.sr.ht/~sircmpwn/getopt"
- "github.com/gdamore/tcell"
"git.sr.ht/~sircmpwn/aerc/commands"
"git.sr.ht/~sircmpwn/aerc/widgets"
@@ -72,8 +71,7 @@ func (Move) Execute(aerc *widgets.Aerc, args []string) error {
case *types.Done:
aerc.PushStatus("Message moved to "+joinedArgs, 10*time.Second)
case *types.Error:
- aerc.PushStatus(" "+msg.Error.Error(), 10*time.Second).
- Color(tcell.ColorDefault, tcell.ColorRed)
+ aerc.PushError(" "+msg.Error.Error())
}
})
return nil
diff --git a/commands/msg/pipe.go b/commands/msg/pipe.go
index 9491cf5d..136d5337 100644
--- a/commands/msg/pipe.go
+++ b/commands/msg/pipe.go
@@ -93,8 +93,7 @@ func (Pipe) Execute(aerc *widgets.Aerc, args []string) error {
}()
err = ecmd.Run()
if err != nil {
- aerc.PushStatus(" "+err.Error(), 10*time.Second).
- Color(tcell.ColorDefault, tcell.ColorRed)
+ aerc.PushError(" "+err.Error())
} else {
color := tcell.ColorDefault
if ecmd.ProcessState.ExitCode() != 0 {
diff --git a/commands/msg/read.go b/commands/msg/read.go
index ef075236..8613f3f9 100644
--- a/commands/msg/read.go
+++ b/commands/msg/read.go
@@ -7,8 +7,6 @@ import (
"git.sr.ht/~sircmpwn/getopt"
- "github.com/gdamore/tcell"
-
"git.sr.ht/~sircmpwn/aerc/lib"
"git.sr.ht/~sircmpwn/aerc/models"
"git.sr.ht/~sircmpwn/aerc/widgets"
@@ -95,8 +93,7 @@ func submitReadChange(aerc *widgets.Aerc, store *lib.MessageStore,
case *types.Done:
aerc.PushStatus(msg_success, 10*time.Second)
case *types.Error:
- aerc.PushStatus(" "+msg.Error.Error(), 10*time.Second).
- Color(tcell.ColorDefault, tcell.ColorRed)
+ aerc.PushError(" "+msg.Error.Error())
}
})
}
@@ -109,8 +106,7 @@ func submitReadChangeWg(aerc *widgets.Aerc, store *lib.MessageStore,
case *types.Done:
wg.Done()
case *types.Error:
- aerc.PushStatus(" "+msg.Error.Error(), 10*time.Second).
- Color(tcell.ColorDefault, tcell.ColorRed)
+ aerc.PushError(" "+msg.Error.Error())
*success = false
wg.Done()
}
diff --git a/commands/term.go b/commands/term.go
index 459f4059..c75fda5f 100644
--- a/commands/term.go
+++ b/commands/term.go
@@ -2,12 +2,10 @@ package commands
import (
"os/exec"
- "time"
- "git.sr.ht/~sircmpwn/aerc/widgets"
-
- "github.com/gdamore/tcell"
"github.com/riywo/loginshell"
+
+ "git.sr.ht/~sircmpwn/aerc/widgets"
)
type Term struct{}
@@ -48,8 +46,7 @@ func TermCore(aerc *widgets.Aerc, args []string) error {
term.OnClose = func(err error) {
aerc.RemoveTab(term)
if err != nil {
- aerc.PushStatus(" "+err.Error(), 10*time.Second).
- Color(tcell.ColorDefault, tcell.ColorRed)
+ aerc.PushError(" "+err.Error())
}
}
return nil