When an entityModel has a field marked as a primary key, it should also be set 
to NotNull
-----------------------------------------------------------------------------------------

                 Key: OFBIZ-3154
                 URL: https://issues.apache.org/jira/browse/OFBIZ-3154
             Project: OFBiz
          Issue Type: Bug
          Components: framework
    Affects Versions: SVN trunk
            Reporter: Bob Morley


We were doing some work with the ModelField and noticed that when a field is 
marked as a primary key it can still have its "NotNull" property set to "false" 
(the default).  Because we were doing work based on if a field was nullable it 
was causing a bug on primary keys.  The suggestion is to set NotNull = "true" 
when a field is marked as a primary key.  This should have no impact to current 
code as (in most places) a redundant check was being done (see creation of 
tables where it will add "NOT NULL" based on both isNotNull and isPk -- ideally 
it should be just based on the former).

This change makes minor modifications to two files ModelField and ModelEntity.  
ModelField.setIsPk() will call setIsNotNull(true) when the primary key 
indicator is true.  ModelEntity will call field.isNotNull = true when it is 
reading the primary key elements in the entity definition.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to