Re: moving forward: thanks for feedback

On 9/28/06, Lukas Kahwe Smith <lsmith@php.net> wrote:

Brett Bieber wrote: The text fields with no length needed to be changed to clobs with no default value. Of those text fields, only one was indexed... which I modified to index 255. Also looks like MDB2_Schema doesn't like CURRENT_TIMESTAMP for a default value, nor 'releases' or 'order.' I'm not sure if there are options for quoting field names for the creation statements, but attached is what I had to do to get MDB2_Schema to successfully create from the xml file. The largest issues were field names I had to modify to get MDB2_Schema to create.... order -> ordering and release->release_v 1) CURRENT_TIMESTAMP Someone needs to sit down and write code to add a trigger for this case. MySQL unfortunately only supports CURRENT_TIMESTAMP on timestamp and not on datetime fields (which are more portable). 2) reserved words Change those names in the application. Really its a good idea! Otherwise set the "quote_identifier" option to true.

I'm all for quote_identifier. :-) -- Justin Patrin