RT3 patch for SQL query capitalization and quoting

As part of our efforts to implement RT3 on Sybase, we’ve generated the attached patches for RT. The changes in this patch are to SQL queries to match the case of the ‘id’ field name to the database, and to remove quotes from integer literals in SQL queries.

These changes were necessary for the Sybase port, and I don’t believe they should have any negative effect on currently supported databases.

Cheers
Chris

rt-3.0.2-capquote.patch (9.44 KB)

Chris,

Apologies if I’m late in on this thread.

I noticed that you submitted some patches rt-devel regarding running on
Sybase. I started to look at this, but not being a SQL expert, I was more
than happy to find that other people had done some work on this.

In particular, I noticed two other places where RT may require changes:

  1. the schema (although I think this is relatively easy)

  2. the fact there are no Sybase specific routines in:

    DBIx::SearchBuilder::Handle
    

    while there are for postgres, mysql and oracle. I’m not sure whether
    this is going to be an issue or not.

Do you know of any thing else which needs attention in order to get RT to
work with Sybase?

          - WarrenOn Thu, 29 May 2003, Audley, Christopher wrote:

As part of our efforts to implement RT3 on Sybase, we’ve generated the
attached patches for RT. The changes in this patch are to SQL queries
to match the case of the ‘id’ field name to the database, and to remove
quotes from integer literals in SQL queries.

These changes were necessary for the Sybase port, and I don’t believe
they should have any negative effect on currently supported databases.

Cheers
Chris