From 9d9493e41a2d9fb79221986507d1789265355d2d Mon Sep 17 00:00:00 2001 From: Adrian Reber Date: Tue, 16 Oct 2018 11:50:10 +0000 Subject: Add checkAllAndLatest() function The check about the --all and --latest option is used and repeated and some commands. Factor it out and put it into common. Signed-off-by: Adrian Reber --- cmd/podman/common.go | 15 +++++++++++++++ 1 file changed, 15 insertions(+) (limited to 'cmd/podman/common.go') diff --git a/cmd/podman/common.go b/cmd/podman/common.go index 8ae1c9e0f..3401b1a93 100644 --- a/cmd/podman/common.go +++ b/cmd/podman/common.go @@ -89,6 +89,21 @@ func validateFlags(c *cli.Context, flags []cli.Flag) error { return nil } +// checkAllAndLatest checks that --all and --latest are used correctly +func checkAllAndLatest(c *cli.Context) error { + argLen := len(c.Args()) + if (c.Bool("all") || c.Bool("latest")) && argLen > 0 { + return errors.Errorf("no arguments are needed with --all or --latest") + } + if c.Bool("all") && c.Bool("latest") { + return errors.Errorf("--all and --latest cannot be used together") + } + if argLen < 1 && !c.Bool("all") && !c.Bool("latest") { + return errors.Errorf("you must provide at least one pod name or id") + } + return nil +} + // getContext returns a non-nil, empty context func getContext() context.Context { return context.TODO() -- cgit v1.2.3-54-g00ecf