Skip Menu |

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

Report information
The Basics
Id: 87731
Status: open
Priority: 0/
Queue: SQL-Translator
People
Owner: Nobody in particular
Requestors: STUIFZAND [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)

History Show all quoted text
  Fri Aug 09 09:08:40 2013 STUIFZAND [...] cpan.org - Ticket created
Subject: No way to specify quoting of names of tables and fields in Schema::Versioned diff
I can specify quote_identifiers in create_ddl_dir, but this option isn't used when creating a diff. It's only used for the complete the schema. $schema->create_ddl_dir(['MySQL'], undef, './sql', $preversion, {quote_identifiers => 1}); It seems SQL::Translator::schema_diff expects producer_args that specify special options for quoting fields and table names. These options aren't passed in. This means there is a difference in the quoting of the complete SQL file for a version and the diff file between verions.
  Tue Jul 29 01:37:41 2014 ribasushi [...] leporine.io - Correspondence added
Subject: ::Producer::MySQL diffing does not respect quote_identifiers (was: No way to specify quoting of names of tables and fields in Schema::Versioned diff)
Part of the problem with used to lie in DBIC, and has been fixed as of However the real issue is that SQLT::Producer::MySQL does not respect qupte_identifiers in ::Diff code, and even when forced via quote_table_names - it still does not DTRT when it comes to DROP statements e.g. Therefore I am reassigning this ticket to the SQLT queue, where the rest of the fixes need to take place. Chers
  Tue Jul 29 01:37:43 2014 The RT System itself - Status changed from 'new' to 'open'
  Tue Jul 29 01:37:45 2014 ribasushi [...] leporine.io - Queue changed from DBIx-Class to SQL-Translator
  Tue Jul 29 02:14:29 2014 ribasushi [...] leporine.io - Correspondence added
Subject: ::Producer::MySQL diffing does not respect quote_identifiers (was: No way to specify quoting of names of tables and fields in Schema::Versioned diff)
Part of the problem used to lie in DBIC, and has been fixed as of https://github.com/dbsrgits/dbix-class/commit/08ac7648 However the real issue is that SQLT::Producer::MySQL does not respect quote_identifiers in ::Diff code, and even when forced via quote_table_names - it still does not DTRT when it comes to e.g. DROP statements. Therefore I am reassigning this ticket to the SQLT queue, where the rest of the fixes need to take place. Chers
  Tue Jul 29 02:15:43 2014 ribasushi [...] leporine.io - Subject changed from 'No way to specify quoting of names of tables and fields in Schema::Versioned diff' to '::Producer::MySQL diffing does not respect quote_identifiers (was: No way to specify quoting of names of tables and fields in Schema::Versioned diff)'