[Maypole-dev] Maypole::Model::CDBI dependencies

From: Dave Howorth (dhoworth at mrc-lmb.cam.ac.uk)
Date: Fri Jan 28 2005 - 13:38:57 GMT


I see from the roadmap that M-M-CDBI is going to get its inheritance
improved (it's a great concept to think about if you could upgrade your
real-life parents like this :)

I'd like to suggest that something else be wrapped in with this.
Actually, I suspect it already is but I just want to be sure. At present
M-M-CDBI starts like this:

package Maypole::Model::CDBI;
use base qw(Maypole::Model::Base Class::DBI);
use Class::DBI::AsForm;
use Class::DBI::FromCGI;
use Class::DBI::Loader;
use Class::DBI::AbstractSearch;
use Class::DBI::Plugin::RetrieveAll;
use Class::DBI::Pager;

use Lingua::EN::Inflect::Number qw(to_PL);
use CGI::Untaint;
use strict;

Looking at that list:

- I use a heavily-patched version of AsForm, which I'd rather rename
something else to avoid mistakes, then I wouldn't want to use AsForm.

- I've just written a do_edit action that uses Data::FormValidator, so I
won't use FromCGI.

- I don't use C-DBI-Loader at present, so I'd rather not have to load that.

- Because I'm planning to use D-FV, I don't need to use CGI::Untaint.

- also, I don't understand 'use Lingua::EN::Inflect::Number qw(to_PL);'
It doesn't seem to be used anywhere? [If it was, I almost certainly
wouldn't be using that code since I've overridden everywhere that
Maypole generates text for display.]

So I'd like some way to not have to load all these modules. Even
something as simple as putting all the use statements in a method so it
could be overridden would be adequate.

Thanks,
Dave

_______________________________________________
maypole-dev mailing list
maypole-dev at lists.netthink.co.uk
http://lists.netthink.co.uk/listinfo/maypole-dev



This archive was generated by hypermail 2.1.3 : Thu Feb 24 2005 - 22:25:57 GMT