Index: lams_tool_forum/conf/language/ApplicationResources.properties =================================================================== RCS file: /usr/local/cvsroot/lams_tool_forum/conf/language/Attic/ApplicationResources.properties,v diff -u -r1.24 -r1.25 --- lams_tool_forum/conf/language/ApplicationResources.properties 29 Sep 2006 02:10:43 -0000 1.24 +++ lams_tool_forum/conf/language/ApplicationResources.properties 10 Oct 2006 04:19:49 -0000 1.25 @@ -167,3 +167,4 @@ #======= End labels: Exported 158 labels for en AU ===== +label.default.user.name=Default \ No newline at end of file Index: lams_tool_forum/db/sql/create_lams_tool_forum.sql =================================================================== RCS file: /usr/local/cvsroot/lams_tool_forum/db/sql/create_lams_tool_forum.sql,v diff -u -r1.26 -r1.27 --- lams_tool_forum/db/sql/create_lams_tool_forum.sql 4 Sep 2006 02:18:43 -0000 1.26 +++ lams_tool_forum/db/sql/create_lams_tool_forum.sql 10 Oct 2006 04:19:48 -0000 1.27 @@ -115,5 +115,10 @@ alter table tl_lafrum11_tool_session add index FK5A04D7AE131CE31E (forum_uid), add constraint FK5A04D7AE131CE31E foreign key (forum_uid) references tl_lafrum11_forum (uid); -INSERT INTO tl_lafrum11_forum (title,instructions,online_instructions,offline_instructions,content_id,allow_anonym,run_offline,lock_on_finished,content_in_use,define_later,allow_edit,allow_rich_editor, allow_new_topic,allow_upload,maximum_reply, minimum_reply,limited_input_flag,limited_of_chars,reflect_on_activity) VALUES("LAMS Forum","Instruction","Online instruction","Offline instruction",${default_content_id},0,0,0,0,0,1,0,1,1,0,0,1,5000,0); +INSERT INTO tl_lafrum11_forum (uid,title,instructions,online_instructions,offline_instructions,content_id,allow_anonym,run_offline,lock_on_finished,content_in_use,define_later,allow_edit,allow_rich_editor, allow_new_topic,allow_upload,maximum_reply, minimum_reply,limited_input_flag,limited_of_chars,reflect_on_activity) +VALUES(1,"LAMS Forum","Instruction","Online instruction","Offline instruction",${default_content_id},0,0,0,0,0,1,0,1,1,0,0,1,5000,0); + +INSERT INTO `tl_lafrum11_message` (`uid`, `create_date`, `last_reply_date`, `update_date`, `create_by`, `modified_by`, `subject`, `body`, `is_authored`, `is_anonymous`, `forum_session_uid`, `parent_uid`, `forum_uid`, `reply_number`, `hide_flag`, `report_id`) VALUES + (1,NOW(),NOW(),NOW(),null,null,'Topic Title','Topic message',1,0,NULL,NULL,1,0,0,NULL); + SET FOREIGN_KEY_CHECKS=1; \ No newline at end of file Index: lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/dto/MessageDTO.java =================================================================== RCS file: /usr/local/cvsroot/lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/dto/MessageDTO.java,v diff -u -r1.15 -r1.16 --- lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/dto/MessageDTO.java 17 Sep 2006 06:22:31 -0000 1.15 +++ lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/dto/MessageDTO.java 10 Oct 2006 04:19:50 -0000 1.16 @@ -61,7 +61,8 @@ MessageDTO dto = new MessageDTO(); dto.setMessage(msg); - dto.setAuthor(msg.getCreatedBy().getFirstName()+" "+msg.getCreatedBy().getLastName()); + if(msg.getCreatedBy() != null) + dto.setAuthor(msg.getCreatedBy().getFirstName()+" "+msg.getCreatedBy().getLastName()); if(msg.getAttachments() == null || msg.getAttachments().isEmpty()) dto.setHasAttachment(false); else @@ -92,7 +93,8 @@ Message msg = (Message) iter.next(); MessageDTO msgDto = new MessageDTO(); msgDto.setMessage(msg); - msgDto.setAuthor(msg.getCreatedBy().getFirstName()+" "+msg.getCreatedBy().getLastName()); + if(msg.getCreatedBy() != null) + msgDto.setAuthor(msg.getCreatedBy().getFirstName()+" "+msg.getCreatedBy().getLastName()); if(msg.getAttachments() == null || msg.getAttachments().isEmpty()) msgDto.setHasAttachment(false); Index: lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/service/ForumService.java =================================================================== RCS file: /usr/local/cvsroot/lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/service/ForumService.java,v diff -u -r1.79 -r1.80 --- lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/service/ForumService.java 5 Oct 2006 00:54:21 -0000 1.79 +++ lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/service/ForumService.java 10 Oct 2006 04:19:49 -0000 1.80 @@ -215,12 +215,18 @@ Message message = getMessage(messageId); if ( message !=null ) { + Long userId = 0L; + String loginName = "Default"; + if(message.getCreatedBy() == null){ + userId = message.getCreatedBy().getUserId(); + loginName = message.getCreatedBy().getLoginName(); + } if ( hideFlag ) { - auditService.logHideEntry(ForumConstants.TOOL_SIGNATURE, message.getCreatedBy().getUserId(), - message.getCreatedBy().getLoginName(), message.toString()); + auditService.logHideEntry(ForumConstants.TOOL_SIGNATURE, userId, + loginName, message.toString()); } else { - auditService.logShowEntry(ForumConstants.TOOL_SIGNATURE, message.getCreatedBy().getUserId(), - message.getCreatedBy().getLoginName(), message.toString()); + auditService.logShowEntry(ForumConstants.TOOL_SIGNATURE,userId, + loginName, message.toString()); } message.setHideFlag(hideFlag); @@ -363,9 +369,9 @@ while(iter.hasNext()){ msg = (Message) iter.next(); if(OLD_FORUM_STYLE) - map.put(msg.getLastReplyDate(),msg); - else map.put(msg.getCreated(),msg); + else + map.put(msg.getLastReplyDate(),msg); } return MessageDTO.getMessageDTO(new ArrayList(map.values())); @@ -813,10 +819,10 @@ } Forum defaultContent = getDefaultForum(); - //save default content by given ID. + //get default content by given ID. Forum content = new Forum(); content = Forum.newInstance(defaultContent,contentID,forumToolContentHandler); - //save topics in this forum + Set topics = content.getMessages(); if(topics != null){ Iterator iter = topics.iterator(); Index: lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/actions/AuthoringAction.java =================================================================== RCS file: /usr/local/cvsroot/lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/actions/AuthoringAction.java,v diff -u -r1.58 -r1.59 --- lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/actions/AuthoringAction.java 3 Oct 2006 01:38:27 -0000 1.58 +++ lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/actions/AuthoringAction.java 10 Oct 2006 04:19:48 -0000 1.59 @@ -182,7 +182,7 @@ //get back the topic list and display them on page forumService = getForumManager(); - List topics = null; + List topics = null; Forum forum = null; try { forum = forumService.getForumByContentId(contentId); @@ -195,13 +195,32 @@ Iterator iter = forum.getMessages().iterator(); while(iter.hasNext()){ Message topic = (Message) iter.next(); + if(topic.getCreatedBy() == null){ + //get login user (author) + HttpSession ss = SessionManager.getSession(); + //get back login user DTO + UserDTO user = (UserDTO) ss.getAttribute(AttributeNames.USER); + ForumUser fuser = new ForumUser(user,null); + topic.setCreatedBy(fuser); + } map.put(topic.getCreated(),topic); } - topics = new ArrayList(map.values()); + topics = MessageDTO.getMessageDTO(new ArrayList(map.values())); }else topics = null; }else{ topics = forumService.getAuthoredTopics(forum.getUid()); + //failure tolerance: if current contentID is defaultID, the createBy will be null. + for (MessageDTO messageDTO : topics) { + if(StringUtils.isBlank(messageDTO.getAuthor())){ + //get login user (author) + HttpSession ss = SessionManager.getSession(); + //get back login user DTO + UserDTO user = (UserDTO) ss.getAttribute(AttributeNames.USER); + ForumUser fuser = new ForumUser(user,null); + messageDTO.setAuthor(fuser.getFirstName()+" "+fuser.getLastName()); + } + } } //initialize attachmentList List attachmentList = getAttachmentList(sessionMap); @@ -217,7 +236,7 @@ //set back STRUTS component value //init it to avoid null exception in following handling if(topics == null) - topics = new ArrayList(); + topics = new ArrayList(); sessionMap.put(ForumConstants.AUTHORING_TOPICS_LIST, topics); return mapping.findForward("success"); Index: lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/actions/ExportServlet.java =================================================================== RCS file: /usr/local/cvsroot/lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/actions/ExportServlet.java,v diff -u -r1.11 -r1.12 --- lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/actions/ExportServlet.java 29 Sep 2006 06:11:18 -0000 1.11 +++ lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/actions/ExportServlet.java 10 Oct 2006 04:19:48 -0000 1.12 @@ -123,6 +123,7 @@ // get root topic list and its children topics List msgDtoList = getSessionTopicList(toolSessionID, directoryName, forumService); + //set author flag, to decide if display mark of topics.Only author allow see his own mark. setAuthorMark(msgDtoList); ForumToolSession session = forumService.getSessionBySessionId(toolSessionID); Index: lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/actions/LearningAction.java =================================================================== RCS file: /usr/local/cvsroot/lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/actions/LearningAction.java,v diff -u -r1.48 -r1.49 --- lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/actions/LearningAction.java 17 Sep 2006 06:22:30 -0000 1.48 +++ lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/actions/LearningAction.java 10 Oct 2006 04:19:48 -0000 1.49 @@ -612,8 +612,14 @@ setAttachment(messageForm, messagePO); if ( makeAuditEntry ) { + Long userId = 0L; + String loginName = "Default"; + if(message.getCreatedBy() == null){ + userId = message.getCreatedBy().getUserId(); + loginName = message.getCreatedBy().getLoginName(); + } forumService.getAuditService().logChange(ForumConstants.TOOL_SIGNATURE, - messagePO.getCreatedBy().getUserId(), messagePO.getCreatedBy().getLoginName(), + userId,loginName, oldMessageString, messagePO.toString()); } Index: lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/actions/MonitoringAction.java =================================================================== RCS file: /usr/local/cvsroot/lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/actions/MonitoringAction.java,v diff -u -r1.28 -r1.29 --- lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/actions/MonitoringAction.java 29 Sep 2006 01:57:01 -0000 1.28 +++ lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/actions/MonitoringAction.java 10 Oct 2006 04:19:48 -0000 1.29 @@ -285,8 +285,7 @@ sheet.setColumnWidth((short) 0, (short) 5000); HSSFRow row = null; HSSFCell cell; - Iterator iter = getTopicsSortedByAuthor(topicList).values() - .iterator(); + Iterator iter = getTopicsSortedByAuthor(topicList).values().iterator(); Iterator dtoIter; boolean first = true; int idx = 0; @@ -481,7 +480,7 @@ request.setAttribute(ForumConstants.PARAM_UPDATE_MODE, updateMode); String mark = markForm.getMark(); - ActionErrors errors = new ActionErrors(); + ActionMessages errors = new ActionMessages(); if (StringUtils.isBlank(mark)) { ActionMessage error = new ActionMessage("error.valueReqd"); errors.add("report.mark", error); @@ -507,7 +506,7 @@ if(!errors.isEmpty()){ // each back to web page request.setAttribute(ForumConstants.ATTR_TOPIC, MessageDTO.getMessageDTO(msg)); - saveMessages(request, errors); + saveErrors(request, errors); return mapping.getInputForward(); } @@ -803,6 +802,8 @@ forumService = getForumService(); while (iter.hasNext()) { MessageDTO dto = (MessageDTO) iter.next(); + if(dto.getMessage().getIsAuthored()) + continue; dto.getMessage().getReport(); ForumUser user = (ForumUser) dto.getMessage().getCreatedBy().clone(); List list = (List) topicsByUser.get(user); Index: lams_tool_forum/web/common/messages.jsp =================================================================== RCS file: /usr/local/cvsroot/lams_tool_forum/web/common/messages.jsp,v diff -u -r1.4 -r1.5 --- lams_tool_forum/web/common/messages.jsp 29 Sep 2006 01:57:01 -0000 1.4 +++ lams_tool_forum/web/common/messages.jsp 10 Oct 2006 04:19:50 -0000 1.5 @@ -1,6 +1,5 @@ <%-- Error Messages --%> -

 


Index: lams_tool_forum/web/jsps/export/exportportfolio.jsp =================================================================== RCS file: /usr/local/cvsroot/lams_tool_forum/web/jsps/export/exportportfolio.jsp,v diff -u -r1.13 -r1.14 --- lams_tool_forum/web/jsps/export/exportportfolio.jsp 28 Sep 2006 23:30:19 -0000 1.13 +++ lams_tool_forum/web/jsps/export/exportportfolio.jsp 10 Oct 2006 04:19:50 -0000 1.14 @@ -50,7 +50,13 @@ - + + + + + + + ${author} - Index: lams_tool_forum/web/jsps/learning/message/topiclist.jsp =================================================================== RCS file: /usr/local/cvsroot/lams_tool_forum/web/jsps/learning/message/topiclist.jsp,v diff -u -r1.7 -r1.8 --- lams_tool_forum/web/jsps/learning/message/topiclist.jsp 24 Aug 2006 00:05:41 -0000 1.7 +++ lams_tool_forum/web/jsps/learning/message/topiclist.jsp 10 Oct 2006 04:19:50 -0000 1.8 @@ -29,7 +29,13 @@ - + + + + + + + ${author} Index: lams_tool_forum/web/jsps/learning/message/topicview.jsp =================================================================== RCS file: /usr/local/cvsroot/lams_tool_forum/web/jsps/learning/message/topicview.jsp,v diff -u -r1.16 -r1.17 --- lams_tool_forum/web/jsps/learning/message/topicview.jsp 24 Aug 2006 00:05:41 -0000 1.16 +++ lams_tool_forum/web/jsps/learning/message/topicview.jsp 10 Oct 2006 04:19:50 -0000 1.17 @@ -20,7 +20,13 @@ - + + + + + + + ${author} - Index: lams_tool_forum/web/jsps/monitoring/updatemarks.jsp =================================================================== RCS file: /usr/local/cvsroot/lams_tool_forum/web/jsps/monitoring/updatemarks.jsp,v diff -u -r1.15 -r1.16 --- lams_tool_forum/web/jsps/monitoring/updatemarks.jsp 29 Sep 2006 06:11:18 -0000 1.15 +++ lams_tool_forum/web/jsps/monitoring/updatemarks.jsp 10 Oct 2006 04:19:47 -0000 1.16 @@ -32,7 +32,13 @@ - + + + + + + + ${author} Index: lams_tool_forum/web/jsps/monitoring/viewmarks.jsp =================================================================== RCS file: /usr/local/cvsroot/lams_tool_forum/web/jsps/monitoring/viewmarks.jsp,v diff -u -r1.3 -r1.4 --- lams_tool_forum/web/jsps/monitoring/viewmarks.jsp 24 Aug 2006 00:05:40 -0000 1.3 +++ lams_tool_forum/web/jsps/monitoring/viewmarks.jsp 10 Oct 2006 04:19:47 -0000 1.4 @@ -50,7 +50,13 @@ - + + + + + + + ${author} Index: lams_tool_forum/web/jsps/monitoring/message/viewtopic.jsp =================================================================== RCS file: /usr/local/cvsroot/lams_tool_forum/web/jsps/monitoring/message/viewtopic.jsp,v diff -u -r1.2 -r1.3 --- lams_tool_forum/web/jsps/monitoring/message/viewtopic.jsp 24 Aug 2006 00:05:41 -0000 1.2 +++ lams_tool_forum/web/jsps/monitoring/message/viewtopic.jsp 10 Oct 2006 04:19:47 -0000 1.3 @@ -12,7 +12,14 @@ - + + + + + + + ${author} + - Index: lams_tool_forum/web/jsps/monitoring/message/viewtopictree.jsp =================================================================== RCS file: /usr/local/cvsroot/lams_tool_forum/web/jsps/monitoring/message/viewtopictree.jsp,v diff -u -r1.2 -r1.3 --- lams_tool_forum/web/jsps/monitoring/message/viewtopictree.jsp 24 Aug 2006 00:05:41 -0000 1.2 +++ lams_tool_forum/web/jsps/monitoring/message/viewtopictree.jsp 10 Oct 2006 04:19:47 -0000 1.3 @@ -13,7 +13,14 @@ - + + + + + + + ${author} + - Index: lams_tool_larsrc/db/sql/create_lams_tool_rsrc.sql =================================================================== RCS file: /usr/local/cvsroot/lams_tool_larsrc/db/sql/create_lams_tool_rsrc.sql,v diff -u -r1.16 -r1.17 --- lams_tool_larsrc/db/sql/create_lams_tool_rsrc.sql 7 Sep 2006 04:04:34 -0000 1.16 +++ lams_tool_larsrc/db/sql/create_lams_tool_rsrc.sql 10 Oct 2006 04:19:40 -0000 1.17 @@ -111,6 +111,14 @@ -INSERT INTO `tl_larsrc11_resource` (`uid`, `create_date`, `update_date`, `create_by`, `title`, `run_offline`, `lock_on_finished`, `instructions`, `online_instructions`, `offline_instructions`, `content_in_use`, `define_later`, `content_id`, `allow_add_files`, `allow_add_urls`, `mini_view_resource_number`, `allow_auto_run`,`reflect_on_activity`) VALUES - (2,NULL,NULL,NULL,'Shared Resources','0','0','Instruction','Online instruction','Offline instruction',0,0,${default_content_id},1,1,0,0,0); +INSERT INTO `tl_larsrc11_resource` (`uid`, `create_date`, `update_date`, `create_by`, `title`, `run_offline`, `lock_on_finished`, + `instructions`, `online_instructions`, `offline_instructions`, `content_in_use`, `define_later`, `content_id`, `allow_add_files`, + `allow_add_urls`, `mini_view_resource_number`, `allow_auto_run`,`reflect_on_activity`) VALUES + (1,NULL,NULL,NULL,'Shared Resources','0','0','Instruction','Online instruction','Offline instruction',0,0,${default_content_id},0,0,0,0,0); + +INSERT INTO `tl_larsrc11_resource_item` (`uid`, `file_uuid`, `file_version_id`, `description`, `ims_schema`, `init_item`, `organization_xml`, `title`, `url`, `create_by`, `create_date`, `create_by_author`, `is_hide`, `item_type`, `file_type`, `file_name`, `open_url_new_window`, `resource_uid`, `session_uid`) VALUES + (1,NULL,NULL,NULL,NULL,NULL,NULL,'Web Search','http://www.google.com ',null,NOW(),1,0,1,NULL,NULL,0,1,NULL); +INSERT INTO `tl_larsrc11_item_instruction` (`uid`, `description`, `sequence_id`, `item_uid`) VALUES + (1,'Use Google to search on the web ',0,1); + SET FOREIGN_KEY_CHECKS=1; \ No newline at end of file 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.49 -r1.50 --- lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/service/ResourceServiceImpl.java 4 Oct 2006 08:54:40 -0000 1.49 +++ lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/service/ResourceServiceImpl.java 10 Oct 2006 04:19:40 -0000 1.50 @@ -586,12 +586,19 @@ public void setItemVisible(Long itemUid, boolean visible) { ResourceItem item = resourceItemDao.getByUid(itemUid); if ( item != null ) { + //createBy should be null for system default value. + Long userId = 0L; + String loginName = "No user"; + if(item.getCreateBy() != null){ + userId = item.getCreateBy().getUserId(); + loginName = item.getCreateBy().getLoginName(); + } if ( visible ) { - auditService.logShowEntry(ResourceConstants.TOOL_SIGNATURE, item.getCreateBy().getUserId(), - item.getCreateBy().getLoginName(), item.toString()); + auditService.logShowEntry(ResourceConstants.TOOL_SIGNATURE,userId, + loginName, item.toString()); } else { - auditService.logHideEntry(ResourceConstants.TOOL_SIGNATURE, item.getCreateBy().getUserId(), - item.getCreateBy().getLoginName(), item.toString()); + auditService.logHideEntry(ResourceConstants.TOOL_SIGNATURE, userId, + loginName, item.toString()); } item.setHide(!visible); resourceItemDao.saveObject(item); Index: lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/web/action/AuthoringAction.java =================================================================== RCS file: /usr/local/cvsroot/lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/web/action/AuthoringAction.java,v diff -u -r1.41 -r1.42 --- lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/web/action/AuthoringAction.java 3 Oct 2006 07:18:17 -0000 1.41 +++ lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/web/action/AuthoringAction.java 10 Oct 2006 04:19:39 -0000 1.42 @@ -369,7 +369,7 @@ // get back the resource and item list and display them on page IResourceService service = getResourceService(); - List items = null; + List items = null; Resource resource = null; ResourceForm resourceForm = (ResourceForm)form; @@ -388,7 +388,7 @@ if(resource == null){ resource = service.getDefaultContent(contentId); if(resource.getResourceItems() != null){ - items = new ArrayList(resource.getResourceItems()); + items = new ArrayList(resource.getResourceItems()); }else items = null; }else @@ -407,8 +407,22 @@ //init it to avoid null exception in following handling if(items == null) - items = new ArrayList(); - + items = new ArrayList(); + else{ + ResourceUser resourceUser = null; + //handle system default question: createBy is null, now set it to current user + for (ResourceItem item : items) { + if(item.getCreateBy() == null){ + if(resourceUser == null){ + //get back login user DTO + HttpSession ss = SessionManager.getSession(); + UserDTO user = (UserDTO) ss.getAttribute(AttributeNames.USER); + resourceUser = new ResourceUser(user,resource); + } + item.setCreateBy(resourceUser); + } + } + } //init resource item list SortedSet resourceItemList = getResourceItemList(sessionMap); resourceItemList.clear(); 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.22 -r1.23 --- lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/web/action/LearningAction.java 17 Sep 2006 06:25:37 -0000 1.22 +++ lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/web/action/LearningAction.java 10 Oct 2006 04:19:39 -0000 1.23 @@ -214,7 +214,8 @@ for(ResourceItem item : items){ //becuase in webpage will use this login name. Here is just //initial it to avoid session close error in proxy object. - item.getCreateBy().getLoginName(); + if(item.getCreateBy() != null) + item.getCreateBy().getLoginName(); if(!item.isHide()){ resourceItemList.add(item); } Index: lams_tool_larsrc/web/pages/learning/learning.jsp =================================================================== RCS file: /usr/local/cvsroot/lams_tool_larsrc/web/pages/learning/learning.jsp,v diff -u -r1.30 -r1.31 --- lams_tool_larsrc/web/pages/learning/learning.jsp 6 Oct 2006 02:48:19 -0000 1.30 +++ lams_tool_larsrc/web/pages/learning/learning.jsp 10 Oct 2006 04:19:40 -0000 1.31 @@ -95,7 +95,7 @@ ${item.title} - + [${item.createBy.loginName}] Index: lams_tool_survey/db/sql/create_lams_tool_survey.sql =================================================================== RCS file: /usr/local/cvsroot/lams_tool_survey/db/sql/create_lams_tool_survey.sql,v diff -u -r1.6 -r1.7 --- lams_tool_survey/db/sql/create_lams_tool_survey.sql 26 Sep 2006 07:12:41 -0000 1.6 +++ lams_tool_survey/db/sql/create_lams_tool_survey.sql 10 Oct 2006 04:19:51 -0000 1.7 @@ -110,6 +110,26 @@ INSERT INTO `tl_lasurv11_survey` (`uid`, `create_date`, `update_date`, `create_by`, `title`, `run_offline`, `lock_on_finished`, `instructions`, `online_instructions`, `offline_instructions`, `content_in_use`, `define_later`, `content_id`,`show_questions_on_one_page`,`reflect_on_activity`) VALUES - (2,NULL,NULL,NULL,'Survey','0','0','Instruction','Online instruction','Offline instruction',0,0,${default_content_id},1,0); + (1,NULL,NULL,NULL,'Survey','0','0','Instruction','Online instruction','Offline instruction',0,0,${default_content_id},1,0); +INSERT INTO `tl_lasurv11_question` (`uid`, `sequence_id`, `description`, `create_by`, `create_date`, `question_type`, `append_text`, `optional`, `allow_multiple_answer`, `survey_uid`) VALUES + (1,1,'Sample Multiple choice - only one response allowed?',null,NOW(),1,0,0,0,1); +INSERT INTO `tl_lasurv11_question` (`uid`, `sequence_id`, `description`, `create_by`, `create_date`, `question_type`, `append_text`, `optional`, `allow_multiple_answer`, `survey_uid`) VALUES + (2,2,'Sample Multiple choice - multiple response allowed?',null,NOW(),2,0,0,1,1); +INSERT INTO `tl_lasurv11_question` (`uid`, `sequence_id`, `description`, `create_by`, `create_date`, `question_type`, `append_text`, `optional`, `allow_multiple_answer`, `survey_uid`) VALUES + (3,3,'Sample Free text question?',null,NOW(),3,0,0,0,1); + + +INSERT INTO `tl_lasurv11_option` (`uid`, `description`, `sequence_id`, `question_uid`) VALUES + (1,'Option 1',0,1); +INSERT INTO `tl_lasurv11_option` (`uid`, `description`, `sequence_id`, `question_uid`) VALUES + (2,'Option 2',1,1); +INSERT INTO `tl_lasurv11_option` (`uid`, `description`, `sequence_id`, `question_uid`) VALUES + (3,'Option 3',2,1); +INSERT INTO `tl_lasurv11_option` (`uid`, `description`, `sequence_id`, `question_uid`) VALUES + (4,'Option 1',0,2); +INSERT INTO `tl_lasurv11_option` (`uid`, `description`, `sequence_id`, `question_uid`) VALUES + (5,'Option 2',1,2); +INSERT INTO `tl_lasurv11_option` (`uid`, `description`, `sequence_id`, `question_uid`) VALUES + (6,'Option 3',2,2); SET FOREIGN_KEY_CHECKS=1; Index: lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/web/action/AuthoringAction.java =================================================================== RCS file: /usr/local/cvsroot/lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/web/action/AuthoringAction.java,v diff -u -r1.11 -r1.12 --- lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/web/action/AuthoringAction.java 6 Oct 2006 01:10:08 -0000 1.11 +++ lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/web/action/AuthoringAction.java 10 Oct 2006 04:19:51 -0000 1.12 @@ -374,7 +374,7 @@ // get back the survey and item list and display them on page ISurveyService service = getSurveyService(); - List questions = null; + List questions = null; Survey survey = null; SurveyForm surveyForm = (SurveyForm)form; @@ -392,11 +392,11 @@ if(survey == null){ survey = service.getDefaultContent(contentId); if(survey.getQuestions() != null){ - questions = new ArrayList(survey.getQuestions()); + questions = new ArrayList(survey.getQuestions()); }else questions = null; }else - questions = new ArrayList(survey.getQuestions()); + questions = new ArrayList(survey.getQuestions()); surveyForm.setSurvey(survey); @@ -408,7 +408,21 @@ //init it to avoid null exception in following handling if(questions == null) questions = new ArrayList(); - + else{ + SurveyUser surveyUser = null; + //handle system default question: createBy is null, now set it to current user + for (SurveyQuestion question : questions) { + if(question.getCreateBy() == null){ + if(surveyUser == null){ + //get back login user DTO + HttpSession ss = SessionManager.getSession(); + UserDTO user = (UserDTO) ss.getAttribute(AttributeNames.USER); + surveyUser = new SurveyUser(user,survey); + } + question.setCreateBy(surveyUser); + } + } + } //init survey item list SortedSet surveyItemList = getSurveyItemList(sessionMap); surveyItemList.clear(); Index: lams_tool_survey/web/pages/monitoring/listanswers.jsp =================================================================== RCS file: /usr/local/cvsroot/lams_tool_survey/web/pages/monitoring/listanswers.jsp,v diff -u -r1.3 -r1.4 --- lams_tool_survey/web/pages/monitoring/listanswers.jsp 28 Sep 2006 23:38:35 -0000 1.3 +++ lams_tool_survey/web/pages/monitoring/listanswers.jsp 10 Oct 2006 04:19:51 -0000 1.4 @@ -24,7 +24,7 @@ <%-- --%> - + <%-- @@ -33,7 +33,7 @@ --%> - -
+ <%= SurveyConstants.OPTION_SHORT_HEADER %>${optStatus.count} @@ -43,7 +43,7 @@
+