inteligr8/alfresco-community-repo
1
0
Fork 0
mirror of https://github.com/Alfresco/alfresco-community-repo.git synced 2025-08-07 17:49:17 +00:00
Code Issues Packages Projects Releases Wiki Activity

NodeHandler was mishandling multiple associations of the same type.

Browse Source 
git-svn-id: https://svn.alfresco.com/repos/alfresco-enterprise/alfresco/HEAD/root@13752 c4b6b30b-aa2e-2d43-bbcb-ca4b014f7261
This commit is contained in:
Neil McErlean
2009-03-26 15:49:01 +00:00
parent 8a3e96597c
commit b534affa68
1 changed files with 12 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
source/java/org/alfresco/repo/forms/processor/NodeHandler.java
View File

@@ -409,15 +409,23 @@ public class NodeHandler extends AbstractHandler
assocFieldDefs.put(assocName, fieldDef); assocFieldDefs.put(assocName, fieldDef);
} }
String prefixedAssocName = ASSOC_PREFIX + assocName;
if (fieldDef.isEndpointMany()) if (fieldDef.isEndpointMany())
{ {
List<String> targets = null;
// add the value as a List (or add to the list if the form data // add the value as a List (or add to the list if the form data
// is already present) // is already present)
List<String> targets = (List<String>)formData.getData().get(assocName); FieldData fieldData = formData.getData().get(prefixedAssocName);
if (targets == null) if (fieldData == null)
{ {
targets = new ArrayList<String>(4); targets = new ArrayList<String>(4);
formData.addData(ASSOC_PREFIX + assocName, targets); formData.addData(prefixedAssocName, targets);
}
else
{
targets = (List<String>)fieldData.getValue();
} }
// add the assoc value to the list // add the assoc value to the list
@@ -426,7 +434,7 @@ public class NodeHandler extends AbstractHandler
else else
{ {
// there should only be one value // there should only be one value
formData.addData(ASSOC_PREFIX + assocName, assocValue); formData.addData(prefixedAssocName, assocValue);
} }
} }
} }