From 0c475a0cf33c01dd17befed8b5af9c89d056694d Mon Sep 17 00:00:00 2001
From: Vincent Batts <vbatts@redhat.com>
Date: Fri, 30 May 2014 16:42:37 -0400
Subject: [PATCH] filters: don't encode empty set. API docs

Docker-DCO-1.1-Signed-off-by: Vincent Batts <vbatts@redhat.com> (github: vbatts)
Upstream-commit: 02255ddaa4948fd94450fd5d9d2ee9b147e88352
Component: engine
---
 components/engine/api/client/commands.go      | 27 ++++++++++---------
 .../reference/api/docker_remote_api_v1.12.md  | 10 +++++++
 components/engine/server/server.go            |  2 +-
 3 files changed, 26 insertions(+), 13 deletions(-)

diff --git a/components/engine/api/client/commands.go b/components/engine/api/client/commands.go
index 4e013b8fe8..4f0915ba79 100644
--- a/components/engine/api/client/commands.go
+++ b/components/engine/api/client/commands.go
@@ -1172,13 +1172,15 @@ func (cli *DockerCli) CmdImages(args ...string) error {
 	matchName := cmd.Arg(0)
 	// FIXME: --viz and --tree are deprecated. Remove them in a future version.
 	if *flViz || *flTree {
-		filterJson, err := filters.ToParam(imageFilterArgs)
-		if err != nil {
-			return err
-		}
 		v := url.Values{
-			"all":     []string{"1"},
-			"filters": []string{filterJson},
+			"all": []string{"1"},
+		}
+		if len(imageFilterArgs) > 0 {
+			filterJson, err := filters.ToParam(imageFilterArgs)
+			if err != nil {
+				return err
+			}
+			v.Set("filters", filterJson)
 		}
 
 		body, _, err := readBody(cli.call("GET", "/images/json?"+v.Encode(), nil, false))
@@ -1242,12 +1244,13 @@ func (cli *DockerCli) CmdImages(args ...string) error {
 			fmt.Fprintf(cli.out, " base [style=invisible]\n}\n")
 		}
 	} else {
-		filterJson, err := filters.ToParam(imageFilterArgs)
-		if err != nil {
-			return err
-		}
-		v := url.Values{
-			"filters": []string{filterJson},
+		v := url.Values{}
+		if len(imageFilterArgs) > 0 {
+			filterJson, err := filters.ToParam(imageFilterArgs)
+			if err != nil {
+				return err
+			}
+			v.Set("filters", filterJson)
 		}
 
 		if cmd.NArg() == 1 {
diff --git a/components/engine/docs/sources/reference/api/docker_remote_api_v1.12.md b/components/engine/docs/sources/reference/api/docker_remote_api_v1.12.md
index 9cc84f4ffc..23afa36bcf 100644
--- a/components/engine/docs/sources/reference/api/docker_remote_api_v1.12.md
+++ b/components/engine/docs/sources/reference/api/docker_remote_api_v1.12.md
@@ -712,6 +712,16 @@ Copy files or folders of container `id`
           }
         ]
 
+
+    Query Parameters:
+
+     
+
+    -   **all** – 1/True/true or 0/False/false, default false
+    -   **filters** – a json encoded value of the filters (a map[string][]string) to process on the images list.
+        
+
+
 ### Create an image
 
 `POST /images/create`
diff --git a/components/engine/server/server.go b/components/engine/server/server.go
index 8c70842a47..ff9d370308 100644
--- a/components/engine/server/server.go
+++ b/components/engine/server/server.go
@@ -712,7 +712,7 @@ func (srv *Server) Images(job *engine.Job) engine.Status {
 		}
 	}
 
-	if job.GetenvBool("all") && !filt_tagged {
+	if job.GetenvBool("all") && filt_tagged {
 		allImages, err = srv.daemon.Graph().Map()
 	} else {
 		allImages, err = srv.daemon.Graph().Heads()