Re: Issues with ERXArrayUtilities.arrayMinusArray() and arrayMinusObject()
Re: Issues with ERXArrayUtilities.arrayMinusArray() and arrayMinusObject()
- Subject: Re: Issues with ERXArrayUtilities.arrayMinusArray() and arrayMinusObject()
- From: Paul Hoadley <email@hidden>
- Date: Sat, 16 Apr 2016 08:57:52 +0930
On 16 Apr 2016, at 8:42 am, Chuck Hill <email@hidden> wrote:Good catch! It should use removeAll() not remove().
Even using removeObject() (which is what the original implementation used) reverts the behaviour. Johann, would you have any objection to me making this change to restore the old behaviour of arrayMinusObject()?
And while we’re here:
1. With the new method signature for arrayMinusArray(), WOUnit’s MockEditingContext can’t even find that method:
java.lang.NoSuchMethodError: er.extensions.foundation.ERXArrayUtilities.arrayMinusArray(Lcom/webobjects/foundation/NSArray;Lcom/webobjects/foundation/NSArray;)Lcom/webobjects/foundation/NSArray;
at com.wounit.rules.MockEditingContext.saveChanges(MockEditingContext.java:310)
It’s not immediately clear to me why this would be the case, as obviously NSArray implements Collection, and the new signature is:
public static <T> NSArray<T> arrayMinusArray(Collection<T> array, Collection<?> minus)
Any thoughts on this one? It’s a bit of a show-stopper for WOUnit at the moment.
Presumably the intent was that the new signature would be a superset of the old one—can anyone suggest why WOUnit is complaining?
|
_______________________________________________
Do not post admin requests to the list. They will be ignored.
Webobjects-dev mailing list (email@hidden)
Help/Unsubscribe/Update your Subscription:
This email sent to email@hidden