Skip Menu |
 

This queue is for tickets about the Jifty CPAN distribution.

Report information
The Basics
Id: 24788
Status: resolved
Priority: 0/
Queue: Jifty

People
Owner: Nobody in particular
Requestors: jesse [...] bestpractical.com
Cc:
AdminCc:

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



Subject: Fw: [jifty-devel] Re: [Jifty-commit] r2084 -jifty/trunk/lib/Jifty/Action
Date: Mon, 05 Feb 2007 09:02:22 +1300
To: bug-jifty [...] rt.cpan.org
From: "Jesse Vincent" <jesse [...] bestpractical.com>
Download (untitled) / with headers
text/plain 2.2k
Add to docs. things referenced in schema, even indirectly need to be defined first. or could we fix this by creating a metaclass that's autoloaded and a blank childclass? Show quoted text
------- Original message ------- From: Gaal Yahas <gaal@forum2.org> Sent: 10/30, 17:10 It's in something I still haven't figured out totally: I have an action that's a subclass of J::A::R::Update, *and* I want to augment it with an action param schema of my own. What happened was that my sub record_class {} needed to come before defining the schema, and when it didn't, I was getting that error. On Mon, Oct 30, 2006 at 09:53:07AM -0500, Jesse Vincent wrote:
> > I 100% appreciate the improved error message, but how did you run into > the error condition? > > -j > > On Mon, Oct 30, 2006 at 03:08:11AM -0500, jifty-commit@lists.jifty.org wrote:
> > Author: gaal > > Date: Mon Oct 30 03:08:10 2006 > > New Revision: 2084 > > > > Modified: > > jifty/trunk/lib/Jifty/Action/Record.pm > > > > Log: > > improve an error message > > > > Modified: jifty/trunk/lib/Jifty/Action/Record.pm > > ============================================================================== > > --- jifty/trunk/lib/Jifty/Action/Record.pm?(original) > > +++ jifty/trunk/lib/Jifty/Action/Record.pm?Mon Oct 30 03:08:10 2006 > > @@ -42,7 +42,11 @@ > > > > sub record_class { > > my $self = shift; > > - $self->log->fatal("Jifty::Action::Record must be subclassed to be used"); > > + my $class = ref $self; > > + my $hint = $class eq __PACKAGE__ ? "" : > > + " (did you forget to override record_class in $class?)"; > > + $self->log->fatal("Jifty::Action::Record must be subclassed to be used" . > > + $hint); > > } > > > > =head2 new PARAMHASH > > _______________________________________________ > > Jifty-commit mailing list > > Jifty-commit@lists.jifty.org > > http://lists.jifty.org/cgi-bin/mailman/listinfo/jifty-commit > >
> > -- > _______________________________________________ > jifty-devel mailing list > jifty-devel@lists.jifty.org > http://lists.jifty.org/cgi-bin/mailman/listinfo/jifty-devel
-- Gaal Yahas <gaal@forum2.org> http://gaal.livejournal.com/
_______________________________________________ jifty-devel mailing list jifty-devel@lists.jifty.org http://lists.jifty.org/cgi-bin/mailman/listinfo/jifty-devel


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.