Index: lams_tool_larsrc/.classpath
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_larsrc/.classpath,v
diff -u -r1.7 -r1.8
--- lams_tool_larsrc/.classpath	1 Jul 2009 14:16:53 -0000	1.7
+++ lams_tool_larsrc/.classpath	18 Aug 2009 04:29:08 -0000	1.8
@@ -25,5 +25,5 @@
 	<classpathentry combineaccessrules="false" kind="src" path="/lams_central"/>
 	<classpathentry combineaccessrules="false" kind="src" path="/lams_learning"/>
 	<classpathentry kind="var" path="JBOSS_HOME/common/lib/log4j.jar"/>
-	<classpathentry kind="output" path="web/WEB-INF/classes"/>
+	<classpathentry kind="output" path="build/classes"/>
 </classpath>
Index: lams_tool_larsrc/build.properties
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_larsrc/build.properties,v
diff -u -r1.14 -r1.15
--- lams_tool_larsrc/build.properties	20 Jan 2009 16:18:54 -0000	1.14
+++ lams_tool_larsrc/build.properties	18 Aug 2009 04:29:08 -0000	1.15
@@ -13,7 +13,7 @@
 weblib=lib
 
 #project version
-tool.version=20090115
+tool.version=20090818
 
 # hide tool option
 hideTool=false
Index: lams_tool_larsrc/build.xml
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_larsrc/build.xml,v
diff -u -r1.25 -r1.26
--- lams_tool_larsrc/build.xml	6 Apr 2009 17:07:39 -0000	1.25
+++ lams_tool_larsrc/build.xml	18 Aug 2009 04:29:08 -0000	1.26
@@ -503,12 +503,6 @@
         		<include name="**/*.*"/>
         	</fileset>
         </copy>
-        <copy verbose="yes" todir="${jboss.deploy}/${product}.war/WEB-INF/classes">
-	    	<fileset dir="${build.classes.java}">
-	    		<include name="**/*.properties"/>
-	    		<include name="**/*.xml"/>
-	    	</fileset>
-        </copy>
     </target>
 
 	<!-- end tasks **************************************************************************************************-->
Index: lams_tool_larsrc/conf/xdoclet/struts-actions.xml
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_larsrc/conf/xdoclet/struts-actions.xml,v
diff -u -r1.33 -r1.34
--- lams_tool_larsrc/conf/xdoclet/struts-actions.xml	13 Feb 2009 05:56:15 -0000	1.33
+++ lams_tool_larsrc/conf/xdoclet/struts-actions.xml	18 Aug 2009 04:29:08 -0000	1.34
@@ -155,6 +155,15 @@
 			scope="request">
 			<forward name="success" path="/pages/authoring/pedagogicalPlannerForm.jsp" />
 		</action>
+		<action path="/authoring/switchResourceItemPosition"
+			type="org.lamsfoundation.lams.tool.rsrc.web.action.AuthoringAction"
+			name="resourceItemForm" 
+			validate="false"
+			parameter="switchResourceItemPosition"
+			scope="request">
+			<forward name="success" path="/pages/authoring/parts/itemlist.jsp" />
+		</action>
+		
 		<!-- ==========  Display Item  ========== -->
 		<action path="/reviewItem"
 			type="org.lamsfoundation.lams.tool.rsrc.web.action.ViewItemAction"
Index: lams_tool_larsrc/db/sql/create_lams_tool_rsrc.sql
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_larsrc/db/sql/create_lams_tool_rsrc.sql,v
diff -u -r1.21 -r1.22
--- lams_tool_larsrc/db/sql/create_lams_tool_rsrc.sql	7 Jul 2009 00:39:35 -0000	1.21
+++ lams_tool_larsrc/db/sql/create_lams_tool_rsrc.sql	18 Aug 2009 04:29:08 -0000	1.22
@@ -66,6 +66,7 @@
    open_url_new_window tinyint,
    resource_uid bigint,
    session_uid bigint,
+   order_id integer,
    primary key (uid)
 )type=innodb;
 create table tl_larsrc11_item_log (
@@ -117,8 +118,8 @@
  `allow_add_urls`, `mini_view_resource_number`, `allow_auto_run`,`reflect_on_activity`) VALUES
   (1,NULL,NULL,NULL,'Resources','0','0','Instructions ',null,null,0,0,${default_content_id},0,0,0,0,0);
   
-INSERT INTO `tl_larsrc11_resource_item` (`uid`, `file_uuid`, `file_version_id`, `description`, `ims_schema`, `init_item`, `organization_xml`, `title`, `url`, `create_by`, `create_date`, `create_by_author`, `is_hide`, `item_type`, `file_type`, `file_name`, `open_url_new_window`, `resource_uid`, `session_uid`) VALUES 
-  (1,NULL,NULL,NULL,NULL,NULL,NULL,'Web Search','http://www.google.com ',null,NOW(),1,0,1,NULL,NULL,0,1,NULL);
+INSERT INTO `tl_larsrc11_resource_item` (`uid`, `file_uuid`, `file_version_id`, `description`, `ims_schema`, `init_item`, `organization_xml`, `title`, `url`, `create_by`, `create_date`, `create_by_author`, `is_hide`, `item_type`, `file_type`, `file_name`, `open_url_new_window`, `resource_uid`, `session_uid`, `order_id`) VALUES 
+  (1,NULL,NULL,NULL,NULL,NULL,NULL,'Web Search','http://www.google.com ',null,NOW(),1,0,1,NULL,NULL,0,1,NULL, 1);
 INSERT INTO `tl_larsrc11_item_instruction` (`uid`, `description`, `sequence_id`, `item_uid`) VALUES 
   (1,'Use Google to search the web',0,1);
     
Index: lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/dbupdates/patch20090818_updateFrom231.sql
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/dbupdates/patch20090818_updateFrom231.sql,v
diff -u
--- /dev/null	1 Jan 1970 00:00:00 -0000
+++ lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/dbupdates/patch20090818_updateFrom231.sql	18 Aug 2009 04:29:08 -0000	1.1
@@ -0,0 +1,15 @@
+-- SQL statements to update from LAMS 2.3.1
+
+-- Turn off autocommit, so nothing is committed if there is an error
+SET AUTOCOMMIT = 0;
+
+----------------------Put all sql statements below here-------------------------
+
+-- LDEV-2421 Adding sorting for resource items
+ALTER TABLE tl_larsrc11_resource_item ADD COLUMN order_id integer DEFAULT NULL;
+
+----------------------Put all sql statements above here-------------------------
+
+-- If there were no errors, commit and restore autocommit to on
+COMMIT;
+SET AUTOCOMMIT = 1;
\ No newline at end of file
Index: lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/model/ResourceItem.hbm.xml
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/model/Attic/ResourceItem.hbm.xml,v
diff -u -r1.12 -r1.13
--- lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/model/ResourceItem.hbm.xml	24 Oct 2006 00:05:36 -0000	1.12
+++ lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/model/ResourceItem.hbm.xml	18 Aug 2009 04:29:08 -0000	1.13
@@ -178,6 +178,14 @@
             column="open_url_new_window"
         />
 
+        <property
+            name="orderId"
+            type="java.lang.Integer"
+            update="true"
+            insert="true"
+            column="order_id"
+        />
+
         <!--
             To add non XDoclet property mappings, create a file named
                 hibernate-properties-ResourceItem.xml
Index: lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/model/ResourceItem.java
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/model/ResourceItem.java,v
diff -u -r1.16 -r1.17
--- lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/model/ResourceItem.java	24 Oct 2006 00:05:36 -0000	1.16
+++ lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/model/ResourceItem.java	18 Aug 2009 04:29:08 -0000	1.17
@@ -33,282 +33,326 @@
 
 /**
  * Resource
+ * 
  * @author Dapeng Ni
- *
- * @hibernate.class  table="tl_larsrc11_resource_item"
- *
+ * 
+ * @hibernate.class table="tl_larsrc11_resource_item"
+ * 
  */
-public class ResourceItem  implements Cloneable{
-	private static final Logger log = Logger.getLogger(ResourceItem.class);
-	
-	private Long uid;
-	//Resource Type:1=URL,2=File,3=Website,4=Learning Object
-	private short type;
-	
-	private String title;
+public class ResourceItem implements Cloneable {
+    private static final Logger log = Logger.getLogger(ResourceItem.class);
 
-	private String description;
-	
-	private String url;
-	
-	private boolean openUrlNewWindow;
+    private Long uid;
+    // Resource Type:1=URL,2=File,3=Website,4=Learning Object
+    private short type;
 
-	private String imsSchema;
+    private String title;
 
-	private String initialItem;
+    private String description;
 
-	private String organizationXml;
+    private String url;
 
-	private Long fileUuid;
+    private boolean openUrlNewWindow;
 
-	private Long fileVersionId;
+    private String imsSchema;
 
-	private String fileName;
-	
-	private String fileType;
-	
-	private Set itemInstructions;
-	
-	private boolean isHide;
-	private boolean isCreateByAuthor;
-	
-	private Date createDate;
-	private ResourceUser createBy;
-	
-	//***********************************************
-	//DTO fields:
-	private boolean complete;
-	
-    public Object clone(){
-    	ResourceItem obj = null;
-		try {
-			obj = (ResourceItem) super.clone();
-//			clone attachment
-  			if(itemInstructions != null){
-  				Iterator iter = itemInstructions.iterator();
-  				Set set = new HashSet();
-  				while(iter.hasNext()){
-  					ResourceItemInstruction instruct = (ResourceItemInstruction)iter.next(); 
-  					ResourceItemInstruction newInsruct = (ResourceItemInstruction) instruct.clone();
-					set.add(newInsruct);
-  				}
-  				obj.itemInstructions = set;
-  			}
-			((ResourceItem)obj).setUid(null);
-  			//clone ReourceUser as well
-  			if(this.createBy != null)
-  				((ResourceItem)obj).setCreateBy((ResourceUser) this.createBy.clone());
-  			
-		} catch (CloneNotSupportedException e) {
-			log.error("When clone " + ResourceItem.class + " failed");
+    private String initialItem;
+
+    private String organizationXml;
+
+    private Long fileUuid;
+
+    private Long fileVersionId;
+
+    private String fileName;
+
+    private String fileType;
+
+    private Set itemInstructions;
+
+    private Integer orderId;
+
+    private boolean isHide;
+    private boolean isCreateByAuthor;
+
+    private Date createDate;
+    private ResourceUser createBy;
+
+    // ***********************************************
+    // DTO fields:
+    private boolean complete;
+
+    public Object clone() {
+	ResourceItem obj = null;
+	try {
+	    obj = (ResourceItem) super.clone();
+	    // clone attachment
+	    if (itemInstructions != null) {
+		Iterator iter = itemInstructions.iterator();
+		Set set = new HashSet();
+		while (iter.hasNext()) {
+		    ResourceItemInstruction instruct = (ResourceItemInstruction) iter.next();
+		    ResourceItemInstruction newInsruct = (ResourceItemInstruction) instruct.clone();
+		    set.add(newInsruct);
 		}
-		
-		return obj;
-	}	
-//    **********************************************************
-	  	//		Get/Set methods
-//	  **********************************************************
-		/**
-		 * @hibernate.id generator-class="native" type="java.lang.Long" column="uid"
-		 * @return Returns the uid.
-		 */
-		public Long getUid() {
-			return uid;
-		}
-		/**
-		 * @param uid The uid to set.
-		 */
-		public void setUid(Long userID) {
-			this.uid = userID;
-		}
-		/**
-		 * @hibernate.property column="file_uuid"
-		 * 
-		 * @return
-		 */
-		public Long getFileUuid() {
-			return fileUuid;
-		}
-		public void setFileUuid(Long crUuid) {
-			this.fileUuid = crUuid;
-		}
-		/**
-		 * @hibernate.property column="file_version_id"
-		 * @return
-		 */
-		public Long getFileVersionId() {
-			return fileVersionId;
-		}
-		public void setFileVersionId(Long crVersionId) {
-			this.fileVersionId = crVersionId;
-		}
-		/**
-		 * @hibernate.property column="description"
-		 * @return
-		 */
-		public String getDescription() {
-			return description;
-		}
-		public void setDescription(String description) {
-			this.description = description;
-		}
-		/**
-		 * @hibernate.property column="ims_schema"
-		 * @return
-		 */
-		public String getImsSchema() {
-			return imsSchema;
-		}
-		public void setImsSchema(String imsSchema) {
-			this.imsSchema = imsSchema;
-		}
-		/**
-		 * @hibernate.property column="init_item"
-		 * @return
-		 */
-		public String getInitialItem() {
-			return initialItem;
-		}
-		public void setInitialItem(String initialItem) {
-			this.initialItem = initialItem;
-		}
-		/**
-	     * @hibernate.set   lazy="false"
-	     * 					cascade="all-delete-orphan"
-	     * 					inverse="false"
-	     * 					order-by="sequence_id asc"
-	     * @hibernate.collection-key column="item_uid"
-	     * @hibernate.collection-one-to-many
-	     * 			class="org.lamsfoundation.lams.tool.rsrc.model.ResourceItemInstruction"
-	     * @return
-		 */
-		public Set getItemInstructions() {
-			return itemInstructions;
-		}
-		public void setItemInstructions(Set itemInstructions) {
-			this.itemInstructions = itemInstructions;
-		}
-		/**
-		 * @hibernate.property
-	     *	column="organization_xml"
-	     *  length="65535"
-		 * @return
-		 */
-		public String getOrganizationXml() {
-			return organizationXml;
-		}
-		public void setOrganizationXml(String organizationXml) {
-			this.organizationXml = organizationXml;
-		}
-		/**
-		 * @hibernate.property column="title" length="255"
-		 * @return
-		 */
-		public String getTitle() {
-			return title;
-		}
-		public void setTitle(String title) {
-			this.title = title;
-		}
-		/**
-		 * @hibernate.property column="url" length="65535"
-		 * @return
-		 */
-		public String getUrl() {
-			return url;
-		}
-		public void setUrl(String url) {
-			this.url = url;
-		}
-		/**
-	     * @hibernate.many-to-one
-	     *     cascade="none"
-	     * 		column="create_by"
-		 * 
-		 * @return
-		 */
-		public ResourceUser getCreateBy() {
-			return createBy;
-		}
-		public void setCreateBy(ResourceUser createBy) {
-			this.createBy = createBy;
-		}
-		/**
-		 * @hibernate.property column="create_date" 
-		 * @return
-		 */
-		public Date getCreateDate() {
-			return createDate;
-		}
-		public void setCreateDate(Date createDate) {
-			this.createDate = createDate;
-		}
-		/**
-		 * @hibernate.property column="create_by_author" 
-		 * @return
-		 */
-		public boolean isCreateByAuthor() {
-			return isCreateByAuthor;
-		}
-		public void setCreateByAuthor(boolean isCreateByAuthor) {
-			this.isCreateByAuthor = isCreateByAuthor;
-		}
-		/**
-		 * @hibernate.property column="is_hide" 
-		 * @return
-		 */
-		public boolean isHide() {
-			return isHide;
-		}
-		public void setHide(boolean isHide) {
-			this.isHide = isHide;
-		}
-		/**
-		 * @hibernate.property column="item_type" 
-		 * @return
-		 */
-		public short getType() {
-			return type;
-		}
-		public void setType(short type) {
-			this.type = type;
-		}
-		  /**
-	     * @hibernate.property column="file_type"
-	     */
-	    public String getFileType() {
-	        return fileType;
+		obj.itemInstructions = set;
 	    }
+	    ((ResourceItem) obj).setUid(null);
+	    // clone ReourceUser as well
+	    if (this.createBy != null)
+		((ResourceItem) obj).setCreateBy((ResourceUser) this.createBy.clone());
 
-	    public void setFileType(String type) {
-	        this.fileType = type;
-	    }
+	} catch (CloneNotSupportedException e) {
+	    log.error("When clone " + ResourceItem.class + " failed");
+	}
 
-	    /**
-	     * @hibernate.property column="file_name"
-	     */
-	    public String getFileName() {
-	        return fileName;
-	    }
+	return obj;
+    }
 
-	    public void setFileName(String name) {
-	        this.fileName = name;
-	    }
-	    /**
-	     * @hibernate.property  column="open_url_new_window" 
-	     * @return
-	     */
-	    public boolean isOpenUrlNewWindow() {
-	    	return openUrlNewWindow;
-	    }
-	    public void setOpenUrlNewWindow(boolean openUrlNewWindow) {
-	    	this.openUrlNewWindow = openUrlNewWindow;
-	    }
-	    
-		public void setComplete(boolean complete) {
-			this.complete=complete;
-		}
-		public boolean isComplete() {
-			return complete;
-		}
+    // **********************************************************
+    // Get/Set methods
+    // **********************************************************
+    /**
+     * @hibernate.id generator-class="native" type="java.lang.Long" column="uid"
+     * @return Returns the uid.
+     */
+    public Long getUid() {
+	return uid;
+    }
+
+    /**
+     * @param uid
+     *            The uid to set.
+     */
+    public void setUid(Long userID) {
+	this.uid = userID;
+    }
+
+    /**
+     * @hibernate.property column="file_uuid"
+     * 
+     * @return
+     */
+    public Long getFileUuid() {
+	return fileUuid;
+    }
+
+    public void setFileUuid(Long crUuid) {
+	this.fileUuid = crUuid;
+    }
+
+    /**
+     * @hibernate.property column="file_version_id"
+     * @return
+     */
+    public Long getFileVersionId() {
+	return fileVersionId;
+    }
+
+    public void setFileVersionId(Long crVersionId) {
+	this.fileVersionId = crVersionId;
+    }
+
+    /**
+     * @hibernate.property column="description"
+     * @return
+     */
+    public String getDescription() {
+	return description;
+    }
+
+    public void setDescription(String description) {
+	this.description = description;
+    }
+
+    /**
+     * @hibernate.property column="ims_schema"
+     * @return
+     */
+    public String getImsSchema() {
+	return imsSchema;
+    }
+
+    public void setImsSchema(String imsSchema) {
+	this.imsSchema = imsSchema;
+    }
+
+    /**
+     * @hibernate.property column="init_item"
+     * @return
+     */
+    public String getInitialItem() {
+	return initialItem;
+    }
+
+    public void setInitialItem(String initialItem) {
+	this.initialItem = initialItem;
+    }
+
+    /**
+     * @hibernate.set lazy="false" cascade="all-delete-orphan" inverse="false"
+     *                order-by="sequence_id asc"
+     * @hibernate.collection-key column="item_uid"
+     * @hibernate.collection-one-to-many 
+     *                                   class="org.lamsfoundation.lams.tool.rsrc.model.ResourceItemInstruction"
+     * @return
+     */
+    public Set getItemInstructions() {
+	return itemInstructions;
+    }
+
+    public void setItemInstructions(Set itemInstructions) {
+	this.itemInstructions = itemInstructions;
+    }
+
+    /**
+     * @hibernate.property column="organization_xml" length="65535"
+     * @return
+     */
+    public String getOrganizationXml() {
+	return organizationXml;
+    }
+
+    public void setOrganizationXml(String organizationXml) {
+	this.organizationXml = organizationXml;
+    }
+
+    /**
+     * @hibernate.property column="title" length="255"
+     * @return
+     */
+    public String getTitle() {
+	return title;
+    }
+
+    public void setTitle(String title) {
+	this.title = title;
+    }
+
+    /**
+     * @hibernate.property column="url" length="65535"
+     * @return
+     */
+    public String getUrl() {
+	return url;
+    }
+
+    public void setUrl(String url) {
+	this.url = url;
+    }
+
+    /**
+     * @hibernate.many-to-one cascade="none" column="create_by"
+     * 
+     * @return
+     */
+    public ResourceUser getCreateBy() {
+	return createBy;
+    }
+
+    public void setCreateBy(ResourceUser createBy) {
+	this.createBy = createBy;
+    }
+
+    /**
+     * @hibernate.property column="create_date"
+     * @return
+     */
+    public Date getCreateDate() {
+	return createDate;
+    }
+
+    public void setCreateDate(Date createDate) {
+	this.createDate = createDate;
+    }
+
+    /**
+     * @hibernate.property column="create_by_author"
+     * @return
+     */
+    public boolean isCreateByAuthor() {
+	return isCreateByAuthor;
+    }
+
+    public void setCreateByAuthor(boolean isCreateByAuthor) {
+	this.isCreateByAuthor = isCreateByAuthor;
+    }
+
+    /**
+     * @hibernate.property column="is_hide"
+     * @return
+     */
+    public boolean isHide() {
+	return isHide;
+    }
+
+    public void setHide(boolean isHide) {
+	this.isHide = isHide;
+    }
+
+    /**
+     * @hibernate.property column="item_type"
+     * @return
+     */
+    public short getType() {
+	return type;
+    }
+
+    public void setType(short type) {
+	this.type = type;
+    }
+
+    /**
+     * @hibernate.property column="file_type"
+     */
+    public String getFileType() {
+	return fileType;
+    }
+
+    public void setFileType(String type) {
+	this.fileType = type;
+    }
+
+    /**
+     * @hibernate.property column="file_name"
+     */
+    public String getFileName() {
+	return fileName;
+    }
+
+    public void setFileName(String name) {
+	this.fileName = name;
+    }
+
+    /**
+     * @hibernate.property column="open_url_new_window"
+     * @return
+     */
+    public boolean isOpenUrlNewWindow() {
+	return openUrlNewWindow;
+    }
+
+    public void setOpenUrlNewWindow(boolean openUrlNewWindow) {
+	this.openUrlNewWindow = openUrlNewWindow;
+    }
+    
+    /**
+     * @hibernate.property column="order_id"
+     * @return
+     */
+    public Integer getOrderId() {
+        return orderId;
+    }
+
+    public void setOrderId(Integer orderId) {
+        this.orderId = orderId;
+    }
+
+    public void setComplete(boolean complete) {
+	this.complete = complete;
+    }
+
+    public boolean isComplete() {
+	return complete;
+    }
 }
Index: lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/util/ResourceItemComparator.java
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/util/ResourceItemComparator.java,v
diff -u -r1.2 -r1.3
--- lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/util/ResourceItemComparator.java	18 Aug 2009 00:33:27 -0000	1.2
+++ lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/util/ResourceItemComparator.java	18 Aug 2009 04:29:08 -0000	1.3
@@ -12,12 +12,19 @@
 public class ResourceItemComparator implements Comparator<ResourceItem> {
 
     public int compare(ResourceItem o1, ResourceItem o2) {
-	if (o1 != null && o2 != null & o1.getCreateDate() != null && o2.getCreateDate() != null) {
+    
+	if (o1 != null && o2 != null & o1.getOrderId() != null && o2.getOrderId() != null){
+	    if (o1.getOrderId() > o2.getOrderId()) {
+		return 1;
+	    } else {
+		return -1;
+	    }
+	}
+	else if (o1 != null && o2 != null & o1.getCreateDate() != null && o2.getCreateDate() != null) {
 	    return (o1.getCreateDate().getTime() - o2.getCreateDate().getTime()) > 0 ? 1 : -1;
 	} else if (o1 != null)
 	    return 1;
 	else
 	    return -1;
     }
-
 }
Index: lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/web/action/AuthoringAction.java
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/web/action/AuthoringAction.java,v
diff -u -r1.54 -r1.55
--- lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/web/action/AuthoringAction.java	18 Aug 2009 00:30:35 -0000	1.54
+++ lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/web/action/AuthoringAction.java	18 Aug 2009 04:29:08 -0000	1.55
@@ -177,6 +177,9 @@
 	if (param.equals("saveOrUpdatePedagogicalPlannerForm")) {
 	    return saveOrUpdatePedagogicalPlannerForm(mapping, form, request, response);
 	}
+	if (param.equals("switchResourceItemPosition")) {
+	    return switchResourceItemPosition(mapping, form, request, response);
+	}
 
 	return mapping.findForward(ResourceConstants.ERROR);
     }
@@ -458,6 +461,16 @@
 	SortedSet<ResourceItem> resourceItemList = getResourceItemList(sessionMap);
 	resourceItemList.clear();
 	resourceItemList.addAll(items);
+	
+	
+	// If there is no order id, set it up
+	int i = 1;
+	for (ResourceItem resourceItem : resourceItemList) {
+	    if (resourceItem.getOrderId() == null || resourceItem.getOrderId() != i) {
+		resourceItem.setOrderId(i);
+	    }
+	    i++;
+	}
 
 	sessionMap.put(ResourceConstants.ATTR_RESOURCE_FORM, resourceForm);
 	return mapping.findForward(ResourceConstants.SUCCESS);
@@ -540,7 +553,7 @@
 		// get back UID
 		resourcePO.setUid(uid);
 	    } else { // if it is Teacher, then just update basic tab content
-		     // (definelater)
+		// (definelater)
 		resourcePO.setInstructions(resource.getInstructions());
 		resourcePO.setTitle(resource.getTitle());
 		// change define later status
@@ -1046,6 +1059,7 @@
 	if (itemIdx == -1) { // add
 	    item = new ResourceItem();
 	    item.setCreateDate(new Timestamp(new Date().getTime()));
+	    item.setOrderId(resourceList.size() + 1);
 	    resourceList.add(item);
 	} else { // edit
 	    List<ResourceItem> rList = new ArrayList<ResourceItem>(resourceList);
@@ -1349,4 +1363,35 @@
 	plannerForm.setFileVersion(insertIndex, null);
 	return mapping.findForward(ResourceConstants.SUCCESS);
     }
+
+    private ActionForward switchResourceItemPosition(ActionMapping mapping, ActionForm form, HttpServletRequest request, HttpServletResponse response) {
+
+	String sessionMapID = WebUtil.readStrParam(request, "sessionMapID");
+	int resourceItemOrderID1 = WebUtil.readIntParam(request, "resourceItemOrderID1");
+	int resourceItemOrderID2 = WebUtil.readIntParam(request, "resourceItemOrderID2");
+
+	SessionMap sessionMap = (SessionMap) request.getSession().getAttribute(sessionMapID);
+	// check whether it is "edit(old item)" or "add(new item)"
+	SortedSet<ResourceItem> resourceList = getResourceItemList(sessionMap);
+
+	for (ResourceItem item : resourceList) {
+	    if (item.getOrderId() == resourceItemOrderID1) {
+		item.setOrderId(resourceItemOrderID2);
+		continue;
+	    }
+	    if (item.getOrderId() == resourceItemOrderID2) {
+		item.setOrderId(resourceItemOrderID1);
+		continue;
+	    }
+	}
+
+	SortedSet<ResourceItem> newItems = new TreeSet<ResourceItem>(new ResourceItemComparator());
+	newItems.addAll(resourceList);
+	sessionMap.put(ResourceConstants.ATTR_RESOURCE_ITEM_LIST, newItems);
+	request.setAttribute(ResourceConstants.ATTR_SESSION_MAP_ID, sessionMapID);
+
+	// return null to close this window
+	return mapping.findForward(ResourceConstants.SUCCESS);
+    }
+
 }
\ No newline at end of file
Index: lams_tool_larsrc/web/pages/authoring/basic.jsp
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_larsrc/web/pages/authoring/basic.jsp,v
diff -u -r1.24 -r1.25
--- lams_tool_larsrc/web/pages/authoring/basic.jsp	3 Nov 2006 04:56:53 -0000	1.24
+++ lams_tool_larsrc/web/pages/authoring/basic.jsp	18 Aug 2009 04:29:08 -0000	1.25
@@ -72,6 +72,25 @@
 		hideBusy(resourceListTargetDiv);
 	}
 
+	function switchItem(resourceItemOrderID1, resourceItemOrderID2, sessionMapID) {
+		resourceItemOrderID1++;
+		resourceItemOrderID2++;
+		
+		var url = "<c:url value="/authoring/switchResourceItemPosition.do"/>";
+		var param = "sessionMapID=" + sessionMapID + "&resourceItemOrderID1=" + resourceItemOrderID1 + "&resourceItemOrderID2=" + resourceItemOrderID2;
+		deleteItemLoading();
+		var myAjax = new Ajax.Updater(
+	    	resourceListTargetDiv,
+	    	url,
+	    	{
+	    		method:'get',
+	    		parameters:param,
+	    		onComplete:deleteItemComplete,
+	    		evalScripts:true
+	    	}
+	    );
+	}
+
 </script>
 <!-- Basic Tab Content -->
 <table>
Index: lams_tool_larsrc/web/pages/authoring/parts/itemlist.jsp
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_larsrc/web/pages/authoring/parts/itemlist.jsp,v
diff -u -r1.15 -r1.16
--- lams_tool_larsrc/web/pages/authoring/parts/itemlist.jsp	9 Nov 2006 05:45:58 -0000	1.15
+++ lams_tool_larsrc/web/pages/authoring/parts/itemlist.jsp	18 Aug 2009 04:29:08 -0000	1.16
@@ -12,10 +12,10 @@
 		<img src="${ctxPath}/includes/images/indicator.gif"
 			style="display:none" id="resourceListArea_Busy" />
 	</h2>
-
+	
 	<table class="alternative-color" id="itemTable" cellspacing="0">
 		<c:set var="sessionMap" value="${sessionScope[sessionMapID]}" />
-
+		<c:set var="listSize" value="${fn:length(sessionMap.resourceList)}" />
 		<c:forEach var="resource" items="${sessionMap.resourceList}"
 			varStatus="status">
 			<tr>
@@ -31,6 +31,38 @@
 							onclick="previewItem(1,${status.index},'${sessionMapID}')"> <fmt:message
 								key="label.authoring.basic.resource.verify.url" /> </a>
 					</td>
+					<td width="10%">
+						<%-- Don't display down icon if last line --%>
+						<c:choose>
+							<c:when test="${0 != status.index}">
+								<img src="<html:rewrite page='/includes/images/uparrow.gif'/>"
+									border="0" title="<fmt:message key="label.up"/>"
+									onclick="switchItem(${status.index}, ${status.index - 1}, '${sessionMapID}')">
+							</c:when>
+							<c:otherwise>
+								<img
+									src="<html:rewrite page='/includes/images/uparrow_disabled.gif'/>"
+									border="0" title="<fmt:message key="label.up"/>">
+							</c:otherwise>
+						</c:choose>
+
+						<c:choose>
+							<c:when test="${listSize != status.count}">
+								<img src="<html:rewrite page='/includes/images/downarrow.gif'/>"
+									border="0" title="<fmt:message key="label.down"/>"
+									onclick="switchItem(${status.index}, ${status.index + 1}, '${sessionMapID}')">
+							</c:when>
+
+							<c:otherwise>
+								<img
+									src="<html:rewrite page='/includes/images/downarrow_disabled.gif'/>"
+									border="0" title="<fmt:message key="label.down"/>">
+							</c:otherwise>
+
+						</c:choose>
+
+						<%-- Don't display down icon if last line --%>
+					</td>
 					<td width="5%">
 						<img src="${tool}includes/images/edit.gif"
 							title="<fmt:message key="label.authoring.basic.resource.edit" />"
@@ -55,6 +87,38 @@
 							onclick="previewItem(2,${status.index},'${sessionMapID}')"> <fmt:message
 								key="label.authoring.basic.resource.preview" /> </a>
 					</td>
+					<td width="10%">
+						<%-- Don't display down icon if last line --%>
+						<c:choose>
+							<c:when test="${0 != status.index}">
+								<img src="<html:rewrite page='/includes/images/uparrow.gif'/>"
+									border="0" title="<fmt:message key="label.up"/>"
+									onclick="switchItem(${status.index}, ${status.index - 1}, '${sessionMapID}')">
+							</c:when>
+							<c:otherwise>
+								<img
+									src="<html:rewrite page='/includes/images/uparrow_disabled.gif'/>"
+									border="0" title="<fmt:message key="label.up"/>">
+							</c:otherwise>
+						</c:choose>
+
+						<c:choose>
+							<c:when test="${listSize != status.count}">
+								<img src="<html:rewrite page='/includes/images/downarrow.gif'/>"
+									border="0" title="<fmt:message key="label.down"/>"
+									onclick="switchItem(${status.index}, ${status.index + 1}, '${sessionMapID}')">
+							</c:when>
+
+							<c:otherwise>
+								<img
+									src="<html:rewrite page='/includes/images/downarrow_disabled.gif'/>"
+									border="0" title="<fmt:message key="label.down"/>">
+							</c:otherwise>
+
+						</c:choose>
+
+						<%-- Don't display down icon if last line --%>
+					</td>
 					<td>
 						<img src="${tool}includes/images/edit.gif"
 							title="<fmt:message key="label.authoring.basic.resource.edit" />"
@@ -79,6 +143,38 @@
 							onclick="previewItem(3,${status.index},'${sessionMapID}')"> <fmt:message
 								key="label.authoring.basic.resource.preview" /> </a>
 					</td>
+					<td width="10%">
+						<%-- Don't display down icon if last line --%>
+						<c:choose>
+							<c:when test="${0 != status.index}">
+								<img src="<html:rewrite page='/includes/images/uparrow.gif'/>"
+									border="0" title="<fmt:message key="label.up"/>"
+									onclick="switchItem(${status.index}, ${status.index - 1}, '${sessionMapID}')">
+							</c:when>
+							<c:otherwise>
+								<img
+									src="<html:rewrite page='/includes/images/uparrow_disabled.gif'/>"
+									border="0" title="<fmt:message key="label.up"/>">
+							</c:otherwise>
+						</c:choose>
+
+						<c:choose>
+							<c:when test="${listSize != status.count}">
+								<img src="<html:rewrite page='/includes/images/downarrow.gif'/>"
+									border="0" title="<fmt:message key="label.down"/>"
+									onclick="switchItem(${status.index}, ${status.index + 1}, '${sessionMapID}')">
+							</c:when>
+
+							<c:otherwise>
+								<img
+									src="<html:rewrite page='/includes/images/downarrow_disabled.gif'/>"
+									border="0" title="<fmt:message key="label.down"/>">
+							</c:otherwise>
+
+						</c:choose>
+
+						<%-- Don't display down icon if last line --%>
+					</td>
 					<td>
 						<img src="${tool}includes/images/edit.gif"
 							title="<fmt:message key="label.authoring.basic.resource.edit" />"
@@ -102,6 +198,38 @@
 							onclick="previewItem(4,${status.index},'${sessionMapID}')"> <fmt:message
 								key="label.authoring.basic.resource.preview" /> </a>
 					</td>
+					<td width="10%">
+						<%-- Don't display down icon if last line --%>
+						<c:choose>
+							<c:when test="${0 != status.index}">
+								<img src="<html:rewrite page='/includes/images/uparrow.gif'/>"
+									border="0" title="<fmt:message key="label.up"/>"
+									onclick="switchItem(${status.index}, ${status.index - 1}, '${sessionMapID}')">
+							</c:when>
+							<c:otherwise>
+								<img
+									src="<html:rewrite page='/includes/images/uparrow_disabled.gif'/>"
+									border="0" title="<fmt:message key="label.up"/>">
+							</c:otherwise>
+						</c:choose>
+
+						<c:choose>
+							<c:when test="${listSize != status.count}">
+								<img src="<html:rewrite page='/includes/images/downarrow.gif'/>"
+									border="0" title="<fmt:message key="label.down"/>"
+									onclick="switchItem(${status.index}, ${status.index + 1}, '${sessionMapID}')">
+							</c:when>
+
+							<c:otherwise>
+								<img
+									src="<html:rewrite page='/includes/images/downarrow_disabled.gif'/>"
+									border="0" title="<fmt:message key="label.down"/>">
+							</c:otherwise>
+
+						</c:choose>
+
+						<%-- Don't display down icon if last line --%>
+					</td>
 					<td>
 						<img src="${tool}includes/images/edit.gif"
 							title="<fmt:message key="label.authoring.basic.resource.edit" />"