Importing archives via web services is not possible without minor change to SamigoAssessmentHandler.java
GENERAL
TESTING
GENERAL
TESTING
Description
Attachments
1
cloned as
is depended on by
is duplicated by
Activity
Show:
Steve Swinsburg January 28, 2013 at 5:34 PM
Please set fix version.
Hui Tsao September 21, 2011 at 11:14 AM
Sam,
I made the change to SamigoPoolHandler.java too. Because it's not easy for me to test it (I make sure the build is successful), can you please verify for me? Thanks.
Hui Tsao September 20, 2011 at 4:30 PM
Sure. I will work on it right after I finish my current task.
Sam Ottenhoff September 20, 2011 at 4:27 PM
It looks like this may be required in ./samigo-archive/sam-handlers/src/java/org/sakaiproject/importer/impl/handlers/SamigoPoolHandler.java also
Sam Ottenhoff September 20, 2011 at 4:18 PM
Hi Karen,
This should be a pretty straightforward patch. It reliably uses siteId instead of attempting to use the ToolManager.getCurrentPlacement().getContext() as the site id. The ToolManager.getCurrentPlacement().getContext() is unreliable because an assessment could be imported via web services instead of via the browser.
--Sam
Several changes were made to allow web services to interact with Samigo. Now that there is a new method signature for as.createAssessmentWithoutDefaultSection, SamigoAssessmentHandler should use the new method signature to guarantee siteId is passed.
Patch attached.