From c227a9a0fe5eef5baefd605aefb85310872de1ac Mon Sep 17 00:00:00 2001 From: Abdulrahman Hawa Date: Tue, 3 Jan 2017 15:29:00 +0400 Subject: [PATCH] Add check for mapped related keys In the case of a relation key mapping to a non-primary key in the related model, an exception will happen. For example: 'related' => array(self::BELONGS_TO, 'RelatedModel', array('field' => 'non-pk')), --- EActiveRecordRelationBehavior.php | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/EActiveRecordRelationBehavior.php b/EActiveRecordRelationBehavior.php index 15931ea..484660e 100644 --- a/EActiveRecordRelationBehavior.php +++ b/EActiveRecordRelationBehavior.php @@ -297,7 +297,8 @@ protected function populateBelongsToAttribute($name, $relation, $beforeSave = tr } // @todo add support for composite primary keys - if (!is_array($pk)) { + // @todo add support for relation keys mapping + if (!is_array($pk) && !is_array($relation[2])) { $this->owner->setAttribute($relation[2], $pk); } }