svn commit: r1387884 - /ofbiz/trunk/framework/widget/src/org/ofbiz/widget/tree/ModelTree.java

[jleroux]

Author: jleroux
Date: Thu Sep 20 07:06:17 2012
New Revision: 1387884

URL: http://svn.apache.org/viewvc?rev=1387884&view=rev
Log:
A patch from Daniel Riquelme " Improve Tree Widget to support Multiple PK 
Entities" https://issues.apache.org/jira/browse/OFBIZ-4971

The PKs are used to keep track of which nodes have been rendered when 
performing recursion inside Tree Widget rendering code.
Currently Multiple PKs Entities can't be used as nodes in a Tree Widget because 
the rendering code does not handle them.


Modified:
    ofbiz/trunk/framework/widget/src/org/ofbiz/widget/tree/ModelTree.java

Modified: ofbiz/trunk/framework/widget/src/org/ofbiz/widget/tree/ModelTree.java
URL: 
http://svn.apache.org/viewvc/ofbiz/trunk/framework/widget/src/org/ofbiz/widget/tree/ModelTree.java?rev=1387884&r1=1387883&r2=1387884&view=diff
==============================================================================
--- ofbiz/trunk/framework/widget/src/org/ofbiz/widget/tree/ModelTree.java 
(original)
+++ ofbiz/trunk/framework/widget/src/org/ofbiz/widget/tree/ModelTree.java Thu 
Sep 20 07:06:17 2012
@@ -664,7 +664,13 @@ public class ModelTree extends ModelWidg
                     ModelField modelField = modelEntity.getOnlyPk();
                     this.pkName = modelField.getName();
                 } else {
-                    // TODO: what to do here?
+                    List<String> pkFieldsName = modelEntity.getPkFieldNames();
+                    StringBuilder sb = new StringBuilder();
+                    for (String pk: pkFieldsName) {
+                            sb.append(pk);
+                            sb.append("|");
+                    }
+                    this.pkName = sb.toString();
                 }
             }
         }