[Catalyst] Catalyst::Model::CDBI::CRUD observation

[Will Hawes]

Simon

Sorry, no patch I'm afraid.

By the way, if you intend to fix this, could I add my vote to what Michael said earlier in this thread - the correct place for all these functions list, view, etc should really be the controller, not the model.

WH

----- Original Message -----
From: Vsevolod (Simon) Ilyushchenko <simonf at cshl.edu>
To: Will Hawes <info at whawes.co.uk>
Cc: catalyst at lists.rawmode.org
Sent: Fri, 29 Apr 2005 20:41:04 +0100
Subject: Re: [Catalyst] Catalyst::Model::CDBI::CRUD observation


> Will,
> 
> Good point. Will fix.
> 
> Have you been able to use the 'limit' clause successfully with Hops? Do 
> you have a patch?
> 
> Thanks,
> Simon
> 
> Will Hawes wrote on 04/29/2005 12:14 PM:
> > With regard to the handling of CDBI objects in the above module. The process()
> method (line 124) stashes the class for the current request and, if an id is
> present in the URL, does a retrieve and stashes the returned item. But if no id
> is present, no CDBI objects are created.
> > 
> > In Hops, the objects are actually created in the "list" template, which does
> class->retrieve_all and builds a HTML table. So the data is being retrieved in
> the View part of the app?!
> > 
> > It's late on Friday and maybe I'm missing something here, but surely CRUD.pm
> should be following Maypole's example and stashing an array(ref) of CDBI objects
> to pass to the template instead.
> > 
> > Regards
> > 
> > WH
> > 
> > 
> > _______________________________________________
> > Catalyst mailing list
> > Catalyst at lists.rawmode.org
> > http://lists.rawmode.org/mailman/listinfo/catalyst
> > 
> > 
> 
> -- 
> 
> Simon (Vsevolod ILyushchenko)   simonf at cshl.edu
> 				http://www.simonf.com
> 
> Terrorism is a tactic and so to declare war on terrorism
> is equivalent to Roosevelt's declaring war on blitzkrieg.
> 
> Zbigniew Brzezinski, U.S. national security advisor, 1977-81
> 
>