On 16 Apr 2016, at 8:42 am, Chuck Hill <[email protected]> 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: > 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? -- Paul Hoadley http://logicsquad.net/
_______________________________________________ Do not post admin requests to the list. They will be ignored. Webobjects-dev mailing list ([email protected]) Help/Unsubscribe/Update your Subscription: https://lists.apple.com/mailman/options/webobjects-dev/archive%40mail-archive.com This email sent to [email protected]
