From c12df414eabda0721fdf6ceebdff09f6d99c238c Mon Sep 17 00:00:00 2001
From: Kohsuke Kawaguchi <kk@kohsuke.org>
Date: Wed, 25 Apr 2012 23:58:54 -0700
Subject: [PATCH] [JENKINS-13536] I think the proper thing to do is to use the
 Accept header, not the Content-Type header.

---
 .../java/hudson/model/ParametersDefinitionProperty.java   | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/core/src/main/java/hudson/model/ParametersDefinitionProperty.java b/core/src/main/java/hudson/model/ParametersDefinitionProperty.java
index c3188e577e..0bc967dd1b 100644
--- a/core/src/main/java/hudson/model/ParametersDefinitionProperty.java
+++ b/core/src/main/java/hudson/model/ParametersDefinitionProperty.java
@@ -148,7 +148,7 @@ public class ParametersDefinitionProperty extends JobProperty<AbstractProject<?,
         Jenkins.getInstance().getQueue().schedule(
                 owner, owner.getDelay(req), new ParametersAction(values), owner.getBuildCause(req));
 
-        if (req.getContentType() != null && req.getContentType().contains("application/json")) {
+        if (requestWantsJson(req)) {
             rsp.setContentType("application/json");
             rsp.serveExposedBean(req, owner, Flavor.JSON);
         } else {
@@ -157,6 +157,12 @@ public class ParametersDefinitionProperty extends JobProperty<AbstractProject<?,
         }
     }
 
+    private boolean requestWantsJson(StaplerRequest req) {
+        String a = req.getHeader("Accept");
+        if (a==null)    return false;
+        return !a.contains("text/html") && a.contains("application/json");
+    }
+
     /**
      * Gets the {@link ParameterDefinition} of the given name, if any.
      */