|
|
|
Created an attachment (id=791)
Initial patch based upon above comments. I'd really appreciate if somebody familiar with postgreSQL could let us know I haven't actually tested the patch but the code in
GeneralFactory.doubleQuoteName looks fine to me. I'm a former co-worker of the original submitter ![]() Pawel, would downloading the latest sources (from CVS), applying the patch,
building Castor and verifying whether it resolves the problem be an option for you ? More generally, can somebody please post a test case (incl. mapping file et
alias) that coudl be used to verify that things work ? All right, I'll try and do that. A test case should not be hard to come up with
but unfortunately I can't promise you I'll manage to find time to prepare it. If you wanted, I could send you a couple of files that I normally use to write a
test case, i.e.
Let me know whether this would be of any use to you. Yes, it certainly will. I would really appreciate it.
OK. I conducted tests on my code-base which includes 50+ classes and tables. All
table definitions use PostgreSQL schemas. The patch works for me just fine. I just sent a test case to Werner, based on his code. Thanks
![]() Patch applied as it. Will add test case anywhen later (but before 0.9.6 releae).
|
quote names with the three components schema name, table name and column name.
It should instead call the generic function genericFactory.doubleQuoteName()
like OracleFactory.quoteName() and SybaseFactor.quoteName():
public String quoteName( String name )
{ return doubleQuoteName( name ); }