On Jan 4, 2007, at 11:31 AM, David Blevins wrote:

On Jan 4, 2007, at 11:06 AM, Mohammad Nour El-Din wrote:

Hi All...

Would someone please review the patch of JIRA
http://issues.apache.org/jira/browse/OPENEJB-368 which solves both
OPENEJB-368 and http://issues.apache.org/jira/browse/OPENEJB-159

Looking into it!

All committed!

Nice patch, Mohammad. I did change a couple things. The lookup method was accepting "java:comp/env" type lookups which it shouldn't, so I just changed it to tack that on before doing the lookup. I also renamed the test case to follow the ContextLookupTests that I created for you earlier and also fixed the lookup strings to not contain "java:comp/env/".

Thank you very much!

-David

Reply via email to