U++ forum: Welcome to the forum

Re: Sql column Ids now escaped [message #35359 is a reply to message #35350]

Mon, 06 February 2012 10:46

mirek
Messages: 14266
Registered: November 2005

Ultimate Member

mirek wrote on Sat, 04 February 2012 09:24

If SqlId does not contain valid sql id, it is not escaped. This is done to support things like:

Select(SqlId("*")).From(TABLE)

which seems to be used in existing code.

Ops, not so easy, above does not cover

Select(SqlId("current_time"))

which leads to SQL error (current_time is not schema name and cannot be escaped).

So I have changed everything in trunk:

static SqlId::UseQuotes switches quotes on for "modern" code, default is off to support legacy.

Modern code should use new SqlTxt to 'inject' SQL terms

Select(SqlTxt("current_time"))

and both of it makes checking id for being real id, which is time consuming, unnecessarry, so it is not there anymore...

Report message to a moderator