> Unfortunately, I don't think it's a good idea to include this into > Sequel, as it modifies the number of arguments to the :eager_loader > proc, which would break all custom eager loaders (and I don't like > hacks like checking proc arity). There would have to be wide support > for Sequel users for this patch's inclusion before I would consider > it.
I agree the ability to pass options to the eager loaded dataset on runtime is a good idea. Is there a way to achieve this without changing the number of arguments to :eager_loader? I'm thinking that eager_loading_dataset can be instance_execed from post_load so that calling_ds doesn't have to be passed in. In order to do this, the default :eager_loader would also need to remove its dependency on the model variable. Would this work? -- You received this message because you are subscribed to the Google Groups "sequel-talk" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/sequel-talk?hl=en.
