svn commit: r1597239 - /ofbiz/trunk/framework/entity/src/org/ofbiz/entity/model/ModelEntity.java

Author: jleroux
Date: Sat May 24 05:09:39 2014
New Revision: 1597239

URL: http://svn.apache.org/r1597239
Log:
Closes OFBIZ-5648 - Extend primary keys

Modified:
    ofbiz/trunk/framework/entity/src/org/ofbiz/entity/model/ModelEntity.java

Modified: ofbiz/trunk/framework/entity/src/org/ofbiz/entity/model/ModelEntity.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/entity/src/org/ofbiz/entity/model/ModelEntity.java?rev=1597239&r1=1597238&r2=1597239&view=diff
==============================================================================
--- ofbiz/trunk/framework/entity/src/org/ofbiz/entity/model/ModelEntity.java (original)
+++ ofbiz/trunk/framework/entity/src/org/ofbiz/entity/model/ModelEntity.java Sat May 24 05:09:39 2014
@@ -47,8 +47,8 @@ import org.ofbiz.entity.Delegator;
 import org.ofbiz.entity.GenericEntity;
 import org.ofbiz.entity.GenericEntityException;
 import org.ofbiz.entity.GenericValue;
-import org.ofbiz.entity.config.model.Datasource;
 import org.ofbiz.entity.config.EntityConfigUtil;
+import org.ofbiz.entity.config.model.Datasource;
 import org.ofbiz.entity.jdbc.DatabaseUtil;
 import org.ofbiz.entity.model.ModelIndex.Field;
 import org.w3c.dom.Document;
@@ -366,11 +366,15 @@ public class ModelEntity implements Comp
                 this.fieldsList.add(newField);
                 this.fieldsMap.put(newField.getName(), newField);
                 if (!newField.getIsPk()) {
-                    // this will always be true for now as extend-entity fields are always nonpks
                     if (existingField != null) {
                         this.nopks.remove(existingField);
                     }
                     this.nopks.add(newField);
+                } else {
+                    if (existingField != null) {
+                        this.pks.remove(existingField);
+                    }
+                    this.pks.add(newField);
                 }
             }
         }