From 5522f6008caf2160a092285a7d416d94611f05aa Mon Sep 17 00:00:00 2001
From: Martin Pokorny <89339813+mPokornyETM@users.noreply.github.com>
Date: Mon, 2 Jan 2023 14:10:17 -0400
Subject: [PATCH] Allow formatted markup for node descriptions (#6511)

Co-authored-by: Adrien Lecharpentier <adrien.lecharpentier@gmail.com>
Co-authored-by: Daniel Beck <1831569+daniel-beck@users.noreply.github.com>
Co-authored-by: Alexander Brandes <mc.cache@web.de>
Co-authored-by: Daniel Beck <daniel-beck@users.noreply.github.com>
Co-authored-by: Basil Crow <me@basilcrow.com>
Co-authored-by: Tim Jacomb <timjacomb1@gmail.com>
Co-authored-by: Tim Jacomb <21194782+timja@users.noreply.github.com>
---
 .../java/hudson/slaves/SlaveComputer.java     | 17 +++++++++++++++
 .../AbstractModelObject/descriptionForm.jelly | 12 +++++++++--
 .../hudson/model/Computer/index-top.jelly     | 11 ++++++++++
 .../hudson/model/Computer/index.jelly         | 11 +---------
 .../slaves/DumbSlave/configure-entries.jelly  |  8 ++++---
 .../slaves/SlaveComputer/systemInfo.jelly     |  7 ++-----
 .../Jenkins/MasterComputer/index-top.jelly    | 13 ++++++++++++
 .../MasterComputer/index-top.properties       |  4 ++++
 .../lib/hudson/editable-description.js        | 16 ++++++++++++++
 .../lib/hudson/editableDescription.jelly      | 21 ++++++++++++++-----
 .../main/webapp/scripts/hudson-behavior.js    | 10 ++++++++-
 11 files changed, 104 insertions(+), 26 deletions(-)
 create mode 100644 core/src/main/resources/hudson/model/Computer/index-top.jelly
 create mode 100644 core/src/main/resources/jenkins/model/Jenkins/MasterComputer/index-top.jelly
 create mode 100644 core/src/main/resources/jenkins/model/Jenkins/MasterComputer/index-top.properties
 create mode 100644 core/src/main/resources/lib/hudson/editable-description.js

diff --git a/core/src/main/java/hudson/slaves/SlaveComputer.java b/core/src/main/java/hudson/slaves/SlaveComputer.java
index daaba7a211d7..ccc90ec0873b 100644
--- a/core/src/main/java/hudson/slaves/SlaveComputer.java
+++ b/core/src/main/java/hudson/slaves/SlaveComputer.java
@@ -780,6 +780,23 @@ public List<LogRecord> getLogRecords() throws IOException, InterruptedException
             return channel.call(new SlaveLogFetcher());
     }
 
+    /**
+     * Inline editing of description
+     */
+    @RequirePOST
+    @Restricted(NoExternalUse.class)
+    public synchronized void doSubmitDescription(StaplerResponse rsp, @QueryParameter String description) throws IOException {
+        checkPermission(CONFIGURE);
+
+        final Slave node = this.getNode();
+        if (node != null) {
+            node.setNodeDescription(description);
+        } else { // Node has been disabled/removed during other session tries to change the description.
+            throw new IOException("Description will be not set. The node " + nodeName + " does not exist (anymore).");
+        }
+        rsp.sendRedirect(".");
+    }
+
     @RequirePOST
     public HttpResponse doDoDisconnect(@QueryParameter String offlineMessage) {
         if (channel != null) {
diff --git a/core/src/main/resources/hudson/model/AbstractModelObject/descriptionForm.jelly b/core/src/main/resources/hudson/model/AbstractModelObject/descriptionForm.jelly
index 2d793ab4b91d..633e67f3568f 100644
--- a/core/src/main/resources/hudson/model/AbstractModelObject/descriptionForm.jelly
+++ b/core/src/main/resources/hudson/model/AbstractModelObject/descriptionForm.jelly
@@ -30,11 +30,19 @@ THE SOFTWARE.
   <j:if test="${it.getPrimaryView() != null}">
     <j:set var="it" value="${it.getPrimaryView()}"/>
   </j:if>
+  <j:set var="submissionUrl" value="${request.getParameter('submissionUrl')}" defaultValue="submitDescription"/>
+  <j:invokeStatic className="hudson.Util" method="isSafeToRedirectTo" var="isSafeToRedirectTo">
+    <j:arg value="${submissionUrl}"/>
+  </j:invokeStatic>
+  <j:if test="${!isSafeToRedirectTo}">
+    <j:set var="submissionUrl" value="submitDescription"/>
+  </j:if>
+  <j:set var="initialDescription" value="${request.getParameter('description')}" defaultValue="${it.description}"/>
   <l:ajax>
-    <form action="submitDescription" method="post">
+    <form action="${submissionUrl}" method="post">
       <table>
         <f:entry help="${app.markupFormatter.helpUrl}">
-          <f:textarea name="description" value="${it.description}"
+          <f:textarea name="description" value="${initialDescription}"
                       codemirror-mode="${app.markupFormatter.codeMirrorMode}" codemirror-config="${app.markupFormatter.codeMirrorConfig}" previewEndpoint="/markupFormatter/previewDescription"/>
         </f:entry>
       </table>
diff --git a/core/src/main/resources/hudson/model/Computer/index-top.jelly b/core/src/main/resources/hudson/model/Computer/index-top.jelly
new file mode 100644
index 000000000000..2d2e116c059a
--- /dev/null
+++ b/core/src/main/resources/hudson/model/Computer/index-top.jelly
@@ -0,0 +1,11 @@
+<?jelly escape-by-default='true'?>
+<j:jelly xmlns:j="jelly:core" xmlns:l="/lib/layout" xmlns:t="/lib/hudson">
+  <!-- Node name and status -->
+  <h1>
+    <l:icon src="${it.iconClassName}" class="icon-xlg" />
+    ${it.caption}
+  </h1>
+
+  <!-- Node description -->
+  <t:editableDescription permission="${it.CONFIGURE}" description="${it.node.nodeDescription}" submissionUrl="submitDescription" />
+</j:jelly>
diff --git a/core/src/main/resources/hudson/model/Computer/index.jelly b/core/src/main/resources/hudson/model/Computer/index.jelly
index 2961982f37ed..ed11d71d44c4 100644
--- a/core/src/main/resources/hudson/model/Computer/index.jelly
+++ b/core/src/main/resources/hudson/model/Computer/index.jelly
@@ -28,6 +28,7 @@ THE SOFTWARE.
     <st:include page="sidepanel.jelly" />
     <!-- no need for additional breadcrumb here as we're on an index page already including breadcrumb -->
     <l:main-panel>
+      <st:include page="index-top.jelly" it="${it}" />
 
       <!-- temporarily offline switch -->
       <div style="float:right">
@@ -53,16 +54,6 @@ THE SOFTWARE.
         </j:choose>
       </div>
 
-      <h1>
-        <span class="icon-lg">
-          <l:icon src="${it.iconClassName}" />
-        </span>
-        ${it.caption}
-        <j:if test="${!empty(it.node.nodeDescription)}">
-          <span style="font-size:smaller">(${it.node.nodeDescription})</span>
-        </j:if>
-      </h1>
-
       <j:if test="${it.offlineCause!=null and it.offline and !it.connecting}">
         <st:include it="${it.offlineCause}" page="cause.jelly" />
       </j:if>
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 1e0b8624fc60..bb6ca2e99838 100644
--- a/core/src/main/resources/hudson/slaves/DumbSlave/configure-entries.jelly
+++ b/core/src/main/resources/hudson/slaves/DumbSlave/configure-entries.jelly
@@ -28,9 +28,11 @@ 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">
-  <f:entry title="${%Description}" help="/help/system-config/master-slave/description.html">
-    <f:textbox field="nodeDescription" />
-  </f:entry>
+  <div class="jenkins-section">
+    <f:entry title="${%Description}" help="/help/system-config/master-slave/description.html">
+      <f:textarea name="nodeDescription" value="${it.nodeDescription}" codemirror-mode="${app.markupFormatter.codeMirrorMode}" codemirror-config="${app.markupFormatter.codeMirrorConfig}" previewEndpoint="/markupFormatter/previewDescription"/>
+    </f:entry>
+  </div>
 
   <f:entry title="${%Number of executors}" field="numExecutors">
     <f:number clazz="positive-number-required" min="1" step="1" default="1"/>
diff --git a/core/src/main/resources/hudson/slaves/SlaveComputer/systemInfo.jelly b/core/src/main/resources/hudson/slaves/SlaveComputer/systemInfo.jelly
index 692e93d645f1..41946d8be476 100644
--- a/core/src/main/resources/hudson/slaves/SlaveComputer/systemInfo.jelly
+++ b/core/src/main/resources/hudson/slaves/SlaveComputer/systemInfo.jelly
@@ -38,11 +38,8 @@ THE SOFTWARE.
       <h1>
         <span class="icon-lg">
           <l:icon src="${it.icon}"/>
-        </span>
-        ${it.caption} ${%System Information}
-        <j:if test="${!empty(it.node.nodeDescription)}">
-          <span style="font-size:smaller">(${it.node.nodeDescription})</span>
-        </j:if>
+        </span>${it.caption}
+        ${%System Information}
       </h1>
 
       <j:choose>
diff --git a/core/src/main/resources/jenkins/model/Jenkins/MasterComputer/index-top.jelly b/core/src/main/resources/jenkins/model/Jenkins/MasterComputer/index-top.jelly
new file mode 100644
index 000000000000..d3b03a03b22e
--- /dev/null
+++ b/core/src/main/resources/jenkins/model/Jenkins/MasterComputer/index-top.jelly
@@ -0,0 +1,13 @@
+<?jelly escape-by-default='true'?>
+<j:jelly xmlns:j="jelly:core" xmlns:l="/lib/layout">
+  <!-- Node name and status -->
+  <h1>
+    <l:icon src="${it.iconClassName}" class="icon-xlg" />
+    ${it.caption}
+  </h1>
+
+  <!-- Node description -->
+  <div class="jenkins-!-margin-bottom-5">
+    ${%blurb}
+  </div>
+</j:jelly>
diff --git a/core/src/main/resources/jenkins/model/Jenkins/MasterComputer/index-top.properties b/core/src/main/resources/jenkins/model/Jenkins/MasterComputer/index-top.properties
new file mode 100644
index 000000000000..f66557ace431
--- /dev/null
+++ b/core/src/main/resources/jenkins/model/Jenkins/MasterComputer/index-top.properties
@@ -0,0 +1,4 @@
+blurb = This is the Jenkins controller''s built-in node. \
+  Builds running on this node will execute on the same system and as the same user as the Jenkins controller. \
+  This is appropriate e.g. for special jobs performing backups, but in general you should run builds on agents. \
+  <a rel="noopener noreferrer nofollow" target="_blank" href="http://www.jenkins.io/redirect/distributed-builds/">Learn more about distributed builds.</a>
diff --git a/core/src/main/resources/lib/hudson/editable-description.js b/core/src/main/resources/lib/hudson/editable-description.js
new file mode 100644
index 000000000000..30a5e01efdaf
--- /dev/null
+++ b/core/src/main/resources/lib/hudson/editable-description.js
@@ -0,0 +1,16 @@
+/* global replaceDescription */
+(function () {
+  document.addEventListener("DOMContentLoaded", function () {
+    let descriptionLink = document.querySelector("#description-link");
+    descriptionLink.addEventListener("click", function (e) {
+      e.preventDefault();
+      let url = descriptionLink.getAttribute("data-url");
+      let description = descriptionLink.getAttribute("data-description");
+      if (url == null && description == null) {
+        return replaceDescription();
+      } else {
+        return replaceDescription(description, url);
+      }
+    });
+  });
+})();
diff --git a/core/src/main/resources/lib/hudson/editableDescription.jelly b/core/src/main/resources/lib/hudson/editableDescription.jelly
index 2ed80ba5e91c..f3c5bb6d76dc 100644
--- a/core/src/main/resources/lib/hudson/editableDescription.jelly
+++ b/core/src/main/resources/lib/hudson/editableDescription.jelly
@@ -32,19 +32,30 @@ THE SOFTWARE.
       If the current user doesn't have this permission, the description
       is not editable.
     </st:attribute>
+    <st:attribute name="description">
+      Optional value of the description. If not specified, will use '${it.description}'.
+    </st:attribute>
+    <st:attribute name="submissionUrl">
+      Optional submission URL. If not specified, will use 'submitDescription'.
+    </st:attribute>
   </st:documentation>
 
-  <div id="description">
+  <j:set var="actualDescription" value="${description}" defaultValue="${it.description}"/>
+
+  <div id="description" class="jenkins-!-margin-bottom-0">
     <div>
-      <j:out value="${it.description!=null ? app.markupFormatter.translate(it.description) : ''}" />
+      <j:out value="${actualDescription!=null ? app.markupFormatter.translate(actualDescription) : ''}" />
     </div>
 
     <l:hasPermission permission="${permission}">
       <div class="jenkins-buttons-row jenkins-buttons-row--invert">
-        <a class="jenkins-button jenkins-button--tertiary" id="description-link" href="editDescription" onclick="return replaceDescription();">
-          <l:icon src="symbol-edit" />
+        <st:adjunct includes="lib.hudson.editable-description"/>
+        <a class="jenkins-button jenkins-button--tertiary" id="description-link" href="editDescription" data-description="${actualDescription}" data-url="${submissionUrl}">
+          <span class="leading-icon">
+            <l:icon src="symbol-edit" />
+          </span>
           <j:choose>
-            <j:when test="${empty(it.description)}">
+            <j:when test="${empty(actualDescription)}">
               ${%add description}
             </j:when>
             <j:otherwise>
diff --git a/war/src/main/webapp/scripts/hudson-behavior.js b/war/src/main/webapp/scripts/hudson-behavior.js
index ee337bb10763..3a0c86c7e2e9 100644
--- a/war/src/main/webapp/scripts/hudson-behavior.js
+++ b/war/src/main/webapp/scripts/hudson-behavior.js
@@ -1841,10 +1841,18 @@ function xor(a, b) {
 }
 
 // used by editableDescription.jelly to replace the description field with a form
-function replaceDescription() {
+function replaceDescription(initialDescription, submissionUrl) {
   var d = document.getElementById("description");
   $(d).down().next().innerHTML = "<div class='jenkins-spinner'></div>";
+  let parameters = {};
+  if (initialDescription !== undefined && submissionUrl !== undefined) {
+    parameters = {
+      description: initialDescription,
+      submissionUrl: submissionUrl,
+    };
+  }
   new Ajax.Request("./descriptionForm", {
+    parameters: parameters,
     onComplete: function (x) {
       d.innerHTML = x.responseText;
       evalInnerHtmlScripts(x.responseText, function () {