Marcus Ramberg wrote:
> Simon Flack wrote:
>
>> Marcus Ramberg wrote:
>>
>>> Your proposed solution feels like a hack. The fact is, ::Base is too
>>> CDBI centric, and this would be a good way to start remedying that..
>>> populate_objects would be an abstract method in ::Base, and
>>> implemented with retrieve in ::CDBI ... That means that CDBI is
>>> optimised for the common cases (retrieving by PK from args) and
>>> easily overridable for custom actions ( not populating at all,
>>> populating from node, whatever you like). And the best part is, this
>>> won't break anything in existing apps.
>>
>>
>>
>> Ok, lets go with this. But I think I'd rather it be fetch_objects,
>> i.e. it shouldn't set objects itself, just return them. I think
>> that'll make it more reusable.
>
>
> oki.
>
> $r->objects([$self->fetch_objects($r)]);
>
> in process? I'll implement this.
Yep. Do we still need get_object_id, or is it redundant now?
--simonflk
_______________________________________________
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