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

[Will Hawes]

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