Index: lams_tool_forum/build.properties
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_forum/build.properties,v
diff -u -r1.24.8.2 -r1.24.8.3
--- lams_tool_forum/build.properties	27 Oct 2010 15:18:44 -0000	1.24.8.2
+++ lams_tool_forum/build.properties	3 Mar 2011 19:43:35 -0000	1.24.8.3
@@ -9,7 +9,7 @@
 ### project properties ###
 signature=lafrum11
 project.displayname = lams forum tool
-tool.version=20101025
+tool.version=20110228
 
 # hide tool option
 hideTool=false
Index: lams_tool_forum/conf/hibernate/mappings/org/lamsfoundation/lams/tool/forum/persistence/Forum.hbm.xml
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_forum/conf/hibernate/mappings/org/lamsfoundation/lams/tool/forum/persistence/Attic/Forum.hbm.xml,v
diff -u -r1.19.10.1 -r1.19.10.2
--- lams_tool_forum/conf/hibernate/mappings/org/lamsfoundation/lams/tool/forum/persistence/Forum.hbm.xml	27 Oct 2010 15:18:44 -0000	1.19.10.1
+++ lams_tool_forum/conf/hibernate/mappings/org/lamsfoundation/lams/tool/forum/persistence/Forum.hbm.xml	3 Mar 2011 19:43:35 -0000	1.19.10.2
@@ -221,6 +221,14 @@
         />
 
         <property
+            name="allowRateMessages"
+            type="boolean"
+            update="true"
+            insert="true"
+            column="allow_rate_messages"
+        />
+
+        <property
             name="maximumReply"
             type="int"
             update="true"
Fisheye: Tag 1.1 refers to a dead (removed) revision in file `lams_tool_forum/conf/hibernate/mappings/org/lamsfoundation/lams/tool/forum/persistence/MessageRating.hbm.xml'.
Fisheye: No comparison available.  Pass `N' to diff?
Index: lams_tool_forum/conf/xdoclet/struts-actions.xml
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_forum/conf/xdoclet/struts-actions.xml,v
diff -u -r1.47.8.1 -r1.47.8.2
--- lams_tool_forum/conf/xdoclet/struts-actions.xml	1 Apr 2010 17:08:32 -0000	1.47.8.1
+++ lams_tool_forum/conf/xdoclet/struts-actions.xml	3 Mar 2011 19:43:34 -0000	1.47.8.2
@@ -275,6 +275,11 @@
 			scope="request">
 			<forward name="success" path="tiles:/learning/forum/viewTopic" />
 		</action>	
+		
+		<action path="/learning/rateMessage"
+			type="org.lamsfoundation.lams.tool.forum.web.actions.LearningAction"
+			parameter="rateMessage" scope="request">
+		</action>
 				
 		<action path="/learning/finish"
 			type="org.lamsfoundation.lams.tool.forum.web.actions.LearningAction"
Index: lams_tool_forum/db/sql/cleanup_lams_tool_forum.sql
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_forum/db/sql/cleanup_lams_tool_forum.sql,v
diff -u -r1.8 -r1.8.8.1
--- lams_tool_forum/db/sql/cleanup_lams_tool_forum.sql	2 Feb 2009 03:21:24 -0000	1.8
+++ lams_tool_forum/db/sql/cleanup_lams_tool_forum.sql	3 Mar 2011 19:43:34 -0000	1.8.8.1
@@ -5,5 +5,6 @@
 delete from  tl_lafrum11_forum_user;
 delete from  tl_lafrum11_message;
 delete from  tl_lafrum11_message_seq;
+delete from  tl_lafrum11_message_rating;
 delete from  tl_lafrum11_tool_session;
 delete from  tl_lafrum11_timestamp;
\ No newline at end of file
Index: lams_tool_forum/db/sql/create_lams_tool_forum.sql
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_forum/db/sql/create_lams_tool_forum.sql,v
diff -u -r1.33.8.2 -r1.33.8.3
--- lams_tool_forum/db/sql/create_lams_tool_forum.sql	27 Oct 2010 15:18:44 -0000	1.33.8.2
+++ lams_tool_forum/db/sql/create_lams_tool_forum.sql	3 Mar 2011 19:43:34 -0000	1.33.8.3
@@ -7,6 +7,7 @@
 drop table if exists tl_lafrum11_message;
 drop table if exists tl_lafrum11_message_seq;
 drop table if exists tl_lafrum11_report;
+drop table if exists tl_lafrum11_message_rating;
 drop table if exists tl_lafrum11_tool_session;
 drop table if exists tl_lafrum11_timestamp;
 create table tl_lafrum11_attachment (
@@ -38,7 +39,8 @@
    allow_edit smallint,
    allow_rich_editor smallint,
    allow_new_topic smallint, 
-   allow_upload smallint, 
+   allow_upload smallint,
+   allow_rate_messages smallint,
    maximum_reply integer, 
    minimum_reply integer,
    limited_of_chars integer,
@@ -95,6 +97,19 @@
    mark float,
    primary key (uid)
 )TYPE=InnoDB;
+CREATE TABLE tl_lafrum11_message_rating (
+       uid BIGINT(20) NOT NULL AUTO_INCREMENT
+     , rating float
+     , user_id BIGINT(20) NOT NULL
+     , message_id BIGINT(20) NOT NULL
+     , PRIMARY KEY (uid)
+     , INDEX (user_id)
+     , CONSTRAINT FK_tl_lafrum11_message_rating_1 FOREIGN KEY (user_id)
+                  REFERENCES tl_lafrum11_forum_user (uid)
+     , INDEX (message_id)
+     , CONSTRAINT FK_tl_lafrum11_message_rating_2 FOREIGN KEY (message_id)
+                  REFERENCES tl_lafrum11_message (uid)
+)TYPE=InnoDB;
 create table tl_lafrum11_tool_session (
    uid bigint not null auto_increment,
    version integer not null,
@@ -155,8 +170,8 @@
 alter table tl_lafrum11_timestamp add index MessageFK (message_uid), add constraint MessageFK foreign key (message_uid) references tl_lafrum11_message (uid);
 
 INSERT INTO tl_lafrum11_forum (uid,title,instructions,online_instructions,offline_instructions,content_id,allow_anonym,run_offline,lock_on_finished,content_in_use,define_later,allow_edit,allow_rich_editor,
- allow_new_topic,allow_upload,maximum_reply, minimum_reply,limited_input_flag,limited_of_chars,notify_learners_on_forum_posting,notify_teachers_on_forum_posting,reflect_on_activity) 
-VALUES(1,"Forum","Instructions",null,null,${default_content_id},0,0,0,0,0,1,0,1,0,1,0,1,5000,0,0,0);
+ allow_new_topic,allow_upload,allow_rate_messages,maximum_reply, minimum_reply,limited_input_flag,limited_of_chars,notify_learners_on_forum_posting,notify_teachers_on_forum_posting,reflect_on_activity) 
+VALUES(1,"Forum","Instructions",null,null,${default_content_id},0,0,0,0,0,1,0,1,0,0,1,0,1,5000,0,0,0);
 
 INSERT INTO `tl_lafrum11_message` (`uid`, `create_date`, `last_reply_date`, `update_date`, `create_by`, `modified_by`, `subject`, `body`, `sequence_id`, `is_authored`, `is_anonymous`, `forum_session_uid`, `parent_uid`, `forum_uid`, `reply_number`, `hide_flag`, `report_id`) VALUES 
   (1,NOW(),NOW(),NOW(),null,null,'Topic Heading','Topic message',1,1,0,NULL,NULL,1,0,0,NULL);
Index: lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/forumApplicationContext.xml
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/forumApplicationContext.xml,v
diff -u -r1.22.2.1.4.1 -r1.22.2.1.4.2
--- lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/forumApplicationContext.xml	27 Oct 2010 15:18:44 -0000	1.22.2.1.4.1
+++ lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/forumApplicationContext.xml	3 Mar 2011 19:41:21 -0000	1.22.2.1.4.2
@@ -53,6 +53,12 @@
 			<ref bean="coreSessionFactory" />
 		</property>
 	</bean>
+	<bean id="messageRatingDao"
+		class="org.lamsfoundation.lams.tool.forum.persistence.MessageRatingDAO">
+		<property name="sessionFactory">
+			<ref bean="coreSessionFactory" />
+		</property>
+	</bean>
 	<bean id="forumToolSessionDao"
 		class="org.lamsfoundation.lams.tool.forum.persistence.ForumToolSessionDao">
 		<property name="sessionFactory">
@@ -91,6 +97,9 @@
 		<property name="messageSeqDao">
 			<ref local="messageSeqDao" />
 		</property>
+		<property name="messageRatingDao">
+			<ref local="messageRatingDao" />
+		</property>	
 		<property name="timestampDao">
 			<ref local="timestampDao" />
 		</property>
@@ -160,6 +169,7 @@
 
 				<prop key="replyTopic">PROPAGATION_REQUIRED,-java.lang.Exception</prop>
 				<prop key="releaseMarksForSession">PROPAGATION_REQUIRED,-java.lang.Exception</prop>
+				<prop key="rateMessage">PROPAGATION_REQUIRED,-java.lang.Exception</prop>
 				<prop key="finishUserSession">PROPAGATION_REQUIRED,-java.lang.Exception</prop>
 				<!--  Tool Content Manager and Tool Session Manager calls -->
 				<prop key="copyToolContent">PROPAGATION_REQUIRED,-java.lang.Exception</prop>
Fisheye: Tag 1.1 refers to a dead (removed) revision in file `lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/dbupdates/patch20110228_updateTo236.sql'.
Fisheye: No comparison available.  Pass `N' to diff?
Fisheye: Tag 1.1 refers to a dead (removed) revision in file `lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/dto/AverageRatingDTO.java'.
Fisheye: No comparison available.  Pass `N' to diff?
Index: lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/dto/MessageDTO.java
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/dto/MessageDTO.java,v
diff -u -r1.18 -r1.18.8.1
--- lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/dto/MessageDTO.java	2 Feb 2009 03:21:24 -0000	1.18
+++ lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/dto/MessageDTO.java	3 Mar 2011 19:41:21 -0000	1.18.8.1
@@ -46,6 +46,10 @@
 	private String comment;
 	private boolean released;
 	
+	//rating fields
+	private String averageRating;
+	private String numberOfVotes;
+	
 	// number of posts the learner has made in this topic.
 	// used when this message is a root topic.
 	private int numOfPosts;   
@@ -181,6 +185,22 @@
 	public void setReleased(boolean isReleased) {
 		this.released = isReleased;
 	}
+	
+	public String getAverageRating() {
+	    return averageRating;
+	}
+	
+	public void setAverageRating(String averageRating) {
+	    this.averageRating = averageRating;
+	}
+	
+	public String getNumberOfVotes() {
+	    return numberOfVotes;
+	}
+	
+	public void setNumberOfVotes(String numberOfVotes) {
+	    this.numberOfVotes = numberOfVotes;
+	}
 
 	public String getAttachmentLocalUrl() {
 		return attachmentLocalUrl;
Index: lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/persistence/Forum.hbm.xml
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/persistence/Attic/Forum.hbm.xml,v
diff -u -r1.19.10.1 -r1.19.10.2
--- lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/persistence/Forum.hbm.xml	27 Oct 2010 15:18:43 -0000	1.19.10.1
+++ lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/persistence/Forum.hbm.xml	3 Mar 2011 19:41:21 -0000	1.19.10.2
@@ -221,6 +221,14 @@
         />
 
         <property
+            name="allowRateMessages"
+            type="boolean"
+            update="true"
+            insert="true"
+            column="allow_rate_messages"
+        />
+
+        <property
             name="maximumReply"
             type="int"
             update="true"
Index: lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/persistence/Forum.java
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/persistence/Forum.java,v
diff -u -r1.31.10.1 -r1.31.10.2
--- lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/persistence/Forum.java	27 Oct 2010 15:18:43 -0000	1.31.10.1
+++ lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/persistence/Forum.java	3 Mar 2011 19:41:21 -0000	1.31.10.2
@@ -67,6 +67,8 @@
     private boolean allowNewTopic;
 
     private boolean allowUpload;
+    
+    private boolean allowRateMessages;
 
     private int maximumReply;
 
@@ -549,7 +551,20 @@
     public void setAllowUpload(boolean allowUpload) {
 	this.allowUpload = allowUpload;
     }
+    
+    /**
+     * @hibernate.property column="allow_rate_messages"
+     * @return
+     */
+    public boolean isAllowRateMessages() {
+	return allowRateMessages;
+    }
 
+    public void setAllowRateMessages(boolean allowRateMessages) {
+	this.allowRateMessages = allowRateMessages;
+    }
+    
+
     /**
      * @hibernate.property column="maximum_reply"
      * @return
Fisheye: Tag 1.1 refers to a dead (removed) revision in file `lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/persistence/MessageRating.hbm.xml'.
Fisheye: No comparison available.  Pass `N' to diff?
Fisheye: Tag 1.1 refers to a dead (removed) revision in file `lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/persistence/MessageRating.java'.
Fisheye: No comparison available.  Pass `N' to diff?
Fisheye: Tag 1.1 refers to a dead (removed) revision in file `lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/persistence/MessageRatingDAO.java'.
Fisheye: No comparison available.  Pass `N' to diff?
Index: lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/service/ForumService.java
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/service/ForumService.java,v
diff -u -r1.99.4.3.4.4 -r1.99.4.3.4.5
--- lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/service/ForumService.java	19 Nov 2010 19:56:12 -0000	1.99.4.3.4.4
+++ lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/service/ForumService.java	3 Mar 2011 19:41:21 -0000	1.99.4.3.4.5
@@ -78,6 +78,7 @@
 import org.lamsfoundation.lams.tool.exception.DataMissingException;
 import org.lamsfoundation.lams.tool.exception.SessionDataExistsException;
 import org.lamsfoundation.lams.tool.exception.ToolException;
+import org.lamsfoundation.lams.tool.forum.dto.AverageRatingDTO;
 import org.lamsfoundation.lams.tool.forum.dto.MessageDTO;
 import org.lamsfoundation.lams.tool.forum.persistence.Attachment;
 import org.lamsfoundation.lams.tool.forum.persistence.AttachmentDao;
@@ -93,6 +94,8 @@
 import org.lamsfoundation.lams.tool.forum.persistence.ForumUserDao;
 import org.lamsfoundation.lams.tool.forum.persistence.Message;
 import org.lamsfoundation.lams.tool.forum.persistence.MessageDao;
+import org.lamsfoundation.lams.tool.forum.persistence.MessageRating;
+import org.lamsfoundation.lams.tool.forum.persistence.MessageRatingDAO;
 import org.lamsfoundation.lams.tool.forum.persistence.MessageSeq;
 import org.lamsfoundation.lams.tool.forum.persistence.MessageSeqDao;
 import org.lamsfoundation.lams.tool.forum.persistence.PersistenceException;
@@ -132,6 +135,8 @@
     private TimestampDao timestampDao;
 
     private MessageSeqDao messageSeqDao;
+    
+    private MessageRatingDAO messageRatingDao;
 
     private ForumUserDao forumUserDao;
 
@@ -588,7 +593,29 @@
 	currentUser.setSessionFinished(true);
 	forumUserDao.save(currentUser);
     }
+    
+    public AverageRatingDTO rateMessage(Long messageId, Long userId, Long toolSessionID, float rating) {
+	ForumUser imageGalleryUser = getUserByUserAndSession(userId, toolSessionID);
+	MessageRating messageRating = messageRatingDao.getRatingByMessageAndUser(messageId, userId);
+	Message message = messageDao.getById(messageId);	
 
+	//persist MessageRating changes in DB
+	if (messageRating == null) { // add
+	    messageRating = new MessageRating();
+	    messageRating.setUser(imageGalleryUser);
+	    messageRating.setMessage(message);
+	}
+	messageRating.setRating(rating);
+	messageRatingDao.saveObject(messageRating);
+	
+	//to make available new changes be visible in jsp page
+	return messageRatingDao.getAverageRatingDTOByMessage(messageId);
+    }
+    
+    public AverageRatingDTO getAverageRatingDTOByMessage(Long messageId) {
+	return messageRatingDao.getAverageRatingDTOByMessage(messageId);
+    }
+
     // ***************************************************************************************************************
     // Private methods
     // ***************************************************************************************************************
@@ -599,13 +626,23 @@
     private List getSortedMessageDTO(SortedMap<MessageSeq, Message> map) {
 	Iterator iter;
 	MessageSeq msgSeq;
+	Message message;
 	List<MessageDTO> msgDtoList = new ArrayList<MessageDTO>();
 	iter = map.entrySet().iterator();
 	while (iter.hasNext()) {
 	    Map.Entry entry = (Entry) iter.next();
 	    msgSeq = (MessageSeq) entry.getKey();
-	    MessageDTO dto = MessageDTO.getMessageDTO((Message) entry.getValue());
+	    message = (Message) entry.getValue();
+	    MessageDTO dto = MessageDTO.getMessageDTO(message);
 	    dto.setLevel(msgSeq.getMessageLevel());
+	    
+	    //set averageRating 
+	    if (message.getForum().isAllowRateMessages()) {
+		AverageRatingDTO averageRating = getAverageRatingDTOByMessage(message.getUid());
+		dto.setAverageRating(averageRating.getRating());
+		dto.setNumberOfVotes(averageRating.getNumberOfVotes());
+	    }
+	    
 	    msgDtoList.add(dto);
 	}
 	return msgDtoList;
@@ -1038,6 +1075,7 @@
 	    toolContentObj.setAllowNewTopic(Boolean.TRUE);
 	    toolContentObj.setAllowRichEditor(Boolean.FALSE);
 	    toolContentObj.setAllowUpload(Boolean.TRUE); // this is the default value
+	    toolContentObj.setAllowRateMessages(Boolean.FALSE); // this is the default value
 	    toolContentObj.setContentId(toolContentId);
 	    toolContentObj.setContentInUse(Boolean.FALSE);
 	    toolContentObj.setCreated(now);
@@ -1230,7 +1268,15 @@
     public void setMessageSeqDao(MessageSeqDao messageSeqDao) {
 	this.messageSeqDao = messageSeqDao;
     }
+    
+    public MessageRatingDAO getMessageRatingDao() {
+	return messageRatingDao;
+    }
 
+    public void setMessageRatingDao(MessageRatingDAO messageRatingDao) {
+	this.messageRatingDao = messageRatingDao;
+    }
+
     public ForumToolSessionDao getForumToolSessionDao() {
 	return forumToolSessionDao;
     }
Index: lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/service/IForumService.java
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/service/IForumService.java,v
diff -u -r1.44.4.2.4.2 -r1.44.4.2.4.3
--- lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/service/IForumService.java	27 Oct 2010 15:18:44 -0000	1.44.4.2.4.2
+++ lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/service/IForumService.java	3 Mar 2011 19:41:21 -0000	1.44.4.2.4.3
@@ -31,6 +31,7 @@
 import org.apache.struts.upload.FormFile;
 import org.lamsfoundation.lams.events.IEventNotificationService;
 import org.lamsfoundation.lams.notebook.model.NotebookEntry;
+import org.lamsfoundation.lams.tool.forum.dto.AverageRatingDTO;
 import org.lamsfoundation.lams.tool.forum.dto.MessageDTO;
 import org.lamsfoundation.lams.tool.forum.persistence.Attachment;
 import org.lamsfoundation.lams.tool.forum.persistence.Forum;
@@ -433,4 +434,8 @@
     public void deleteCondition(ForumCondition condition) throws PersistenceException;
     
     void sendNotificationsOnNewPosting(Long forumId, Long sessionId, Message message);
+    
+    AverageRatingDTO rateMessage(Long messageId, Long userId, Long toolSessionID, float rating);
+    
+    AverageRatingDTO getAverageRatingDTOByMessage(Long responseId);
 }
Index: lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/util/ForumConstants.java
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/util/ForumConstants.java,v
diff -u -r1.52.4.1.2.1 -r1.52.4.1.2.2
--- lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/util/ForumConstants.java	19 Nov 2010 19:56:13 -0000	1.52.4.1.2.1
+++ lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/util/ForumConstants.java	3 Mar 2011 19:41:21 -0000	1.52.4.1.2.2
@@ -76,6 +76,8 @@
     public static final String ATTR_ALLOW_NEW_TOPICS = "allowNewTopics";
 
     public static final String ATTR_ALLOW_RICH_EDITOR = "allowRichEditor";
+    
+    public static final String ATTR_ALLOW_RATE_MESSAGES = "allowRateMessages";
 
     public static final String ATTR_LIMITED_CHARS = "limitedChars";
 
Index: lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/actions/LearningAction.java
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/actions/LearningAction.java,v
diff -u -r1.65.8.3 -r1.65.8.4
--- lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/actions/LearningAction.java	6 Feb 2011 11:15:51 -0000	1.65.8.3
+++ lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/actions/LearningAction.java	3 Mar 2011 19:41:21 -0000	1.65.8.4
@@ -49,13 +49,16 @@
 import org.apache.struts.action.ActionMessage;
 import org.apache.struts.action.ActionMessages;
 import org.hibernate.Session;
+import org.json.JSONException;
+import org.json.JSONObject;
 import org.lamsfoundation.lams.events.IEventNotificationService;
 import org.lamsfoundation.lams.notebook.model.NotebookEntry;
 import org.lamsfoundation.lams.notebook.service.CoreNotebookConstants;
 import org.lamsfoundation.lams.tool.ToolAccessMode;
 import org.lamsfoundation.lams.tool.ToolSessionManager;
 import org.lamsfoundation.lams.tool.exception.DataMissingException;
 import org.lamsfoundation.lams.tool.exception.ToolException;
+import org.lamsfoundation.lams.tool.forum.dto.AverageRatingDTO;
 import org.lamsfoundation.lams.tool.forum.dto.MessageDTO;
 import org.lamsfoundation.lams.tool.forum.persistence.Attachment;
 import org.lamsfoundation.lams.tool.forum.persistence.Forum;
@@ -129,6 +132,9 @@
 		if (param.equals("updateMessageHideFlag")) {
 			return updateMessageHideFlag(mapping, form, request, response);
 		}
+		if (param.equals("rateMessage")) {
+			return rateMessage(mapping, form, request, response);
+		}
 
 		//================ Reflection =======================
 		if (param.equals("newReflection")) {
@@ -211,6 +217,7 @@
 		sessionMap.put(ForumConstants.ATTR_USER_FINISHED, forumUser.isSessionFinished());
 		sessionMap.put(ForumConstants.ATTR_ALLOW_EDIT, forum.isAllowEdit());
 		sessionMap.put(ForumConstants.ATTR_ALLOW_UPLOAD, forum.isAllowUpload());
+		sessionMap.put(ForumConstants.ATTR_ALLOW_RATE_MESSAGES, forum.isAllowRateMessages());
 		sessionMap.put(ForumConstants.ATTR_ALLOW_NEW_TOPICS, forum.isAllowNewTopic());
 		sessionMap.put(ForumConstants.ATTR_ALLOW_RICH_EDITOR, allowRichEditor);
 		sessionMap.put(ForumConstants.ATTR_LIMITED_CHARS, new Integer(allowNumber));
@@ -895,7 +902,41 @@
 		
 		return mapping.findForward("success");
 	}
+	
+    /**
+     * Rates postings submitted by other learners.
+     * 
+     * @param mapping
+     * @param form
+     * @param request
+     * @param response
+     * @return
+     * @throws JSONException
+     * @throws IOException
+     * @throws ServletException
+     * @throws ToolException
+     */
+    public ActionForward rateMessage(ActionMapping mapping, ActionForm form, HttpServletRequest request,
+	    HttpServletResponse response) throws JSONException, IOException {
 
+	forumService = getForumManager();
+
+	float rating = Float.parseFloat((String) request.getParameter("rate"));
+	Long responseId = WebUtil.readLongParam(request, "idBox");
+	Long toolSessionID = WebUtil.readLongParam(request, AttributeNames.PARAM_TOOL_SESSION_ID);
+	UserDTO user = (UserDTO) SessionManager.getSession().getAttribute(AttributeNames.USER);
+	Long userId = new Long(user.getUserID().intValue());
+
+	AverageRatingDTO averageRatingDTO = forumService.rateMessage(responseId, userId, toolSessionID, rating);
+
+	JSONObject JSONObject = new JSONObject();
+	JSONObject.put("averageRating", averageRatingDTO.getRating());
+	JSONObject.put("numberOfVotes", averageRatingDTO.getNumberOfVotes());
+	response.setContentType("application/x-json");
+	response.getWriter().print(JSONObject);
+	return null;
+    }
+
 	// ==========================================================================================
 	// Utility methods
 	// ==========================================================================================
Index: lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/forms/ForumForm.java
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/forms/ForumForm.java,v
diff -u -r1.17 -r1.17.12.1
--- lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/forms/ForumForm.java	17 Sep 2006 06:22:30 -0000	1.17
+++ lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/forms/ForumForm.java	3 Mar 2011 19:41:21 -0000	1.17.12.1
@@ -118,6 +118,7 @@
     	forum.setLockWhenFinished(false);
     	forum.setAllowNewTopic(false);
     	forum.setAllowUpload(false);
+    	forum.setAllowRateMessages(false);
     	forum.setReflectOnActivity(false);
     }
     //-------------------------get/set methods----------------
Index: lams_tool_forum/web/WEB-INF/struts-config.xml
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_forum/web/WEB-INF/Attic/struts-config.xml,v
diff -u -r1.54.8.1 -r1.54.8.2
--- lams_tool_forum/web/WEB-INF/struts-config.xml	1 Apr 2010 17:08:32 -0000	1.54.8.1
+++ lams_tool_forum/web/WEB-INF/struts-config.xml	3 Mar 2011 19:43:34 -0000	1.54.8.2
@@ -317,6 +317,10 @@
 			scope="request">
 			<forward name="success" path="tiles:/learning/forum/viewTopic" />
 		</action>	
+		<action path="/learning/rateMessage"
+			type="org.lamsfoundation.lams.tool.forum.web.actions.LearningAction"
+			parameter="rateMessage" scope="request">
+		</action>
 		<action path="/learning/finish"
 			type="org.lamsfoundation.lams.tool.forum.web.actions.LearningAction"
 			input="tiles:/learning/forum/viewForum" 
Index: lams_tool_forum/web/WEB-INF/struts/tiles-defs.xml
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_forum/web/WEB-INF/struts/Attic/tiles-defs.xml,v
diff -u -r1.33 -r1.33.12.1
--- lams_tool_forum/web/WEB-INF/struts/tiles-defs.xml	17 Sep 2006 06:22:31 -0000	1.33
+++ lams_tool_forum/web/WEB-INF/struts/tiles-defs.xml	3 Mar 2011 19:43:34 -0000	1.33.12.1
@@ -71,10 +71,7 @@
     	<put name="body" value="/jsps/learning/definelater.jsp"/>
 	</definition>
 	
-	<definition name="tiles:/learning/forum/viewTopic" extends="tiles:learner:layout">
-    	<put name="pageTitleKey" value="title.message.view.topic"/>
-    	<put name="body" value="/jsps/learning/viewtopic.jsp"/>
-	</definition>
+	<definition name="tiles:/learning/forum/viewTopic" path="/jsps/learning/viewtopic.jsp" />
 	
     <definition name="tiles:/learning/forum/newTopic" extends="tiles:learner:layout">
     	<put name="pageTitleKey" value="title.message.add"/>
Index: lams_tool_forum/web/images/bg_jRatingInfos.png
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_forum/web/images/Attic/bg_jRatingInfos.png,v
diff -u -r1.1 -r1.1.2.1
Binary files differ
Index: lams_tool_forum/web/images/small.png
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_forum/web/images/Attic/small.png,v
diff -u -r1.1 -r1.1.2.1
Binary files differ
Index: lams_tool_forum/web/images/stars.png
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_forum/web/images/Attic/stars.png,v
diff -u -r1.1 -r1.1.2.1
Binary files differ
Fisheye: Tag 1.1 refers to a dead (removed) revision in file `lams_tool_forum/web/includes/css/jRating.jquery.css'.
Fisheye: No comparison available.  Pass `N' to diff?
Fisheye: Tag 1.1 refers to a dead (removed) revision in file `lams_tool_forum/web/includes/css/ratingStars.css'.
Fisheye: No comparison available.  Pass `N' to diff?
Fisheye: Tag 1.1 refers to a dead (removed) revision in file `lams_tool_forum/web/includes/javascript/jRating.jquery.js'.
Fisheye: No comparison available.  Pass `N' to diff?
Fisheye: Tag 1.1 refers to a dead (removed) revision in file `lams_tool_forum/web/includes/javascript/jquery.js'.
Fisheye: No comparison available.  Pass `N' to diff?
Index: lams_tool_forum/web/jsps/authoring/advance.jsp
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_forum/web/jsps/authoring/advance.jsp,v
diff -u -r1.24.10.2 -r1.24.10.3
--- lams_tool_forum/web/jsps/authoring/advance.jsp	27 Oct 2010 15:18:44 -0000	1.24.10.2
+++ lams_tool_forum/web/jsps/authoring/advance.jsp	3 Mar 2011 19:43:34 -0000	1.24.10.3
@@ -23,6 +23,15 @@
 </p>
 
 <p>
+	<html:checkbox property="forum.allowRateMessages" styleClass="noBorder"
+		styleId="allowRateMessages">
+	</html:checkbox>
+	<label for="allowRateMessages">
+		<fmt:message key="label.authoring.advance.allow.rate.postings" />
+	</label>
+</p>
+
+<p>
 	<html:checkbox property="forum.allowUpload" styleClass="noBorder"
 		styleId="allowUpload">
 	</html:checkbox>
Index: lams_tool_forum/web/jsps/learning/viewtopic.jsp
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_forum/web/jsps/learning/viewtopic.jsp,v
diff -u -r1.17.6.1.4.2 -r1.17.6.1.4.3
--- lams_tool_forum/web/jsps/learning/viewtopic.jsp	1 Apr 2010 17:08:32 -0000	1.17.6.1.4.2
+++ lams_tool_forum/web/jsps/learning/viewtopic.jsp	3 Mar 2011 19:43:35 -0000	1.17.6.1.4.3
@@ -1,92 +1,177 @@
+<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01//EN"
+        "http://www.w3.org/TR/html4/strict.dtd">
+
 <%@ include file="/common/taglibs.jsp"%>
+<c:set var="lams">
+	<lams:LAMSURL />
+</c:set>
+<c:set var="tool">
+	<lams:WebAppURL />
+</c:set>
+<c:set var="ctxPath" value="${pageContext.request.contextPath}" scope="request" />
 <c:set var="sessionMap" value="${sessionScope[sessionMapID]}" />
 
-<script type="text/javascript">
-	function refreshTopic(){
-		var reqIDVar = new Date();
-		location.href= "<html:rewrite page="/learning/viewTopic.do?sessionMapID=${sessionMapID}&topicID=${sessionMap.rootUid}&hideReflection=${sessionMap.hideReflection}&reqUid=" />"+reqIDVar.getTime();;
-	}
-</script>
+<lams:html>
+	<lams:head>
+		<title><fmt:message key="activity.title" /></title>
+		
+		<!-- ********************  CSS ********************** -->
+		<c:choose>
+			<c:when test="${not empty localLinkPath}">
+				<lams:css localLinkPath="${localLinkPath}" />
+			</c:when>
+			<c:otherwise>
+				<lams:css />
+			</c:otherwise>
+		</c:choose>
+		<link rel="stylesheet" href="<html:rewrite page='/includes/css/jRating.jquery.css'/>"  type="text/css" />
+		<link rel="stylesheet" href="<html:rewrite page='/includes/css/ratingStars.css'/>"  type="text/css" />
+		
+		<!-- ********************  javascript ********************** -->
+		<script type="text/javascript" src="${lams}includes/javascript/common.js"></script>
+		<script type="text/javascript" src="${lams}includes/javascript/tabcontroller.js"></script>
+		
+		<%@ include file="/common/fckeditorheader.jsp"%>
+		
+		<script type="text/javascript">
+			var pathToImageFolder = "<html:rewrite page='/images/'/>"; 
+			var removeItemAttachmentUrl = "<html:rewrite page="/learning/deleteAttachment.do" />";
+		</script>		
+		<script type="text/javascript" src="<html:rewrite page='/includes/javascript/jquery.js'/>"></script>
+		<script type="text/javascript" src="<html:rewrite page='/includes/javascript/jRating.jquery.js'/>"></script>
+		<script type="text/javascript" src="${tool}includes/javascript/message.js"></script>
+		<script type="text/javascript">
+			$(document).ready(function(){
+				$(".ratingStars").jRating({
+				    phpPath : "<c:url value='/learning/rateMessage.do'/>?toolSessionID=${sessionMap.toolSessionID}",
+				    rateMax : 5,
+				    decimalLength : 1,
+					onSuccess : function(data, messageId){
+					    $("#averageRating" + messageId).html(data.averageRating);
+					    $("#numberOfVotes" + messageId).html(data.numberOfVotes);
+					},
+					onError : function(){
+					    jError('Error : please retry');
+					}
+				});
+			    $(".ratingStarsDisabled").jRating({
+			    	rateMax : 5,
+			    	isDisabled : true
+				});
+			});
+	
+			function refreshTopic(){
+				var reqIDVar = new Date();
+				location.href= "<html:rewrite page="/learning/viewTopic.do?sessionMapID=${sessionMapID}&topicID=${sessionMap.rootUid}&hideReflection=${sessionMap.hideReflection}&reqUid=" />"+reqIDVar.getTime();;
+			}
+		
+			function removeAtt(mapID){
+				removeItemAttachmentUrl =  removeItemAttachmentUrl + "?sessionMapID="+ mapID;
+				removeItemAttachment();
+			}
+			
+			function closeAndRefreshParentMonitoringWindow() {
+				refreshParentMonitoringWindow();
+				window.close();
+			} 
+		</script>		
+		
+	</lams:head>
+	<body class="stripes">
+		<div id="content">
 
+			<h1>
+				${sessionMap.title}
+			</h1>
+		
+			<div>
+				<div class="right-buttons">
+					<c:set var="backToForum">
+						<html:rewrite
+							page="/learning/viewForum.do?mode=${sessionMap.mode}&sessionMapID=${sessionMapID}&toolSessionID=${sessionMap.toolSessionID}&hideReflection=${sessionMap.hideReflection}" />
+					</c:set>
+					<html:button property="backToForum"
+						onclick="javascript:location.href='${backToForum}';"
+						styleClass="button">
+						<fmt:message key="label.back.to.forum" />
+					</html:button>
+				</div>
+				<h2>
+					<fmt:message key="title.message.view.topic" />
+				</h2>
+		
+			</div>
+		
+			<c:if test="${sessionMap.mode == 'author' || sessionMap.mode == 'learner'}">
+				<c:choose>
+				  <c:when test="${not sessionMap.allowNewTopics and (sessionMap.minimumReply ne 0 and sessionMap.maximumReply ne 0)}">
+					<div class="info">
+						<fmt:message key="label.postingLimits.topic.reminder">
+							<fmt:param value="${sessionMap.minimumReply}" />
+							<fmt:param value="${sessionMap.maximumReply}" />
+							<fmt:param value="${numOfPosts}" />
+							<fmt:param value="${sessionMap.maximumReply - numOfPosts}" />
+						</fmt:message>
+					</div>
+		                  </c:when> 
+		                  <c:when test="${not sessionMap.allowNewTopics and (sessionMap.minimumReply ne 0 or sessionMap.maximumReply eq 0)}">
+		                        <div class="info">
+		                                <fmt:message key="label.postingLimits.topic.reminder.min">
+		                                        <fmt:param value="${sessionMap.minimumReply}" />
+		                                        <fmt:param value="${numOfPosts}" />
+		                                        <fmt:param value="${sessionMap.maximumReply - numOfPosts}" />
+		                                </fmt:message>
+		                        </div>
+		                  </c:when> 
+		                  <c:when test="${not sessionMap.allowNewTopics and (sessionMap.minimumReply eq 0 or sessionMap.maximumReply ne 0)}">
+		                        <div class="info">
+		                                <fmt:message key="label.postingLimits.topic.reminder.max">
+		                                        <fmt:param value="${sessionMap.maximumReply}" />
+		                                        <fmt:param value="${numOfPosts}" />
+		                                        <fmt:param value="${sessionMap.maximumReply - numOfPosts}" />
+		                                </fmt:message>
+		                        </div>
+		                  </c:when>
+		                </c:choose>
+			</c:if>
+			<br>
+			
+			<%@ include file="message/topicview.jsp"%>
+			<c:set var="refreshTopicURL">
+			</c:set>
+		
+			<div class="space-bottom-top">
+		
+				<div class="right-buttons">
+					<c:set var="backToForum">
+						<html:rewrite
+							page="/learning/viewForum.do?mode=${sessionMap.mode}&sessionMapID=${sessionMapID}&toolSessionID=${sessionMap.toolSessionID}&hideReflection=${sessionMap.hideReflection}" />
+					</c:set>
+					<html:button property="backToForum"
+						onclick="javascript:location.href='${backToForum}';"
+						styleClass="button">
+						<fmt:message key="label.back.to.forum" />
+					</html:button>
+				</div>
+		
+				<a href="javascript:refreshTopic();" class="button"> <fmt:message
+						key="label.refresh" /> </a>
+		
+			</div>
+		</div>
 
-<div id="content">
+	</body>
+</lams:html>
 
-	<h1>
-		${sessionMap.title}
-	</h1>
 
-	<div>
-		<div class="right-buttons">
-			<c:set var="backToForum">
-				<html:rewrite
-					page="/learning/viewForum.do?mode=${sessionMap.mode}&sessionMapID=${sessionMapID}&toolSessionID=${sessionMap.toolSessionID}&hideReflection=${sessionMap.hideReflection}" />
-			</c:set>
-			<html:button property="backToForum"
-				onclick="javascript:location.href='${backToForum}';"
-				styleClass="button">
-				<fmt:message key="label.back.to.forum" />
-			</html:button>
-		</div>
-		<h2>
-			<fmt:message key="title.message.view.topic" />
-		</h2>
 
-	</div>
 
-	<c:if test="${sessionMap.mode == 'author' || sessionMap.mode == 'learner'}">
-		<c:choose>
-		  <c:when test="${not sessionMap.allowNewTopics and (sessionMap.minimumReply ne 0 and sessionMap.maximumReply ne 0)}">
-			<div class="info">
-				<fmt:message key="label.postingLimits.topic.reminder">
-					<fmt:param value="${sessionMap.minimumReply}" />
-					<fmt:param value="${sessionMap.maximumReply}" />
-					<fmt:param value="${numOfPosts}" />
-					<fmt:param value="${sessionMap.maximumReply - numOfPosts}" />
-				</fmt:message>
-			</div>
-                  </c:when> 
-                  <c:when test="${not sessionMap.allowNewTopics and (sessionMap.minimumReply ne 0 or sessionMap.maximumReply eq 0)}">
-                        <div class="info">
-                                <fmt:message key="label.postingLimits.topic.reminder.min">
-                                        <fmt:param value="${sessionMap.minimumReply}" />
-                                        <fmt:param value="${numOfPosts}" />
-                                        <fmt:param value="${sessionMap.maximumReply - numOfPosts}" />
-                                </fmt:message>
-                        </div>
-                  </c:when> 
-                  <c:when test="${not sessionMap.allowNewTopics and (sessionMap.minimumReply eq 0 or sessionMap.maximumReply ne 0)}">
-                        <div class="info">
-                                <fmt:message key="label.postingLimits.topic.reminder.max">
-                                        <fmt:param value="${sessionMap.maximumReply}" />
-                                        <fmt:param value="${numOfPosts}" />
-                                        <fmt:param value="${sessionMap.maximumReply - numOfPosts}" />
-                                </fmt:message>
-                        </div>
-                  </c:when>
-                </c:choose>
-	</c:if>
-	<br>
-	
-	<%@ include file="message/topicview.jsp"%>
-	<c:set var="refreshTopicURL">
-	</c:set>
 
-	<div class="space-bottom-top">
 
-		<div class="right-buttons">
-			<c:set var="backToForum">
-				<html:rewrite
-					page="/learning/viewForum.do?mode=${sessionMap.mode}&sessionMapID=${sessionMapID}&toolSessionID=${sessionMap.toolSessionID}&hideReflection=${sessionMap.hideReflection}" />
-			</c:set>
-			<html:button property="backToForum"
-				onclick="javascript:location.href='${backToForum}';"
-				styleClass="button">
-				<fmt:message key="label.back.to.forum" />
-			</html:button>
-		</div>
 
-		<a href="javascript:refreshTopic();" class="button"> <fmt:message
-				key="label.refresh" /> </a>
 
-	</div>
-</div>
+
+
+
+
+
Index: lams_tool_forum/web/jsps/learning/message/topicview.jsp
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_forum/web/jsps/learning/message/topicview.jsp,v
diff -u -r1.32 -r1.32.8.1
--- lams_tool_forum/web/jsps/learning/message/topicview.jsp	16 Dec 2008 04:09:33 -0000	1.32
+++ lams_tool_forum/web/jsps/learning/message/topicview.jsp	3 Mar 2011 19:43:34 -0000	1.32.8.1
@@ -96,16 +96,26 @@
 			<tr>
 				<td>
 					<div class="right-buttons">
+						<!--  Rating stars -->
+						<c:choose>
+							<c:when test='${(sessionMap.mode == "teacher") || msgDto.isAuthor || sessionMap.finishedLock}'>
+								<%@ include file="/jsps/learning/rating/ratingStarsDisabled.jsp"%>
+							</c:when>
+							<c:otherwise>
+								<%@ include file="/jsps/learning/rating/ratingStars.jsp"%>
+							</c:otherwise>
+						</c:choose>
+					
 						<!--  Hide/Unhide Button -->
 						<c:if test='${sessionMap.mode == "teacher"}'>
 							<c:set var="updateMark">
 								<html:rewrite
 									page="/monitoring/editMark.do?sessionMapID=${sessionMapID}&topicID=${msgDto.message.uid}&updateMode=viewForum&hideReflection=${sessionMap.hideReflection}" />
 							</c:set>
-						
 							<html:link href="${updateMark}" styleClass="button">
 								<fmt:message key="lable.topic.title.mark" />
 							</html:link>
+							
 							<!--  call the hide action -->
 							<c:choose>
 								<c:when test="${hidden}">
@@ -139,7 +149,7 @@
 									<html:rewrite
 										page="/learning/editTopic.do?sessionMapID=${sessionMapID}&topicID=${msgDto.message.uid}&rootUid=${sessinoMap.rootUid}&create=${msgDto.message.created.time}&hideReflection=${sessionMap.hideReflection}" />
 								</c:set>
-								<html:link href="${edittopic}" styleClass="button">
+								<html:link href="${edittopic}" styleClass="button" style="margin-top:10px;">
 									<fmt:message key="label.edit" />
 								</html:link>
 							</c:if>
Fisheye: Tag 1.1 refers to a dead (removed) revision in file `lams_tool_forum/web/jsps/learning/rating/ratingStars.jsp'.
Fisheye: No comparison available.  Pass `N' to diff?
Fisheye: Tag 1.1 refers to a dead (removed) revision in file `lams_tool_forum/web/jsps/learning/rating/ratingStarsDisabled.jsp'.
Fisheye: No comparison available.  Pass `N' to diff?
Index: lams_tool_forum/web/jsps/monitoring/summary.jsp
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_forum/web/jsps/monitoring/summary.jsp,v
diff -u -r1.15.10.1 -r1.15.10.2
--- lams_tool_forum/web/jsps/monitoring/summary.jsp	28 Oct 2010 07:49:28 -0000	1.15.10.1
+++ lams_tool_forum/web/jsps/monitoring/summary.jsp	3 Mar 2011 19:43:35 -0000	1.15.10.2
@@ -97,6 +97,23 @@
 	
 	<tr>
 		<td>
+			<fmt:message key="label.authoring.advance.allow.rate.postings" />
+		</td>
+		
+		<td>
+			<c:choose>
+				<c:when test="${forum.allowRateMessages}">
+					<fmt:message key="label.on" />
+				</c:when>
+				<c:otherwise>
+					<fmt:message key="label.off" />
+				</c:otherwise>
+			</c:choose>	
+		</td>
+	</tr>
+	
+	<tr>
+		<td>
 			<fmt:message key="label.authoring.advance.allow.upload" />
 		</td>