|
From: Robert M. <rob...@gm...> - 2010-01-31 22:58:35
|
Christian, I just applied the patch (minus the redundant DB2 custom types). In the end I decided to replace all occurrences with getColumnName since I do not have a system against which to test. Try the latest snapshot : https://sourceforge.net/projects/squirrel-sql/files/3-snapshots/Snapshot-20100131_2249/squirrel-sql-Snapshot-20100131_2249-install.jar/download Let us know whether or not it addresses the issue. Rob On Thu, Jan 28, 2010 at 8:26 PM, Robert Manning <rob...@gm...> wrote: > Thanks for confirming that Christian - I suspected that but I wanted > to have another look to be sure. We will take care of this issue > before the 3.1 release in February and when we do there will be a > snapshot that you (and others who might be interested in the impact of > this change) to try out. > > Rob > > On Thu, Jan 28, 2010 at 2:02 PM, Christian Gruenberg > <chr...@di...> wrote: >>>> I also think that if we do this in core, then probably >>>> most if not all of Christian's custom DB2 datatypes are unnecessary. >>>> I skimmed over them so it is possible that I missed something but they >>>> seemed to be merely overriding the core implementation of >>>> getWhereClauseValue() and the like in *DataTypeComponent. I'll take >>>> another look. >> >> You are right, they are unnecessary now if you'll corret the basic datatypes. >> >> Christain >> > |
