Fix urls when 2 clouds with same name exist

When 2 clouds are created with the same name, then accessing the second
cloud produces a different url.

Author: mawinter69

core/src/main/java/hudson/slaves/Cloud.java

@@ -140,7 +140,11 @@ public String getDisplayName() {
      * @return Jenkins relative URL.
      */
     public @NonNull String getUrl() {
-        return "cloud/" + Util.rawEncode(name) + "/";
+        if (Jenkins.get().getCloud(name) != this) { // this cloud is not the first occurrence with this name
+            return "cloud/cloudByIndex/" + Jenkins.get().clouds.indexOf(this) + "/";
+        } else {
+            return "cloud/" + Util.rawEncode(name) + "/";
+        }
     }
 
     @Override

core/src/main/java/jenkins/agents/CloudSet.java

@@ -113,11 +113,7 @@ public ManagementLink getManagementLink() {
     @Restricted(DoNotUse.class) // stapler
     public String getCloudUrl(StaplerRequest2 request, Jenkins jenkins, Cloud cloud) {
         String context = Functions.getNearestAncestorUrl(request, jenkins);
-        if (Jenkins.get().getCloud(cloud.name) != cloud) { // this cloud is not the first occurrence with this name
-            return context + "/cloud/cloudByIndex/" + getClouds().indexOf(cloud) + "/";
-        } else {
-            return context + "/" + cloud.getUrl();
-        }
+        return context + "/" + cloud.getUrl();
     }
 
     /**

core/src/main/resources/hudson/slaves/Cloud/sidepanel.jelly

@@ -27,9 +27,9 @@ THE SOFTWARE.
     <l:header />
     <l:side-panel>
         <l:tasks>
-            <j:set var="url" value="${h.getNearestAncestorUrl(request2,it)}"/>
-            <l:task contextMenu="false" href="${url}/" icon="symbol-computer" title="${%Status}"/>
-            <l:task href="${url}/configure" icon="symbol-settings"
+            <j:set var="url" value="${rootURL}/${it.url}"/>
+            <l:task contextMenu="false" href="${url}" icon="symbol-computer" title="${%Status}"/>
+            <l:task href="${url}configure" icon="symbol-settings"
                     title="${app.hasPermission(app.ADMINISTER) ? '%Configure' : '%View Configuration'}"/>
             <l:delete permission="${app.ADMINISTER}" title="${%Delete Cloud}" message="${%delete.cloud(it.displayName)}" urlPrefix="${url}"/>
             <t:actions />

core/src/main/resources/jenkins/agents/CloudSet/index.jelly

@@ -64,11 +64,11 @@ THE SOFTWARE.
                   </td>
                   <td>
                     <input type="hidden" name="name" value="${cloud.name}" />
-                    <a href="${it.getCloudUrl(request2,app,cloud)}" class="jenkins-table__link model-link inside">${cloud.name}</a>
+                    <a href="../${cloud.url}" class="jenkins-table__link model-link inside">${cloud.name}</a>
                   </td>
                   <td class="jenkins-table__cell--tight">
                     <div class="jenkins-table__cell__button-wrapper">
-                      <a href="${it.getCloudUrl(request2,app,cloud)}configure" class="jenkins-button">
+                      <a href="../${cloud.url}configure" class="jenkins-button">
                         <l:icon src="symbol-settings"/>
                       </a>
                     </div>

core/src/main/resources/lib/layout/delete.jelly

@@ -75,7 +75,8 @@ THE SOFTWARE.
 
   <j:set var="url" value="doDelete" />
   <j:if test="${attrs.urlPrefix != null}">
-    <j:set var="url" value="${urlPrefix}/doDelete" />
+    <j:set var="basePrefix" value="${urlPrefix.endsWith('/') ? urlPrefix.substring(0, urlPrefix.length() - 1) : urlPrefix}" />
+    <j:set var="url" value="${basePrefix}/doDelete" />
   </j:if>