Bug #61117 for DBD-SQLite: Supporting database as an alias for dbname in DSN

Mon Sep 06 12:15:53 2010 davidp [...] preshweb.co.uk - Ticket created

Subject:

Supporting database as an alias for dbname in DSN

The documentation for DBI recommends that DBD::* modules should understand the parameter database= in a DSN, but DBD::SQLite requires dbname=. Quoting from DBI's docs: It is recommended that drivers support the ODBC style, shown in the last example above. It is also recommended that that they support the three common names 'host', 'port', and 'database' (plus 'db' as an alias for database). This simplifies automatic construction of basic DSNs: "dbi:$driver:database=$db;host=$host;port=$port". Drivers should aim to 'do something reasonable' when given a DSN in this form [...] DBD::Pg also expects dbname, but recognises database (and db) also - it would be great if DBD::SQLite would do the same. I'd be happy to provide a patch if requested, although I expect it'd be trivial to implement - the question is more whether you consider it a good idea.

Thu Sep 16 03:31:23 2010 ishigaki [...] cpan.org - Correspondence added

Implemented in the trunk. It's a good idea that fixes an overlooked compat issue. Thanks. On Mon Sep 06 12:15:53 2010, BIGPRESH wrote: Show quoted text

> The documentation for DBI recommends that DBD::* modules should > understand the parameter > database= in a DSN, but DBD::SQLite requires dbname=. > > Quoting from DBI's docs: > > It is recommended that drivers support the ODBC style, shown in > the > last example above. It is also recommended that that they support > the > three common names 'host', 'port', and 'database' (plus 'db' as an > alias for database). This simplifies automatic construction of > basic > DSNs: "dbi:$driver:database=$db;host=$host;port=$port". Drivers > should aim to 'do something reasonable' when given a DSN in this > form > [...] > > DBD::Pg also expects dbname, but recognises database (and db) also - > it would be great if > DBD::SQLite would do the same. > > I'd be happy to provide a patch if requested, although I expect it'd > be trivial to > implement - the question is more whether you consider it a good idea.

Thu Sep 16 03:31:24 2010 The RT System itself - Status changed from 'new' to 'open'

Thu Sep 16 03:31:24 2010 ishigaki [...] cpan.org - Status changed from 'open' to 'patched'

Fri Dec 10 01:47:56 2010 ishigaki [...] cpan.org - Correspondence added

DBD::SQLite 1.32_01 with this fix is out. Thanks. On 2010-9-16 Thu 03:31:23, ISHIGAKI wrote: Show quoted text

> Implemented in the trunk. It's a good idea that fixes an overlooked > compat issue. Thanks. > > On Mon Sep 06 12:15:53 2010, BIGPRESH wrote:

> > The documentation for DBI recommends that DBD::* modules should > > understand the parameter > > database= in a DSN, but DBD::SQLite requires dbname=. > > > > Quoting from DBI's docs: > > > > It is recommended that drivers support the ODBC style, shown in > > the > > last example above. It is also recommended that that they

support Show quoted text

> > the > > three common names 'host', 'port', and 'database' (plus 'db' as

an Show quoted text

> > alias for database). This simplifies automatic construction of > > basic > > DSNs: "dbi:$driver:database=$db;host=$host;port=$port". Drivers > > should aim to 'do something reasonable' when given a DSN in this > > form > > [...] > > > > DBD::Pg also expects dbname, but recognises database (and db) also - > > it would be great if > > DBD::SQLite would do the same. > > > > I'd be happy to provide a patch if requested, although I expect it'd > > be trivial to > > implement - the question is more whether you consider it a good

idea. Show quoted text

> >

Fri Dec 10 01:47:57 2010 The RT System itself - Status changed from 'patched' to 'open'

Fri Dec 10 01:47:57 2010 ishigaki [...] cpan.org - Status changed from 'open' to 'resolved'