Simon Flack wrote:
> Certainly, my only concern is about "fixing" something in Maypole,
> that's fixable by using documented Class::DBI methods. If setting
> 'Essentials' columns doesn't work, then perhaps we need to look at
> how Class::DBI::AsForm can be improved.
>
> I'm not trying to pass the buck, but I'm not sure that this is a
> problem Maypole needs to address directly other than perhaps
> documenting the issue in a FAQ. I didn't mean to imply that such
> discussions weren't welcome here.
offlist...
Hi Simon,
I agree with you about this. I had two motivations for the angle I was
pushing on the list:
(1) I wanted to encourage Brian, and anybody else, to post code to the list.
(2) IMHO, Brian's solution is better than the code in AsForm. There
really is no need to instantiate a potentially large bunch of objects
just so you can extract one field from them! And again IMHO there's no
reason to presuppose that the column you need for this purpose is the
same as the ones that are 'Essential' when you are actually doing some
object manipulation elsewhere in the application. They're potentially
different concepts so why bind them together if it isn't necessary?
But I certainly don't feel my understanding of the code is anything like
as good as Simon or Tony or many of the other luminaries on the list.
I have to declare a vested interest. I have some other issues with
AsForm, but have had no luck getting any feedback at all. I wonder if
I've annoyed people or broken some rule of netiquette, but the result is
I'm spending a lot more time getting intimate with modules that I'd
rather just be a user of.
So I'm certainly not feeling brave enough to have said the last sentence
of your first paragraph even though I absolutely agree with it!
Cheers, Dave
This archive was generated by hypermail 2.1.3 : Thu Feb 24 2005 - 22:25:57 GMT