[Opensource] DBObject.haveAllKeys() test?

[larry hamel]

The second test below in haveAllKeys() doesn't make sense to me.  seems like DBObject.haveAllKeys() should not care about && isAutoIncremented() in that test.

am I missing something?

larry
--------------------------------

    public boolean haveAllKeys()
            throws DBException {
        DBField oneField = null;

        for (Iterator i = this.getJDBCMetaData().getAllKeysMap().values().iterator(); i.hasNext();) {
            oneField = (DBField) i.next();
            DataField df = this.getDataField(oneField.getName());
            if (df == null || ((df.isNull() && !oneField.isAutoIncremented()))) {
                return false;
            }
        }

        return true;
    } /* haveAllKeys() */


Larry Hamel
----------------------
Free link-test tool (special handling for Expresso sites): http://morebot.org
Custom web site development: http://codeguild.com