[Catalyst] Re: Catalyst::Plugin::DBIC::QueryLog

[Jonathan Rockway]

Fayland Lam wrote:
> Matt S Trout wrote:
>> No, it's awful. There's no reason at all for this to be a plugin.
>>
>> I'd suggest you make it a mixin for Catalyst::Model::DBIC::Schema
>> instead.
> but it depends on every request. it start at the sub prepare and end
> before finalize_body. due to my limited knowledge, I'm not sure if
> it's OK as a mixin for Catalyst::Model::DBIC::Schema

I got about 90% through composing the post that mst wrote, and decided
against it because of the per-request requirement.  However, just out of
curiosity, why do you care about making it per-request?

> that's right. but there is more than one way to do it. :)
Careful with that.  For every good way to do something, there are 1000
bad ways.  For throw-away code, TMTOWTDI is fine, but if you want every
DBIC user to use your code, do things the good way :)

Regards,
Jonathan Rockway

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 370 bytes
Desc: OpenPGP digital signature
Url : http://lists.scsys.co.uk/pipermail/catalyst/attachments/20070904/91ba=
4715/signature.pgp