On Thu, Oct 22, 2009 at 8:07 AM, Dan Wilson <[email protected]> wrote: > I'm not sure I agree it is a bug. > > Frankly it seems like a slight change in semantics because the thing being > modified is the event, not just the variable. > > So the following line; > > <cfset something = event.getValue("Beer", "Leffe") /> > > Actually reads as: Please go into the event context, grab the thing named > beer, and put it inside the something variable. Should there not be anything > named beer, then please use Leffe as the value in the event context.
I think that any method called getSomething() that actually modifies the object is very poor semantics. getSomething() should always be effectively a readonly operation. FWIW, ColdBox, Fusebox and Mach-II all treat event.getValue() / event.getArg() as pure readonly operations that do NOT modify the event object which is definitely what I think most OO folks would expect. Model-Glue stands alone in its behavior. -- Sean A Corfield -- (904) 302-SEAN Railo Technologies US -- http://getrailo.com/ An Architect's View -- http://corfield.org/ "If you're not annoying somebody, you're not really alive." -- Margaret Atwood --~--~---------~--~----~------------~-------~--~----~ Model-Glue Sites: Home Page: http://www.model-glue.com Documentation: http://docs.model-glue.com Bug Tracker: http://bugs.model-glue.com Blog: http://www.model-glue.com/blog 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 -~----------~----~----~----~------~----~------~--~---
