Reading this code I can't help but try to evaluate it in my head. Question: is there a bug for the assignment of party_not_at_fault? Would you want party2 to b

Reading this code I can't help but try to evaluate it in my head. Question: is there a bug for the assignment of party_not_at_fault? Would you want party2 to be selected if the condition is true? Other languages I've used have the first part be the true branch, and the other the false branch.