Index: lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/dao/AssessmentUserDAO.java =================================================================== diff -u -rfeb6062a5128a445b1fa0095b8c9d6cf232af90f -r5c770157545dea3dbc72e22a1dd1107c56cf22eb --- lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/dao/AssessmentUserDAO.java (.../AssessmentUserDAO.java) (revision feb6062a5128a445b1fa0095b8c9d6cf232af90f) +++ lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/dao/AssessmentUserDAO.java (.../AssessmentUserDAO.java) (revision 5c770157545dea3dbc72e22a1dd1107c56cf22eb) @@ -43,7 +43,7 @@ int getCountUsersBySession(Long sessionId, String searchString); List getPagedUsersBySessionAndQuestion(Long sessionId, Long questionUid, int page, int size, - String sortBy, String sortOrder, String searchString); + String sortBy, String sortOrder, String searchString, IUserManagementService userManagementService); List getRawUserMarksBySession(Long sessionId); Index: lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/dao/hibernate/AssessmentUserDAOHibernate.java =================================================================== diff -u -rfeb6062a5128a445b1fa0095b8c9d6cf232af90f -r5c770157545dea3dbc72e22a1dd1107c56cf22eb --- lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/dao/hibernate/AssessmentUserDAOHibernate.java (.../AssessmentUserDAOHibernate.java) (revision feb6062a5128a445b1fa0095b8c9d6cf232af90f) +++ lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/dao/hibernate/AssessmentUserDAOHibernate.java (.../AssessmentUserDAOHibernate.java) (revision 5c770157545dea3dbc72e22a1dd1107c56cf22eb) @@ -209,8 +209,9 @@ } - private static String LOAD_USERS_ORDERED_BY_SESSION_QUESTION = "SELECT DISTINCT question_result.uid, user.last_name, user.first_name, user.login_name, question_result.mark" - + " FROM tl_laasse10_user user" + " INNER JOIN tl_laasse10_session session" + private static String LOAD_USERS_ORDERED_BY_SESSION_QUESTION_SELECT = "SELECT DISTINCT question_result.uid, user.last_name, user.first_name, user.login_name, question_result.mark"; + private static String LOAD_USERS_ORDERED_BY_SESSION_QUESTION_FROM = " FROM tl_laasse10_user user"; + private static String LOAD_USERS_ORDERED_BY_SESSION_QUESTION_JOIN = " INNER JOIN tl_laasse10_session session" + " ON user.session_uid=session.uid" + " LEFT OUTER JOIN tl_laasse10_assessment_result result " + " ON result.user_uid = user.uid" @@ -226,9 +227,15 @@ @SuppressWarnings("unchecked") @Override public List getPagedUsersBySessionAndQuestion(Long sessionId, Long questionUid, int page, - int size, String sortBy, String sortOrder, String searchString) { + int size, String sortBy, String sortOrder, String searchString, IUserManagementService userManagementService) { - StringBuilder bldr = new StringBuilder(LOAD_USERS_ORDERED_BY_SESSION_QUESTION); + String[] portraitStrings = userManagementService.getPortraitSQL("user.user_id"); + + StringBuilder bldr = new StringBuilder(LOAD_USERS_ORDERED_BY_SESSION_QUESTION_SELECT) + .append(portraitStrings[0]) + .append(LOAD_USERS_ORDERED_BY_SESSION_QUESTION_FROM) + .append(portraitStrings[1]) + .append(LOAD_USERS_ORDERED_BY_SESSION_QUESTION_JOIN); if ( "grade".equalsIgnoreCase(sortBy) ) bldr.append(LOAD_USERS_ORDERED_ORDER_BY_RESULT); else @@ -254,6 +261,7 @@ String lastName = (String) element[2]; String login = (String) element[3]; float grade = element[4] == null ? 0 : ((Number) element[4]).floatValue(); + Long portraitId = element[5] == null ? null : ((Number) element[5]).longValue(); AssessmentUserDTO userDto = new AssessmentUserDTO(); userDto.setQuestionResultUid(questionResultUid); @@ -262,6 +270,7 @@ userDto.setLastName(lastName); userDto.setLogin(login); userDto.setGrade(grade); + userDto.setPortraitId(portraitId); userDtos.add(userDto); } Index: lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/service/AssessmentServiceImpl.java =================================================================== diff -u -rf899620dd37f54eb375d29a5b15a7793c5dc3bb4 -r5c770157545dea3dbc72e22a1dd1107c56cf22eb --- lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/service/AssessmentServiceImpl.java (.../AssessmentServiceImpl.java) (revision f899620dd37f54eb375d29a5b15a7793c5dc3bb4) +++ lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/service/AssessmentServiceImpl.java (.../AssessmentServiceImpl.java) (revision 5c770157545dea3dbc72e22a1dd1107c56cf22eb) @@ -348,7 +348,7 @@ public List getPagedUsersBySessionAndQuestion(Long sessionId, Long questionUid, int page, int size, String sortBy, String sortOrder, String searchString) { return assessmentUserDao.getPagedUsersBySessionAndQuestion(sessionId, questionUid, page, size, sortBy, - sortOrder, searchString); + sortOrder, searchString, userManagementService); } @Override Index: lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/web/action/MonitoringAction.java =================================================================== diff -u -rfeb6062a5128a445b1fa0095b8c9d6cf232af90f -r5c770157545dea3dbc72e22a1dd1107c56cf22eb --- lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/web/action/MonitoringAction.java (.../MonitoringAction.java) (revision feb6062a5128a445b1fa0095b8c9d6cf232af90f) +++ lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/web/action/MonitoringAction.java (.../MonitoringAction.java) (revision 5c770157545dea3dbc72e22a1dd1107c56cf22eb) @@ -500,14 +500,15 @@ userData.put(questionResult.getMaxMark()); userData.put(fullName); userData.put(AssessmentEscapeUtils.printResponsesForJqgrid(questionResult)); - // show confidence levels if this feature is turned ON if (assessment.isEnableConfidenceLevels()) { userData.put(questionResult.getConfidenceLevel()); } userData.put(questionResult.getMark()); - + if (userDto.getPortraitId() != null ) + userData.put(userDto.getPortraitId()); + } else { userData.put(""); userData.put(""); Index: lams_tool_assessment/web/pages/monitoring/parts/questionsummary.jsp =================================================================== diff -u -rfeb6062a5128a445b1fa0095b8c9d6cf232af90f -r5c770157545dea3dbc72e22a1dd1107c56cf22eb --- lams_tool_assessment/web/pages/monitoring/parts/questionsummary.jsp (.../questionsummary.jsp) (revision feb6062a5128a445b1fa0095b8c9d6cf232af90f) +++ lams_tool_assessment/web/pages/monitoring/parts/questionsummary.jsp (.../questionsummary.jsp) (revision 5c770157545dea3dbc72e22a1dd1107c56cf22eb) @@ -16,6 +16,7 @@ +