Skip Menu |

This queue is for tickets about the DBIx-Class CPAN distribution.

Report information
The Basics
Id: 42550
Status: rejected
Priority: 0/
Queue: DBIx-Class
People
Owner: Nobody in particular
Requestors: ddascalescu+perl [...] gmail.com
Cc:
AdminCc:
Bug Information
Severity: Unimportant
Broken in: 0.08099_05
Fixed in: (no value)

History Show all quoted text
  Tue Jan 20 03:41:56 2009 ddascalescu+perl [...] gmail.com - Ticket created
Subject: DOC: "Adding Indexes And Functions To Your SQL" in the Cookbook needs fixes
The "Adding Indexes And Functions To Your SQL" section mentions an out of date way of determining the database type: $sqlt_table->schema->translator->producer_type SQL::Translator returns just "CODE" nowadays. (also, the statement doesn't compile - the open parenthesis needs to be moved before 'my'). Is there a workaround way to determine the database type in sub sqlt_deploy_hook { ... } ?
  Tue Jan 20 19:02:13 2009 ddascalescu+perl [...] gmail.com - Correspondence added
From: ddascalescu+perl [...] gmail.com
Show quoted text
> Is there a workaround way to determine the database type in sub > sqlt_deploy_hook { ... } ?
$self->storage->connect_info->[0] Patching the doc with this change and the moved parenthesis should be sufficient to close this ticket.
  Sun Feb 15 08:47:29 2009 ribasushi [...] leporine.io - Correspondence added
$sqlt_table->schema->translator->producer_type in fact does return the current db type as can be seen here (and by `prove`ing t/86sqlt.t): http://dev.catalyst.perl.org/svnweb/bast/revision/?rev=5466 No doc patches are necessary, please provide us with a code example where producer_type indeed returns 'CODE'
  Sun Feb 15 08:47:30 2009 The RT System itself - Status changed from 'new' to 'open'
  Sun Feb 15 08:47:31 2009 ribasushi [...] leporine.io - Status changed from 'open' to 'rejected'