Skip Menu |

This queue is for tickets about the Catalyst-Model-Adaptor CPAN distribution.

Report information
The Basics
Id: 78663
Status: open
Priority: 0/
Queue: Catalyst-Model-Adaptor

People
Owner: Nobody in particular
Requestors: forstreuter [...] belwue.de
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



Subject: Bug in Catalyst-Model-Adaptor
Date: Mon, 30 Jul 2012 12:40:03 +0200
To: bug-Catalyst-Model-Adaptor [...] rt.cpan.org
From: Jörg Forstreuter <forstreuter [...] belwue.de>
Hi, Catalyst::Model::Adaptor causes an eror on an external ModelClass which uses MooseX::StrictConstructor, because catalyst default model has some extra args it passes to the constructor. A fix and a longer description of the problem is send via github as a pull request. Best regards Jörg Forstreuter -- Jörg Forstreuter, BelWü-Koordination, Universität Stuttgart Industriestr. 28, 70565 Stuttgart Tel: 0711/685-62509 - Fax: 0711/678-8363 http://www.belwue.de
Any plan for a fix here? I'm seeing $arg here is the config -- so this has the effect of doing %new_config = ( %{$config->{args}}, %{$config} ); And there's a few modules that complain about extra configs. (Cache::Memcached::libmemcached is one). sub prepare_arguments { my ($self, $app, $arg) = @_; return exists $self->{args} ? { %{$self->{args}}, %$arg, } : $arg; }
I note that this issue was reported over 6 years ago and a patched version has still not been released. If you are no longer able or interested in maintaining this module, I am willing to adopt it. You can just assign COMAINT to me in PAUSE. My CPAN id is MSCHOUT. Thanks. -- Regards, Michael Schout
Subject: Re: [rt.cpan.org #78663] Bug in Catalyst-Model-Adaptor
Date: Wed, 24 Oct 2018 10:30:13 +0200
To: MSCHOUT via RT <bug-Catalyst-Model-Adaptor [...] rt.cpan.org>
From: Jörg Forstreuter <forstreuter [...] belwue.de>
Hi, thanks for looking into this, I am not maintaining this module, I even dont have a PAUSE account. The patch I had proposed (with help of mst) is a pull request in https://github.com/bobtfish/catalyst-model-adaptor Best regards Jörg On Tue, Oct 23, 2018 at 01:16:11PM -0400, MSCHOUT via RT wrote: Show quoted text
> <URL: https://rt.cpan.org/Ticket/Display.html?id=78663 > > > I note that this issue was reported over 6 years ago and a patched version has still not been released. > > If you are no longer able or interested in maintaining this module, I am willing to adopt it. You can just assign COMAINT to me in PAUSE. My CPAN id is MSCHOUT. > > Thanks. > -- > Regards, > Michael Schout
-- Jörg Forstreuter, BelWü-Koordination, Universität Stuttgart