Index: lams_build/lib/lams/lams.jar
===================================================================
RCS file: /usr/local/cvsroot/lams_build/lib/lams/lams.jar,v
diff -u -r1.420.2.72 -r1.420.2.73
Binary files differ
Index: lams_central/conf/language/lams/ApplicationResources.properties
===================================================================
RCS file: /usr/local/cvsroot/lams_central/conf/language/lams/ApplicationResources.properties,v
diff -u -r1.138.2.22 -r1.138.2.23
--- lams_central/conf/language/lams/ApplicationResources.properties	16 Mar 2016 12:14:10 -0000	1.138.2.22
+++ lams_central/conf/language/lams/ApplicationResources.properties	21 Apr 2016 09:13:16 -0000	1.138.2.23
@@ -677,4 +677,10 @@
 label.no.comments=No Comments
 label.newest.first=Newest First
 label.top.comments=Top Comments
+
+label.private.notifications.title =Notifications
+label.private.notifications.messages =Messages
+label.private.notifications.read =Read
+label.private.notifications.read.hint =Mark notification as read
+label.private.notifications.read.all.hint =Mark all notifications as read
 #======= End labels: Exported 439 labels for en AU =====
Fisheye: Tag 1.1 refers to a dead (removed) revision in file `lams_central/src/java/org/lamsfoundation/lams/web/NotificationAction.java'.
Fisheye: No comparison available.  Pass `N' to diff?
Index: lams_central/web/main.jsp
===================================================================
RCS file: /usr/local/cvsroot/lams_central/web/main.jsp,v
diff -u -r1.77.2.19 -r1.77.2.20
--- lams_central/web/main.jsp	16 Mar 2016 12:13:47 -0000	1.77.2.19
+++ lams_central/web/main.jsp	21 Apr 2016 09:13:16 -0000	1.77.2.20
@@ -68,6 +68,16 @@
 					NAVIGATE_AWAY_CONFIRM : decoderDiv.html('<c:out value="${NAVIGATE_AWAY_CONFIRM_VAR}" />').text(),
 					<fmt:message key="index.monitoring.title" var="MONITORING_TITLE_VAR"/>
 					MONITORING_TITLE : '<c:out value="${MONITORING_TITLE_VAR}" />',
+					<fmt:message key="label.private.notifications.title" var="PRIVATE_NOTIFICATIONS_TITLE_VAR"/>
+					PRIVATE_NOTIFICATIONS_TITLE : '<c:out value="${PRIVATE_NOTIFICATIONS_TITLE_VAR}" />',
+					<fmt:message key="label.private.notifications.messages" var="PRIVATE_NOTIFICATIONS_MESSAGES_VAR"/>
+					PRIVATE_NOTIFICATIONS_MESSAGES : '<c:out value="${PRIVATE_NOTIFICATIONS_MESSAGES_VAR}" />',
+					<fmt:message key="label.private.notifications.read" var="PRIVATE_NOTIFICATIONS_READ_VAR"/>
+					PRIVATE_NOTIFICATIONS_READ : '<c:out value="${PRIVATE_NOTIFICATIONS_READ_VAR}" />',
+					<fmt:message key="label.private.notifications.read.hint" var="PRIVATE_NOTIFICATIONS_READ_HINT_VAR"/>
+					PRIVATE_NOTIFICATIONS_READ_HINT : '<c:out value="${PRIVATE_NOTIFICATIONS_READ_HINT_VAR}" />',
+					<fmt:message key="label.private.notifications.read.all.hint" var="PRIVATE_NOTIFICATIONS_READ_ALL_HINT_VAR"/>
+					PRIVATE_NOTIFICATIONS_READ_ALL_HINT : '<c:out value="${PRIVATE_NOTIFICATIONS_READ_ALL_HINT_VAR}" />',
 				},
 				
 				tabName = '${tab}',
@@ -192,12 +202,13 @@
 					</c:if>
 				</td>
 				<td class="linksCell">
-					<c:if test="${not empty orgDTOs}">
-						<div id="refreshButton" class="ui-button" title="<fmt:message key="index.refresh.hint"/>"
-							 onClick="javascript:loadOrgTab(null, true)">
-								<fmt:message key="index.refresh" />
-						</div>
-					</c:if>
+					<div id="refreshButton" class="ui-button" title="<fmt:message key="index.refresh.hint"/>"
+						 onClick="javascript:refreshPrivateNotificationCount();loadOrgTab(null, true)">
+							<fmt:message key="index.refresh" />
+					</div>
+					<div id="notificationsButton" class="ui-button" onClick="javascript:showPrivateNotificationsDialog()">
+						Notifications <span id="notificationsPendingCount"></span>
+					</div>
 					<div id="logoutButton" class="ui-button" onClick="javascript:closeAllChildren();document.location.href='home.do?method=logout'">
 							<fmt:message key="index.logout" />
 					</div>
@@ -248,6 +259,5 @@
 		</p>
 	</div>
 </div>
-
 </body>
 </lams:html>
\ No newline at end of file
Index: lams_central/web/css/index.css
===================================================================
RCS file: /usr/local/cvsroot/lams_central/web/css/index.css,v
diff -u -r1.10.2.6 -r1.10.2.7
--- lams_central/web/css/index.css	2 Mar 2016 13:22:44 -0000	1.10.2.6
+++ lams_central/web/css/index.css	21 Apr 2016 09:13:16 -0000	1.10.2.7
@@ -143,4 +143,36 @@
 body > .addLessonDialog .ui-dialog-titlebar {
 	width: 25px;
 	right : 10px;
+}
+
+#dialogPrivateNotifications table {
+}
+
+#dialogPrivateNotifications table tr:first-child td {
+	font-weight: bold;
+	text-align: center;
+}
+
+#dialogPrivateNotifications td.notificationsReadCell {
+	font-size: 16px;
+	width: 30px;
+	text-align: center;
+}
+
+#dialogPrivateNotifications td.notificationsPendingCell {
+	font-weight: bold;
+}
+
+#dialogPrivateNotifications td.notificationsClickableCell:hover {
+	cursor: pointer;
+	
+	border-color: #9be0f9;
+	background-color: #F0F8FF;
+	/* CSS3 glow effect: */
+	-moz-border-radius: 6px;
+	-webkit-border-radius: 6px;
+	border-radius: 6px;
+	-moz-box-shadow: 0 0 5px #A6E5FD;
+	-webkit-box-shadow: 0 0 5px #A6E5FD;
+	box-shadow: 0 0 5px #A6E5FD;
 }
\ No newline at end of file
Index: lams_central/web/includes/javascript/groupDisplay.js
===================================================================
RCS file: /usr/local/cvsroot/lams_central/web/includes/javascript/groupDisplay.js,v
diff -u -r1.37.2.14 -r1.37.2.15
--- lams_central/web/includes/javascript/groupDisplay.js	16 Mar 2016 12:14:10 -0000	1.37.2.14
+++ lams_central/web/includes/javascript/groupDisplay.js	21 Apr 2016 09:13:16 -0000	1.37.2.15
@@ -15,6 +15,8 @@
 	$("#actionAccord").accordion({
 		'heightStyle' : 'content'
 	});
+	
+	refreshPrivateNotificationCount();
 }
 
 function loadOrgTab(orgTab, refresh) {
@@ -198,7 +200,7 @@
 						+ 'home.do?method=addLesson&organisationID='
 						+ $(this).dialog('option', 'orgID'));
 			
-			//in case of mobile devices allow iframe scrolling
+			// in case of mobile devices allow iframe scrolling
 			if( /Android|webOS|iPhone|iPad|iPod|BlackBerry|IEMobile|Opera Mini/i.test(navigator.userAgent) ) {
 			    setTimeout(function() {
 			    	 $('.dialogContainer').css('overflow-y','scroll');
@@ -280,8 +282,7 @@
 		'open' : function() {
 			var dialog = $(this),
 				lessonID = dialog.dialog('option', 'lessonID');
-			// if lesson ID is given, use lesson view; otherwise
-			// use course view
+			// if lesson ID is given, use lesson view; otherwise use course view
 			if (lessonID) {
 				// load contents after opening the dialog
 				$('iframe', dialog).attr('src', LAMS_URL
@@ -297,6 +298,127 @@
 	}, true);
 }
 
+
+function showPrivateNotificationsDialog(){
+	showDialog("dialogPrivateNotifications", {
+		'height' : 470,
+		'width' : 600,
+		'title' : LABELS.PRIVATE_NOTIFICATIONS_TITLE,
+		'close' : function(){
+			refreshPrivateNotificationCount();
+			// completely delete the dialog
+			$(this).remove();
+		},
+		'open' : function() {
+			// build the table from the scratch
+			var dialog = $(this),
+				table = $('<table />').appendTo(dialog),
+				// table header
+				headerRow = $('<tr />').appendTo(table);
+			$('<td />').text(LABELS.PRIVATE_NOTIFICATIONS_MESSAGES).appendTo(headerRow);
+			// click it to mark all notifications as read
+			$('<td class="notificationsClickableCell"/>').text(LABELS.PRIVATE_NOTIFICATIONS_READ)
+														 .attr('title', LABELS.PRIVATE_NOTIFICATIONS_READ_ALL_HINT)
+														 .click(markAllPrivateNotificationsAsRead)
+														 .appendTo(headerRow);
+			$('iframe', dialog).remove();
+			$.ajax({
+				cache : false,
+				url : LAMS_URL + "notification.do",
+				dataType : 'json',
+				data : {
+					'method' : 'getNotificationSubscriptions',
+					// maybe it will change for paging; "offset" param is also available
+					'limit'  : 10
+				},
+				success : function(notifications) {
+					if (!notifications) {
+						return;
+					}
+					
+					// build notification rows one by one
+					$.each(notifications, function(){
+						var notification = this,
+							row = $('<tr />').attr('id', 'subscription-' + notification.subscriptionUid)
+											 .appendTo(table),
+							messageCell = $('<td />').appendTo(row),
+							readCell = $('<td class="notificationsReadCell" />')
+											.appendTo(row);
+						// is it a link?
+						if (notification.message.indexOf('<a ') === 0) {
+							var link = $(notification.message);
+							// make it navigable
+							messageCell.text(link.text()).addClass('notificationsClickableCell').click(function(){
+								if (!readCell.text()) {
+									markPrivateNotificationAsRead(notification.subscriptionUid);
+								}
+								// open in a new tab/window
+								window.open(link.attr('href'), '_blank');
+							});
+						} else {
+							messageCell.text(notification.message);
+						}
+						// was it read already?
+						if (notification.pending) {
+							messageCell.addClass('notificationsPendingCell');
+							readCell.addClass('notificationsClickableCell')
+									.attr('title', LABELS.PRIVATE_NOTIFICATIONS_READ_HINT)
+									.click(function(){
+										markPrivateNotificationAsRead(notification.subscriptionUid);
+									});
+						} else {
+							readCell.html('&#10004;');
+						}
+					});
+				}
+			});
+		}
+	}, true);
+}
+
+function markAllPrivateNotificationsAsRead(){
+	$('#dialogPrivateNotifications tr[id^=subscription-]').each(function(){
+		var row = $(this),
+			read = $('td', row).last().text();
+		if (!read) {
+			markPrivateNotificationAsRead(row.attr('id').split('-')[1]);
+		}
+	});
+}
+
+function markPrivateNotificationAsRead(subscriptionUid){
+	$.ajax({
+		cache : false,
+		url : LAMS_URL + "notification.do",
+		data : {
+			'method' 		  : 'markNotificationAsRead',
+			'subscriptionUid' : subscriptionUid
+		},
+		success : function() {
+			// mark the message as read
+			$('#dialogPrivateNotifications tr#subscription-' + subscriptionUid + ' > td')
+				// message cell
+				.first().removeClass('notificationsPendingCell')
+				// read cell
+				.next().html('&#10004;').removeClass('notificationsClickableCell').attr('title', null).off('click');
+		}
+	});
+}
+
+function refreshPrivateNotificationCount(){
+	$.ajax({
+		cache : false,
+		url : LAMS_URL + "notification.do",
+		dataType : 'text',
+		data : {
+			'method' : 'getPendingNotificationCount'
+		},
+		success : function(count) {
+			$('#notificationsPendingCount').text(count == 0 ? '' : '(' + count + ')');
+		}
+	});
+}
+
 function showGradebookCourseDialog(orgID){
 	var id = "dialoGradebookCourse" + orgID;
 	showDialog(id, {
@@ -431,12 +553,15 @@
 				// check if the LD was created successfully
 				if (learningDesignID) {
 					var frame = $('iframe', dialog);
-					// disable previous onload handler, set in showAddSingleActivityLessonDialog()
+					// disable previous onload handler, set in
+					// showAddSingleActivityLessonDialog()
 					frame.off('load').load(function(){
-						// disable current onload handler as closing the dialog reloads the iframe
+						// disable current onload handler as closing the dialog
+						// reloads the iframe
 						frame.off('load');
 						
-						// call svgGenerator.jsp code to store LD SVG on the server
+						// call svgGenerator.jsp code to store LD SVG on the
+						// server
 						var win = frame[0].contentWindow || frame[0].contentDocument;
 						win.GeneralLib.saveLearningDesignImage();
 						
@@ -463,7 +588,6 @@
 	$("#" + id).dialog('close');
 }
 
-
 /**
  * Loads contents to already open organisation groups dialog.
  */
@@ -486,7 +610,8 @@
 }
 
 /**
- * Called from within Course Groups dialog, it saves groups and loads grouping page.
+ * Called from within Course Groups dialog, it saves groups and loads grouping
+ * page.
  */
 function saveOrgGroups() {
 	var groupsSaved = $('#dialogOrgGroup iframe')[0].contentWindow.saveGroups();
Index: lams_common/build.xml
===================================================================
RCS file: /usr/local/cvsroot/lams_common/build.xml,v
diff -u -r1.89.2.9 -r1.89.2.10
--- lams_common/build.xml	16 Mar 2016 12:13:24 -0000	1.89.2.9
+++ lams_common/build.xml	21 Apr 2016 09:12:53 -0000	1.89.2.10
@@ -14,6 +14,7 @@
 		<!-- Event Notification and Pedagogical Planner -->
 		<include name="**/Event.java" />
 		<include name="**/Subscription.java" />
+		<include name="**/DeliveryPending.java" />
 		<include name="**/LogEvent.java" />
 		<include name="**/PedagogicalPlanner*.java" />
 		<include name="**/DataFlowObject.java" />
Index: lams_common/src/java/org/lamsfoundation/lams/beanRefContext.xml
===================================================================
RCS file: /usr/local/cvsroot/lams_common/src/java/org/lamsfoundation/lams/beanRefContext.xml,v
diff -u -r1.10.2.5 -r1.10.2.6
--- lams_common/src/java/org/lamsfoundation/lams/beanRefContext.xml	12 Jan 2016 12:20:17 -0000	1.10.2.5
+++ lams_common/src/java/org/lamsfoundation/lams/beanRefContext.xml	21 Apr 2016 09:12:53 -0000	1.10.2.6
@@ -43,7 +43,6 @@
 			<value>/org/lamsfoundation/lams/toolApplicationContext.xml</value>
 			<value>/org/lamsfoundation/lams/comments/commentsContext.xml</value>
 			<value>/org/lamsfoundation/lams/contentrepository/applicationContext.xml</value>
-			<value>/org/lamsfoundation/lams/lesson/lessonApplicationContext.xml</value>
 			<value>/org/lamsfoundation/lams/learning/learningApplicationContext.xml</value>
 			
 			<value>classpath*:org/lamsfoundation/lams/tool/**/*pplicationContext.xml</value>
Index: lams_common/src/java/org/lamsfoundation/lams/commonContext.xml
===================================================================
RCS file: /usr/local/cvsroot/lams_common/src/java/org/lamsfoundation/lams/commonContext.xml,v
diff -u -r1.100.2.25 -r1.100.2.26
--- lams_common/src/java/org/lamsfoundation/lams/commonContext.xml	16 Mar 2016 12:13:24 -0000	1.100.2.25
+++ lams_common/src/java/org/lamsfoundation/lams/commonContext.xml	21 Apr 2016 09:12:53 -0000	1.100.2.26
@@ -106,6 +106,36 @@
 		</property>
 	</bean>
 	
+	<bean id="lessonServiceTarget" class="org.lamsfoundation.lams.lesson.service.LessonService">
+		<property name="lessonDAO"><ref bean="lessonDAO"/></property>
+		<property name="lessonClassDAO"><ref bean="lessonClassDAO"/></property>
+		<property name="groupingDAO"><ref bean="groupingDAO"/></property>
+		<property name="baseDAO"><ref bean="baseDAO"/></property>
+		<property name="learnerProgressDAO"><ref bean="learnerProgressDAO"/></property>
+		<property name="messageService"><ref bean="commonMessageService"/></property>
+	</bean>
+	
+	<bean id="lessonService" class="org.springframework.transaction.interceptor.TransactionProxyFactoryBean">
+		<property name="transactionManager"><ref bean="transactionManager"/></property>
+		<property name="target"><ref bean="lessonServiceTarget"/></property>
+		<property name="proxyTargetClass"><value>true</value></property> 
+		<property name="preInterceptors">
+			<list>
+				<ref bean="transactionRetryInterceptor" />			
+			</list>
+		</property>
+		<property name="transactionAttributes">
+			<props>
+				<prop key="perform*">PROPAGATION_REQUIRED</prop>
+				<prop key="remove*">PROPAGATION_REQUIRED</prop>
+				<prop key="get*">PROPAGATION_REQUIRED</prop>
+				<prop key="add*">PROPAGATION_REQUIRED</prop>
+				<prop key="create*">PROPAGATION_REQUIRED</prop>
+				<prop key="update*">PROPAGATION_REQUIRED</prop>
+			</props>
+		</property>
+	</bean>
+	
 	<bean id="themeServiceTarget" class="org.lamsfoundation.lams.themes.service.ThemeService">
 		<property name="themeDAO"><ref bean="themeDAO"/></property>
 		<property name="userManagementService"><ref bean="userManagementService"/></property>
@@ -327,15 +357,16 @@
 		</property>
 		<property name="transactionAttributes">
 			<props>
-				<prop key="createEvent">PROPAGATION_REQUIRED</prop>
+				<prop key="create*">PROPAGATION_REQUIRED</prop>
 				<prop key="eventExists">PROPAGATION_REQUIRED</prop>
 				<prop key="sendMessage">PROPAGATION_REQUIRED</prop>
 				<prop key="resendMessages">PROPAGATION_REQUIRED</prop>
-				<prop key="notifyLessonMonitors">PROPAGATION_REQUIRED</prop>
+				<prop key="notify*">PROPAGATION_REQUIRED</prop>
 				<prop key="subscribe">PROPAGATION_REQUIRED</prop>
 				<prop key="unsubscribe">PROPAGATION_REQUIRED</prop>
 				<prop key="trigger*">PROPAGATION_REQUIRED</prop>
 				<prop key="isSubscribed">PROPAGATION_REQUIRED</prop>
+				<prop key="getNotification*">PROPAGATION_REQUIRED</prop>
 			</props>
 		</property>
 	</bean>
@@ -443,8 +474,22 @@
 	</bean>
 	<bean id="licenseDAO" class="org.lamsfoundation.lams.learningdesign.dao.hibernate.LicenseDAO">
 		<property name="sessionFactory"><ref bean="coreSessionFactory"/></property>
-	</bean>	
+	</bean>
 	
+	<!-- Lesson related beans -->
+	
+	<bean id="lessonDAO" class="org.lamsfoundation.lams.lesson.dao.hibernate.LessonDAO">
+		<property name="sessionFactory"><ref bean="coreSessionFactory"/></property>
+	</bean>
+
+	<bean id="lessonClassDAO" class="org.lamsfoundation.lams.lesson.dao.hibernate.LessonClassDAO">
+		<property name="sessionFactory"><ref bean="coreSessionFactory"/></property>
+	</bean>
+
+	<bean id="learnerProgressDAO" class="org.lamsfoundation.lams.lesson.dao.hibernate.LearnerProgressDAO">
+		<property name="sessionFactory"><ref bean="coreSessionFactory"/></property>
+	</bean>
+	
 	<!--User management related beans -->
 	
 	<bean id="baseDAO" class="org.lamsfoundation.lams.dao.hibernate.LAMSBaseDAO">
Index: lams_common/src/java/org/lamsfoundation/lams/events/AbstractDeliveryMethod.java
===================================================================
RCS file: /usr/local/cvsroot/lams_common/src/java/org/lamsfoundation/lams/events/AbstractDeliveryMethod.java,v
diff -u -r1.3 -r1.3.2.1
--- lams_common/src/java/org/lamsfoundation/lams/events/AbstractDeliveryMethod.java	4 Jul 2013 11:35:26 -0000	1.3
+++ lams_common/src/java/org/lamsfoundation/lams/events/AbstractDeliveryMethod.java	21 Apr 2016 09:12:52 -0000	1.3.2.1
@@ -1,8 +1,12 @@
 package org.lamsfoundation.lams.events;
 
 import java.security.InvalidParameterException;
+import java.util.HashMap;
+import java.util.Iterator;
+import java.util.Map;
+import java.util.Map.Entry;
 
-import org.apache.commons.lang.StringUtils;
+import org.apache.log4j.Logger;
 
 /**
  * Provides methods to notify users of an event.
@@ -12,16 +16,6 @@
  */
 public abstract class AbstractDeliveryMethod {
     /**
-     * Short name for the delivery method
-     */
-    protected final String signature;
-
-    /**
-     * Short description of the delivery method.
-     */
-    protected final String description;
-
-    /**
      * Unique identifier of the delivery method.
      */
     protected final short id;
@@ -31,24 +25,16 @@
      */
     protected long sendTimeout = Long.MAX_VALUE;
 
+    protected final Map<Long, String> errors = new HashMap<Long, String>();
+    protected static final long ERROR_SILENCE_TIMEOUT = 5000;
+
     /**
      * Standard constructor.
      * 
      * @param id
      *            ID of the delivery method
-     * @param signature
-     *            signature of the delivery method
-     * @param description
-     *            short description of the delivery method
-     * @throws InvalidParameterException
-     *             if signature is blank
      */
-    protected AbstractDeliveryMethod(short id, String signature, String description) throws InvalidParameterException {
-	if (StringUtils.isEmpty(signature)) {
-	    throw new InvalidParameterException("Signature must not be blank.");
-	}
-	this.signature = signature;
-	this.description = description;
+    protected AbstractDeliveryMethod(short id) throws InvalidParameterException {
 	this.id = id;
     }
 
@@ -69,18 +55,13 @@
     protected abstract String send(Integer fromUserId, Integer toUserId, String subject, String message,
 	    boolean isHtmlFormat) throws InvalidParameterException;
 
-    public String getSignature() {
-	return signature;
-    }
+    protected abstract Logger getLog();
 
-    public String getDescription() {
-	return description;
-    }
+    protected abstract boolean lastOperationFailed(Subscription subscription);
 
     @Override
     public boolean equals(Object o) {
-	return (o instanceof AbstractDeliveryMethod)
-		&& ((AbstractDeliveryMethod) o).signature.equalsIgnoreCase(signature);
+	return (o instanceof AbstractDeliveryMethod) && (((AbstractDeliveryMethod) o).id == this.id);
     }
 
     public short getId() {
@@ -94,4 +75,29 @@
     public void setSendTimeout(long sendTimeout) {
 	this.sendTimeout = sendTimeout;
     }
+
+    /**
+     * Checks if the error was already logged recently. If not, logs it.
+     */
+    protected synchronized void logError(String error) {
+	Iterator<Entry<Long, String>> errorIterator = errors.entrySet().iterator();
+	boolean logError = true;
+	long currentTime = System.currentTimeMillis();
+	// same errors younger than this will be silenced
+	long silencePeriod = currentTime - AbstractDeliveryMethod.ERROR_SILENCE_TIMEOUT;
+	while (errorIterator.hasNext()) {
+	    Entry<Long, String> entry = errorIterator.next();
+	    if (entry.getKey() < silencePeriod) {
+		// clear the map from garbage
+		errorIterator.remove();
+	    } else if (error.equals(entry.getValue())) {
+		logError = false;
+		// do not break, let the clean up go through the whole map
+	    }
+	}
+	if (logError) {
+	    errors.put(currentTime, error);
+	    getLog().error("Error while notifying users: " + error);
+	}
+    }
 }
\ No newline at end of file
Index: lams_common/src/java/org/lamsfoundation/lams/events/DeliveryMethodMail.java
===================================================================
RCS file: /usr/local/cvsroot/lams_common/src/java/org/lamsfoundation/lams/events/DeliveryMethodMail.java,v
diff -u -r1.6.2.4 -r1.6.2.5
--- lams_common/src/java/org/lamsfoundation/lams/events/DeliveryMethodMail.java	3 Mar 2016 12:24:17 -0000	1.6.2.4
+++ lams_common/src/java/org/lamsfoundation/lams/events/DeliveryMethodMail.java	21 Apr 2016 09:12:53 -0000	1.6.2.5
@@ -2,10 +2,6 @@
 
 import java.io.UnsupportedEncodingException;
 import java.security.InvalidParameterException;
-import java.util.HashMap;
-import java.util.Iterator;
-import java.util.Map;
-import java.util.Map.Entry;
 
 import javax.mail.MessagingException;
 import javax.mail.internet.AddressException;
@@ -28,47 +24,44 @@
 public class DeliveryMethodMail extends AbstractDeliveryMethod {
     private static final Logger log = Logger.getLogger(DeliveryMethodMail.class);
 
-    private static final long ERROR_SILENCE_TIMEOUT = 5000;
-
     private static DeliveryMethodMail instance = null;
-    private final EmailValidator emailValidator = EmailValidator.getInstance();
-    private final Map<Long, String> errors = new HashMap<Long, String>();
+    private static final EmailValidator emailValidator = EmailValidator.getInstance();
 
-    private IUserManagementService userManagementService = null;
+    private static IUserManagementService userManagementService = null;
 
-    public static DeliveryMethodMail getInstance() {
+    protected static DeliveryMethodMail getInstance() {
 	if (DeliveryMethodMail.instance == null) {
 	    DeliveryMethodMail.instance = new DeliveryMethodMail();
 	}
 	return DeliveryMethodMail.instance;
     }
 
     private DeliveryMethodMail() {
-	super((short) 1, "MAIL", "Messages will be send by email");
+	super((short) 1);
     }
 
     @Override
-    public String send(Integer fromUserId, Integer toUserId, String subject, String message, boolean isHtmlFormat)
+    protected String send(Integer fromUserId, Integer toUserId, String subject, String message, boolean isHtmlFormat)
 	    throws InvalidParameterException {
 	try {
-	    User toUser = (User) userManagementService.findById(User.class, toUserId);
+	    User toUser = (User) DeliveryMethodMail.userManagementService.findById(User.class, toUserId);
 	    if (toUser == null) {
 		return "Target user with ID " + toUserId + " was not found.";
 	    }
 	    String toEmail = toUser.getEmail();
-	    if (!emailValidator.isValid(toEmail)) {
+	    if (!DeliveryMethodMail.emailValidator.isValid(toEmail)) {
 		return "Target user's e-mail address is invalid.";
 	    }
 
 	    if (fromUserId == null) {
 		Emailer.sendFromSupportEmail(subject, toEmail, message, isHtmlFormat);
 	    } else {
-		User fromUser = (User) userManagementService.findById(User.class, fromUserId);
+		User fromUser = (User) DeliveryMethodMail.userManagementService.findById(User.class, fromUserId);
 		if (fromUser == null) {
 		    return "Source user with ID " + fromUserId + " was not found.";
 		}
 		String fromEmail = fromUser.getEmail();
-		if (!emailValidator.isValid(fromEmail)) {
+		if (!DeliveryMethodMail.emailValidator.isValid(fromEmail)) {
 		    return "Source user's e-mail address is invalid.";
 		}
 
@@ -82,10 +75,14 @@
 	}
     }
 
-    public void setUserManagementService(IUserManagementService userManagementService) {
-	this.userManagementService = userManagementService;
+    protected boolean lastOperationFailed(Subscription subscription) {
+	return subscription.getLastOperationMessage() != null;
     }
 
+    static void setUserManagementService(IUserManagementService userManagementService) {
+	DeliveryMethodMail.userManagementService = userManagementService;
+    }
+
     /**
      * Sends an email to the address provided by the admin.
      * 
@@ -113,28 +110,8 @@
 	}
     }
 
-    /**
-     * Checks if the error was already logged recently. If not, logs it.
-     */
-    private synchronized void logError(String error) {
-	Iterator<Entry<Long, String>> errorIterator = errors.entrySet().iterator();
-	boolean logError = true;
-	long currentTime = System.currentTimeMillis();
-	// same errors younger than this will be silenced
-	long silencePeriod = currentTime - DeliveryMethodMail.ERROR_SILENCE_TIMEOUT;
-	while (errorIterator.hasNext()) {
-	    Entry<Long, String> entry = errorIterator.next();
-	    if (entry.getKey() < silencePeriod) {
-		// clear the map from garbage
-		errorIterator.remove();
-	    } else if (error.equals(entry.getValue())) {
-		logError = false;
-		// do not break, let the clean up go through the whole map
-	    }
-	}
-	if (logError) {
-	    errors.put(currentTime, error);
-	    DeliveryMethodMail.log.error("Error while sending emails: " + error);
-	}
+    @Override
+    protected Logger getLog() {
+	return DeliveryMethodMail.log;
     }
 }
\ No newline at end of file
Fisheye: Tag 1.1 refers to a dead (removed) revision in file `lams_common/src/java/org/lamsfoundation/lams/events/DeliveryMethodNotification.java'.
Fisheye: No comparison available.  Pass `N' to diff?
Index: lams_common/src/java/org/lamsfoundation/lams/events/Event.java
===================================================================
RCS file: /usr/local/cvsroot/lams_common/src/java/org/lamsfoundation/lams/events/Event.java,v
diff -u -r1.5.2.1 -r1.5.2.2
--- lams_common/src/java/org/lamsfoundation/lams/events/Event.java	16 Feb 2016 16:03:21 -0000	1.5.2.1
+++ lams_common/src/java/org/lamsfoundation/lams/events/Event.java	21 Apr 2016 09:12:53 -0000	1.5.2.2
@@ -154,7 +154,7 @@
 
     /**
      * 
-     * @hibernate.set cascade="all-delete-orphan" order-by="last_operation_time desc" outer-join="true"
+     * @hibernate.set cascade="all-delete-orphan" outer-join="true"
      * @hibernate.collection-key column="event_uid"
      * @hibernate.collection-one-to-many class="org.lamsfoundation.lams.events.Subscription"
      * 
Index: lams_common/src/java/org/lamsfoundation/lams/events/EventNotificationService.java
===================================================================
RCS file: /usr/local/cvsroot/lams_common/src/java/org/lamsfoundation/lams/events/EventNotificationService.java,v
diff -u -r1.7.2.6 -r1.7.2.7
--- lams_common/src/java/org/lamsfoundation/lams/events/EventNotificationService.java	19 Apr 2016 06:52:35 -0000	1.7.2.6
+++ lams_common/src/java/org/lamsfoundation/lams/events/EventNotificationService.java	21 Apr 2016 09:12:52 -0000	1.7.2.7
@@ -20,6 +20,7 @@
 import org.lamsfoundation.lams.util.Configuration;
 import org.lamsfoundation.lams.util.ConfigurationKeys;
 import org.lamsfoundation.lams.util.MessageService;
+import org.lamsfoundation.lams.util.hibernate.HibernateSessionManager;
 
 /**
  * Provides tools for managing events and notifing users.
@@ -33,6 +34,7 @@
 
     static {
 	IEventNotificationService.availableDeliveryMethods.add(IEventNotificationService.DELIVERY_METHOD_MAIL);
+	IEventNotificationService.availableDeliveryMethods.add(IEventNotificationService.DELIVERY_METHOD_NOTIFICATION);
     }
 
     private EventDAO eventDAO = null;
@@ -43,14 +45,37 @@
 
     @Override
     public void createEvent(String scope, String name, Long eventSessionId, String defaultSubject,
-	    String defaultMessage, boolean isHtmlFormat) throws InvalidParameterException {
+	    String defaultMessage, boolean isHtmlFormat) {
 	if (!eventExists(scope, name, eventSessionId)) {
 	    Event event = new Event(scope, name, eventSessionId, defaultSubject, defaultMessage, isHtmlFormat);
-	    eventDAO.saveEvent(event);
+	    eventDAO.insertOrUpdate(event);
 	}
     }
 
+    @SuppressWarnings("unchecked")
     @Override
+    public void createLessonEvent(String scope, String name, Long toolContentId, String defaultSubject,
+	    String defaultMessage, boolean isHtmlFormat, AbstractDeliveryMethod deliveryMethod)
+	    throws InvalidParameterException {
+	Lesson lesson = getLessonByToolContentId(toolContentId);
+	if (!eventExists(scope, name, lesson.getLessonId())) {
+	    Event event = new Event(scope, name, lesson.getLessonId(), defaultSubject, defaultMessage, isHtmlFormat);
+	    Set<User> users = null;
+	    if (scope.equals(IEventNotificationService.LESSON_MONITORS_SCOPE)) {
+		users = lesson.getLessonClass().getStaffGroup().getUsers();
+	    } else if (scope.equals(IEventNotificationService.LESSON_LEARNERS_SCOPE)) {
+		users = lesson.getLessonClass().getLearners();
+	    }
+	    if (users != null) {
+		for (User user : users) {
+		    event.getSubscriptions().add(new Subscription(user.getUserId(), deliveryMethod));
+		}
+		eventDAO.insert(event);
+	    }
+	}
+    }
+
+    @Override
     public boolean eventExists(String scope, String name, Long eventSessionId) throws InvalidParameterException {
 	if (scope == null) {
 	    throw new InvalidParameterException("Scope should not be null.");
@@ -157,7 +182,7 @@
 		    IEventNotificationService.DELIVERY_METHOD_MAIL.notifyAdmin(
 			    messageService.getMessage("mail.resend.abandon.subject"), body.toString(),
 			    event.isHtmlFormat());
-		    eventDAO.deleteEvent(event);
+		    eventDAO.delete(event);
 		}
 	    }
 
@@ -179,7 +204,7 @@
 		String.valueOf(System.currentTimeMillis()), null, subject, message, isHtmlFormat);
 	subscribe(event, toUserId, deliveryMethod);
 	event.setFailTime(new Date());
-	eventDAO.saveEvent(event);
+	eventDAO.insertOrUpdate(event);
 	return false;
     }
 
@@ -195,6 +220,8 @@
 	}
 	// create a new thread to send the messages as it can take some time
 	new Thread(() -> {
+	    HibernateSessionManager.bindHibernateSessionToCurrentThread(false);
+
 	    Event event = null;
 	    for (Integer id : toUserIds) {
 		String result = deliveryMethod.send(fromUserId, id, subject, message, isHtmlFormat);
@@ -206,11 +233,22 @@
 	    }
 	    if (event != null) {
 		event.setFailTime(new Date());
-		eventDAO.saveEvent(event);
+		eventDAO.insertOrUpdate(event);
 	    }
 	}).start();
     }
 
+    @Override
+    public List<Subscription> getNotificationSubscriptions(Long lessonId, Integer userId, Integer limit,
+	    Integer offset) {
+	return eventDAO.getLessonEventSubscriptions(lessonId, userId, limit, offset);
+    }
+
+    @Override
+    public long getNotificationPendingCount(Long lessonId, Integer userId) {
+	return eventDAO.getPendingNotificationCount(lessonId, userId);
+    }
+
     public void setEventDAO(EventDAO eventDAO) {
 	this.eventDAO = eventDAO;
     }
@@ -229,7 +267,7 @@
 
     public void setUserManagementService(IUserManagementService userManagementService) {
 	this.userManagementService = userManagementService;
-	IEventNotificationService.DELIVERY_METHOD_MAIL.setUserManagementService(userManagementService);
+	DeliveryMethodMail.setUserManagementService(userManagementService);
     }
 
     /**
@@ -248,16 +286,15 @@
 	List<Subscription> subscriptionList = new ArrayList<Subscription>(event.getSubscriptions());
 	for (int index = 0; index < subscriptionList.size(); index++) {
 	    Subscription subscription = subscriptionList.get(index);
-	    if (subscription.getUserId().equals(userId)
-		    && subscription.getDeliveryMethod().equals(deliveryMethod.getId())) {
+	    if (subscription.getUserId().equals(userId) && subscription.getDeliveryMethod().equals(deliveryMethod)) {
 		substriptionFound = true;
 		break;
 	    }
 	}
 	if (!substriptionFound) {
 	    event.getSubscriptions().add(new Subscription(userId, deliveryMethod));
 	}
-	eventDAO.saveEvent(event);
+	eventDAO.update(event);
     }
 
     @Override
@@ -291,12 +328,14 @@
 
 	// create a new thread to send the messages as it can take some time
 	new Thread(() -> {
+	    HibernateSessionManager.bindHibernateSessionToCurrentThread(false);
+
 	    Event eventFailCopy = null;
 	    Iterator<Subscription> subscriptionIterator = event.getSubscriptions().iterator();
 	    while (subscriptionIterator.hasNext()) {
 		Subscription subscription = subscriptionIterator.next();
 		notifyUser(subscription, subjectToSend, messageToSend, event.isHtmlFormat());
-		if (subscription.getLastOperationMessage() == null) {
+		if (subscription.getDeliveryMethod().lastOperationFailed(subscription)) {
 		    if (event.getFailTime() != null) {
 			subscriptionIterator.remove();
 		    }
@@ -308,9 +347,9 @@
 		}
 	    }
 	    if (event.getSubscriptions().isEmpty()) {
-		eventDAO.deleteEvent(event);
+		eventDAO.delete(event);
 	    } else {
-		eventDAO.saveEvent(event);
+		eventDAO.insertOrUpdate(event);
 	    }
 
 	    /*
@@ -321,7 +360,7 @@
 		eventFailCopy.setFailTime(new Date());
 		eventFailCopy.setSubject(subjectToSend);
 		eventFailCopy.setMessage(messageToSend);
-		eventDAO.saveEvent(eventFailCopy);
+		eventDAO.insertOrUpdate(eventFailCopy);
 	    }
 	}).start();
     }
@@ -366,6 +405,12 @@
     }
 
     @Override
+    public void triggerLessonEvent(String scope, String name, Long toolContentId, String subject, String message) {
+	Lesson lesson = getLessonByToolContentId(toolContentId);
+	trigger(scope, name, lesson.getLessonId(), subject, message);
+    }
+
+    @Override
     public void trigger(String scope, String name, Long eventSessionId, String title, String message)
 	    throws InvalidParameterException {
 	if (scope == null) {
@@ -381,30 +426,32 @@
 	trigger(event, title, message);
     }
 
-    /**
-     * See {@link IEventNotificationService#triggerForSingleUser(String, String, Long, Long)}
-     */
-    private void triggerForSingleUser(Event event, Integer userId, String subject, String message)
-	    throws InvalidParameterException {
-	final String subjectToSend = subject == null ? event.getSubject() : subject;
-	final String messageToSend = message == null ? event.getMessage() : message;
-
+    private void triggerForSingleUser(Event event, Integer userId, String subject, String message) {
 	for (Subscription subscription : event.getSubscriptions()) {
 	    if (subscription.getUserId().equals(userId)) {
-		notifyUser(subscription, subject, message, event.isHtmlFormat());
-		if (subscription.getLastOperationMessage() != null) {
-		    Event eventFailCopy = (Event) event.clone();
-		    eventFailCopy.setFailTime(new Date());
-		    eventFailCopy.setSubject(subjectToSend);
-		    eventFailCopy.setMessage(messageToSend);
-		    subscribe(eventFailCopy, subscription.getUserId(), subscription.getDeliveryMethod());
-		    eventDAO.saveEvent(eventFailCopy);
-		}
+		triggerForSingleUser(subscription.getUid(), subject, message);
 	    }
 	}
     }
 
     @Override
+    public void triggerForSingleUser(Long subscriptionUid, String subject, String message) {
+	Subscription subscription = (Subscription) eventDAO.find(Subscription.class, subscriptionUid);
+	Event event = subscription.getEvent();
+	String subjectToSend = subject == null ? event.getSubject() : subject;
+	String messageToSend = message == null ? event.getMessage() : message;
+	notifyUser(subscription, subject, message, event.isHtmlFormat());
+	if (subscription.getDeliveryMethod().lastOperationFailed(subscription)) {
+	    Event eventFailCopy = (Event) event.clone();
+	    eventFailCopy.setFailTime(new Date());
+	    eventFailCopy.setSubject(subjectToSend);
+	    eventFailCopy.setMessage(messageToSend);
+	    subscribe(eventFailCopy, subscription.getUserId(), subscription.getDeliveryMethod());
+	    eventDAO.insertOrUpdate(eventFailCopy);
+	}
+    }
+
+    @Override
     public void triggerForSingleUser(String scope, String name, Long eventSessionId, Integer userId)
 	    throws InvalidParameterException {
 	if (scope == null) {
@@ -484,9 +531,9 @@
 	    }
 	}
 	if (event.getSubscriptions().isEmpty()) {
-	    eventDAO.deleteEvent(event);
+	    eventDAO.delete(event);
 	} else {
-	    eventDAO.saveEvent(event);
+	    eventDAO.insertOrUpdate(event);
 	}
     }
 
@@ -509,9 +556,9 @@
 	    }
 	}
 	if (event.getSubscriptions().isEmpty()) {
-	    eventDAO.deleteEvent(event);
+	    eventDAO.delete(event);
 	} else {
-	    eventDAO.saveEvent(event);
+	    eventDAO.insertOrUpdate(event);
 	}
     }
 
@@ -555,4 +602,10 @@
 	}
 	unsubscribe(event, userId, deliveryMethod);
     }
+
+    private Lesson getLessonByToolContentId(Long toolContentId) {
+	ToolActivity activity = (ToolActivity) eventDAO
+		.findByProperty(ToolActivity.class, "toolContentId", toolContentId).get(0);
+	return (Lesson) activity.getLearningDesign().getLessons().iterator().next();
+    }
 }
\ No newline at end of file
Index: lams_common/src/java/org/lamsfoundation/lams/events/IEventNotificationService.java
===================================================================
RCS file: /usr/local/cvsroot/lams_common/src/java/org/lamsfoundation/lams/events/IEventNotificationService.java,v
diff -u -r1.4.2.4 -r1.4.2.5
--- lams_common/src/java/org/lamsfoundation/lams/events/IEventNotificationService.java	19 Apr 2016 06:52:35 -0000	1.4.2.4
+++ lams_common/src/java/org/lamsfoundation/lams/events/IEventNotificationService.java	21 Apr 2016 09:12:53 -0000	1.4.2.5
@@ -2,6 +2,7 @@
 
 import java.security.InvalidParameterException;
 import java.util.HashSet;
+import java.util.List;
 import java.util.Set;
 
 /**
@@ -15,9 +16,19 @@
     /**
      * Scope for the events that are common for the whole LAMS environment.
      */
-    static final String CORE_EVENTS_SCOPE = "CORE";
+    static final String CORE_SCOPE = "CORE";
 
     /**
+     * Scope for the events that are meant for given lesson staff.
+     */
+    static final String LESSON_MONITORS_SCOPE = "LESSON_MONITORS";
+
+    /**
+     * Scope for the events that are meant for given lesson learners.
+     */
+    static final String LESSON_LEARNERS_SCOPE = "LESSON_LEARNERS";
+
+    /**
      * Scope for events that were created after {@link #sendMessage(Long, AbstractDeliveryMethod, String, String)}
      * failed.
      */
@@ -34,6 +45,8 @@
      */
     static final DeliveryMethodMail DELIVERY_METHOD_MAIL = DeliveryMethodMail.getInstance();
 
+    static final DeliveryMethodNotification DELIVERY_METHOD_NOTIFICATION = DeliveryMethodNotification.getInstance();
+
     static final Set<AbstractDeliveryMethod> availableDeliveryMethods = new HashSet<AbstractDeliveryMethod>(2);
 
     /**
@@ -51,13 +64,13 @@
      *            body of the message send to users; it can be altered when triggering the event
      * @param isHtmlFormat
      *            whether the message is of HTML content-type or plain text
-     * @return <code>true</code> if the event did not exist and was correctly created
-     * @throws InvalidParameterException
-     *             if scope was <code>null</code> or name was blank
      */
     void createEvent(String scope, String name, Long eventSessionId, String defaultSubject, String defaultMessage,
-	    boolean isHtmlFormat) throws InvalidParameterException;
+	    boolean isHtmlFormat);
 
+    void createLessonEvent(String scope, String name, Long toolSessionId, String defaultSubject, String defaultMessage,
+	    boolean isHtmlFormat, AbstractDeliveryMethod deliveryMethod);
+
     /**
      * Checks if event with the given parameters exists in the database.
      * 
@@ -80,6 +93,10 @@
      */
     Set<AbstractDeliveryMethod> getAvailableDeliveryMethods();
 
+    List<Subscription> getNotificationSubscriptions(Long lessonId, Integer userId, Integer limit, Integer offset);
+
+    long getNotificationPendingCount(Long lessonId, Integer userId);
+
     /**
      * Checks if an user is subscribed to the given event.
      * 
@@ -206,6 +223,8 @@
     void trigger(String scope, String name, Long eventSessionId, Object[] parameterValues)
 	    throws InvalidParameterException;
 
+    void triggerLessonEvent(String scope, String name, Long toolContentId, String subject, String message);
+
     /**
      * Triggers the event with given subject and message. Each subscribed user is notified. Default message and subject
      * are overridden.
@@ -226,6 +245,8 @@
     void trigger(String scope, String name, Long eventSessionId, String subject, String message)
 	    throws InvalidParameterException;
 
+    void triggerForSingleUser(Long subscriptionUid, String subject, String message);
+
     /**
      * Notifies only a single user of the event using the default subject and message. Does not set the event as
      * "triggered".
Index: lams_common/src/java/org/lamsfoundation/lams/events/dao/EventDAO.java
===================================================================
RCS file: /usr/local/cvsroot/lams_common/src/java/org/lamsfoundation/lams/events/dao/EventDAO.java,v
diff -u -r1.1 -r1.1.14.1
--- lams_common/src/java/org/lamsfoundation/lams/events/dao/EventDAO.java	25 Aug 2008 08:15:43 -0000	1.1
+++ lams_common/src/java/org/lamsfoundation/lams/events/dao/EventDAO.java	21 Apr 2016 09:12:53 -0000	1.1.14.1
@@ -2,26 +2,29 @@
 
 import java.util.List;
 
+import org.lamsfoundation.lams.dao.IBaseDAO;
 import org.lamsfoundation.lams.events.Event;
+import org.lamsfoundation.lams.events.Subscription;
 
-public interface EventDAO {
-	/**
-	 * Gets an instance of the event.
-	 * @param scope
-	 * @param name
-	 * @param eventSessionId
-	 * @return
-	 */
-	Event getEvent(String scope, String name, Long sessionId);
+public interface EventDAO extends IBaseDAO {
+    /**
+     * Gets an instance of the event.
+     * 
+     * @param scope
+     * @param name
+     * @param eventSessionId
+     * @return
+     */
+    Event getEvent(String scope, String name, Long sessionId);
 
-	/**
-	 * Gets events with messages that need to be resend.
-	 * Either they failed to be send or they should be repeated.
-	 * @return list of events
-	 */
-	List<Event> getEventsToResend();
+    /**
+     * Gets events with messages that need to be resend. Either they failed to be send or they should be repeated.
+     * 
+     * @return list of events
+     */
+    List<Event> getEventsToResend();
 
-	void deleteEvent(Event event);
+    List<Subscription> getLessonEventSubscriptions(Long lessonId, Integer userId, Integer limit, Integer offset);
 
-	void saveEvent(Event event);
+    long getPendingNotificationCount(Long lessonId, Integer userId);
 }
\ No newline at end of file
Index: lams_common/src/java/org/lamsfoundation/lams/events/dao/hibernate/EventDAOHibernate.java
===================================================================
RCS file: /usr/local/cvsroot/lams_common/src/java/org/lamsfoundation/lams/events/dao/hibernate/EventDAOHibernate.java,v
diff -u -r1.3.2.5 -r1.3.2.6
--- lams_common/src/java/org/lamsfoundation/lams/events/dao/hibernate/EventDAOHibernate.java	19 Apr 2016 06:52:35 -0000	1.3.2.5
+++ lams_common/src/java/org/lamsfoundation/lams/events/dao/hibernate/EventDAOHibernate.java	21 Apr 2016 09:12:53 -0000	1.3.2.6
@@ -3,8 +3,11 @@
 import java.security.InvalidParameterException;
 import java.util.List;
 
+import org.hibernate.Query;
 import org.lamsfoundation.lams.dao.hibernate.LAMSBaseDAO;
+import org.lamsfoundation.lams.events.DeliveryMethodNotification;
 import org.lamsfoundation.lams.events.Event;
+import org.lamsfoundation.lams.events.Subscription;
 import org.lamsfoundation.lams.events.dao.EventDAO;
 import org.springframework.stereotype.Repository;
 
@@ -17,6 +20,14 @@
     private static final String GET_EVENTS_TO_RESEND_QUERY = "SELECT DISTINCT e FROM " + Event.class.getName()
 	    + " AS e WHERE e.failTime IS NOT NULL";
 
+    private static final String GET_LESSON_EVENT_SUBSCRIPTIONS = "FROM " + Subscription.class.getName()
+	    + " AS s WHERE s.userId = ? AND s.event.scope LIKE 'LESSON_%'";
+
+    private static final String COUNT_PENDING_NOTIFICATIONS = "SELECT COUNT(*) FROM " + Subscription.class.getName()
+	    + " AS s WHERE (s.lastOperationMessage IS NULL OR s.lastOperationMessage != '"
+	    + DeliveryMethodNotification.LAST_OPERATION_SEEN + "') AND s.userId = ? AND s.event.scope LIKE 'LESSON_%'";
+
+    @Override
     @SuppressWarnings("unchecked")
     public Event getEvent(String scope, String name, Long sessionId) throws InvalidParameterException {
 	List<Event> events = (List<Event>) doFind(EventDAOHibernate.GET_EVENT_QUERY,
@@ -30,16 +41,46 @@
 	return events.get(0);
     }
 
+    @Override
     @SuppressWarnings("unchecked")
     public List<Event> getEventsToResend() {
 	return (List<Event>) doFind(EventDAOHibernate.GET_EVENTS_TO_RESEND_QUERY);
     }
 
-    public void deleteEvent(Event event) {
-	getSession().delete(event);
+    @Override
+    @SuppressWarnings("unchecked")
+    public List<Subscription> getLessonEventSubscriptions(Long lessonId, Integer userId, Integer limit,
+	    Integer offset) {
+	String query = EventDAOHibernate.GET_LESSON_EVENT_SUBSCRIPTIONS;
+	if (lessonId != null) {
+	    query += " AND s.event.eventSessionId = ?";
+	}
+	query += " ORDER BY ISNULL(s.lastOperationMessage) DESC, uid ASC";
+	Query queryObject = getSession().createQuery(query);
+	queryObject.setInteger(0, userId);
+	if (lessonId != null) {
+	    queryObject.setLong(1, lessonId);
+	}
+	if (limit != null) {
+	    queryObject.setMaxResults(limit);
+	}
+	if (offset != null) {
+	    queryObject.setFirstResult(offset);
+	}
+	return queryObject.list();
     }
 
-    public void saveEvent(Event event) {
-	getSession().saveOrUpdate(event);
+    @Override
+    public long getPendingNotificationCount(Long lessonId, Integer userId) {
+	String query = EventDAOHibernate.COUNT_PENDING_NOTIFICATIONS;
+	if (lessonId != null) {
+	    query += " AND s.event.eventSessionId = ?";
+	}
+	Query queryObject = getSession().createQuery(query);
+	queryObject.setInteger(0, userId);
+	if (lessonId != null) {
+	    queryObject.setLong(1, lessonId);
+	}
+	return (Long) queryObject.uniqueResult();
     }
 }
\ No newline at end of file
Fisheye: Tag 1.8.18.8 refers to a dead (removed) revision in file `lams_common/src/java/org/lamsfoundation/lams/lesson/lessonApplicationContext.xml'.
Fisheye: No comparison available.  Pass `N' to diff?
Index: lams_tool_larsrc/conf/language/lams/ApplicationResources.properties
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_larsrc/conf/language/lams/ApplicationResources.properties,v
diff -u -r1.35.2.2 -r1.35.2.3
--- lams_tool_larsrc/conf/language/lams/ApplicationResources.properties	7 Apr 2016 13:38:42 -0000	1.35.2.2
+++ lams_tool_larsrc/conf/language/lams/ApplicationResources.properties	21 Apr 2016 09:13:15 -0000	1.35.2.3
@@ -155,9 +155,11 @@
 monitor.summary.th.advancedSettings  =Advanced settings
 monitor.summary.td.addNotebook  =Add a notebook at end of Shared Resources
 monitor.summary.td.notebookInstructions  =Notebook instructions
-label.authoring.advanced.notify.onassigmentsubmit  =Notify instructors when a learner submits a resource
+label.authoring.advanced.notify.onassigmentsubmit  =Email instructors when a learner submits a resource
+label.authoring.advanced.notify.onfileupload =Notify instructors when a learner uploads a file
 event.assigment.submit.subject  =LAMS: A learner submitted an assignment in a Shared Resources tool
 event.assigment.submit.body  =The learner {0} submitted an assignment in a Shared Resources tool.\n\nThis message was sent automatically, following the tool''s advanced settings.
+event.file.upload ={0} uploaded a file \"{1}\"
 error.planner.no.resource.save  =There has to be at least one resource to save.
 error.planner.url.blank  =In resource {0} URL can not be blank.
 error.planner.file.blank  =In resource {0} file can not be blank.
Index: lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/rsrcApplicationContext.xml
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/rsrcApplicationContext.xml,v
diff -u -r1.22.2.6 -r1.22.2.7
--- lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/rsrcApplicationContext.xml	10 Aug 2015 15:24:12 -0000	1.22.2.6
+++ lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/rsrcApplicationContext.xml	21 Apr 2016 09:13:15 -0000	1.22.2.7
@@ -119,7 +119,7 @@
                 <prop key="setItemComplete">PROPAGATION_REQUIRED,-java.lang.Exception</prop>
                 <prop key="setItemVisible">PROPAGATION_REQUIRED,-java.lang.Exception</prop>
                 <prop key="setItemAccess">PROPAGATION_REQUIRED,-java.lang.Exception</prop>
-                <prop key="notifyTeachersOnAssigmentSumbit">PROPAGATION_REQUIRED,-java.lang.Exception</prop>
+                <prop key="notifyTeachers*">PROPAGATION_REQUIRED,-java.lang.Exception</prop>
                 <prop key="copyToolContent">PROPAGATION_REQUIRED,-java.lang.Exception</prop>
                 <prop key="leaveToolSession">PROPAGATION_REQUIRED,-java.lang.Exception</prop>
                 <prop key="finishToolSession">PROPAGATION_REQUIRED,-java.lang.Exception</prop>
Fisheye: Tag 1.1 refers to a dead (removed) revision in file `lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/dbupdates/patch20160421.sql'.
Fisheye: No comparison available.  Pass `N' to diff?
Index: lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/model/Resource.java
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/model/Resource.java,v
diff -u -r1.21.2.1 -r1.21.2.2
--- lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/model/Resource.java	14 Dec 2015 22:59:17 -0000	1.21.2.1
+++ lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/model/Resource.java	21 Apr 2016 09:13:15 -0000	1.21.2.2
@@ -23,19 +23,14 @@
 /* $Id$ */
 package org.lamsfoundation.lams.tool.rsrc.model;
 
-import java.util.ArrayList;
 import java.util.Date;
 import java.util.HashSet;
 import java.util.Iterator;
-import java.util.List;
 import java.util.Set;
 
-import org.apache.commons.lang.StringUtils;
 import org.apache.commons.lang.builder.EqualsBuilder;
 import org.apache.commons.lang.builder.HashCodeBuilder;
 import org.apache.log4j.Logger;
-import org.lamsfoundation.lams.contentrepository.client.IToolContentHandler;
-import org.lamsfoundation.lams.tool.rsrc.util.ResourceToolContentHandler;
 
 /**
  * Resource
@@ -77,6 +72,8 @@
 
     private boolean notifyTeachersOnAssigmentSumbit;
 
+    private boolean notifyTeachersOnFileUpload;
+
     // general infomation
     private Date created;
 
@@ -444,7 +441,6 @@
 
     /**
      * @hibernate.property column="assigment_submit_notify"
-     * @return
      */
     public boolean isNotifyTeachersOnAssigmentSumbit() {
 	return notifyTeachersOnAssigmentSumbit;
@@ -453,4 +449,15 @@
     public void setNotifyTeachersOnAssigmentSumbit(boolean notifyTeachersOnAssigmentSumbit) {
 	this.notifyTeachersOnAssigmentSumbit = notifyTeachersOnAssigmentSumbit;
     }
-}
+
+    /**
+     * @hibernate.property column="file_upload_notify"
+     */
+    public boolean isNotifyTeachersOnFileUpload() {
+	return notifyTeachersOnFileUpload;
+    }
+
+    public void setNotifyTeachersOnFileUpload(boolean notifyTeachersOnFileUpload) {
+	this.notifyTeachersOnFileUpload = notifyTeachersOnFileUpload;
+    }
+}
\ No newline at end of file
Index: lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/service/IResourceService.java
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/service/IResourceService.java,v
diff -u -r1.30.2.2 -r1.30.2.3
--- lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/service/IResourceService.java	24 Feb 2016 12:36:17 -0000	1.30.2.2
+++ lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/service/IResourceService.java	21 Apr 2016 09:13:15 -0000	1.30.2.3
@@ -258,6 +258,9 @@
 
     void notifyTeachersOnAssigmentSumbit(Long sessionId, ResourceUser resourceUser);
 
+    void notifyTeachersOnFileUpload(Long toolContentId, Long toolSessionId, String sessionMapId, String userName,
+	    Long itemUid, String fileName);
+
     /**
      * Returns whether activity is grouped and therefore it is expected more than one tool session.
      * 
Index: lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/service/ResourceServiceImpl.java
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/service/ResourceServiceImpl.java,v
diff -u -r1.77.2.10 -r1.77.2.11
--- lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/service/ResourceServiceImpl.java	7 Apr 2016 13:38:42 -0000	1.77.2.10
+++ lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/service/ResourceServiceImpl.java	21 Apr 2016 09:13:15 -0000	1.77.2.11
@@ -68,6 +68,7 @@
 import org.lamsfoundation.lams.notebook.service.ICoreNotebookService;
 import org.lamsfoundation.lams.rest.RestTags;
 import org.lamsfoundation.lams.rest.ToolRestManager;
+import org.lamsfoundation.lams.tool.ToolAccessMode;
 import org.lamsfoundation.lams.tool.ToolContentManager;
 import org.lamsfoundation.lams.tool.ToolOutput;
 import org.lamsfoundation.lams.tool.ToolOutputDefinition;
@@ -85,6 +86,7 @@
 import org.lamsfoundation.lams.tool.rsrc.dto.ResourceItemDTO;
 import org.lamsfoundation.lams.tool.rsrc.dto.SessionDTO;
 import org.lamsfoundation.lams.tool.rsrc.dto.VisitLogDTO;
+import org.lamsfoundation.lams.tool.rsrc.ims.ImscpApplicationException;
 import org.lamsfoundation.lams.tool.rsrc.ims.SimpleContentPackageConverter;
 import org.lamsfoundation.lams.tool.rsrc.model.Resource;
 import org.lamsfoundation.lams.tool.rsrc.model.ResourceItem;
@@ -100,9 +102,11 @@
 import org.lamsfoundation.lams.usermanagement.service.IUserManagementService;
 import org.lamsfoundation.lams.util.JsonUtil;
 import org.lamsfoundation.lams.util.MessageService;
+import org.lamsfoundation.lams.util.WebUtil;
 import org.lamsfoundation.lams.util.audit.IAuditService;
 import org.lamsfoundation.lams.util.zipfile.ZipFileUtil;
 import org.lamsfoundation.lams.util.zipfile.ZipFileUtilException;
+import org.lamsfoundation.lams.web.util.AttributeNames;
 
 /**
  * @author Dapeng.Ni
@@ -563,6 +567,23 @@
 	eventNotificationService.notifyLessonMonitors(sessionId, message, false);
     }
 
+    @Override
+    public void notifyTeachersOnFileUpload(Long toolContentId, Long toolSessionId, String sessionMapId, String userName,
+	    Long itemUid, String fileName) {
+	String eventName = new StringBuilder("resources_file_upload_").append(toolContentId).append("_")
+		.append(System.currentTimeMillis()).toString();
+	String url = new StringBuilder("<a href='").append(WebUtil.getBaseServerURL())
+		.append("/lams/tool/larsrc11/reviewItem.do?").append(ResourceConstants.ATTR_SESSION_MAP_ID).append("=")
+		.append(sessionMapId).append("&").append(AttributeNames.ATTR_MODE).append("=")
+		.append(ToolAccessMode.TEACHER.toString()).append("&").append(ResourceConstants.ATTR_TOOL_SESSION_ID)
+		.append("=").append(toolSessionId).append("&").append(ResourceConstants.ATTR_RESOURCE_ITEM_UID)
+		.append("=").append(itemUid).append("'>")
+		.append(getLocalisedMessage("event.file.upload", new Object[] { userName, fileName })).append("</a>")
+		.toString();
+	eventNotificationService.createLessonEvent(IEventNotificationService.LESSON_MONITORS_SCOPE, eventName,
+		toolContentId, null, url, true, IEventNotificationService.DELIVERY_METHOD_NOTIFICATION);
+    }
+
     // *****************************************************************************
     // private methods
     // *****************************************************************************
@@ -1144,6 +1165,8 @@
 	resource.setMiniViewResourceNumber(JsonUtil.opt(toolContentJSON, "minViewResourceNumber", 0));
 	resource.setNotifyTeachersOnAssigmentSumbit(
 		JsonUtil.opt(toolContentJSON, "notifyTeachersOnAssigmentSubmit", Boolean.FALSE));
+	resource.setNotifyTeachersOnAssigmentSumbit(
+		JsonUtil.opt(toolContentJSON, "notifyTeachersOnFileUpload", Boolean.FALSE));
 	resource.setReflectOnActivity(JsonUtil.opt(toolContentJSON, RestTags.REFLECT_ON_ACTIVITY, Boolean.FALSE));
 	resource.setReflectInstructions(JsonUtil.opt(toolContentJSON, RestTags.REFLECT_INSTRUCTIONS, (String) null));
 	resource.setRunAuto(JsonUtil.opt(toolContentJSON, "runAuto", Boolean.FALSE));
Index: lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/web/action/LearningAction.java
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/web/action/LearningAction.java,v
diff -u -r1.42 -r1.42.2.1
--- lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/web/action/LearningAction.java	21 Jul 2014 18:19:18 -0000	1.42
+++ lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/web/action/LearningAction.java	21 Apr 2016 09:13:15 -0000	1.42.2.1
@@ -48,8 +48,6 @@
 import org.apache.struts.action.ActionMapping;
 import org.apache.struts.action.ActionMessage;
 import org.apache.struts.action.ActionMessages;
-import org.lamsfoundation.lams.events.DeliveryMethodMail;
-import org.lamsfoundation.lams.events.IEventNotificationService;
 import org.lamsfoundation.lams.learning.web.bean.ActivityPositionDTO;
 import org.lamsfoundation.lams.learning.web.util.LearningWebUtil;
 import org.lamsfoundation.lams.notebook.model.NotebookEntry;
@@ -66,7 +64,6 @@
 import org.lamsfoundation.lams.tool.rsrc.util.ResourceItemComparator;
 import org.lamsfoundation.lams.tool.rsrc.web.form.ReflectionForm;
 import org.lamsfoundation.lams.tool.rsrc.web.form.ResourceItemForm;
-import org.lamsfoundation.lams.usermanagement.User;
 import org.lamsfoundation.lams.usermanagement.dto.UserDTO;
 import org.lamsfoundation.lams.util.FileUtil;
 import org.lamsfoundation.lams.util.FileValidatorUtil;
@@ -86,6 +83,8 @@
 
     private static Logger log = Logger.getLogger(LearningAction.class);
 
+    private static IResourceService resourceService;
+
     @Override
     public ActionForward execute(ActionMapping mapping, ActionForm form, HttpServletRequest request,
 	    HttpServletResponse response) throws IOException, ServletException {
@@ -162,7 +161,7 @@
 	// get back the resource and item list and display them on page
 	IResourceService service = getResourceService();
 	ResourceUser resourceUser = null;
-	if (mode != null && mode.isTeacher()) {
+	if ((mode != null) && mode.isTeacher()) {
 	    // monitoring mode - user is specified in URL
 	    // resourceUser may be null if the user was force completed.
 	    resourceUser = getSpecifiedUser(service, sessionId,
@@ -177,14 +176,14 @@
 	resource = service.getResourceBySessionId(sessionId);
 
 	// check whehter finish lock is on/off
-	boolean lock = resource.getLockWhenFinished() && resourceUser != null && resourceUser.isSessionFinished();
+	boolean lock = resource.getLockWhenFinished() && (resourceUser != null) && resourceUser.isSessionFinished();
 
 	// check whether there is only one resource item and run auto flag is true or not.
 	boolean runAuto = false;
 	int itemsNumber = 0;
 	if (resource.getResourceItems() != null) {
 	    itemsNumber = resource.getResourceItems().size();
-	    if (resource.isRunAuto() && itemsNumber == 1) {
+	    if (resource.isRunAuto() && (itemsNumber == 1)) {
 		ResourceItem item = (ResourceItem) resource.getResourceItems().iterator().next();
 		// only visible item can be run auto.
 		if (!item.isHide()) {
@@ -209,7 +208,8 @@
 	sessionMap.put(ResourceConstants.ATTR_RESOURCE_INSTRUCTION, resource.getInstructions());
 	sessionMap.put(ResourceConstants.ATTR_FINISH_LOCK, lock);
 	sessionMap.put(ResourceConstants.ATTR_LOCK_ON_FINISH, resource.getLockWhenFinished());
-	sessionMap.put(ResourceConstants.ATTR_USER_FINISHED, resourceUser != null && resourceUser.isSessionFinished());
+	sessionMap.put(ResourceConstants.ATTR_USER_FINISHED,
+		(resourceUser != null) && resourceUser.isSessionFinished());
 
 	sessionMap.put(AttributeNames.PARAM_TOOL_SESSION_ID, sessionId);
 	sessionMap.put(AttributeNames.ATTR_MODE, mode);
@@ -360,7 +360,7 @@
 		return mapping.findForward(ResourceConstants.ERROR);
 	    }
 	    item.setOpenUrlNewWindow(itemForm.isOpenUrlNewWindow());
-	    
+
 	} else if (type == ResourceConstants.RESOURCE_TYPE_URL) {
 	    item.setUrl(itemForm.getUrl());
 	    item.setOpenUrlNewWindow(itemForm.isOpenUrlNewWindow());
@@ -392,7 +392,13 @@
 	if (resource.isNotifyTeachersOnAssigmentSumbit()) {
 	    service.notifyTeachersOnAssigmentSumbit(sessionId, resourceUser);
 	}
-	
+
+	if (resource.isNotifyTeachersOnFileUpload() && (type == ResourceConstants.RESOURCE_TYPE_FILE)) {
+	    service.notifyTeachersOnFileUpload(resource.getContentId(), sessionId, sessionMapID,
+		    resourceUser.getFirstName() + " " + resourceUser.getLastName(), item.getUid(),
+		    itemForm.getFile().getFileName());
+	}
+
 	return mapping.findForward(ResourceConstants.SUCCESS);
     }
 
@@ -490,8 +496,8 @@
 	// if current user view less than reqired view count number, then just return error message.
 	if (miniViewFlag > 0) {
 	    ActionErrors errors = new ActionErrors();
-	    errors.add(ActionMessages.GLOBAL_MESSAGE, new ActionMessage("lable.learning.minimum.view.number.less",
-		    miniViewFlag));
+	    errors.add(ActionMessages.GLOBAL_MESSAGE,
+		    new ActionMessage("lable.learning.minimum.view.number.less", miniViewFlag));
 	    this.addErrors(request, errors);
 	    return false;
 	}
@@ -500,9 +506,12 @@
     }
 
     private IResourceService getResourceService() {
-	WebApplicationContext wac = WebApplicationContextUtils.getRequiredWebApplicationContext(getServlet()
-		.getServletContext());
-	return (IResourceService) wac.getBean(ResourceConstants.RESOURCE_SERVICE);
+	if (LearningAction.resourceService == null) {
+	    WebApplicationContext wac = WebApplicationContextUtils
+		    .getRequiredWebApplicationContext(getServlet().getServletContext());
+	    LearningAction.resourceService = (IResourceService) wac.getBean(ResourceConstants.RESOURCE_SERVICE);
+	}
+	return LearningAction.resourceService;
     }
 
     /**
@@ -616,11 +625,11 @@
 	// if(StringUtils.isBlank(itemForm.getDescription()))
 	// errors.add(ActionMessages.GLOBAL_MESSAGE,new ActionMessage(ResourceConstants.ERROR_MSG_DESC_BLANK));
 	// }
-	if (itemForm.getItemType() == ResourceConstants.RESOURCE_TYPE_WEBSITE
-		|| itemForm.getItemType() == ResourceConstants.RESOURCE_TYPE_LEARNING_OBJECT
-		|| itemForm.getItemType() == ResourceConstants.RESOURCE_TYPE_FILE) {
+	if ((itemForm.getItemType() == ResourceConstants.RESOURCE_TYPE_WEBSITE)
+		|| (itemForm.getItemType() == ResourceConstants.RESOURCE_TYPE_LEARNING_OBJECT)
+		|| (itemForm.getItemType() == ResourceConstants.RESOURCE_TYPE_FILE)) {
 
-	    if (itemForm.getFile() != null && FileUtil.isExecutableFile(itemForm.getFile().getFileName())) {
+	    if ((itemForm.getFile() != null) && FileUtil.isExecutableFile(itemForm.getFile().getFileName())) {
 		ActionMessage msg = new ActionMessage("error.attachment.executable");
 		errors.add(ActionMessages.GLOBAL_MESSAGE, msg);
 	    }
@@ -630,7 +639,7 @@
 
 	    // for edit validate: file already exist
 	    if (!itemForm.isHasFile()
-		    && (itemForm.getFile() == null || StringUtils.isEmpty(itemForm.getFile().getFileName()))) {
+		    && ((itemForm.getFile() == null) || StringUtils.isEmpty(itemForm.getFile().getFileName()))) {
 		errors.add(ActionMessages.GLOBAL_MESSAGE, new ActionMessage(ResourceConstants.ERROR_MSG_FILE_BLANK));
 	    }
 	}
@@ -666,5 +675,4 @@
 	    }
 	}
     }
-
-}
+}
\ No newline at end of file
Index: lams_tool_larsrc/web/pages/authoring/advance.jsp
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_larsrc/web/pages/authoring/advance.jsp,v
diff -u -r1.14 -r1.14.14.1
--- lams_tool_larsrc/web/pages/authoring/advance.jsp	3 Sep 2008 02:11:20 -0000	1.14
+++ lams_tool_larsrc/web/pages/authoring/advance.jsp	21 Apr 2016 09:13:14 -0000	1.14.14.1
@@ -73,6 +73,15 @@
 </p>
 
 <p>
+	<html:checkbox property="resource.notifyTeachersOnFileUpload"
+		styleClass="noBorder" styleId="notifyTeachersOnFileUpload">
+	</html:checkbox>
+	<label for="notifyTeachersOnFileUpload">
+		<fmt:message key="label.authoring.advanced.notify.onfileupload" />
+	</label>
+</p>
+
+<p>
 	<html:checkbox property="resource.reflectOnActivity"
 		styleClass="noBorder" styleId="reflectOn">
 	</html:checkbox>
Index: lams_tool_larsrc/web/pages/monitoring/advanceoptions.jsp
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_larsrc/web/pages/monitoring/advanceoptions.jsp,v
diff -u -r1.2 -r1.2.2.1
--- lams_tool_larsrc/web/pages/monitoring/advanceoptions.jsp	3 Apr 2014 17:23:49 -0000	1.2
+++ lams_tool_larsrc/web/pages/monitoring/advanceoptions.jsp	21 Apr 2016 09:13:14 -0000	1.2.2.1
@@ -107,6 +107,22 @@
 	</tr>
 	<tr>
 		<td>
+			<fmt:message key="label.authoring.advanced.notify.onfileupload" />
+		</td>
+		
+		<td>
+			<c:choose>
+				<c:when test="${sessionMap.resource.notifyTeachersOnFileUpload == true}">
+					<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="monitor.summary.td.addNotebook" />
 		</td>