[Catalyst] Bump Catalyst::Model::Adaptor

[neil.lunn]

On 16/12/2013 12:53 PM, Bill Moseley wrote:
>
>
>
> On Sun, Dec 15, 2013 at 4:49 PM, neil.lunn <neil at mylunn.id.au =

> <mailto:neil at mylunn.id.au>> wrote:
>
>
>     Isn't this just a case of adding to your model class:
>
>
> Yes, that's good.   Thanks.
>
>
>     sub mangle_arguments {
>         my ( $self, $args ) =3D @_;
>         return $args->{args};
>     }
>

An alternate is overriding prepare_arguments as in the previous release:
http://cpansearch.perl.org/src/BOBTFISH/Catalyst-Model-Adaptor-0.09/lib/Cat=
alyst/Model/Adaptor/Base.pm

But I prefer the mangle_arguments particulary for classes that do not =

use a Moose like contructor.
The change from the old code seems to be deliberate in passing in extra =

arguments to the model constructor, so it seems best to override these =

hooks where needed rather than patch the base.


>
>     Thus overriding the default with what you want to do for your
>     constructor.
>     This clears the error for me.
>
>>
>>     In this case it is Cache::Memcached::libmemcached complaining.
>>
>>     I think there's a suggested patch in one of these.   Is there
>>     someone that can review and maybe push out a new version?
>>
>>     https://rt.cpan.org/Public/Bug/Display.html?id=3D67078
>>     https://rt.cpan.org/Public/Bug/Display.html?id=3D78663
>>
>>
>>
>>     -- =

>>     Bill Moseley
>>     moseley at hank.org <mailto:moseley at hank.org>
>>
>>
>>     _______________________________________________
>>     List:Catalyst at lists.scsys.co.uk  <mailto:Catalyst at lists.scsys.co.uk>
>>     Listinfo:http://lists.scsys.co.uk/cgi-bin/mailman/listinfo/catalyst
>>     Searchable archive:http://www.mail-archive.com/catalyst@lists.scsys.=
co.uk/
>>     Dev site:http://dev.catalyst.perl.org/
>
>
>
>     ---------------------------------------------------------------------=
---
>     <http://www.avast.com/> 	=

>
>     This email is free from viruses and malware because avast!
>     Antivirus <http://www.avast.com/> protection is active.
>
>
>
>     _______________________________________________
>     List: Catalyst at lists.scsys.co.uk <mailto:Catalyst at lists.scsys.co.uk>
>     Listinfo: http://lists.scsys.co.uk/cgi-bin/mailman/listinfo/catalyst
>     Searchable archive:
>     http://www.mail-archive.com/catalyst@lists.scsys.co.uk/
>     Dev site: http://dev.catalyst.perl.org/
>
>
>
>
> -- =

> Bill Moseley
> moseley at hank.org <mailto:moseley at hank.org>
>
>
> _______________________________________________
> List: Catalyst at lists.scsys.co.uk
> Listinfo: http://lists.scsys.co.uk/cgi-bin/mailman/listinfo/catalyst
> Searchable archive: http://www.mail-archive.com/catalyst@lists.scsys.co.u=
k/
> Dev site: http://dev.catalyst.perl.org/



---
This email is free from viruses and malware because avast! Antivirus protec=
tion is active.
http://www.avast.com
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.scsys.co.uk/pipermail/catalyst/attachments/20131216/b57b5=
ce4/attachment.htm