John Lenton
2004-07-07 08:37:01 UTC
Attached is a patch that quotes entites and attributes in the sql. It
has several problems, still:
- I haven't fixed the tests (and I'm not certain it's even possible
without rewriting them), so if you use postgres suddenly a whole
slew of tests fail.
- Schema generation *almost* works: I'm not quoting the drop
statements for the primary key indexes.
- I changed an unrelated bit: I set useAllCaps to default to 0 in
Entity. This makes the previous bug hide under the carpet, plus
you don't have to quote things when using 'psql' (the latter is the
reson for this change).
- No documentation.
- Only done for postgres (but the per-adapter changes are small)
- Some of the lines are way too long, especially when cutting a line
would mean creating an intermediate object---in those cases I
defer to Sébastien to decide if it's worth it---or when the original
was pretty long to start with :)
however, it's working for me (so far!), and I'm using it for
development.
has several problems, still:
- I haven't fixed the tests (and I'm not certain it's even possible
without rewriting them), so if you use postgres suddenly a whole
slew of tests fail.
- Schema generation *almost* works: I'm not quoting the drop
statements for the primary key indexes.
- I changed an unrelated bit: I set useAllCaps to default to 0 in
Entity. This makes the previous bug hide under the carpet, plus
you don't have to quote things when using 'psql' (the latter is the
reson for this change).
- No documentation.
- Only done for postgres (but the per-adapter changes are small)
- Some of the lines are way too long, especially when cutting a line
would mean creating an intermediate object---in those cases I
defer to Sébastien to decide if it's worth it---or when the original
was pretty long to start with :)
however, it's working for me (so far!), and I'm using it for
development.
--
John Lenton (***@gmail.com) -- Random fortune:
bash: fortune: command not found
John Lenton (***@gmail.com) -- Random fortune:
bash: fortune: command not found