Delegated Access functionality is broken with the fix
There were 2 files modified:
1. delegatedaccess/impl/src/java/org/sakaiproject/delegatedaccess/logic/ProjectLogicImpl.java - This file had an issue where Concurrent hashmaps throw NPE when storing null values https://jira.sakaiproject.org/browse/SAK-28988. Fix for this has been merged
SAK-28988 into the Sakai 10.x Subversion maintenance branch
2. delegatedaccess/impl/src/java/org/sakaiproject/delegatedaccess/dao/impl/DelegatedAccessDaoImpl.java - This file was modified mostly to use list of arguments (ArrayList) as an input to bind to the query method of the JDBCTemplate. As a result of the change , getNodesBySiteRef method is totally broken and is not returning right results.
e.g getNodesBySiteRef - The sql use in this method is as below:
select.hierarchyNode = Select ID, title From HIERARCHY_NODE_META where title in and hierarchyId = ? and isDisabled = 0
But when values are added in List, hierarchyId is added first and which as a result does not return any data.
And what was the intent of changes in this class as it has also removed given below changes which was the reported JIRA issue.
inParams += "'" + subSiteRefs[i].replace("'", "''") + "'"; //escape apostrophe