On Wed, Oct 8, 2008 at 7:23 PM, Jared Rypka-Hauer wrote: > > Are you getting an error? Perhaps sending in the error regarding > makeEventBean() would be better in this case than sending in a patch. > Your patch isn't going to work because the CollectionBeanMaker is set > up to handle typed structs for passing to remoting endpoints. In > order for that to work, CollectionBeanMaker doesn't need a setter to > inject a value from the collection into the target and your patch > would break it.
This is what I ran into: using Railo (dunno if it's specific), when I do a makeEventBean on a reactor record object (without passing in fields to set), it's setting all the event variables on the object. So there's eventName, myself, etc.. stuck in there, which I honestly wouldn't have noticed had not there been a variable named "Save" that is a string, that replaced the Reactor record object's save(). I'm sure just passing in the field names would solve the problem, although I didn't try. I thought that *only* setters would be set, see. Didn't expect all the viewstate/event variables to be added if you didn't specify what to set. Or at least that's the behavior I'm seeing, but it sounds like it's expected, and needed. I can use one of your ideas if I really want that "no arg" flexibility. Pros and cons, anyways. [snip excellent suggestions] > As for your ReactorAdapter patch, I believe you're correct. I don't > actually have commit permissions to the MG:G repo at the moment (JOE! > heh), but I'll follow up to make sure Dan or Joe get this committed > if our discussion proves it to be a valid patch. Yeah, I haven't vetted either patch very much, so, whatever's clever. The Reactor one looks like an old one, and seems fine so far, and I'm only using makeEventBean for ORM beans, really, so I'd never notice the break that would've introduced. Thanks for the run-down on the BeanMaker! > In any case, thanks for submitting patches at all... you're a shining > example to the rest of the community. Gold star for your efforts! ;) Haha! Thanks, J! Stars and smilies are what I live for. =] Peace for now, :Denny --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "model-glue" 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/model-glue?hl=en For more about Model-Glue, check http://www.model-glue.com . -~----------~----~----~----~------~----~------~--~---
