|
History
| Log In
|
|
|
|
How about logging start / open an assessment as well?
Also for consistency with other tools, an event for creating a new assessment, and revising an assessment.
commit -m "
Sending sakai/sam/samigo-app/src/java/org/sakaiproject/tool/assessment/ui/listener/author/AuthorAssessmentListener.java Sending sakai/sam/samigo-app/src/java/org/sakaiproject/tool/assessment/ui/listener/samlite/AssessmentListener.java Transmitting file data ... Committed revision 34610. commit -m "
Sending sakai/sam/samigo-app/src/java/org/sakaiproject/tool/assessment/ui/listener/author/ItemAddListener.java Sending sakai/sam/samigo-app/src/java/org/sakaiproject/tool/assessment/ui/listener/author/SaveAssessmentSettings.java Sending sakai/sam/samigo-app/src/java/org/sakaiproject/tool/assessment/ui/listener/author/SavePartListener.java Sending sakai/sam/samigo-app/src/java/org/sakaiproject/tool/assessment/ui/listener/delivery/DeliveryActionListener.java Transmitting file data ... Committed revision 34612. To verify this, you will need to look into db (sakai_event) to see if these events are logged correctly. The Samigo events should start with "sam" and followed by the event you just fire. For example, if you just publish an assessment, you should see a record in sakai_event table as:
EVENT_DATE EVENT REF (the date/time you publish the assessment) sam.publish assessmentId=xxx, publishedAssessmentId=xxx Looks like my format of the above example doesn't work. Let me redo in this wayL
EVENT_DATE: (the date/time you publish the assessment) EVENT: sam.publish REF: assessmentId=xxx, publishedAssessmentId=xxx Huong,
Here is the query you want to use: SELECT * FROM sakai_event s Tested on Stanford nightly: http://coursework-dev4.stanford.edu:8090/portal
Action Expected Log Current Log 1. Creating a New Assessment sam.createAssessment sam.createAssessment 2. Add Question sam.addItem sam.addItem 3. Preview an Assessment nothing sam.takeAssessment 4. Revising an Assessment sam.editAssessment sam.addItem 5. EditSetting sam.editSetting sam.editSetting 6. Publish an Assessment sam.publish sam.publish 7. Start Taking Assessment sam.takeAssessment sam.takeAssessment 8. Submitting an Assessment sam.submit sam.submit 1,2, 5, 6, 7 and 8 are currently correct logged. 3 and 4 are needed to be fixed Good catch! Thanks. I will take out the log for preview.
For item 4, because I don't want to put in logic to distinguish add or edit (this will affect performance), I will do following changes: 2. Add Question sam.addOrEditItem 4. Revising an Assessment sam.addOrEditItem 5. EditSetting sam.addOrEditSetting Please let me know if this is fine with you. Thanks. Look good! let me re write the expect values:
Action| Log Creating a New Assessment | sam.createAssessment Revising an Assessment, add/edit question, add/edit Part | sam.addOrEditItem EditSetting|sam.addOrEditSetting Publish An Assessment | sam. publish Starting taking An assessment | sam.takeAssessment Submitting an Assessment | sam.submit No log for Prviewing An Assessment. Do we log Assessment types: - Creating An assessment type - Editing Assessment Type? final draft for action and log for Samigo
Action | Log Creating a New Assessment | sam.assessment.create Revising an Assessment: add/edit question, add/edit Part | sam.assessment.revise EditSetting | sam.setting.edit Publish An Assessment | sam.assessment.publish Starting taking An assessment | sam.assessment.take Submitting an Assessment | sam.assessment.submit commit -m "
Sending sakai/sam/samigo-app/src/java/org/sakaiproject/tool/assessment/ui/bean/author/ItemAuthorBean.java Sending sakai/sam/samigo-app/src/java/org/sakaiproject/tool/assessment/ui/bean/delivery/DeliveryBean.java Sending sakai/sam/samigo-app/src/java/org/sakaiproject/tool/assessment/ui/bean/delivery/ItemContentsBean.java Sending sakai/sam/samigo-app/src/java/org/sakaiproject/tool/assessment/ui/listener/author/AuthorAssessmentListener.java Sending sakai/sam/samigo-app/src/java/org/sakaiproject/tool/assessment/ui/listener/author/ItemAddListener.java Sending sakai/sam/samigo-app/src/java/org/sakaiproject/tool/assessment/ui/listener/author/PublishAssessmentListener.java Sending sakai/sam/samigo-app/src/java/org/sakaiproject/tool/assessment/ui/listener/author/RemovePartListener.java Sending sakai/sam/samigo-app/src/java/org/sakaiproject/tool/assessment/ui/listener/author/SaveAssessmentSettings.java Sending sakai/sam/samigo-app/src/java/org/sakaiproject/tool/assessment/ui/listener/author/SavePartListener.java Sending sakai/sam/samigo-app/src/java/org/sakaiproject/tool/assessment/ui/listener/delivery/DeliveryActionListener.java Sending sakai/sam/samigo-app/src/java/org/sakaiproject/tool/assessment/ui/listener/samlite/AssessmentListener.java Transmitting file data ... Committed revision 36268. Verified on Stanford nightly, event logs show correctly as desired.
Because this bug cannot be verified in Official QA instance (need database access to verify the fix), our QA verified it in our local instance. However, she is too efficient that she closed this bug before the fix can get merged into branch :) Therefore, I re-open this bug and request to be merged to the next tag.
OOps, sorry! So, I change it to resolve until it is merged to the branch.
|
|||||||||||||||||||||||||||||||||||||||||||||||||||
 |
|
Task
Closed
Major
Add event logging for Samigo user operations
SAK-11137" C:/sakai/sam/samigo-app/pom.xml C:/sakai/sam/samigo-app/src/java/org/sakaiproject/tool/assessment/ui/bean/delivery/DeliveryBean.java C:/sakai/sam/samigo-app/src/java/org/sakaiproject/tool/assessment/ui/bean/util/EmailBean.java C:/sakai/sam/samigo-app/src/java/org/sakaiproject/tool/assessment/ui/listener/author/PublishAssessmentListener.javaSending sakai/sam/samigo-app/pom.xml
Sending sakai/sam/samigo-app/src/java/org/sakaiproject/tool/assessment/ui/bean/delivery/DeliveryBean.java
Sending sakai/sam/samigo-app/src/java/org/sakaiproject/tool/assessment/ui/bean/util/EmailBean.java
Sending sakai/sam/samigo-app/src/java/org/sakaiproject/tool/assessment/ui/listener/author/PublishAssessmentListener.java
Transmitting file data ...
Committed revision 34600.