Skip to content
Permalink
Browse files

Fixing problems found in review

  • Loading branch information...
lorenzo committed Dec 18, 2013
1 parent 8f6b962 commit 1a9f262f90e95f4f0f94993399b955765fe7438c
@@ -332,10 +332,10 @@ public function saveStrategy($strategy = null) {
* between each side of this association. It will not destroy existing ones even
* though they may not be present in the array of entities to be saved.
*
* When using the 'replace' strategy, existing links will be removed and new ones
* will be created in between both tables in the association. If there exists
* links in the database to some of the entities intended to be saved by this method,
* they will be updated, not deleted.
* When using the 'replace' strategy, existing links will be removed and new links
* will be created in the joint table. If there exists links in the database to some
* of the entities intended to be saved by this method, they will be updated,
* not deleted.
*
* @param \Cake\ORM\Entity $entity an entity from the source table
* @param array|\ArrayObject $options options to be passed to the save method in
@@ -1106,7 +1106,7 @@ public function testSaveWithReplaceReturnFalse() {
$assoc->expects($this->once())->method('replaceLinks')
->with($entity, $entity->tags, $options)
->will($this->returnValue(false));
$this->assertSame(false, $assoc->save($entity, $options));
$this->assertFalse($assoc->save($entity, $options));
}
}

0 comments on commit 1a9f262

Please sign in to comment.
You can’t perform that action at this time.