From 3e396876eafddfb279839cfb62e91d1c1ee1bb42 Mon Sep 17 00:00:00 2001
From: kohsuke <kohsuke@71c3de6d-444a-0410-be80-ed276b4c234a>
Date: Mon, 23 Mar 2009 18:18:18 +0000
Subject: [PATCH] replaced h.ifThenElse with "x?y:z" (and occasionally with
 "x?:y")

git-svn-id: https://hudson.dev.java.net/svn/hudson/trunk/hudson/main@16486 71c3de6d-444a-0410-be80-ed276b4c234a
---
 .../src/main/resources/hudson/PluginManager/installed.jelly | 6 +++---
 .../hudson/matrix/MatrixProject/configure-entries.jelly     | 2 +-
 .../resources/hudson/model/AbstractProject/sidepanel.jelly  | 2 +-
 .../hudson/model/Cause/UpstreamCause/description.jelly      | 2 +-
 core/src/main/resources/hudson/model/Hudson/configure.jelly | 4 ++--
 core/src/main/resources/hudson/model/Run/console.jelly      | 2 +-
 core/src/main/resources/hudson/model/View/builds.jelly      | 2 +-
 core/src/main/resources/hudson/model/View/cc.xml.jelly      | 2 +-
 core/src/main/resources/hudson/model/View/index.jelly       | 2 +-
 .../hudson/slaves/DumbSlave/configure-entries.jelly         | 4 ++--
 core/src/main/resources/hudson/tasks/Mailer/global.jelly    | 2 +-
 .../main/resources/hudson/triggers/SCMTrigger/global.jelly  | 2 +-
 .../resources/hudson/views/LastDurationColumn/column.jelly  | 2 +-
 .../resources/hudson/views/LastFailureColumn/column.jelly   | 2 +-
 .../resources/hudson/views/LastSuccessColumn/column.jelly   | 2 +-
 .../main/resources/hudson/widgets/HistoryWidget/entry.jelly | 6 +++---
 core/src/main/resources/lib/form/descriptorList.jelly       | 2 +-
 core/src/main/resources/lib/form/descriptorRadioList.jelly  | 2 +-
 core/src/main/resources/lib/form/hetero-list.jelly          | 2 +-
 core/src/main/resources/lib/form/option.jelly               | 2 +-
 core/src/main/resources/lib/form/password.jelly             | 2 +-
 core/src/main/resources/lib/form/radio.jelly                | 2 +-
 core/src/main/resources/lib/form/radioBlock.jelly           | 2 +-
 core/src/main/resources/lib/form/repeatable.jelly           | 2 +-
 core/src/main/resources/lib/form/rowSet.jelly               | 2 +-
 core/src/main/resources/lib/form/textarea.jelly             | 2 +-
 core/src/main/resources/lib/form/textbox.jelly              | 2 +-
 core/src/main/resources/lib/hudson/buildHealth.jelly        | 4 ++--
 core/src/main/resources/lib/hudson/buildLink.jelly          | 2 +-
 .../src/main/resources/lib/hudson/editableDescription.jelly | 2 +-
 core/src/main/resources/lib/hudson/executors.jelly          | 4 ++--
 core/src/main/resources/lib/hudson/progressBar.jelly        | 2 +-
 core/src/main/resources/lib/hudson/progressiveText.jelly    | 2 +-
 core/src/main/resources/lib/hudson/project/config-scm.jelly | 2 +-
 core/src/main/resources/lib/hudson/projectView.jelly        | 2 +-
 core/src/main/resources/lib/hudson/projectViewNested.jelly  | 2 +-
 core/src/main/resources/lib/hudson/projectViewRow.jelly     | 2 +-
 core/src/main/resources/lib/hudson/queue.jelly              | 4 ++--
 core/src/main/resources/lib/hudson/setIconSize.jelly        | 2 +-
 core/src/main/resources/lib/hudson/summary.jelly            | 4 ++--
 core/src/main/resources/lib/layout/task.jelly               | 2 +-
 41 files changed, 51 insertions(+), 51 deletions(-)

diff --git a/core/src/main/resources/hudson/PluginManager/installed.jelly b/core/src/main/resources/hudson/PluginManager/installed.jelly
index 41ee582747..1fdd719fdb 100644
--- a/core/src/main/resources/hudson/PluginManager/installed.jelly
+++ b/core/src/main/resources/hudson/PluginManager/installed.jelly
@@ -45,15 +45,15 @@ THE SOFTWARE.
             <j:forEach var="p" items="${app.pluginManager.plugins}">
               <tr>
                 <td class="center pane">
-                  <j:set var="state" value="${h.ifThenElse(p.enabled,'true',null)}"/>
+                  <j:set var="state" value="${p.enabled?'true':null}"/>
                   <input type="checkbox" checked="${state}" onclick="flip(event)"
                          url="plugin/${p.shortName}"
-                         original="${h.ifThenElse(p.active,'true','false')}"/>
+                         original="${p.active?'true':'false'}"/>
                 </td>
                 <td class="pane">
                   <div>
                     <a href="${p.url}">
-                      ${h.ifThenElse(p.updateInfo!=null, p.updateInfo.title, p.longName)}
+                      ${p.updateInfo.title?:p.longName}
                     </a>
                   </div>
                   <div class="excerpt">
diff --git a/core/src/main/resources/hudson/matrix/MatrixProject/configure-entries.jelly b/core/src/main/resources/hudson/matrix/MatrixProject/configure-entries.jelly
index e9b9073355..3ccc9d8506 100644
--- a/core/src/main/resources/hudson/matrix/MatrixProject/configure-entries.jelly
+++ b/core/src/main/resources/hudson/matrix/MatrixProject/configure-entries.jelly
@@ -67,7 +67,7 @@ THE SOFTWARE.
               var labels = new YAHOO.widget.TextNode("${%Labels}", tree.getRoot(), false);
 
               <j:forEach var="l" items="${app.labels}">
-                new YAHOO.widget.TextNode('&lt;input type="checkbox" name="label.${l.name}" id="label.${l.name}" <j:if test="${labels.contains(l)}">checked</j:if>&gt;<label for="label.${l.name}">${h.jsStringEscape(l.name)} (${h.jsStringEscape(l.description)})</label>', ${h.ifThenElse(l.isSelfLabel(),'machines','labels')}, false);
+                new YAHOO.widget.TextNode('&lt;input type="checkbox" name="label.${l.name}" id="label.${l.name}" <j:if test="${labels.contains(l)}">checked</j:if>&gt;<label for="label.${l.name}">${h.jsStringEscape(l.name)} (${h.jsStringEscape(l.description)})</label>', ${l.isSelfLabel()?'machines':'labels'}, false);
               </j:forEach>
 
               tree.draw();
diff --git a/core/src/main/resources/hudson/model/AbstractProject/sidepanel.jelly b/core/src/main/resources/hudson/model/AbstractProject/sidepanel.jelly
index 88001f5a98..9aa3fd4288 100644
--- a/core/src/main/resources/hudson/model/AbstractProject/sidepanel.jelly
+++ b/core/src/main/resources/hudson/model/AbstractProject/sidepanel.jelly
@@ -43,7 +43,7 @@ THE SOFTWARE.
       <j:if test="${it.configurable}">
         <j:if test="${!it.disabled}">
           <l:task icon="images/24x24/clock.gif" href="${url}/build?delay=0sec" title="${%Build Now}"
-                  onclick="${h.ifThenElse(it.parameterized,null,'return build(this)')}" permission="${it.BUILD}" />
+                  onclick="${it.parameterized?null:'return build(this)'}" permission="${it.BUILD}" />
           <script>
             function build(a) {
               new Ajax.Request(a.href,{method:"post"});
diff --git a/core/src/main/resources/hudson/model/Cause/UpstreamCause/description.jelly b/core/src/main/resources/hudson/model/Cause/UpstreamCause/description.jelly
index 327262d7a4..49178b059e 100644
--- a/core/src/main/resources/hudson/model/Cause/UpstreamCause/description.jelly
+++ b/core/src/main/resources/hudson/model/Cause/UpstreamCause/description.jelly
@@ -23,5 +23,5 @@ 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">
     <!-- upstreamUrl added in 1.284, so handle missing value -->
-    <p>${h.ifThenElse(it.upstreamUrl!=null,"%started_by_project(it.upstreamProject,it.upstreamBuild.toString(),it.upstreamUrl,rootURL)",it.shortDescription)}</p>
+    <p>${it.upstreamUrl!=null ? "%started_by_project(it.upstreamProject,it.upstreamBuild.toString(),it.upstreamUrl,rootURL)" : it.shortDescription}</p>
 </j:jelly>
diff --git a/core/src/main/resources/hudson/model/Hudson/configure.jelly b/core/src/main/resources/hudson/model/Hudson/configure.jelly
index 2d85979fc2..d4c168e532 100644
--- a/core/src/main/resources/hudson/model/Hudson/configure.jelly
+++ b/core/src/main/resources/hudson/model/Hudson/configure.jelly
@@ -58,8 +58,8 @@ THE SOFTWARE.
                    checked="${it.slaveAgentPort &gt; 0}" onclick="$('sat.port').disabled=false"/>
           <label for="sat.fixed">${%Fixed}</label> :
           <input type="text" class="number" name="slaveAgentPort" id="sat.port"
-             value="${h.ifThenElse(it.slaveAgentPort &gt; 0, it.slaveAgentPort, null)}"
-             disabled="${h.ifThenElse(it.slaveAgentPort &gt; 0, null, 'true')}"/>
+             value   ="${it.slaveAgentPort gt 0 ? it.slaveAgentPort : null}"
+             disabled="${it.slaveAgentPort gt 0 ? null : 'true'}"/>
 
           <st:nbsp />
 
diff --git a/core/src/main/resources/hudson/model/Run/console.jelly b/core/src/main/resources/hudson/model/Run/console.jelly
index be3207f000..7cbb259d2c 100644
--- a/core/src/main/resources/hudson/model/Run/console.jelly
+++ b/core/src/main/resources/hudson/model/Run/console.jelly
@@ -42,7 +42,7 @@ THE SOFTWARE.
         <j:arg value="150" />
       </j:invokeStatic>
       <!-- Show at most last 150KB (can override with system property) unless consoleFull is set -->
-      <j:set var="offset" value="${h.ifThenElse(empty(consoleFull),it.logFile.length()-threshold*1024,0)}" />
+      <j:set var="offset" value="${empty(consoleFull) ? it.logFile.length()-threshold*1024 : 0}" />
       <j:choose>
         <j:when test="${offset > 0}">
           ${%skipSome(offset/1024,"consoleFull")}
diff --git a/core/src/main/resources/hudson/model/View/builds.jelly b/core/src/main/resources/hudson/model/View/builds.jelly
index 85df55ed05..f7870b9c9a 100644
--- a/core/src/main/resources/hudson/model/View/builds.jelly
+++ b/core/src/main/resources/hudson/model/View/builds.jelly
@@ -28,7 +28,7 @@ THE SOFTWARE.
     <l:main-panel>
       <h1>
         <img src="${imagesURL}/48x48/notepad.gif" alt=""/>
-        ${%buildHistory(h.ifThenElse(it.class.name=='hudson.model.AllView',app.displayName,it.displayName))}
+        ${%buildHistory(it.class.name=='hudson.model.AllView' ? app.displayName : it.displayName)}
       </h1>
       <t:buildListTable builds="${it.builds}"/>
     </l:main-panel>
diff --git a/core/src/main/resources/hudson/model/View/cc.xml.jelly b/core/src/main/resources/hudson/model/View/cc.xml.jelly
index 365f912d4e..099f48340b 100644
--- a/core/src/main/resources/hudson/model/View/cc.xml.jelly
+++ b/core/src/main/resources/hudson/model/View/cc.xml.jelly
@@ -35,7 +35,7 @@ THE SOFTWARE.
       <j:set var="lb" value="${p.lastCompletedBuild}"/>
       <j:if test="${lb!=null}">
         <Project  name="${p.displayName}"
-                  activity="${h.ifThenElse(p.isBuilding(),'Building','Sleeping')}"
+                  activity="${p.isBuilding() ? 'Building' : 'Sleeping'}"
                   lastBuildStatus="${h.toCCStatus(p)}"
                   lastBuildLabel="${lb.number}"
                   lastBuildTime="${lb.timestampString2}"
diff --git a/core/src/main/resources/hudson/model/View/index.jelly b/core/src/main/resources/hudson/model/View/index.jelly
index c89cfc76fc..8194bd8f58 100644
--- a/core/src/main/resources/hudson/model/View/index.jelly
+++ b/core/src/main/resources/hudson/model/View/index.jelly
@@ -25,7 +25,7 @@ THE SOFTWARE.
 <st:compress 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">
 	<st:header name="X-Hudson" value="${servletContext.getAttribute('version')}" />
   <j:new var="h" className="hudson.Functions" /><!-- instead of JSP functions -->
-	<l:layout title="${h.ifThenElse(it.class.name=='hudson.model.AllView','%Dashboard',it.viewName)}">
+	<l:layout title="${it.class.name=='hudson.model.AllView' ? '%Dashboard' : it.viewName}">
     <st:include page="sidepanel.jelly" />
 		<l:main-panel>
 		  <div id="view-message">
diff --git a/core/src/main/resources/hudson/slaves/DumbSlave/configure-entries.jelly b/core/src/main/resources/hudson/slaves/DumbSlave/configure-entries.jelly
index e28eb46f52..ae5fa43c24 100644
--- a/core/src/main/resources/hudson/slaves/DumbSlave/configure-entries.jelly
+++ b/core/src/main/resources/hudson/slaves/DumbSlave/configure-entries.jelly
@@ -54,7 +54,7 @@ THE SOFTWARE.
                            title="${d.displayName}">
         <j:set var="descriptor" value="${d}"/>
         <j:set var="instance"
-               value="${h.ifThenElse(it.launcher.descriptor==d,it.launcher,null)}"/>
+               value="${it.launcher.descriptor==d ? it.launcher : null}"/>
         <tr><td>
           <input type="hidden" name="stapler-class" value="${d.clazz.name}" />
         </td></tr>
@@ -74,7 +74,7 @@ THE SOFTWARE.
                                title="${d.displayName}">
             <j:set var="descriptor" value="${d}"/>
             <j:set var="instance"
-                   value="${h.ifThenElse(it.retentionStrategy.descriptor==d,it.retentionStrategy,null)}"/>
+                   value="${it.retentionStrategy.descriptor==d ? it.retentionStrategy : null}"/>
             <tr><td>
               <input type="hidden" name="stapler-class" value="${d.clazz.name}" />
             </td></tr>
diff --git a/core/src/main/resources/hudson/tasks/Mailer/global.jelly b/core/src/main/resources/hudson/tasks/Mailer/global.jelly
index d8f96c554c..db7003eb40 100644
--- a/core/src/main/resources/hudson/tasks/Mailer/global.jelly
+++ b/core/src/main/resources/hudson/tasks/Mailer/global.jelly
@@ -48,7 +48,7 @@ THE SOFTWARE.
     </f:entry>
     <f:entry title="${%Hudson URL}"
              help="/help/tasks/mailer/url.html">
-      <f:textbox field="url" value="${h.ifThenElse(descriptor.url!=null,descriptor.url,h.inferHudsonURL(request))}" />
+      <f:textbox field="url" value="${descriptor.url ?: h.inferHudsonURL(request)}" />
     </f:entry>
     <f:advanced>
       <f:optionalBlock name="mailer.useSMTPAuth" title="${%Use SMTP Authentication}" checked="${descriptor.smtpAuthUserName!=null}"
diff --git a/core/src/main/resources/hudson/triggers/SCMTrigger/global.jelly b/core/src/main/resources/hudson/triggers/SCMTrigger/global.jelly
index a46e905ca7..9f62674d92 100644
--- a/core/src/main/resources/hudson/triggers/SCMTrigger/global.jelly
+++ b/core/src/main/resources/hudson/triggers/SCMTrigger/global.jelly
@@ -32,7 +32,7 @@ THE SOFTWARE.
       <f:entry title="${%Max # of concurrent polling}"
                help="/help/triggers/SCMTrigger/concurrency.html">
         <input class="setting-input" name="poll_scm_threads"
-          type="text" value="${h.ifThenElse(descriptor.pollingThreadCount==0,'',descriptor.pollingThreadCount)}"/>
+          type="text" value="${descriptor.pollingThreadCount==0 ? '' : descriptor.pollingThreadCount}"/>
       </f:entry>
     </f:section>
   </j:if>
diff --git a/core/src/main/resources/hudson/views/LastDurationColumn/column.jelly b/core/src/main/resources/hudson/views/LastDurationColumn/column.jelly
index 1b2863580a..1444b07c0b 100644
--- a/core/src/main/resources/hudson/views/LastDurationColumn/column.jelly
+++ b/core/src/main/resources/hudson/views/LastDurationColumn/column.jelly
@@ -23,7 +23,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">
-      <td data="${h.ifThenElse(lsBuild==null,h.ifThenElse(lfBuild==null,'0',lfBuild.duration),lsBuild.duration)}">
+      <td data="${lsBuild.duration ?: lfBuild.duration ?: '0'}">
           <j:choose>
               <j:when test="${lsBuild!=null}">
                   ${lsBuild.durationString}
diff --git a/core/src/main/resources/hudson/views/LastFailureColumn/column.jelly b/core/src/main/resources/hudson/views/LastFailureColumn/column.jelly
index 5e43d50485..939f822645 100644
--- a/core/src/main/resources/hudson/views/LastFailureColumn/column.jelly
+++ b/core/src/main/resources/hudson/views/LastFailureColumn/column.jelly
@@ -23,7 +23,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">
-    <td data="${h.ifThenElse(lfBuild==null,'-',lfBuild.timestampString2)}">
+    <td data="${lfBuild.timestampString2 ?: '-'}">
           <j:choose>
               <j:when test="${lfBuild!=null}">
                   ${lfBuild.timestampString}
diff --git a/core/src/main/resources/hudson/views/LastSuccessColumn/column.jelly b/core/src/main/resources/hudson/views/LastSuccessColumn/column.jelly
index ff301babe2..8d860af47e 100644
--- a/core/src/main/resources/hudson/views/LastSuccessColumn/column.jelly
+++ b/core/src/main/resources/hudson/views/LastSuccessColumn/column.jelly
@@ -23,7 +23,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">
-    <td data="${h.ifThenElse(lsBuild==null,'-',lsBuild.timestampString2)}">
+    <td data="${lsBuild.timestampString2 ?: '-'}">
           <j:choose>
               <j:when test="${lsBuild!=null}">
                   ${lsBuild.timestampString}
diff --git a/core/src/main/resources/hudson/widgets/HistoryWidget/entry.jelly b/core/src/main/resources/hudson/widgets/HistoryWidget/entry.jelly
index 167f53a3d5..25102e03c1 100644
--- a/core/src/main/resources/hudson/widgets/HistoryWidget/entry.jelly
+++ b/core/src/main/resources/hudson/widgets/HistoryWidget/entry.jelly
@@ -27,7 +27,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">
   <j:set var="link" value="${it.baseUrl}/${build.number}/" />
-  <tr class="build-row ${h.ifThenElse(build.building,'transitive',null)}">
+  <tr class="build-row ${build.building ? 'transitive' : null}">
     <td nowrap="nowrap">
       <img width="16" height="16" src="${imagesURL}/16x16/${build.buildStatusUrl}" alt="${build.iconColor.description}" tooltip="${build.iconColor.description}" /><st:nbsp/>
       #${build.number}
@@ -53,7 +53,7 @@ THE SOFTWARE.
           <t:buildProgressBar build="${build}"/>
         </td><td style="padding:0">
           <!-- Check ABORT permission for Project, Admin permission otherwise -->
-          <j:if test="${h.ifThenElse(empty(it.owner.ABORT),h.hasPermission(app.ADMINISTER),it.owner.hasPermission(it.owner.ABORT))}">
+          <j:if test="${empty(it.owner.ABORT) ? h.hasPermission(app.ADMINISTER) : it.owner.hasPermission(it.owner.ABORT)}">
             <a href="${link}stop"><img src="${imagesURL}/16x16/stop.gif" alt="[cancel]"/></a>
           </j:if>
         </td></tr>
@@ -61,7 +61,7 @@ THE SOFTWARE.
     </td></tr>
   </j:if>
   <j:if test="${!empty build.description}">
-    <tr class="${h.ifThenElse(build.building,'transitive',null)}">
+    <tr class="${build.building ? 'transitive' : null}">
       <td></td>
       <td style="padding:0;white-space:normal">
         ${build.truncatedDescription}
diff --git a/core/src/main/resources/lib/form/descriptorList.jelly b/core/src/main/resources/lib/form/descriptorList.jelly
index 827e3a9f2e..96bc4183b3 100644
--- a/core/src/main/resources/lib/form/descriptorList.jelly
+++ b/core/src/main/resources/lib/form/descriptorList.jelly
@@ -53,7 +53,7 @@ THE SOFTWARE.
   </st:documentation>
 
   <j:set var="targetType" value="${h.defaulted(attrs.targetType,it.class)}"/>
-  <j:set var="instances" value="${h.ifThenElse(attrs.instances!=null,attrs.instances,instance[field])}"/>
+  <j:set var="instances" value="${attrs.instances ?: instance[field]}"/>
 
   <f:section title="${attrs.title}" name="${h.defaulted(attrs.field,attrs.name)}">
     <j:if test="${attrs.field!=null}">
diff --git a/core/src/main/resources/lib/form/descriptorRadioList.jelly b/core/src/main/resources/lib/form/descriptorRadioList.jelly
index c542ce5341..71a632938b 100644
--- a/core/src/main/resources/lib/form/descriptorRadioList.jelly
+++ b/core/src/main/resources/lib/form/descriptorRadioList.jelly
@@ -53,7 +53,7 @@ THE SOFTWARE.
       <f:radioBlock name="${varName}" help="${d.helpFile}" value="${loop.index}"
         title="${d.displayName}" checked="${instance.descriptor==d}">
         <j:set var="descriptor" value="${d}" />
-	      <j:set var="instance" value="${h.ifThenElse(instance.descriptor==d,instance,null)}" />
+	      <j:set var="instance" value="${instance.descriptor==d?instance:null}" />
 	      <st:include from="${d}" page="${d.configPage}" optional="true" />
 	    </f:radioBlock>
 	  </j:forEach>
diff --git a/core/src/main/resources/lib/form/hetero-list.jelly b/core/src/main/resources/lib/form/hetero-list.jelly
index 6ec14433c2..673167ef1a 100644
--- a/core/src/main/resources/lib/form/hetero-list.jelly
+++ b/core/src/main/resources/lib/form/hetero-list.jelly
@@ -75,7 +75,7 @@ THE SOFTWARE.
   </d:taglib>
 
   <j:set var="targetType" value="${h.defaulted(attrs.targetType,it.class)}"/>
-  <div class="hetero-list-container ${h.ifThenElse(attrs.hasHeader!=null,'with-drag-drop','')}">
+  <div class="hetero-list-container ${attrs.hasHeader!=null?'with-drag-drop':''}">
     <!-- display existing items -->
     <j:forEach var="i" items="${attrs.items}">
       <j:set var="descriptor" value="${i.descriptor}" />
diff --git a/core/src/main/resources/lib/form/option.jelly b/core/src/main/resources/lib/form/option.jelly
index e05b7a8c22..ddbb468649 100644
--- a/core/src/main/resources/lib/form/option.jelly
+++ b/core/src/main/resources/lib/form/option.jelly
@@ -33,7 +33,7 @@ THE SOFTWARE.
       If true, the option value appears as selected.
     </st:attribute>
   </st:documentation>
-  <option value="${attrs.value}" selected="${h.ifThenElse(attrs.selected,'true',null)}">
+  <option value="${attrs.value}" selected="${attrs.selected?'true':null}">
     <d:invokeBody/>
   </option>
 </j:jelly>
\ No newline at end of file
diff --git a/core/src/main/resources/lib/form/password.jelly b/core/src/main/resources/lib/form/password.jelly
index 38a30bcbbc..838541ffd5 100644
--- a/core/src/main/resources/lib/form/password.jelly
+++ b/core/src/main/resources/lib/form/password.jelly
@@ -48,7 +48,7 @@ THE SOFTWARE.
     </st:attribute>
   </st:documentation>
   <f:prepareDatabinding />
-  <input class="setting-input ${h.ifThenElse(attrs.checkUrl!=null,'validated','')}"
+  <input class="setting-input ${attrs.checkUrl!=null?'validated':''}"
          name ="${h.defaulted(attrs.name,'_.'+attrs.field)}"
          value="${h.defaulted(attrs.value,instance[attrs.field])}"
          id="${attrs.id}"
diff --git a/core/src/main/resources/lib/form/radio.jelly b/core/src/main/resources/lib/form/radio.jelly
index b1e388390b..d3e96c55f5 100644
--- a/core/src/main/resources/lib/form/radio.jelly
+++ b/core/src/main/resources/lib/form/radio.jelly
@@ -28,4 +28,4 @@ THE SOFTWARE.
 	Note that safari doesn't support onchange.
 -->
 <input type="radio" name="${attrs.name}" onclick="${attrs.onclick}" id="${attrs.id}" value="${attrs.value}"
-        checked="${h.ifThenElse(attrs.checked,'true',null)}"/>
\ No newline at end of file
+        checked="${attrs.checked?'true':null}"/>
\ No newline at end of file
diff --git a/core/src/main/resources/lib/form/radioBlock.jelly b/core/src/main/resources/lib/form/radioBlock.jelly
index ba69c70279..64631fc20b 100644
--- a/core/src/main/resources/lib/form/radioBlock.jelly
+++ b/core/src/main/resources/lib/form/radioBlock.jelly
@@ -52,7 +52,7 @@ THE SOFTWARE.
 	<tr id="rb_s${rb_id}" hasHelp="${attrs.help!=null}"><!-- this ID marks the beginning -->
 	  <td colspan="3">
 	    <input type="radio" name="${name}" value="${value}"
-             id="Rb${rb_id}" checked="${h.ifThenElse(checked,'true',null)}" />
+             id="Rb${rb_id}" checked="${checked?'true':null}" />
       <st:nbsp/>
       <label for="Rb${rb_id}">${title}</label>
     </td>
diff --git a/core/src/main/resources/lib/form/repeatable.jelly b/core/src/main/resources/lib/form/repeatable.jelly
index e264e76494..d3bd8f4b51 100644
--- a/core/src/main/resources/lib/form/repeatable.jelly
+++ b/core/src/main/resources/lib/form/repeatable.jelly
@@ -101,7 +101,7 @@ THE SOFTWARE.
       <j:set var="descriptor" value="${descriptor.getPropertyType(field).itemTypeDescriptor}" />
     </j:when>
     <j:otherwise>
-      <j:set var="name" value="${h.ifThenElse(attrs.name!=null,attrs.name,attrs.var)}"/>
+      <j:set var="name" value="${attrs.name?:attrs.var}"/>
     </j:otherwise>
   </j:choose>
 
diff --git a/core/src/main/resources/lib/form/rowSet.jelly b/core/src/main/resources/lib/form/rowSet.jelly
index c5a2d73acf..6276b0aa84 100644
--- a/core/src/main/resources/lib/form/rowSet.jelly
+++ b/core/src/main/resources/lib/form/rowSet.jelly
@@ -44,7 +44,7 @@ THE SOFTWARE.
     <j:otherwise>
       <j:set var="id" value="${h.generateId()}" />
       <j:set var="head" value="rowGroupStart${id}"/>
-      <j:set var="ref" value="${h.ifThenElse(attrs.ref!=null, attrs.ref, head)}"/>
+      <j:set var="ref" value="${attrs.ref?:head}"/>
 
       <tr id="${head}" style="display:none" name="${attrs.name}" />
       <d:invokeBody />
diff --git a/core/src/main/resources/lib/form/textarea.jelly b/core/src/main/resources/lib/form/textarea.jelly
index 726a3822b2..d4a0de5f74 100644
--- a/core/src/main/resources/lib/form/textarea.jelly
+++ b/core/src/main/resources/lib/form/textarea.jelly
@@ -52,7 +52,7 @@ THE SOFTWARE.
   <f:prepareDatabinding />
   <textarea id="${attrs.id}" style="${attrs.style}"
             name ="${h.defaulted(attrs.name,'_.'+attrs.field)}"
-            class="setting-input ${h.ifThenElse(checkUrl!=null,'validated','')}"
+            class="setting-input ${checkUrl!=null?'validated':''}"
             checkUrl="${checkUrl}"
             rows="${h.determineRows(attrs.value)}">
     <st:out value="${h.defaulted(attrs.value,instance[attrs.field])}" />
diff --git a/core/src/main/resources/lib/form/textbox.jelly b/core/src/main/resources/lib/form/textbox.jelly
index 6ae30d8a23..bd0a314379 100644
--- a/core/src/main/resources/lib/form/textbox.jelly
+++ b/core/src/main/resources/lib/form/textbox.jelly
@@ -49,7 +49,7 @@ THE SOFTWARE.
     </st:attribute>
   </st:documentation>
   <f:prepareDatabinding />
-  <input class="setting-input ${h.ifThenElse(attrs.checkUrl!=null,'validated','')}"
+  <input class="setting-input ${attrs.checkUrl!=null?'validated':''}"
          name ="${h.defaulted(attrs.name,'_.'+attrs.field)}"
          value="${h.defaulted(attrs.value,instance[attrs.field])}"
          id="${attrs.id}"
diff --git a/core/src/main/resources/lib/hudson/buildHealth.jelly b/core/src/main/resources/lib/hudson/buildHealth.jelly
index 1086a16dd9..13672a59a0 100644
--- a/core/src/main/resources/lib/hudson/buildHealth.jelly
+++ b/core/src/main/resources/lib/hudson/buildHealth.jelly
@@ -31,13 +31,13 @@ THE SOFTWARE.
 <j:jelly xmlns:j="jelly:core" xmlns:x="jelly:xml" xmlns:st="jelly:stapler" xmlns:d="jelly:define" xmlns:l="/lib/layout" xmlns:t="/lib/hudson" xmlns:f="/lib/form">
     <j:set var="buildHealth" value="${job.buildHealth}"/>
     <j:if test="${td}"><j:set var="useTdElement" value="x"/></j:if>
-    <x:element name="${h.ifThenElse(useTdElement!=null,'td','div')}">
+    <x:element name="${useTdElement!=null?'td':'div'}">
         <x:attribute name="data">${buildHealth.score}</x:attribute>
         <x:attribute name="class">healthReport</x:attribute>
         <x:attribute name="onmouseover">this.className='healthReport hover';return true;
         </x:attribute>
         <x:attribute name="onmouseout">this.className='healthReport';return true;</x:attribute>
-        <a href="${h.ifThenElse(empty(link),'#',link)}" style="${attrs.style}">
+        <a href="${empty(link)?'#':link}" style="${attrs.style}">
             <img src="${rootURL}${buildHealth.getIconUrl(iconSize)}"
                  alt="${buildHealth.score}%"/>
         </a>
diff --git a/core/src/main/resources/lib/hudson/buildLink.jelly b/core/src/main/resources/lib/hudson/buildLink.jelly
index 4014aa1e9e..4de7e09933 100644
--- a/core/src/main/resources/lib/hudson/buildLink.jelly
+++ b/core/src/main/resources/lib/hudson/buildLink.jelly
@@ -45,7 +45,7 @@ THE SOFTWARE.
           #<!-- -->${number}
 	      </j:when>
 	      <j:otherwise>
-	        <a href="${h.ifThenElse(attrs.href!=null,href,rootURL+'/'+r.url)}">
+	        <a href="${attrs.href ?: rootURL+'/'+r.url}">
 	          <img src="${imagesURL}/16x16/${r.buildStatusUrl}"
                  alt="${r.iconColor.description}"/>${jobName_}#<!-- -->${number}</a>
 	      </j:otherwise>
diff --git a/core/src/main/resources/lib/hudson/editableDescription.jelly b/core/src/main/resources/lib/hudson/editableDescription.jelly
index 0a6ebfd8d8..65131513e9 100644
--- a/core/src/main/resources/lib/hudson/editableDescription.jelly
+++ b/core/src/main/resources/lib/hudson/editableDescription.jelly
@@ -39,7 +39,7 @@ THE SOFTWARE.
     </div>
 
     <l:hasPermission permission="${permission}">
-      <div align="right"><a href="editDescription" onclick="${h.ifThenElse(h.isAutoRefresh(request), null, 'return replaceDescription();')}">
+      <div align="right"><a href="editDescription" onclick="${h.isAutoRefresh(request) ? null : 'return replaceDescription();'}">
         <img src="${imagesURL}/16x16/notepad.gif" alt="" />
         <j:choose>
           <j:when test="${empty(it.description)}">
diff --git a/core/src/main/resources/lib/hudson/executors.jelly b/core/src/main/resources/lib/hudson/executors.jelly
index 4983e9c60f..59d990068b 100644
--- a/core/src/main/resources/lib/hudson/executors.jelly
+++ b/core/src/main/resources/lib/hudson/executors.jelly
@@ -37,7 +37,7 @@ THE SOFTWARE.
       <j:if test="${!c.acceptingTasks}"> (${%suspended})</j:if>
     </d:tag>
   </d:taglib>
-  <j:set var="computers" value="${h.ifThenElse(attrs.computers!=null, attrs.computers, app.computers)}" />
+  <j:set var="computers" value="${attrs.computers?:app.computers}" />
 	<l:pane title="&lt;a href='${rootURL}/computer/'>${%Build Executor Status}&lt;/a>" width="3" id="executors">
 	  <j:forEach var="c" items="${computers}" varStatus="cloop">
 	    <j:choose>
@@ -111,7 +111,7 @@ THE SOFTWARE.
     <!-- schedule updates only for the full page reload -->
     <j:if test="${ajax==null and !h.isAutoRefresh(request) and h.hasPermission(app.READ)}">
       <script defer="defer">
-        refreshPart('executors',"${h.ifThenElse(h.hasView(it,'ajaxExecutors'),'.',rootURL)}/ajaxExecutors");
+        refreshPart('executors',"${h.hasView(it,'ajaxExecutors')?'.':rootURL}/ajaxExecutors");
       </script>
     </j:if>
 	</l:pane>
diff --git a/core/src/main/resources/lib/hudson/progressBar.jelly b/core/src/main/resources/lib/hudson/progressBar.jelly
index 718603dcbd..a4ea13eed6 100644
--- a/core/src/main/resources/lib/hudson/progressBar.jelly
+++ b/core/src/main/resources/lib/hudson/progressBar.jelly
@@ -31,7 +31,7 @@ THE SOFTWARE.
     red  : if set to non-null, the progress bar will be drawn in red, to indicate that the processing is likely stuck.
 -->
 <j:jelly xmlns:j="jelly:core" xmlns:x="jelly:xml" xmlns:st="jelly:stapler" xmlns:d="jelly:define" xmlns:l="/lib/layout" xmlns:t="/lib/hudson" xmlns:f="/lib/form">
-  <table class="progress-bar ${h.ifThenElse(attrs.red,'red',null)}" id="${attrs.id}" tooltip="${attrs.tooltip}" href="${attrs.href}" style="${h.ifThenElse(attrs.href!=null,'cursor:pointer',null)}">
+  <table class="progress-bar ${attrs.red?'red':null}" id="${attrs.id}" tooltip="${attrs.tooltip}" href="${attrs.href}" style="${h.ifThenElse(attrs.href!=null,'cursor:pointer',null)}">
     <j:choose>
       <j:when test="${pos lt 0}">
         <!-- style sheet will fill in the progress bar image -->
diff --git a/core/src/main/resources/lib/hudson/progressiveText.jelly b/core/src/main/resources/lib/hudson/progressiveText.jelly
index 28fc8d072d..ea3a5b8293 100644
--- a/core/src/main/resources/lib/hudson/progressiveText.jelly
+++ b/core/src/main/resources/lib/hudson/progressiveText.jelly
@@ -65,7 +65,7 @@ THE SOFTWARE.
 	      });
 	    }
 	  </j:if>
-	  $$("${idref}").fetchedBytes = ${h.ifThenElse(empty(startOffset),0,startOffset)};
+	  $$("${idref}").fetchedBytes = ${empty(startOffset)?0:startOffset};
 	  fetchNext($$("${idref}"),"${href}");
 	</script>
 </j:jelly>
diff --git a/core/src/main/resources/lib/hudson/project/config-scm.jelly b/core/src/main/resources/lib/hudson/project/config-scm.jelly
index 140cb6d2a3..96da937686 100644
--- a/core/src/main/resources/lib/hudson/project/config-scm.jelly
+++ b/core/src/main/resources/lib/hudson/project/config-scm.jelly
@@ -30,7 +30,7 @@ THE SOFTWARE.
       <j:set var="descriptor" value="${scms[idx]}" />
       <j:set var="scmd" value="${descriptor}" /><!-- backward compatibility with <1.238 -->
       <f:radioBlock name="scm" value="${idx}" title="${scmd.displayName}" checked="${it.scm.descriptor==scmd}">
-        <j:set var="instance" value="${h.ifThenElse(it.scm.descriptor==descriptor, it.scm, null)}"/>
+        <j:set var="instance" value="${it.scm.descriptor==descriptor ? it.scm : null}"/>
         <j:set var="scm" value="${instance}" /><!-- backward compatibility with <1.238 -->
         <st:include from="${scmd}" page="${scmd.configPage}"/>
       </f:radioBlock>
diff --git a/core/src/main/resources/lib/hudson/projectView.jelly b/core/src/main/resources/lib/hudson/projectView.jelly
index 31db7ea04d..a25c250893 100644
--- a/core/src/main/resources/lib/hudson/projectView.jelly
+++ b/core/src/main/resources/lib/hudson/projectView.jelly
@@ -49,7 +49,7 @@ THE SOFTWARE.
       <d:invokeBody/>
       <!-- project list -->
       <table id="projectstatus" class="sortable pane bigtable"
-             style="${h.ifThenElse(showViewTabs!=null,'margin-top:0px; border-top: none;',null)}">
+             style="${showViewTabs!=null?'margin-top:0px; border-top: none;':null}">
         <tr style="border-top: 0px;">
           <j:forEach var="col" items="${columnExtensions}">
              <st:include page="columnHeader.jelly" it="${col}" />
diff --git a/core/src/main/resources/lib/hudson/projectViewNested.jelly b/core/src/main/resources/lib/hudson/projectViewNested.jelly
index dcc693a729..6e76d140dd 100644
--- a/core/src/main/resources/lib/hudson/projectViewNested.jelly
+++ b/core/src/main/resources/lib/hudson/projectViewNested.jelly
@@ -38,7 +38,7 @@ THE SOFTWARE.
             <!-- if nested view supports ajaxRows -->
             <a href="#">
               <img src="${imagesURL}/tree/plus.png" state="plus" width="14" height="14" alt="" class="fold-control treeview-fold-control"
-                   url="${rootURL}/${v.url}/ajaxRows?depth=${h.ifThenElse(depth!=null,depth+1,1)}"/>
+                   url="${rootURL}/${v.url}/ajaxRows?depth=${depth!=null?depth+1:1}"/>
               <st:adjunct includes="lib.hudson.projectViewNested" />
             </a>
           </j:otherwise>
diff --git a/core/src/main/resources/lib/hudson/projectViewRow.jelly b/core/src/main/resources/lib/hudson/projectViewRow.jelly
index 4d7354c640..a495a17734 100644
--- a/core/src/main/resources/lib/hudson/projectViewRow.jelly
+++ b/core/src/main/resources/lib/hudson/projectViewRow.jelly
@@ -32,7 +32,7 @@ THE SOFTWARE.
          xmlns:t="/lib/hudson" xmlns:f="/lib/form">
   <j:set var="lsBuild" value="${job.lastSuccessfulBuild}"/>
   <j:set var="lfBuild" value="${job.lastFailedBuild}"/>
-  <tr class="${h.ifThenElse(job.disabled,'disabledJob',null)}">
+  <tr class="${job.disabled?'disabledJob':null}">
       <j:forEach var="col" items="${columnExtensions}">
           <st:include page="column.jelly" it="${col}" />
       </j:forEach>
diff --git a/core/src/main/resources/lib/hudson/queue.jelly b/core/src/main/resources/lib/hudson/queue.jelly
index fe3775f05a..67f4480aad 100644
--- a/core/src/main/resources/lib/hudson/queue.jelly
+++ b/core/src/main/resources/lib/hudson/queue.jelly
@@ -58,7 +58,7 @@ THE SOFTWARE.
           <tr>
             <td class="pane" width="100%" tooltip="${item.why}" style="white-space: normal;">
               <j:set var="stuck" value="${item.isStuck()}" />
-              <a href="${rootURL}/${item.task.url}" style="${h.ifThenElse(stuck,'color:#ef2929',null)}">
+              <a href="${rootURL}/${item.task.url}" style="${stuck?'color:#ef2929':null}">
                 ${item.task.fullDisplayName}
                 <j:if test="${stuck}"> (${%appears to be stuck})</j:if>
               </a>
@@ -75,7 +75,7 @@ THE SOFTWARE.
     <!-- schedule updates only for the full page reload -->
     <j:if test="${ajax==null and !h.isAutoRefresh(request) and h.hasPermission(app.READ)}">
       <script defer="defer">
-        refreshPart('buildQueue',"${h.ifThenElse(h.hasView(it,'ajaxBuildQueue'),'.',rootURL)}/ajaxBuildQueue");
+        refreshPart('buildQueue',"${h.hasView(it,'ajaxBuildQueue')?'.':rootURL}/ajaxBuildQueue");
       </script>
     </j:if>
   </l:pane>
diff --git a/core/src/main/resources/lib/hudson/setIconSize.jelly b/core/src/main/resources/lib/hudson/setIconSize.jelly
index 200d543f10..427c1f9fbc 100644
--- a/core/src/main/resources/lib/hudson/setIconSize.jelly
+++ b/core/src/main/resources/lib/hudson/setIconSize.jelly
@@ -32,5 +32,5 @@ THE SOFTWARE.
     so we try not to make the secondary icons look bigger than the icon.
     we want the user's eyes to go to balls, not the clock.
   -->
-  <j:set scope="parent" var="subIconSize" value="${h.ifThenElse(iconSize=='32x32','24x24',iconSize)}"/>
+  <j:set scope="parent" var="subIconSize" value="${iconSize=='32x32'?'24x24':iconSize}"/>
 </j:jelly>
\ No newline at end of file
diff --git a/core/src/main/resources/lib/hudson/summary.jelly b/core/src/main/resources/lib/hudson/summary.jelly
index 33c2c7dc6a..eecf88189c 100644
--- a/core/src/main/resources/lib/hudson/summary.jelly
+++ b/core/src/main/resources/lib/hudson/summary.jelly
@@ -45,12 +45,12 @@ THE SOFTWARE.
       <tr>
         <td>
           <a href="${attrs.href}">
-            <img src="${h.ifThenElse(icon.startsWith('/'), rootURL+icon, imagesURL+'/48x48/'+icon)}"
+            <img src="${icon.startsWith('/') ? rootURL+icon : imagesURL+'/48x48/'+icon}"
                  alt="" width="48" height="48" style="margin-right:1em" />
           </a>
         </td>
         <td style="vertical-align:middle">
-          <a href="${h.ifThenElse(attrs.iconOnly==null,attrs.href,null)}">
+          <a href="${attrs.iconOnly==null ? attrs.href : null}">
             <d:invokeBody />
           </a>
         </td>
diff --git a/core/src/main/resources/lib/layout/task.jelly b/core/src/main/resources/lib/layout/task.jelly
index e76c2b7577..1a4f9cc6f2 100644
--- a/core/src/main/resources/lib/layout/task.jelly
+++ b/core/src/main/resources/lib/layout/task.jelly
@@ -72,7 +72,7 @@ THE SOFTWARE.
       <j:otherwise>
         <div class="task">
           <a href="${href}" onclick="${attrs.onclick}">
-            <img width="24" height="24" style="margin: 2px;" alt="" src="${rootURL}${h.ifThenElse(icon.startsWith('images/'),h.resourcePath,'')}/${icon}"/>
+            <img width="24" height="24" style="margin: 2px;" alt="" src="${rootURL}${icon.startsWith('images/') ? h.resourcePath : '')}/${icon}"/>
           </a>
           <st:nbsp />
 
-- 
GitLab