pombreda / formalchemy

Automatically exported from code.google.com/p/formalchemy
MIT License
0 stars 0 forks source link

Add support for dynamic loaders #81

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
currently formalchemy supports only relation(), but not dynamic_loaders.

Original issue reported on code.google.com by iElect...@gmail.com on 22 Mar 2009 at 8:07

GoogleCodeExporter commented 9 years ago
http://dpaste.com/18125/

Original comment by iElect...@gmail.com on 23 Mar 2009 at 8:40

GoogleCodeExporter commented 9 years ago
applied.  for future references, please perform the diff from the top-level
formalchemy source directory; this makes it easier for patch to figure out what 
file
is being referenced.

Original comment by jbel...@gmail.com on 23 Mar 2009 at 10:31

GoogleCodeExporter commented 9 years ago
(please elaborate on your comment in #formalchemy that further work is 
required.)

Original comment by jbel...@gmail.com on 23 Mar 2009 at 10:32

GoogleCodeExporter commented 9 years ago
alright:

dynamic loader is basically a replacement for collection on multi-one and 
multi-multi
relations that returns an query object that can be later abused:)

it's main purpose is to eliminate loading of (too)many objects to memory.

it can be simply transformed to normal collection like user.posts.all()

Original comment by iElect...@gmail.com on 23 Mar 2009 at 10:54

GoogleCodeExporter commented 9 years ago
Thinking about this I'm not sure how FA can usefully support dynamic loaders.

Since the point is to show only part of the child objects at a time, and you 
can't
edit this 'view' directly (append and remove are supported but not wholesale 
"this
list should be your new collection of child objects" like normal relations) I'm 
not
what we can usefully do besides "don't crash."

So I've excluded dynamic loaders from the fields list entirely unless you can 
think
of a reason to keep them in.

Original comment by jbel...@gmail.com on 26 Mar 2009 at 10:03