[JENKINS-18427] log formatter error

use more explicit method name
use formattedDescription for all parameter types

core/src/main/java/hudson/model/ParameterDefinition.java

@@ -32,6 +32,7 @@ import hudson.util.DescriptorList;
 
 import java.io.Serializable;
 import java.io.IOException;
+import java.util.logging.Logger;
 
 import jenkins.model.Jenkins;
 import net.sf.json.JSONObject;
@@ -137,11 +138,16 @@ public abstract class ParameterDefinition implements
         return description;
     }
 
-    public String getSafeDescription() {
+    /**
+     * return parameter description, applying the configured MarkupFormatter for jenkins instance.
+     * @since 1.521
+     */
+    public String getFormattedDescription() {
         try {
             return Jenkins.getInstance().getMarkupFormatter().translate(description);
         } catch (IOException e) {
-            return "failed to translate description using configured markup formatter";
+            LOGGER.warning("failed to translate description using configured markup formatter");
+            return "";
         }
     }
 
@@ -249,4 +255,6 @@ public abstract class ParameterDefinition implements
             return "Parameter";
         }
     }
+
+    private static final Logger LOGGER = Logger.getLogger(ParameterDefinition.class.getName());
 }

core/src/main/resources/hudson/model/BooleanParameterDefinition/index.jelly

@@ -26,8 +26,8 @@ THE SOFTWARE.
 <j:jelly xmlns:j="jelly:core" xmlns:st="jelly:stapler" xmlns:d="jelly:define"
 	xmlns:l="/lib/layout" xmlns:t="/lib/hudson" xmlns:f="/lib/form"
 	xmlns:i="jelly:fmt" xmlns:p="/lib/hudson/project">
-	<f:entry title="${it.name}" description="${it.description}">
-		<div name="parameter" description="${it.description}">
+	<f:entry title="${it.name}" description="${it.formattedDescription}">
+		<div name="parameter">
 			<input type="hidden" name="name" value="${it.name}" />
 			<f:checkbox name="value" checked="${it.defaultValue}" />
 		</div>

core/src/main/resources/hudson/model/ChoiceParameterDefinition/index.jelly

@@ -26,8 +26,8 @@ THE SOFTWARE.
 <j:jelly xmlns:j="jelly:core" xmlns:st="jelly:stapler" xmlns:d="jelly:define"
 	xmlns:l="/lib/layout" xmlns:t="/lib/hudson" xmlns:f="/lib/form"
 	xmlns:i="jelly:fmt" xmlns:p="/lib/hudson/project">
-	<f:entry title="${it.name}" description="${it.description}">
-		<div name="parameter" description="${it.description}">
+	<f:entry title="${it.name}" description="${it.formattedDescription}">
+		<div name="parameter">
 			<input type="hidden" name="name" value="${it.name}" />
             <select name="value">
               <j:forEach var="value" items="${it.choices}">

core/src/main/resources/hudson/model/FileParameterDefinition/index.jelly

@@ -26,7 +26,7 @@ THE SOFTWARE.
 <j:jelly xmlns:j="jelly:core" xmlns:st="jelly:stapler" xmlns:d="jelly:define"
 	xmlns:l="/lib/layout" xmlns:t="/lib/hudson" xmlns:f="/lib/form"
 	xmlns:i="jelly:fmt" xmlns:p="/lib/hudson/project">
-	<f:entry title="${it.name}" description="${it.description}">
+	<f:entry title="${it.name}" description="${it.formattedDescription}">
 		<div name="parameter">
 			<input type="hidden" name="name" value="${it.name}" />
 			<input name="file" type="file" jsonAware="true" />

core/src/main/resources/hudson/model/PasswordParameterDefinition/index.jelly

@@ -26,8 +26,8 @@ THE SOFTWARE.
 <j:jelly xmlns:j="jelly:core" xmlns:st="jelly:stapler" xmlns:d="jelly:define"
 	xmlns:l="/lib/layout" xmlns:t="/lib/hudson" xmlns:f="/lib/form"
 	xmlns:i="jelly:fmt" xmlns:p="/lib/hudson/project">
-	<f:entry title="${it.name}" description="${it.description}">
-		<div name="parameter" description="${it.description}">
+	<f:entry title="${it.name}" description="${it.formattedDescription}">
+		<div name="parameter">
 			<input type="hidden" name="name" value="${it.name}" />
 			<f:password name="value" value="${it.defaultValue}" />
 		</div>

core/src/main/resources/hudson/model/RunParameterDefinition/index.jelly

@@ -26,8 +26,8 @@ THE SOFTWARE.
 <j:jelly xmlns:j="jelly:core" xmlns:st="jelly:stapler" xmlns:d="jelly:define"
 	xmlns:l="/lib/layout" xmlns:t="/lib/hudson" xmlns:f="/lib/form"
 	xmlns:i="jelly:fmt" xmlns:p="/lib/hudson/project">
-	<f:entry title="${it.name}" description="${it.description}">
-		<div name="parameter" description="${it.description}">
+	<f:entry title="${it.name}" description="${it.formattedDescription}">
+		<div name="parameter">
 			<input type="hidden" name="name" value="${it.name}" />
             <select name="runId">
               <j:forEach var="run" items="${it.builds}">

core/src/main/resources/hudson/model/StringParameterDefinition/index.jelly

@@ -26,7 +26,7 @@ THE SOFTWARE.
 <j:jelly xmlns:j="jelly:core" xmlns:st="jelly:stapler" xmlns:d="jelly:define"
 	xmlns:l="/lib/layout" xmlns:t="/lib/hudson" xmlns:f="/lib/form"
 	xmlns:i="jelly:fmt" xmlns:p="/lib/hudson/project">
-	<f:entry title="${it.name}" description="${it.safeDescription}">
+	<f:entry title="${it.name}" description="${it.formattedDescription}">
 		<div name="parameter">
 			<input type="hidden" name="name" value="${it.name}" />
 			<f:textbox name="value" value="${it.defaultValue}" />

core/src/main/resources/hudson/model/TextParameterDefinition/index.jelly

@@ -24,8 +24,8 @@ THE SOFTWARE.
 -->
 <?jelly escape-by-default='true'?>
 <j:jelly xmlns:j="jelly:core" xmlns:st="jelly:stapler" xmlns:d="jelly:define" xmlns:l="/lib/layout" xmlns:t="/lib/hudson" xmlns:f="/lib/form" xmlns:i="jelly:fmt" xmlns:p="/lib/hudson/project">
-  <f:entry title="${it.name}" description="${it.description}">
-    <div name="parameter" description="${it.description}">
+  <f:entry title="${it.name}" description="${it.formattedDescription}">
+    <div name="parameter">
       <input type="hidden" name="name" value="${it.name}"/>
       <f:textarea name="value" value="${it.defaultValue}"/>
     </div>

test/src/test/java/hudson/model/ParametersTest.java

@@ -89,7 +89,7 @@ public class ParametersTest extends HudsonTestCase {
 
         HtmlElement element = (HtmlElement) form.selectSingleNode(".//tr[td/div/input/@value='choice']");
         assertNotNull(element);
-        assertEquals("choice description", ((HtmlElement) element.selectSingleNode("td/div")).getAttribute("description"));
+        assertEquals("choice description", ((HtmlElement) element.getNextSibling().getNextSibling().selectSingleNode("td[@class='setting-description']")).getTextContent());
         assertEquals("choice", ((HtmlElement) element.selectSingleNode("td[@class='setting-name']")).getTextContent());
         HtmlOption opt = (HtmlOption)element.selectSingleNode("td/div/select/option[@value='Choice <2>']");
         assertNotNull(opt);