[Catalyst] Catalyst-Authentication-Store-DBIx-Class-User v0.1400 - patch

Tomas Doran bobtfish at bobtfish.net
Tue Nov 16 15:40:36 GMT 2010


On 16 Nov 2010, at 15:28, Anthony Gladdish wrote:
> The from_session() method has a call to load() with no $c param. It  
> did previously, and this change is breaking my own authentication  
> tests.
> Catalyst-Authentication-Store-DBIx-Class 0.1400 test suite passes on  
> my attached patch, and my own authentication tests are now passing.
>
> Is there any reason why $c has been ommited here?

No, it's a bug.

> Can this patch be merged in please?

Can you extract a test from your tests and add it to the patch?

That'll stop the issue ever re-occurring..

Cheers
t0m




More information about the Catalyst mailing list