Just out of curiousity though, can anyone explain why ConvertedLegacyTest was functioning properly before this change?
I changed from "id_i" to "id" for the first version of the patch that was more restrictive then the final committed code. The first version made sure you specify an id regardless of the allowDups value. In the committed code, it only checks the uniqueKey if it needs to. ConvertedLegacyTest can revered to a pre 520088 version with "id_i" and it will still pass. I just noticed that line 734 still referes to "id_i" and should change to "id" (though it does not affect the test) ryan
