Index: lams_monitoring/web/tblmonitor/tblmonitor.jsp =================================================================== diff -u -r2ce1ca7a4995e15743c5800fdd789868b0236836 -rc606e9ef96a924f5389866c6fceb1c7fee959032 --- lams_monitoring/web/tblmonitor/tblmonitor.jsp (.../tblmonitor.jsp) (revision 2ce1ca7a4995e15743c5800fdd789868b0236836) +++ lams_monitoring/web/tblmonitor/tblmonitor.jsp (.../tblmonitor.jsp) (revision c606e9ef96a924f5389866c6fceb1c7fee959032) @@ -119,16 +119,7 @@ $("#tblmonitor-tab-content").load( url, - options, - //callback function fired on complete - function() { - //use jqeury toggle instead of bootstrap collapse - $(".burning-question-title").on('click', function () { - var div = $("#collapse-" + $(this).data("itemuid")); - div.toggleClass("in"); - $(this).toggleClass("collapsed"); - }); - } + options ); } Index: lams_tool_scratchie/src/java/org/lamsfoundation/lams/tool/scratchie/web/controller/MonitoringController.java =================================================================== diff -u -r3eb8dcbf308a6b082c30d57f51e7000ebbe1c184 -rc606e9ef96a924f5389866c6fceb1c7fee959032 --- lams_tool_scratchie/src/java/org/lamsfoundation/lams/tool/scratchie/web/controller/MonitoringController.java (.../MonitoringController.java) (revision 3eb8dcbf308a6b082c30d57f51e7000ebbe1c184) +++ lams_tool_scratchie/src/java/org/lamsfoundation/lams/tool/scratchie/web/controller/MonitoringController.java (.../MonitoringController.java) (revision c606e9ef96a924f5389866c6fceb1c7fee959032) @@ -24,6 +24,8 @@ package org.lamsfoundation.lams.tool.scratchie.web.controller; import java.io.IOException; +import java.util.Collections; +import java.util.Comparator; import java.util.Date; import java.util.LinkedList; import java.util.List; @@ -46,6 +48,7 @@ import org.lamsfoundation.lams.qb.dto.QbStatsActivityDTO; import org.lamsfoundation.lams.qb.service.IQbService; import org.lamsfoundation.lams.tool.scratchie.ScratchieConstants; +import org.lamsfoundation.lams.tool.scratchie.dto.BurningQuestionDTO; import org.lamsfoundation.lams.tool.scratchie.dto.BurningQuestionItemDTO; import org.lamsfoundation.lams.tool.scratchie.dto.GroupSummary; import org.lamsfoundation.lams.tool.scratchie.dto.LeaderResultsDTO; @@ -58,6 +61,7 @@ import org.lamsfoundation.lams.tool.scratchie.service.IScratchieService; import org.lamsfoundation.lams.tool.scratchie.util.ScratchieItemComparator; import org.lamsfoundation.lams.usermanagement.dto.UserDTO; +import org.lamsfoundation.lams.util.AlphanumComparator; import org.lamsfoundation.lams.util.Configuration; import org.lamsfoundation.lams.util.ConfigurationKeys; import org.lamsfoundation.lams.util.DateUtil; @@ -137,7 +141,8 @@ if (scratchie.isBurningQuestionsEnabled()) { List burningQuestionItemDtos = scratchieService.getBurningQuestionDtos(scratchie, null, true, true); - sessionMap.put(ScratchieConstants.ATTR_BURNING_QUESTION_ITEM_DTOS, burningQuestionItemDtos); + MonitoringController.setUpBurningQuestions(burningQuestionItemDtos); + request.setAttribute(ScratchieConstants.ATTR_BURNING_QUESTION_ITEM_DTOS, burningQuestionItemDtos); } // Create reflectList if reflection is enabled. @@ -326,4 +331,28 @@ request.setAttribute(ScratchieConstants.ATTR_SESSION_MAP_ID, sessionMapID); return (SessionMap) request.getSession().getAttribute(sessionMapID); } + + static void setUpBurningQuestions(List burningQuestionItemDtos) { + //unescape previously escaped session names + for (BurningQuestionItemDTO burningQuestionItemDto : burningQuestionItemDtos) { + List burningQuestionDtos = burningQuestionItemDto.getBurningQuestionDtos(); + + for (BurningQuestionDTO burningQuestionDto : burningQuestionItemDto.getBurningQuestionDtos()) { + + String escapedBurningQuestion = StringEscapeUtils + .unescapeJavaScript(burningQuestionDto.getEscapedBurningQuestion()); + burningQuestionDto.setEscapedBurningQuestion(escapedBurningQuestion); + + String sessionName = StringEscapeUtils.unescapeJavaScript(burningQuestionDto.getSessionName()); + burningQuestionDto.setSessionName(sessionName); + } + + Collections.sort(burningQuestionDtos, new Comparator() { + @Override + public int compare(BurningQuestionDTO o1, BurningQuestionDTO o2) { + return new AlphanumComparator().compare(o1.getSessionName(), o2.getSessionName()); + } + }); + } + } } Index: lams_tool_scratchie/src/java/org/lamsfoundation/lams/tool/scratchie/web/controller/TblMonitorController.java =================================================================== diff -u -re743566518bd5c53d6574987565cfb8581a92c66 -rc606e9ef96a924f5389866c6fceb1c7fee959032 --- lams_tool_scratchie/src/java/org/lamsfoundation/lams/tool/scratchie/web/controller/TblMonitorController.java (.../TblMonitorController.java) (revision e743566518bd5c53d6574987565cfb8581a92c66) +++ lams_tool_scratchie/src/java/org/lamsfoundation/lams/tool/scratchie/web/controller/TblMonitorController.java (.../TblMonitorController.java) (revision c606e9ef96a924f5389866c6fceb1c7fee959032) @@ -25,8 +25,6 @@ import java.io.IOException; import java.util.Collection; -import java.util.Collections; -import java.util.Comparator; import java.util.LinkedList; import java.util.List; import java.util.Map; @@ -38,10 +36,8 @@ import javax.servlet.http.HttpServletRequest; import javax.servlet.http.HttpServletResponse; -import org.apache.commons.lang.StringEscapeUtils; import org.apache.log4j.Logger; import org.lamsfoundation.lams.tool.scratchie.ScratchieConstants; -import org.lamsfoundation.lams.tool.scratchie.dto.BurningQuestionDTO; import org.lamsfoundation.lams.tool.scratchie.dto.BurningQuestionItemDTO; import org.lamsfoundation.lams.tool.scratchie.dto.GroupSummary; import org.lamsfoundation.lams.tool.scratchie.dto.ScratchieItemDTO; @@ -51,7 +47,6 @@ import org.lamsfoundation.lams.tool.scratchie.model.ScratchieUser; import org.lamsfoundation.lams.tool.scratchie.service.IScratchieService; import org.lamsfoundation.lams.tool.scratchie.util.ScratchieItemComparator; -import org.lamsfoundation.lams.util.AlphanumComparator; import org.lamsfoundation.lams.util.FileUtil; import org.lamsfoundation.lams.util.WebUtil; import org.lamsfoundation.lams.util.excel.ExcelSheet; @@ -185,36 +180,18 @@ List burningQuestionItemDtos = scratchieService.getBurningQuestionDtos(scratchie, null, true, true); - //unescape previously escaped session names - for (BurningQuestionItemDTO burningQuestionItemDto : burningQuestionItemDtos) { - List burningQuestionDtos = burningQuestionItemDto.getBurningQuestionDtos(); + MonitoringController.setUpBurningQuestions(burningQuestionItemDtos); - for (BurningQuestionDTO burningQuestionDto : burningQuestionItemDto.getBurningQuestionDtos()) { - - String escapedBurningQuestion = StringEscapeUtils - .unescapeJavaScript(burningQuestionDto.getEscapedBurningQuestion()); - burningQuestionDto.setEscapedBurningQuestion(escapedBurningQuestion); - - String sessionName = StringEscapeUtils.unescapeJavaScript(burningQuestionDto.getSessionName()); - burningQuestionDto.setSessionName(sessionName); - } - - Collections.sort(burningQuestionDtos, new Comparator() { - @Override - public int compare(BurningQuestionDTO o1, BurningQuestionDTO o2) { - return new AlphanumComparator().compare(o1.getSessionName(), o2.getSessionName()); - } - }); - } - request.setAttribute(ScratchieConstants.ATTR_BURNING_QUESTION_ITEM_DTOS, burningQuestionItemDtos); ScratchieConfigItem hideTitles = scratchieService.getConfigItem(ScratchieConfigItem.KEY_HIDE_TITLES); request.setAttribute(ScratchieConfigItem.KEY_HIDE_TITLES, Boolean.valueOf(hideTitles.getConfigValue())); } - return "pages/tblmonitoring/burningQuestions"; + request.setAttribute("isTbl", true); + + return "pages/monitoring/parts/burningQuestions"; } /** Index: lams_tool_scratchie/web/pages/monitoring/monitoring.jsp =================================================================== diff -u -r094d165248768ab19f66d75424a6a13ec968b829 -rc606e9ef96a924f5389866c6fceb1c7fee959032 --- lams_tool_scratchie/web/pages/monitoring/monitoring.jsp (.../monitoring.jsp) (revision 094d165248768ab19f66d75424a6a13ec968b829) +++ lams_tool_scratchie/web/pages/monitoring/monitoring.jsp (.../monitoring.jsp) (revision c606e9ef96a924f5389866c6fceb1c7fee959032) @@ -10,6 +10,8 @@ + + Index: lams_tool_scratchie/web/pages/monitoring/parts/burningQuestions.jsp =================================================================== diff -u --- lams_tool_scratchie/web/pages/monitoring/parts/burningQuestions.jsp (revision 0) +++ lams_tool_scratchie/web/pages/monitoring/parts/burningQuestions.jsp (revision c606e9ef96a924f5389866c6fceb1c7fee959032) @@ -0,0 +1,172 @@ +<%@ include file="/common/taglibs.jsp"%> + + + +
+
+ + +

+ +

+
+
+
+ + + +
+
+ + + + + +
+
+

+ + + + + + + + + Q${i.index+1}) + + + + + + + + + + ${itemTitle} + + + + + ${burningQsCount} + + +

+
+ + +
+
+ + +
+
+ + + + + + + + +
+ + + + + + + + + + +
+ + + ${burningQuestionDto.escapedBurningQuestion} + + ${burningQuestionDto.likeCount}   +
+
+
+
+
+ +
+
+ +
+
Index: lams_tool_scratchie/web/pages/monitoring/summary.jsp =================================================================== diff -u -r3b0f9138b73adccc73267089b5430769ffbf490a -rc606e9ef96a924f5389866c6fceb1c7fee959032 --- lams_tool_scratchie/web/pages/monitoring/summary.jsp (.../summary.jsp) (revision 3b0f9138b73adccc73267089b5430769ffbf490a) +++ lams_tool_scratchie/web/pages/monitoring/summary.jsp (.../summary.jsp) (revision c606e9ef96a924f5389866c6fceb1c7fee959032) @@ -32,6 +32,10 @@ -webkit-border-radius:0; -khtml-border-radius:0; } + + #collapseBurning .panel { + margin-bottom: 20px; + }