Cannot override instance methodMissing() in ExpandoMetaClass

Hi,
I am attaching patches for branches 1.5.x and 1.6.x that resolved this issue and an accompanying test case.

There was no issue with specifying methodMissing at instance level. Issue was related to how a method is identified internally as the special MethodMissing method.

A method gets registered as the special missingMethod only if its name is "methodMissing" and it takes 2 parameters - 1st String and 2nd Object.

In the following code, although the name is methodMissing, the types of parameters are [Object, Object] and hence it does not get registered as methodMissing at EMC level:

TestClass.metaClass.methodMissing = { method, args ->
    return method
}

So, one work around is to use the syntax:

TestClass.metaClass.methodMissing = { String method, args ->
    return method
}

Now "new TestClass().test()" correctly invokes missingMethod().

However, with the patch supplied, even if method name's type is not explicitly specified as String, it still gets handled and methodMissing works as expected.

rgds,
Roshan

Attaching the patch for 1.7.x as the issue was found there as well.

rgds,
Roshan

Since the solution is to use the String type for the name parameter I reduce the priority of this bug to minor. It is an annoying gotcha, which is the reason I did not close it as "won't fix". I also think that this might influence meta programming in an unexpected way, so 1.5.x and 1.6.x are for now out of the picture for this bug. I think it should be fixed for 1.7 only. If we have more experience with it we can still backport it.

So, I can just fix it on 1.7 and mark it as Fixed? Or should its status be left open?

apply the patch and mark it as fixed. I prefer followup issues.

Fixed in v1.7.