Bug in jdbcDatabaseSnapshotGenerator.getForeignKeys() ??

Liquibase Development
1

Hi,
I found a bug in jdbcDatabaseSnapshotGenerator.getForeignKeys() that will cause a NullPointerException in this code:

                        if (fkInfo == null) {                         throw new DatabaseException("Database returned out of sequence foreign key column for "+fkInfo.getName());                     }
since fkInfo is null, then it tries to call getName() on a null pointer. I'd fix it myself, but I'm not 100% sure what the original intention here was.  Someone either used the wrong variable name, or meant != instead of ==, or they didn't notice the red X Eclipse put on that line with a warning...
2

In the current trunk code, it has been changed to:

    if (foreignKey == null) { throw new DatabaseException("Database returned out of sequence foreign key column for " + fkInfo.getFkName()); }
which should the problem.

Nathan