[PATCH] Trac does not send object names in quotes to PostgreSQL

[devrim@…]

Hello,

If we create an environment with a dash sign in it, Trac fails to authenticate from PostgreSQL. The reason is that within PostgreSQL, object names cannot have - in it:

test=# CREATE TEMP TABLE t-1 (c1 int);
ERROR:  syntax error at or near "-"
LINE 1: CREATE TEMP TABLE t-1 (c1 int);

If you want to use dash, you need to double quote object name:

test=# CREATE TEMP TABLE "t-1" (c1 int);
CREATE TABLE

This is not a bug -- this is a feature. So trac needs to send all object names in double quotes to PostgreSQL.

Regards, Devrim

[felix.schwarz@…]

This problem is present even in 0.11 and trunk. I attach a patch which solves the problem for me. As I did not find any unit tests in trac for testing SQL generation for specific databases, I don't attach a test case.

[felix.schwarz@…]

patch against 0.11

[felix.schwarz@…]

re-did my patch, this time with unit tests

[felix.schwarz@…]

modifying subject to make my patch more visible

[cboos]

Patch looks OK, would need some double checking before applying.

Postponing to 0.11.3 but if someone feels like testing and applying it before, please do.