Propel icon indicating copy to clipboard operation
Propel copied to clipboard

Incorrect checking if primary key columns are all NULL in equals()

Open dymoksc opened this issue 9 years ago • 1 comments

When model is set to use multiple columns as its primary key it still has if (null === $this->getPrimaryKey() || null === $obj->getPrimaryKey()) condition in generated equals() method instead of using isPrimaryKeyNull method for this test. This leads to confirming, that two models are equal despite the fact they are not.

dymoksc avatar Jul 13 '16 09:07 dymoksc

Care to send us a pull-request?

marcj avatar Sep 01 '16 11:09 marcj