Index: lams_tool_vote/src/java/org/lamsfoundation/lams/tool/vote/dao/IVoteUsrAttemptDAO.java =================================================================== diff -u -rc3e129b94254eed94888271d74007c1604be1521 -rba561d8505e4944008b79b226b564128cae92f28 --- lams_tool_vote/src/java/org/lamsfoundation/lams/tool/vote/dao/IVoteUsrAttemptDAO.java (.../IVoteUsrAttemptDAO.java) (revision c3e129b94254eed94888271d74007c1604be1521) +++ lams_tool_vote/src/java/org/lamsfoundation/lams/tool/vote/dao/IVoteUsrAttemptDAO.java (.../IVoteUsrAttemptDAO.java) (revision ba561d8505e4944008b79b226b564128cae92f28) @@ -47,7 +47,7 @@ Set getUserEntries(final Long voteContentUid); - Set getSessionUserEntriesSet(final Long voteSessionUid); + List getSessionOpenTextUserEntries(final Long voteSessionUid); VoteUsrAttempt getAttemptByUID(Long uid); @@ -69,7 +69,7 @@ Set getAttemptsForUserAndSession(final Long queUsrId, final Long sessionUid); - Set getAttemptsForUserAndSessionUseOpenAnswer(final Long queUsrId, final Long sessionUid); + List getAttemptsForUserAndSessionUseOpenAnswer(final Long queUsrId, final Long sessionUid); List getAttemptsForUserAndQuestionContent(final Long queUsrId, final Long voteQueContentId); Index: lams_tool_vote/src/java/org/lamsfoundation/lams/tool/vote/dao/hibernate/VoteUsrAttemptDAO.java =================================================================== diff -u -rc3e129b94254eed94888271d74007c1604be1521 -rba561d8505e4944008b79b226b564128cae92f28 --- lams_tool_vote/src/java/org/lamsfoundation/lams/tool/vote/dao/hibernate/VoteUsrAttemptDAO.java (.../VoteUsrAttemptDAO.java) (revision c3e129b94254eed94888271d74007c1604be1521) +++ lams_tool_vote/src/java/org/lamsfoundation/lams/tool/vote/dao/hibernate/VoteUsrAttemptDAO.java (.../VoteUsrAttemptDAO.java) (revision ba561d8505e4944008b79b226b564128cae92f28) @@ -66,8 +66,8 @@ private static final String LOAD_USER_ENTRY_RECORDS = "from voteUsrAttempt in class VoteUsrAttempt where voteUsrAttempt.userEntry=:userEntry and voteUsrAttempt.voteQueContent.uid=1 and voteUsrAttempt.voteQueUsr.voteSession.voteContent.uid=:voteContentUid"; - private static final String LOAD_ENTRIES_BY_SESSION_UID = "select att from VoteUsrAttempt att, VoteQueUsr user, VoteSession ses where " - + "att.voteQueUsr=user and user.voteSession=ses and ses.uid=:voteSessionUid"; + private static final String LOAD_OPEN_TEXT_ENTRIES_BY_SESSION_UID = "select att from VoteUsrAttempt att, VoteQueUsr user, VoteSession ses where " + + "att.voteQueUsr=user and user.voteSession=ses and ses.uid=:voteSessionUid and att.userEntry is not null and att.userEntry <> \'\'"; private static final String COUNT_ENTRIES_BY_SESSION_ID = "select count(*) from VoteUsrAttempt att, VoteQueUsr user, VoteSession ses where " + "att.voteQueUsr=user and user.voteSession=ses and ses.uid=:voteSessionUid"; @@ -123,13 +123,9 @@ } @Override - public Set getSessionUserEntriesSet(final Long voteSessionUid) { - List list = getSession().createQuery(VoteUsrAttemptDAO.LOAD_ENTRIES_BY_SESSION_UID) + public List getSessionOpenTextUserEntries(final Long voteSessionUid) { + return (List) getSession().createQuery(VoteUsrAttemptDAO.LOAD_OPEN_TEXT_ENTRIES_BY_SESSION_UID) .setLong("voteSessionUid", voteSessionUid).list(); - - Set sessionUserEntries = new HashSet(); - sessionUserEntries.addAll(list); - return sessionUserEntries; } @Override @@ -226,35 +222,13 @@ } @Override - public Set getAttemptsForUserAndSessionUseOpenAnswer(final Long queUsrId, final Long sessionUid) { + public List getAttemptsForUserAndSessionUseOpenAnswer(final Long queUsrId, final Long sessionUid) { - List list = getSession().createQuery(VoteUsrAttemptDAO.LOAD_ATTEMPT_FOR_USER_AND_SESSION) + return getSession().createQuery(VoteUsrAttemptDAO.LOAD_ATTEMPT_FOR_USER_AND_SESSION) .setLong("queUsrId", queUsrId.longValue()).setLong("sessionUid", sessionUid.longValue()).list(); - String openAnswer = ""; - Set userEntries = new HashSet(); - if ((list != null) && (list.size() > 0)) { - Iterator listIterator = list.iterator(); - while (listIterator.hasNext()) { - VoteUsrAttempt attempt = listIterator.next(); + } - Long questionUid = attempt.getVoteQueContent().getUid(); - if (!questionUid.toString().equals("1")) { - userEntries.add(attempt.getVoteQueContent().getQuestion()); - } else { - // this is a user entered vote - if (attempt.getUserEntry().length() > 0) { - openAnswer = attempt.getUserEntry(); - // adding openAnswer to userEntries - userEntries.add(openAnswer); - } - - } - } - } - return userEntries; - } - @Override public int getSessionEntriesCount(final Long voteSessionUid) { List result = getSession().createQuery(VoteUsrAttemptDAO.COUNT_ENTRIES_BY_SESSION_ID) Index: lams_tool_vote/src/java/org/lamsfoundation/lams/tool/vote/service/VoteServicePOJO.java =================================================================== diff -u -rc3e129b94254eed94888271d74007c1604be1521 -rba561d8505e4944008b79b226b564128cae92f28 --- lams_tool_vote/src/java/org/lamsfoundation/lams/tool/vote/service/VoteServicePOJO.java (.../VoteServicePOJO.java) (revision c3e129b94254eed94888271d74007c1604be1521) +++ lams_tool_vote/src/java/org/lamsfoundation/lams/tool/vote/service/VoteServicePOJO.java (.../VoteServicePOJO.java) (revision ba561d8505e4944008b79b226b564128cae92f28) @@ -26,6 +26,7 @@ import java.util.Collection; import java.util.Date; import java.util.HashMap; +import java.util.HashSet; import java.util.Hashtable; import java.util.Iterator; import java.util.LinkedList; @@ -241,10 +242,13 @@ VoteContent voteContent = this.getVoteContent(toolContentID); int entriesCount = 0; - Set userEntries = null; + List userEntries = null; if (toolSessionUid != null) { entriesCount = voteUsrAttemptDAO.getSessionEntriesCount(toolSessionUid); - userEntries = voteUsrAttemptDAO.getSessionUserEntriesSet(toolSessionUid); + if ( voteContent.isAllowText() ) + userEntries = voteUsrAttemptDAO.getSessionOpenTextUserEntries(toolSessionUid); + else + userEntries = new ArrayList(0); } Long mapIndex = 1L; @@ -729,7 +733,31 @@ @Override public Set getAttemptsForUserAndSessionUseOpenAnswer(final Long userUid, final Long sessionUid) { - return voteUsrAttemptDAO.getAttemptsForUserAndSessionUseOpenAnswer(userUid, sessionUid); + List list = voteUsrAttemptDAO.getAttemptsForUserAndSessionUseOpenAnswer(userUid, sessionUid); + + String openAnswer = ""; + Set userEntries = new HashSet(); + if ((list != null) && (list.size() > 0)) { + Iterator listIterator = list.iterator(); + while (listIterator.hasNext()) { + VoteUsrAttempt attempt = listIterator.next(); + + Long questionUid = attempt.getVoteQueContent().getUid(); + if (!questionUid.toString().equals("1")) { + userEntries.add(attempt.getVoteQueContent().getQuestion()); + } else { + // this is a user entered vote + if (attempt.getUserEntry().length() > 0) { + if ( attempt.isVisible() ) + userEntries.add(attempt.getUserEntry()); + else + userEntries.add(getMessageService().getMessage("label.hidden")); + } + + } + } + } + return userEntries; } @Override Index: lams_tool_vote/web/export/ExportContent.jsp =================================================================== diff -u -r43437f9c52f60b4fe545162b3ed72e6ddf911777 -rba561d8505e4944008b79b226b564128cae92f28 --- lams_tool_vote/web/export/ExportContent.jsp (.../ExportContent.jsp) (revision 43437f9c52f60b4fe545162b3ed72e6ddf911777) +++ lams_tool_vote/web/export/ExportContent.jsp (.../ExportContent.jsp) (revision ba561d8505e4944008b79b226b564128cae92f28) @@ -44,7 +44,10 @@ - + + + + Index: lams_tool_vote/web/learning/AllNominations.jsp =================================================================== diff -u -r8a78ed9d4fddacbbbe874998c08d2138ebd0917b -rba561d8505e4944008b79b226b564128cae92f28 --- lams_tool_vote/web/learning/AllNominations.jsp (.../AllNominations.jsp) (revision 8a78ed9d4fddacbbbe874998c08d2138ebd0917b) +++ lams_tool_vote/web/learning/AllNominations.jsp (.../AllNominations.jsp) (revision ba561d8505e4944008b79b226b564128cae92f28) @@ -165,11 +165,16 @@ - -
- -
-
+
+ + + + + + () + + +
 
Index: lams_tool_vote/web/learning/RevisitedDisplay.jsp =================================================================== diff -u -r8a78ed9d4fddacbbbe874998c08d2138ebd0917b -rba561d8505e4944008b79b226b564128cae92f28 --- lams_tool_vote/web/learning/RevisitedDisplay.jsp (.../RevisitedDisplay.jsp) (revision 8a78ed9d4fddacbbbe874998c08d2138ebd0917b) +++ lams_tool_vote/web/learning/RevisitedDisplay.jsp (.../RevisitedDisplay.jsp) (revision ba561d8505e4944008b79b226b564128cae92f28) @@ -90,7 +90,14 @@
+ + + + + () + +
 
Index: lams_tool_vote/web/learning/mobile/AllNominations.jsp =================================================================== diff -u -r8a78ed9d4fddacbbbe874998c08d2138ebd0917b -rba561d8505e4944008b79b226b564128cae92f28 --- lams_tool_vote/web/learning/mobile/AllNominations.jsp (.../AllNominations.jsp) (revision 8a78ed9d4fddacbbbe874998c08d2138ebd0917b) +++ lams_tool_vote/web/learning/mobile/AllNominations.jsp (.../AllNominations.jsp) (revision ba561d8505e4944008b79b226b564128cae92f28) @@ -199,11 +199,16 @@ - -
- -
-
+
+ + + + + + () + + +
 
Index: lams_tool_vote/web/learning/mobile/RevisitedDisplay.jsp =================================================================== diff -u -r8a78ed9d4fddacbbbe874998c08d2138ebd0917b -rba561d8505e4944008b79b226b564128cae92f28 --- lams_tool_vote/web/learning/mobile/RevisitedDisplay.jsp (.../RevisitedDisplay.jsp) (revision 8a78ed9d4fddacbbbe874998c08d2138ebd0917b) +++ lams_tool_vote/web/learning/mobile/RevisitedDisplay.jsp (.../RevisitedDisplay.jsp) (revision ba561d8505e4944008b79b226b564128cae92f28) @@ -125,7 +125,14 @@
+ + + + + () + +