[Building Sakai] SAK-16568

[Bryan Holladay]

I've been asked to float SAK-16568 to the dev list for review since the patch for this touches kernel and several other tools in sakai...

Anyone can review, but specifically, please review: 
Qian (assign, site-manage)
Horwitz (knl, polls)
Hedrick (linktool)
Swinsburg (knl)
Zeckoski (knl)
Whyte (knl)
Karen Tsao (sam)
Botimer (osp)
Amsler (gb2)
Theriault (knl)
may (msgcntr -- Wagner is transitioning off the project)
Kirschner (knl, calendar)

https://jira.sakaiproject.org/browse/SAK-16568  

Title "Entity Picker Link points to original site when "Import from Site" option is used"

Description: "The new option in the Rich Text Editor, Sakai Entity Link, is not working properly when imported through sites. It points to the original site and any students that do not have permission to that site will be given an error. The links always refer back to the originating site."


About the Patch:
I've extended the kernel Interface "EntityTransferrer" to add another method:

	/**
	 * Takes a map of ref's (fromContextRef -> toContextRef) and replaces any reference to them
	 * 
	 * @param toContext
	 * @param transversalMap
	 */
	void updateEntityReferences(String toContext, Map<String, String> transversalMap); 


Each tool that implements the EntityTransferrer Interface has been updated to either implement this method or just have an empty stub, which is why there are so many tools that has changed.  If a tool wants to implement this, it just loops through the copied over entities and does a simple find/replace in key fields (i.e. description, ect).  The updateEntityReferences method must be called after all tools have completed transferring since a tool can point to other tools (i.e. assignment description can point to msgcntr).

Thanks,
Bryan


On Apr 25, 2011, at 7:40 PM, Anthony Whyte wrote:

> I have emails from Berg on this Jira from back in July, 2010.  The solution provided by the patches (touching kernel, tools, etc.) should in my opinion be floated on sakai-dev given that the fix touches so much.  Can you summarize the fix in an email (e.g., for kernel we do x, for tools we do y) and send it to the following individuals (cc'ing the dev list):
> 
> Qian (assign, site-manage)
> Horwitz (knl, polls)
> Hedrick (linktool)
> Swinsburg (knl)
> Zeckoski (knl)
> Whyte (knl)
> Karen Tsao (sam)
> Botimer (osp)
> Amsler (gb2)
> Theriault (knl)
> may (msgcntr -- Wagner is transitioning off the project)
> Kirschner (knl, calendar)
> 
> After that, perhaps I can help organize a review.
> 
> https://jira.sakaiproject.org/browse/SAK-16568
> 
> 
> Cheers,
> 
> Anth
> 
> 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://collab.sakaiproject.org/pipermail/sakai-dev/attachments/20110426/512ea9b2/attachment.html