Bug #78089 for Catalyst-TraitFor-Component-ConfigPerSite: Additional site config parameters are not accessible

Thu Jun 28 05:40:32 2012 nick [...] webcraftcs.com - Ticket created

Subject:	Additional site config parameters are not accessible
Date:	Thu, 28 Jun 2012 10:40:19 +0100
To:	bug-Catalyst-TraitFor-Component-ConfigPerSite [...] rt.cpan.org
From:	Nick <nick [...] webcraftcs.com>

The Catalyst-TraitFor-Component-ConfigPerSite package does not allow access to config parameters other than the specific ones required for the database connection (in Model::DB) or view (in View::TT). Access to additional parameters is desired to fully utilise this package. Specifically, individual site file paths for use by DBIx::Class::InflateColumn::FS would be useful to be able to keep site files in segregated areas. e.g. <Model::DB> schema_class default::Schema <connect_info> dsn dbi:mysql:defaultdb user xxxx password xxxx </connect_info> file_path /path/to/site/default </Model::DB> Thank you

Thu Jun 28 14:40:28 2012 TEEJAY [...] cpan.org - Status changed from 'new' to 'open'

Thu Jun 28 14:40:28 2012 TEEJAY [...] cpan.org - Given to TEEJAY

Thu Jun 28 14:40:28 2012 TEEJAY [...] cpan.org - Severity Important added

Thu Jun 28 14:40:28 2012 TEEJAY [...] cpan.org - Broken in 0.05 added

Thu Jun 28 14:40:28 2012 TEEJAY [...] cpan.org - Broken in 0.06 added

Thu Jun 28 14:40:28 2012 TEEJAY [...] cpan.org - Broken in 0.07 added

Thu Jun 28 14:40:28 2012 TEEJAY [...] cpan.org - Broken in 0.08 added

Thu Jun 28 14:40:28 2012 TEEJAY [...] cpan.org - Broken in 0.09 added

Thu Jun 28 14:40:29 2012 TEEJAY [...] cpan.org - Broken in 0.10 added

Thu Jul 05 10:20:55 2012 TEEJAY [...] cpan.org - Correspondence added 50 min

Hi Nick, Can you post a bit more example code - i.e. modify the config/code in the tests included in the distribution, as I can't see how to replicate it without a bit more context/code to implement the function you're calling against the schema. The configuration all seems to be correctly copied at the model layer - should I also be throwing it in the dbic schema connect call too? I can't make much more progress without either some more pointers in the code or dedicated a large chunk of my weekend (which won't be likely in the short term) Thanks, Aaron

Fri Jul 13 16:08:57 2012 TEEJAY [...] cpan.org - Correspondence added

On Thu Jun 28 05:40:32 2012, nick@webcraftcs.com wrote: Show quoted text

> > The Catalyst-TraitFor-Component-ConfigPerSite package does not allow > access to config parameters other than the specific ones required for > the database connection (in Model::DB) or view (in View::TT).

Looking again at your original catalyst list post I can see that the problem is due to Catalyst::TraitFor::Model::DBIC::Schema::SchemaProxy loading configuration at start/compile time. I'll see if I can find a work around and/or provide an easy alternative in ConfigPerSite.

Fri Jul 13 16:09:19 2012 TEEJAY [...] cpan.org - Severity Important changed to Normal

Wed Jul 18 12:27:58 2012 nick [...] webcraftcs.com - Correspondence added

Subject:	Re: [rt.cpan.org #78089] Additional site config parameters are not accessible
Date:	Wed, 18 Jul 2012 17:27:41 +0100
To:	bug-Catalyst-TraitFor-Component-ConfigPerSite [...] rt.cpan.org
From:	Nick <nick [...] webcraftcs.com>

Hi Aaron, Apologies for the lack of contact from me which was due to an urgent application deployment and subsequent server failure, but I'm around now and keen to help (if I can!). I was using TraitFor/SchemaProxy so that I could move the file_path out of the Model and ultimately into the config files. I may have overly complicated things and suspect I can simplify it slightly.....will report back shortly... Nick On 13/07/12 21:08, Aaron James Trevena via RT wrote: Show quoted text

> <URL: https://rt.cpan.org/Ticket/Display.html?id=78089 > > > On Thu Jun 28 05:40:32 2012, nick@webcraftcs.com wrote:

>> The Catalyst-TraitFor-Component-ConfigPerSite package does not allow >> access to config parameters other than the specific ones required for >> the database connection (in Model::DB) or view (in View::TT).

> Looking again at your original catalyst list post I can see that the > problem is due to Catalyst::TraitFor::Model::DBIC::Schema::SchemaProxy > loading configuration at start/compile time. > > I'll see if I can find a work around and/or provide an easy alternative > in ConfigPerSite. > >

Thu Jul 19 08:12:24 2012 nick [...] webcraftcs.com - Correspondence added

Subject:	Re: [rt.cpan.org #78089] Additional site config parameters are not accessible
Date:	Thu, 19 Jul 2012 13:12:02 +0100
To:	bug-Catalyst-TraitFor-Component-ConfigPerSite [...] rt.cpan.org
From:	Nick <nick [...] webcraftcs.com>

Hi Aaron, I've spent some time this morning adding the relevant bits to the test directory to at least demonstrate the file_path/SchemaProxy issue. I have attached a zip file with everything in and you will notice some modified files including a couple of additional test files with a suffix "_nick.t". The test databases have also been modified. Hopefully this will be of some use although I suspect you may have done this much already by now! Although it's more a conflict between SchemaProxy and ConfigPerSite than a bug, I do believe that ConfigPerSite would be enhanced by the ability to specify different site file paths (in the config file) for the inflate_columns_fs feature. Thanks Nick On 13/07/12 21:08, Aaron James Trevena via RT wrote: Show quoted text

> <URL: https://rt.cpan.org/Ticket/Display.html?id=78089 > > > On Thu Jun 28 05:40:32 2012, nick@webcraftcs.com wrote:

>> The Catalyst-TraitFor-Component-ConfigPerSite package does not allow >> access to config parameters other than the specific ones required for >> the database connection (in Model::DB) or view (in View::TT).

> Looking again at your original catalyst list post I can see that the > problem is due to Catalyst::TraitFor::Model::DBIC::Schema::SchemaProxy > loading configuration at start/compile time. > > I'll see if I can find a work around and/or provide an easy alternative > in ConfigPerSite. > >

Download Catalyst-TraitFor-Component-ConfigPerSite-0.10_nick.tar.gz
application/x-gzip 17.6k

Message body not shown because it is not plain text.