On 16/12/2013 12:53 PM, Bill Moseley wrote:



On Sun, Dec 15, 2013 at 4:49 PM, neil.lunn <n...@mylunn.id.au <mailto:n...@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 ) = @_;
        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/Catalyst/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=67078
    https://rt.cpan.org/Public/Bug/Display.html?id=78663



-- Bill Moseley
    mose...@hank.org <mailto:mose...@hank.org>


    _______________________________________________
    List:Catalyst@lists.scsys.co.uk  <mailto:Catalyst@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@lists.scsys.co.uk <mailto:Catalyst@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
mose...@hank.org <mailto:mose...@hank.org>


_______________________________________________
List: Catalyst@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/



---
This email is free from viruses and malware because avast! Antivirus protection 
is active.
http://www.avast.com
_______________________________________________
List: Catalyst@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/

Reply via email to