You are viewing a plain text version of this content. The canonical link for it is here.
Posted to ddlutils-dev@db.apache.org by to...@apache.org on 2006/01/11 23:30:43 UTC
svn commit: r368150 -
/db/ddlutils/trunk/src/java/org/apache/ddlutils/platform/SqlBuilder.java
Author: tomdz
Date: Wed Jan 11 14:30:37 2006
New Revision: 368150
URL: http://svn.apache.org/viewcvs?rev=368150&view=rev
Log:
Fixed bug found by Laurent Sacaut where alterDatabase failed when no creation parameters were given
Modified:
db/ddlutils/trunk/src/java/org/apache/ddlutils/platform/SqlBuilder.java
Modified: db/ddlutils/trunk/src/java/org/apache/ddlutils/platform/SqlBuilder.java
URL: http://svn.apache.org/viewcvs/db/ddlutils/trunk/src/java/org/apache/ddlutils/platform/SqlBuilder.java?rev=368150&r1=368149&r2=368150&view=diff
==============================================================================
--- db/ddlutils/trunk/src/java/org/apache/ddlutils/platform/SqlBuilder.java (original)
+++ db/ddlutils/trunk/src/java/org/apache/ddlutils/platform/SqlBuilder.java Wed Jan 11 14:30:37 2006
@@ -282,7 +282,9 @@
Table table = database.getTable(idx);
writeTableComment(table);
- createTable(database, table, (params == null ? null : params.getParametersFor(table)));
+ createTable(database,
+ table,
+ params == null ? null : params.getParametersFor(table));
}
// we're writing the external foreignkeys last to ensure that all referenced tables are already defined
@@ -344,7 +346,9 @@
{
_log.info("Creating table " + desiredTable.getName());
}
- createTable(desiredModel, desiredTable, params.getParametersFor(desiredTable));
+ createTable(desiredModel,
+ desiredTable,
+ params == null ? null : params.getParametersFor(desiredTable));
// we're deferring foreignkey generation
newTables.add(desiredTable);
}