Merge branch 'master' into JENKINS-73960

src/main/java/au/com/centrumsystems/hudson/plugin/buildpipeline/BuildForm.java

@@ -94,7 +94,7 @@ private BuildForm(ItemGroup context, final PipelineBuild pipelineBuild, final Co
         }
         id = hashCode();
         final AbstractProject<?, ?> project = pipelineBuild.getProject();
-        projectId = project.getFullName().hashCode();
+        projectId = project.getFullName().hashCode() & 0x7FFFFFFF;
         final ParametersDefinitionProperty params = project.getProperty(ParametersDefinitionProperty.class);
         final ArrayList<String> paramList = new ArrayList<String>();
         if (params != null && params.getParameterDefinitionNames() != null) {

src/main/java/au/com/centrumsystems/hudson/plugin/buildpipeline/ProjectForm.java

@@ -240,7 +240,7 @@ public boolean equals(final Object obj) {
     }
 
     public int getId() {
-        return name.hashCode();
+        return name.hashCode() & 0x7FFFFFFF;
     }
 
     /**

src/main/resources/au/com/centrumsystems/hudson/plugin/buildpipeline/BuildPipelineView/main_dashboard.jelly

@@ -2,16 +2,5 @@
 <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">
     <st:include page="bpp.jelly" from="${it.getBuildPipelineView()}" />
     <link href="${rootURL}/plugin/build-pipeline-plugin/css/main_dashboard.css" type="text/css" rel="stylesheet" />
-    <script type="text/javascript">
-        // show/hide build details
-        jQuery(document).ready(function() {
-            jQuery(".header").click(function() {
-                var parent = jQuery(this).parent();
-                var ba = parent.find(".build-actions");
-                var bb = parent.find(".build-body");
-
-                ba.add(bb).toggle('slow');
-            });
-        });
-    </script>
+    <st:adjunct includes="au.com.centrumsystems.hudson.plugin.buildpipeline.BuildPipelineView.toggle-build-details" />
 </j:jelly>

src/main/resources/au/com/centrumsystems/hudson/plugin/buildpipeline/BuildPipelineView/toggle-build-details.js

@@ -0,0 +1,10 @@
+// show/hide build details
+jQuery(document).ready(function() {
+    jQuery(".header").click(function() {
+        var parent = jQuery(this).parent();
+        var ba = parent.find(".build-actions");
+        var bb = parent.find(".build-body");
+
+        ba.add(bb).toggle('slow');
+    });
+});

src/main/resources/au/com/centrumsystems/hudson/plugin/buildpipeline/extension/BuildCardExtension/build-card-script-helpers.js

@@ -0,0 +1,7 @@
+Handlebars.registerHelper('startScript', function() {
+    return new Handlebars.SafeString("<script>");
+});
+
+Handlebars.registerHelper('endScript', function() {
+    return new Handlebars.SafeString("</script>");
+});

src/main/resources/au/com/centrumsystems/hudson/plugin/buildpipeline/extension/BuildCardExtension/buildCardHelpers.jelly

@@ -7,17 +7,5 @@
 
         By default we just include helpers to append script tags to your build card
     -->
-    <![CDATA[
-
-    <script type="text/javascript">
-        Handlebars.registerHelper('startScript', function() {
-            return new Handlebars.SafeString("<scr" + "ipt>");
-        });
-
-        Handlebars.registerHelper('endScript', function() {
-            return new Handlebars.SafeString("</scr" + "ipt>");
-        });
-    </script>
-
-    ]]>
+    <st:adjunct includes="au.com.centrumsystems.hudson.plugin.buildpipeline.extension.BuildCardExtension.build-card-script-helpers"/>
 </j:jelly>