search

ahri / ahriphplibs

My PHP Libraries
http://ahri.net
GNU Affero General Public License v3.0
3 stars 0 forks source link

__setup() call for Relationships #33

Closed ahri closed 14 years ago

ahri commented 14 years ago

It works very nicely for TLOs, so why not TLORelationships?

ahri commented 14 years ago

Can't behave in quite the same way as the TLO version (i.e. export the stuff in TLORelationshipResult->fetch() to __setup()) because of the key name stuff.

Could fix this by adding to the select so that we know what the keys are for the relationship, and also what the keys and class are for the relation. Is it any better adding invisible coupling though? Probably not.

In fact it might be a good idea to move the similar stuff (key processing) out of TLO->__setup() and into TLOObjectResult->fetch() for consistency.

ahri commented 14 years ago

Implement TLORelationship->__setup() and refactor stuff so that TLO/TLORelationship (mostly) do initialisation the same way. Closed by b834c93c10061b3d600784c44cbd415e67cef4c7