In PropertySetter.computeAggregationType(), the switch statement should be completed:

if (addMethod != null) {
AggregationType type = computeRawAggregationType(addMethod);
switch (type)

{ case NOT_FOUND: return AggregationType.NOT_FOUND; case AS_BASIC_PROPERTY: return AggregationType.AS_BASIC_PROPERTY_COLLECTION; case AS_COMPLEX_PROPERTY: return AggregationType.AS_COMPLEX_PROPERTY_COLLECTION; }

}

to something like this (unsure whether logic correct, which is why switch statements should be completed, right?):

if (addMethod != null) {
AggregationType type = computeRawAggregationType(addMethod);
switch (type)

{ case NOT_FOUND: return AggregationType.NOT_FOUND; case AS_BASIC_PROPERTY: return AggregationType.AS_BASIC_PROPERTY_COLLECTION; case AS_COMPLEX_PROPERTY: return AggregationType.AS_COMPLEX_PROPERTY_COLLECTION; case AS_BASIC_PROPERTY_COLLECTION: // FALL-THROUGH case AS_COMPLEX_PROPERTY_COLLECTION: throw new IllegalStateException(""Can't happen: " + type); break; }

}