Skip Menu |

This queue is for tickets about the SQL-Abstract-More CPAN distribution.

Report information
The Basics
Id: 108773
Status: resolved
Priority: 0/
Queue: SQL-Abstract-More
People
Owner: Nobody in particular
Requestors: aar [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)

History Show all quoted text
  Thu Nov 12 04:30:25 2015 aar [...] cpan.org - Ticket created
Subject: table_alias() doesn't honor quote_char
It looks like the table_alias() method doesn't use the same quoting logic used by SQL::Abstract. The table alias is not enclosed by the configured quote_char (such as backticks in MySQL). A workaround for this is to supply table_alias => '%s AS `%s`' to the constructor in addition to quote_char but this looks unnecessary and it should at least be documented. (I love this module, by the way)
  Sat Dec 24 15:55:19 2016 DAMI [...] cpan.org - Correspondence added
On Thu Nov 12 04:30:25 2015, AAR wrote: Show quoted text
> It looks like the table_alias() method doesn't use the same quoting > logic used by SQL::Abstract. The table alias is not enclosed by the > configured quote_char (such as backticks in MySQL). > A workaround for this is to supply table_alias => '%s AS `%s`' to the > constructor in addition to quote_char but this looks unnecessary and > it should at least be documented. > > (I love this module, by the way)
Support for quoting whas introduced in v1.28 (05.07.2016). Sorry, I forgot to close the ticket at that moment.
  Sat Dec 24 15:55:20 2016 The RT System itself - Status changed from 'new' to 'open'
  Sat Dec 24 15:55:30 2016 DAMI [...] cpan.org - Status changed from 'open' to 'resolved'
  Sat Dec 24 15:55:45 2016 DAMI [...] cpan.org - Correspondence added
On Thu Nov 12 04:30:25 2015, AAR wrote: Show quoted text
> It looks like the table_alias() method doesn't use the same quoting > logic used by SQL::Abstract. The table alias is not enclosed by the > configured quote_char (such as backticks in MySQL). > A workaround for this is to supply table_alias => '%s AS `%s`' to the > constructor in addition to quote_char but this looks unnecessary and > it should at least be documented. > > (I love this module, by the way)
Support for quoting whas introduced in v1.28 (05.07.2016). Sorry, I forgot to close the ticket at that moment.