Re: [struts] More Struts Insanity

2008-03-05 Thread Chris Pratt
On Wed, Mar 5, 2008 at 10:53 PM, Dale Newfield <[EMAIL PROTECTED]> wrote: > Chris Pratt wrote: > > > > > > > Was that case typo (CurrentMember instead of currentMember) introduced > in transcription? I would assume it would not have produced "true" if > it were in your code... > Yup, it was

Re: [struts] More Struts Insanity

2008-03-05 Thread Dale Newfield
Chris Pratt wrote: Was that case typo (CurrentMember instead of currentMember) introduced in transcription? I would assume it would not have produced "true" if it were in your code... -Dale - To unsubscribe, e-mail: [EM

Re: [struts] More Struts Insanity

2008-03-05 Thread Dave Newton
--- Dale Newfield <[EMAIL PROTECTED]> wrote: > ...some of my models implement Comparable, but not nearly all. Not sure > how that slipped through... Probably most of the time people aren't comparing entire objects, but simple properties like Strings or numbery-things, so it hasn't come up until

Re: [struts] More Struts Insanity

2008-03-05 Thread Dave Newton
--- Dale Newfield <[EMAIL PROTECTED]> wrote: > There's a chance those bugs were introduced between 2.6.11 and 2.7, but > I'm betting not. Nope; it's in 2.6.9 at least. The behavior happens because of the isAssignableFrom sequence I mentioned in my previous post. Dave --

Re: [struts] More Struts Insanity

2008-03-05 Thread Dale Newfield
Dave Newton wrote: --- Chris Pratt <[EMAIL PROTECTED]> wrote: Notice that the last statement (using the OGNL == operator) doesn't return anything at all, which makes it useless in a test. OGNL actually checks for isAssignableFrom; if either class isAssignableFrom the other it checks for Compa

Re: [struts] More Struts Insanity

2008-03-05 Thread Dale Newfield
Chris Pratt wrote: I added some instrumentation and it appears that OGNL is not calling the equals method when I use the == operator, only when I explicitly call .equals(). Look! A clue! I believe the jar file is ognl-2.6.11.jar Yep: http://jira.opensymphony.com/browse/OGNL-96 http://jira

Re: [struts] More Struts Insanity

2008-03-05 Thread Chris Pratt
On Wed, Mar 5, 2008 at 9:56 PM, Dale Newfield <[EMAIL PROTECTED]> wrote: > Chris Pratt wrote: > > > > Just to be pedantic, can you also verify that: > > > > yields: > > > > ? > All correctly implemented equals methods are symmetric, so in the event > OGNL decides that neither of these ar

Re: [struts] More Struts Insanity

2008-03-05 Thread Dale Newfield
Chris Pratt wrote: Just to be pedantic, can you also verify that: yields: ? All correctly implemented equals methods are symmetric, so in the event OGNL decides that neither of these are null and that it should call the equals() method, it could theoretically do so in either order. I