Re: [gnome-db] Per-table meta-data with capital letters in the table name?





2009/7/1 Armin Burgmeier <armin arbur net>
On Tue, 2009-06-30 at 09:08 +0200, Vivien Malerba wrote:
>
>
> 2009/6/29 Murray Cumming <murrayc murrayc com>
>         On Mon, 2009-06-29 at 17:45 +0200, Vivien Malerba wrote:
>         >
>         >
>         > 2009/6/29 Murray Cumming <murrayc murrayc com>
>         >         I should take the time to make a test case, but it
>         looks like
>         >         libgda 4.0
>         >         fails to get the meta-data for a specified table if
>         the table
>         >         name has
>         >         capital letters. I guess that this is a quoting (and
>         escaping)
>         >         problem
>         >         somewhere.
>         >
>         > Yes, you need to surround any SQL identifier which is not
>         case
>         > insensitive (or which has some spaces, etc in it) with
>         double quotes.
>         > You can use gda_sql_identifier_needs_quotes().
>
>
>         So we need to add quotes around the table_name (and escape the
>         text),
>         when setting the table name in GdaMetaContext, as in the
>         libgdamm code
>         below? If so, that's definitely something that should be
>         mentioned in
>         the gda_connection_update_meta_store() and/or GdaMetaContext
>         documentation.
>
> Yes, and I'll correct that documentation point.

Are you sure? The testcase at
http://bugzilla.gnome.org/show_bug.cgi?id=587051#c17 suggests the
contrary.

There is obviously a bug there... Let me correct it.

Vivien
 



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]