[samigo-team] Review rwequest: SAM-2088 GB registration

[David Horwitz]

Hi All,

I've checked a fix into a branch I would appreciate eyes on. It changes the way Samigo registers a assessment id with the gradebook from the format:

[id] - the id of the published assessment
to

/samigo/[id]  - where [id] is the published assessment id

The reasons for this are:

1) Performance: the external integration code in Samigo can identify references that are not part of its domain without going to  storage
2) Safety: it removes the risk that a similarly written integration would collide and give incorrect results in the gradebook


note is will require a db conversion script.

Feedback welcomed

Regards

David


________________________________
UNIVERSITY OF CAPE TOWN

This e-mail is subject to the UCT ICT policies and e-mail disclaimer published on our website at http://www.uct.ac.za/about/policies/emaildisclaimer/ or obtainable from +27 21 650 9111. This e-mail is intended only for the person(s) to whom it is addressed. If the e-mail has reached you in error, please notify the author. If you are not the intended recipient of the e-mail you may not use, disclose, copy, redirect or print the content. If this e-mail is not related to the business of UCT it is sent by the sender in the sender's individual capacity.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://collab.sakaiproject.org/pipermail/samigo-team/attachments/20130522/fdb73b19/attachment.html