lams / lams_tool_nb / src

java

Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 12
Last Commit:
Commits this week: 0
Total Lines of Code (LoC): 5,386
Net change in LoC this week: 0

Commit Activity

Commits By Day In Week
52 week commits volume
Commits By Day In Week
Commits by day
Commits By Day In Week
Commits by hour
Commits By Hour In Day
Commit calendar
 

Most active committers (90 days)

loading
loading
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Thursday 11 Aug 2005
1:16 pm

mtruong

changed the copyFormValues..(), the deleteAttachment method now doesnt call this method, otherwise all fields will be blank.

instead of using hardcoded default content id, makes the call to getDefaultToolContentId

Options
    • -116
    • +156
    ./org/lamsfoundation/lams/tool/noticeboard/web/NbAuthoringV2Action.java
1:15 pm

mtruong

added comments to code

Options
    • -0
    • +36
    ./org/lamsfoundation/lams/tool/noticeboard/web/NbAuthoringAction.java
    • -2
    • +10
    ./org/lamsfoundation/lams/tool/noticeboard/web/NbAuthoringForm.java
    • -1
    • +24
    ./org/lamsfoundation/lams/tool/noticeboard/web/NbAuthoringStarterAction.java
    • -5
    • +5
    ./org/lamsfoundation/lams/tool/noticeboard/web/NbExportAction.java
    • -1
    • +1
    ./org/lamsfoundation/lams/tool/noticeboard/web/NbExportForm.java
    • -0
    • +8
    ./org/lamsfoundation/lams/tool/noticeboard/web/NbLearnerAction.java
    • -0
    • +14
    ./org/lamsfoundation/lams/tool/noticeboard/web/NbLearnerStarterAction.java
    • -7
    • +31
    ./org/lamsfoundation/lams/tool/noticeboard/web/NbMonitoringAction.java
Wednesday 10 Aug 2005
2:53 pm

mtruong

implemented to call to leaveToolSession, indicating that the learner has finished the activity

Options
    • -9
    • +20
    ./org/lamsfoundation/lams/tool/noticeboard/web/NbLearnerAction.java
2:53 pm

mtruong

instead of using the constant for default content id, the service method is called

Options
    • -1
    • +11
    ./org/lamsfoundation/lams/tool/noticeboard/web/NbAuthoringStarterAction.java
2:52 pm

mtruong

removed the TODO

Options
    • -13
    • +26
    ./org/lamsfoundation/lams/tool/noticeboard/util/NbWebUtil.java
2:52 pm

mtruong

added author tag

Options
    • -0
    • +1
    ./org/lamsfoundation/lams/tool/noticeboard/util/NbToolContentHandler.java
    • -0
    • +2
    ./org/lamsfoundation/lams/tool/noticeboard/web/CustomStrutsExceptionHandler.java
2:52 pm

mtruong

methods to return the toolSessionManager and ToolContentManager interface

Options
    • -0
    • +27
    ./org/lamsfoundation/lams/tool/noticeboard/service/NoticeboardServiceProxy.java
2:51 pm

mtruong

implementation of the new function get default content id

in method copyToolSession and copyToolCOntent this method is used instead of the constant

Options
    • -33
    • +52
    ./org/lamsfoundation/lams/tool/noticeboard/service/NoticeboardServicePOJO.java
2:51 pm

mtruong

added comments in the code

and also new method called getDefaultToolContentId which retrieves the default content id instead of hard coding it

Options
    • -1
    • +8
    ./org/lamsfoundation/lams/tool/noticeboard/service/INoticeboardService.java
2:50 pm

mtruong

use the JNDI tool-ds instead od jdbc connection

also added learnerService and ToolService

Options
    • -15
    • +8
    ./org/lamsfoundation/lams/tool/noticeboard/applicationContext.xml
2:49 pm

mtruong

added comments in the code

Options
    • -35
    • +20
    ./org/lamsfoundation/lams/tool/noticeboard/NoticeboardAttachment.java
    • -4
    • +3
    ./org/lamsfoundation/lams/tool/noticeboard/NoticeboardContent.java
    • -1
    • +10
    ./org/lamsfoundation/lams/tool/noticeboard/NoticeboardSession.java
    • -2
    • +10
    ./org/lamsfoundation/lams/tool/noticeboard/NoticeboardUser.java
    • -2
    • +59
    ./org/lamsfoundation/lams/tool/noticeboard/dao/INoticeboardAttachmentDAO.java
    • -3
    • +62
    ./org/lamsfoundation/lams/tool/noticeboard/dao/INoticeboardContentDAO.java
    • -9
    • +94
    ./org/lamsfoundation/lams/tool/noticeboard/dao/INoticeboardSessionDAO.java
    • -1
    • +47
    ./org/lamsfoundation/lams/tool/noticeboard/dao/INoticeboardUserDAO.java
    • -5
    • +10
    ./org/lamsfoundation/lams/tool/noticeboard/dao/hibernate/NoticeboardAttachmentDAO.java
    • -62
    • +17
    ./org/lamsfoundation/lams/tool/noticeboard/dao/hibernate/NoticeboardContentDAO.java
    • -75
    • +23
    ./org/lamsfoundation/lams/tool/noticeboard/dao/hibernate/NoticeboardSessionDAO.java
    • -43
    • +9
    ./org/lamsfoundation/lams/tool/noticeboard/dao/hibernate/NoticeboardUserDAO.java
12:27 pm

mtruong

added AUTHOR_PAGEV2

AND TOOL_SIGNATURE constants

Options
    • -4
    • +9
    ./org/lamsfoundation/lams/tool/noticeboard/NoticeboardConstants.java
Tuesday 09 Aug 2005
1:14 pm

mtruong

used for the authoring page using DHTML

Options
    • -0
    • +535
    ./org/lamsfoundation/lams/tool/noticeboard/web/NbAuthoringV2Action.java
  2. … 10 more files in changeset.
Monday 08 Aug 2005
8:30 am

mtruong

removed BASIC and ADVANCED constants because they are not used.

Options
    • -4
    • +0
    ./org/lamsfoundation/lams/tool/noticeboard/NoticeboardConstants.java
8:29 am

mtruong

removed the FORM constant as it isnt used in this Action class

Options
    • -2
    • +1
    ./org/lamsfoundation/lams/tool/noticeboard/web/NbMonitoringAction.java
8:29 am

mtruong

a new instance of NBMonitoringForm is instantiated in the execute method to ensure that we are using a new form each time this starter action is called

Options
    • -2
    • +5
    ./org/lamsfoundation/lams/tool/noticeboard/web/NbMonitoringStarterAction.java
8:27 am

mtruong

instead placing the toolCotnentId in session scope, it is now stored in the session scope form bean

Options
    • -12
    • +17
    ./org/lamsfoundation/lams/tool/noticeboard/web/NbMonitoringAction.java
8:26 am

mtruong

modified the validate method, the validate method doesnt check whether the title or content fields are empty, because the fckeditor buts in <br> tags even when there is no content

Options
    • -16
    • +8
    ./org/lamsfoundation/lams/tool/noticeboard/web/NbAuthoringForm.java
8:25 am

mtruong

in cleanMonitoringSession, the session scope variable to hold the toolCotnentId is removed, as the session scope formbean now handles this instead

Options
    • -1
    • +4
    ./org/lamsfoundation/lams/tool/noticeboard/util/NbWebUtil.java
8:23 am

mtruong

*** empty log message ***

Options
    • -0
    • +1
    ./org/lamsfoundation/lams/tool/noticeboard/web/NbMonitoringForm.java
  2. … 1 more file in changeset.
Friday 05 Aug 2005
1:03 pm

mtruong

increased the filename field in attachment table to be varchar(255) instead of varchar(50).

Options
    • -1
    • +1
    ./org/lamsfoundation/lams/tool/noticeboard/NoticeboardAttachment.java
  2. … 4 more files in changeset.
12:41 pm

mtruong

*** empty log message ***

Options
    • -6
    • +2
    ./org/lamsfoundation/lams/tool/noticeboard/web/NbMonitoringStarterAction.java
12:40 pm

mtruong

added a new bean property attachments

Options
    • -4
    • +28
    ./org/lamsfoundation/lams/tool/noticeboard/web/NbMonitoringForm.java
12:40 pm

mtruong

modified the struts action "instructions": added code to initialise the attachment map, used to view the attachments that have been made with this specific tool content id

Options
    • -0
    • +17
    ./org/lamsfoundation/lams/tool/noticeboard/web/NbMonitoringAction.java
9:34 am

mtruong

*** empty log message ***

Options
    • -0
    • +1
    ./org/lamsfoundation/lams/tool/noticeboard/web/ApplicationResources.properties
  2. … 1 more file in changeset.
9:33 am

mtruong

fixed the reset method (previously it wasnt defined correctly, and so it wasnt being called by the controller)

the validate method now checks the filesize of the uploaded files.

Options
    • -3
    • +34
    ./org/lamsfoundation/lams/tool/noticeboard/web/NbAuthoringForm.java
9:32 am

mtruong

made some changes to the way the authoring screen is shown.

instead of having 3 separate jsp pages with 3 separate paths, combined these 3 authoring pages into the one page authorContent.jsp

works the same way as the monitoring environment.

removed the action forward nb_basic, nb_advanced and nb_instructions. the only forward path is now authoringContent

Options
    • -14
    • +19
    ./org/lamsfoundation/lams/tool/noticeboard/web/NbAuthoringAction.java
    • -4
    • +3
    ./org/lamsfoundation/lams/tool/noticeboard/web/NbAuthoringStarterAction.java
  3. … 3 more files in changeset.
9:30 am

mtruong

removed the constants BASIC_PAGE, ADVANCED_PAGE AND INSTRUCTIONS_PAGE, since the three pages are now referenced through authorContent.jsp instead

Options
    • -4
    • +6
    ./org/lamsfoundation/lams/tool/noticeboard/NoticeboardConstants.java
Thursday 04 Aug 2005
2:07 pm

mtruong

when dealing with the same files being uploaded, the previous code, updated teh db with the new uuid, however, it wasnt updating instead new entries were being made. Now the code is changed so that if the file already exists (ie. has been uploaded already) then it will delete previously file from content repository, save it into content repository, delete the entry in the db, and create a new entry with the new uuid

Options
    • -6
    • +6
    ./org/lamsfoundation/lams/tool/noticeboard/web/NbAuthoringAction.java
Wednesday 03 Aug 2005
11:11 am

mtruong

the struts exception handles nullPointerExceptions (using customstrutshandler)

Options
    • -0
    • +4
    ./org/lamsfoundation/lams/tool/noticeboard/web/NbMonitoringAction.java
    • -0
    • +5
    ./org/lamsfoundation/lams/tool/noticeboard/web/NbMonitoringStarterAction.java