[Dbix-class] Feature proposal for DBIx::Class::Storage::DBI::connect_info

[Matt S Trout]

On Wed, Aug 27, 2008 at 01:31:16PM +0300, Oleg Kostyuk wrote:
> > Well, if the patch is only designed for Catalyst...
> Not "only" but "mainly" ;)
> And this is just in my case, not more - may be someone else want to
> use Config::General and DBIC without Catalyst, and this patch will be
> good for him too. Because of this I created patch for DBIC and not for
> Catalyst::*. Of course, if by your opinion "we need adapt this to
> Catalyst::Model::DBIC::Schema" - I can do this. But by my personal
> opinion, creating additional parse_connect_info method is more generic
> and so it's preferred way. Moreover, if I understood correctly, it's
> not so hard to do - just slightly modify current patch.

Right. I'd really much rather see the parse_connect_info method added
than anything else.

> I also thinks about possibility to add other parameters, such as
> 'driver', 'host', 'dbname' etc, but this way looks more longer and I
> don't need such features at the moment - so my decision was to quickly
> make short patch that just works (tm).

Right. The reason I'm asking for it to have a method name is to make it
easy for people to go in and add those other parameters later. I'm a
believer in do the simplest thing that can possible work -provided- you
make it easy to do the cooler more complicated thing later :)

> I'm not so deeply know all internals of DBI/DBIC as you, Matt :) - so
> I'm hope you give me good direction for further moving. We can use
> personal emails instead of pollute mail list, if you prefer.

List good. Many people on here with useful ideas. If I thought there was
pollution, I'd create a -dev list like catalyst has, but I haven't felt
the need to yet.

-- 
      Matt S Trout       Need help with your Catalyst or DBIx::Class project?
   Technical Director                    http://www.shadowcat.co.uk/catalyst/
 Shadowcat Systems Ltd.  Want a managed development or deployment platform?
http://chainsawblues.vox.com/            http://www.shadowcat.co.uk/servers/