Get build working on jre6,7,8

build.xml

@@ -2,12 +2,31 @@
   <property file="build.properties"/>
   <description>Builds the scriptcraft.jar file - a plugin for bukkit</description>
   <property name="src.canary" location="src/main/java/canary"/>
+  <property name="lib.canary" location="lib/canary.jar"/>
   <property name="src.bukkit" location="src/main/java/bukkit"/>
   <property name="build" location="target/classes"/>
   <property name="dist" location="target/" />
   <property name="minecraft.dir" location="${dist}/minecraft" />
   <property name="js-plugins-dir" value="scriptcraft"/>
   <property name="http.agent" value="'Walter'" />
+
+  <macrodef name="jscript">
+    <attribute name="src"/>
+    <attribute name="out"/>
+    <attribute name="err"/>
+    <element name="js-args" implicit="yes" optional="true"/>
+    <sequential>
+      <java classname="jscript" failonerror="true" fork="true" output="@{out}"  error="@{err}">
+	<classpath>
+          <pathelement path="${build}"/>      
+	  <pathelement path="${lib.canary}"/>
+	</classpath>
+      <arg value="@{src}"/>
+      <js-args/>
+      </java>
+    </sequential>
+  </macrodef>
+
   <target name="init">
     <property file="build.local.properties"/>
     <tstamp>
@@ -35,7 +54,7 @@
       <src path="${src.canary}"/>
       <src path="${src.bukkit}"/>
       <classpath>
-	<pathelement path="lib/canary.jar" />
+	<pathelement path="${lib.canary}" />
 	<pathelement path="lib/bukkit-1.7.9-R0.2.jar" />
       </classpath>
     </javac>
@@ -45,30 +64,19 @@
   </target>
 
   <target name="compile-docs" depends="init">
-    <javac includeantruntime="false" 
+    <javac includeantruntime="false" srcdir="src/docs/java" destdir="${build}">
-	   srcdir="src/docs/java"
+      <classpath>
-	   destdir="${build}"
+	<pathelement path="${lib.canary}"/>
-	   />
+      </classpath>
+    </javac>
   </target>
 
   <target name="generate-api-ref-entries" depends="copy-js,compile-docs,init">
 
-    <java classname="jscript" failonerror="true" fork="true" output="${dist}/apiref.md">
+    <jscript src="src/docs/js/generateApiDocs.js" out="${dist}/apiref.md" err="${dist}/gen-api-error.log">
-      <classpath>
-        <pathelement path="${build}"/>      
-      </classpath>
-      <arg value="src/docs/js/generateApiDocs.js"/>
       <arg value="${dist}/js"/>      
-    </java>
+    </jscript>
-
+    <jscript src="src/docs/js/generateItemsDoc.js" out="${dist}/items.md" err="${dist}/gen-items-error.log" />
-    <java classname="jscript" failonerror="true" fork="true" output="${dist}/items.md" error="${dist}/genitemserror.log">
-      <classpath>
-        <pathelement path="${build}"/>      
-	<pathelement path="lib/canary.jar"/>
-      </classpath>
-      <arg value="src/docs/js/generateItemsDoc.js"/>
-    </java>
-
     <concat destfile="${dist}/apiref-con.md">
       <fileset file="${dist}/apiref.md" />
       <fileset file="${dist}/items.md" />
@@ -80,32 +88,22 @@
 
   <target name="gen-events-helper-canary" depends="compile-docs,init">
     <mkdir dir="${dist}/js/lib"/>
-    <java classname="jscript" failonerror="true" fork="true" output="${dist}/js/lib/events-helper.js" error="${dist}/geneventserror.log">
+    <jscript src="src/docs/js/generateEventsHelper.js" out="${dist}/js/lib/events-helper.js" err="${dist}/gen-events-canary-error.log">
-      <classpath>
-        <pathelement path="${build}"/>      
-	<pathelement path="lib/canary.jar"/>
-      </classpath>
-      <arg value="src/generateEventsHelper.js"/>
       <arg value="canary"/>
-      <arg value="lib/canary.jar"/>
+      <arg value="${lib.canary}"/>
       <arg value="blockDestroy"/>
       <arg value="net.canarymod.hook.player.BlockDestroyHook"/>
-    </java>
+    </jscript>
   </target>
 
   <target name="gen-events-helper-bukkit" depends="compile-docs,init">
     <mkdir dir="${dist}/js/lib"/>
-    <java classname="jscript" failonerror="true" fork="true" output="${dist}/js/lib/events-helper.js" error="${dist}/geneventserror.log">
+    <jscript src="src/docs/js/generateEventsHelper.js" out="${dist}/js/lib/events-helper.js" err="${dist}/gen-events-bukkit-error.log">
-      <classpath>
+      <arg value="bukkit"/>
-        <pathelement path="${build}"/>      
+      <arg value="${lib.canary}"/>
-	<pathelement path="lib/canary.jar"/>
-      </classpath>
-      <arg value="src/generateEventsHelper.js"/>
-      <arg value="canary"/>
-      <arg value="lib/canary.jar"/>
       <arg value="Break"/>
       <arg value="org.bukkit.event.block.BlockBreakEvent"/>
-    </java>
+    </jscript>
   </target>
 
   <target name="construct-api-ref" depends="gen-toc-apiref,init">
@@ -126,23 +124,15 @@ Walter Higgins
   </target>
   
   <target name="gen-toc-apiref" depends="compile-docs,generate-api-ref-entries, init" description="Generate Table of Contents for API Reference">
-    <java classname="jscript" failonerror="true" fork="true" output="${dist}/toc-apiref.md">
+    <jscript src="src/docs/js/generateTOC.js" out="${dist}/toc-apiref.md"  err="${dist}/gen-toc-error.log">
-      <classpath>
-        <pathelement path="${build}"/>      
-      </classpath>
-      <arg value="src/docs/js/generateTOC.js"/>
       <arg value="${dist}/apiref-con.md"/>
-    </java>
+    </jscript>
   </target>
 
   <target name="gen-toc-ypgpm" depends="compile-docs,init" description="Generate Table of Contents for Young Programmers Guide">
-    <java classname="jscript" failonerror="true" fork="true" output="${dist}/toc-ypgpm.md">
+    <jscript src="src/docs/js/generateTOC.js" out="${dist}/toc-ypgpm.md" err="${dist}/gen-ypgpm-error.log">
-      <classpath>
-        <pathelement path="${build}"/>      
-      </classpath>
-      <arg value="src/docs/js/generateTOC.js"/>
       <arg value="src/docs/templates/ypgpm.md"/>
-    </java>
+    </jscript>
   </target>
 
   <!-- Piece together the Young persons' guide from template and generated table of contents -->

src/docs/java/jscript.java

@@ -1,5 +1,7 @@
 import javax.script.*;
 import java.io.FileReader;
+import net.canarymod.api.Server;
+import net.canarymod.api.inventory.ItemType;
 
 public class jscript
 {
@@ -10,6 +12,7 @@ public class jscript
         java.io.File file = new java.io.File(args[0]);
         engine.put("engine",engine);
         engine.put("args",args);
+	engine.put("itemTypeClass",ItemType.class);
         FileReader fr = new java.io.FileReader(file);
         engine.eval(fr);
         fr.close();

src/docs/js/generateApiDocs.js

@@ -106,11 +106,10 @@ if (typeof importPackage == 'undefined'){
 }
 var dir = args[0];
 
-
+var io = Packages.java.io;
-importPackage(java.io);
+var File = io.File;
-
 var store = [];
-find(new File(dir),store,/\/[a-zA-Z0-9_\-]+\.js$/);
+find(new io.File(dir),store,/\/[a-zA-Z0-9_\-]+\.js$/);
 
 store.sort(sorter([ 
     /lib\/scriptcraft\.js$/, 
@@ -127,7 +126,7 @@ store.sort(sorter([
 
 var contents = [];
 foreach(store, function(filename){
-  var br = new BufferedReader(new FileReader(filename));
+  var br = new io.BufferedReader(new io.FileReader(filename));
   var line ;
   while ( (line = br.readLine()) != null){
     contents.push(line);

src/docs/js/generateEventsHelper.js

@@ -33,12 +33,12 @@ for (var i = 0; i< content.length; i++){
   out.println(content[i]);
 }
 while ( ( entry = zis.nextEntry) != null) { 
-  var name = '' + entry.name;
+  var name = new String( entry.name );
   var re1 = /org\/bukkit\/event\/.+Event\.class$/;
   if (args[0] == 'canary'){
     re1 = /net\/canarymod\/hook\/.+Hook\.class$/;
   }
-  if (name.match(re1)){
+  if ( re1.test(name) ) {
     name = name.replace(/\//g,'.').replace('.class','');
     try { 
       clz = java.lang.Class.forName(name);

src/docs/js/generateItemsDoc.js

@@ -20,13 +20,12 @@ var content = [
 ];
 
 //var ItemType = java.lang.Class.forName('net.canarymod.api.inventory.ItemType');
-var ItemType = Packages.net.canarymod.api.inventory.ItemType;
+var materials = itemTypeClass.getDeclaredFields();
-var materials = ItemType.class.getDeclaredFields();
 
 var enumVals = [];
 for (var i = 0;i < materials.length; i++ ){
 
-  if (materials[i].type != ItemType.class) {
+  if (materials[i].type != itemTypeClass) {
     continue;
   }
   var materialField = materials[i];

src/main/js/modules/canary/items.js

@@ -17,7 +17,10 @@ for (var i = 0;i < materials.length; i++ ){
     continue;
   }
   var materialField = materials[i];
-  var name = (''+materialField.name).replace(/^(.)/,function(a){ return a.toLowerCase() });
+  var name = (''+materialField.name);
+  name = name.replace(/^(.)/,function(a){ 
+    return a.toLowerCase(); 
+  });
   
   items[name] = (function(material){
     return function(amount){