quite a few changes here. removed unused import declarations the learner environment doesnt expect a toolContentId to be passed via url. the tool contentId/toolSessionId relationship is assumed to have been establish when the core made the call to createToolSessions() if no toolSession exist in the table when the learner reaches the activity, the user will see an error screen. if user in this tool session isnt found, thena new entry is created.
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
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
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
