Hi, Panu..<br><br>1. Remeber, this Glorp version is a port of Cincom Glorp Version... and we try follow closely as posible it version.<br>2. Varchar type needed a parameter with the length of field. plantform>>varchar is present only for compability with other types definitions.<br>
3. The SQL generated, depends on the Platform..in the especific platform is the right place to introduces fixes for a especific database. in your case PostgreSQLPlatform i can guess..<br><br>Best. D.<br><br><div class="gmail_quote">
On Wed, Apr 6, 2011 at 4:38 AM, Panu Suominen <span dir="ltr"><<a href="mailto:panu.j.m.suominen@gmail.com">panu.j.m.suominen@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
There is little problem with varchar field type in Glorp.<br>
Following kind of table definitions in DescriptorSystem are not working:<br>
aTable createFieldNamed: 'zipcode' type: platform varChar.<br>
<br>
This is because GlorpVarCharType can return non working SQL.<br>
If one evaluates: (GlorpVarCharType new typeName: 'character varying')<br>
typeString<br>
it returns 'character varying(nil)' which does not compute (at least<br>
not with postgresql).<br>
<br>
This could be fixed by removing the typeString method from GlorpVarCharType<br>
(then GlorpAbstractStringType >> #typeString will be used which works ok).<br>
<br>
Currently I don't have opportunity to test this with other database systems.<br>
<br>
Other options could be setting some default for varchar length or removing<br>
DatabasePlatform >> #varChar and #varchar -methods.<br>
<br>
Any opinions?<br>
--<br>
<font color="#888888">Panu<br>
</font><br>_______________________________________________<br>
SqueakDBX mailing list<br>
<a href="mailto:SqueakDBX@lists.squeakfoundation.org">SqueakDBX@lists.squeakfoundation.org</a><br>
<a href="http://lists.squeakfoundation.org/mailman/listinfo/squeakdbx" target="_blank">http://lists.squeakfoundation.org/mailman/listinfo/squeakdbx</a><br>
<br></blockquote></div><br><div style="visibility: hidden; left: -5000px; position: absolute; z-index: 9999; padding: 0px; margin-left: 0px; margin-top: 0px; overflow: hidden; word-wrap: break-word; color: black; font-size: 10px; text-align: left; line-height: 130%;" id="avg_ls_inline_popup">
</div>