For models with composite primary keys, make the Model#primary_key array immutable. #730

merged 1 commit into from Nov 4, 2013


None yet
2 participants

chanks commented Nov 2, 2013

Modifying this array can lead to bad things, so per IRC conversation, freeze it when it's set.


jeremyevans commented Nov 2, 2013

This looks good. I'll merge and test it on Monday. Thanks!

@jeremyevans jeremyevans merged commit 7dc5425 into jeremyevans:master Nov 4, 2013

chanks deleted the chanks:immutable_composite_primary_key branch Nov 11, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment