Browse Source

#8671 Adding documentation and another assert.

Florian Krämer 10 years ago
parent
commit
85bd4edc0e
2 changed files with 4 additions and 1 deletions
  1. 3 1
      src/ORM/Table.php
  2. 1 0
      tests/TestCase/ORM/TableTest.php

+ 3 - 1
src/ORM/Table.php

@@ -1213,7 +1213,9 @@ class Table implements RepositoryInterface, EventListenerInterface, EventDispatc
      * called allowing you to define additional default values. The new
      * entity will be saved and returned.
      *
-     * @param array|\Cake\ORM\Query $search The criteria to find existing records by.
+     * @param array|\Cake\ORM\Query $search The criteria to find existing
+     *   records by. Note that when you pass a query object you'll have to use
+     *   the 2nd arg of the method to modify the entity data before saving.
      * @param callable|null $callback A callback that will be invoked for newly
      *   created entities. This callback will be called *before* the entity
      *   is persisted.

+ 1 - 0
tests/TestCase/ORM/TableTest.php

@@ -5388,6 +5388,7 @@ class TableTest extends TestCase
         $article = $articles->findOrCreate($query);
         $this->assertEquals('First Article', $article->title);
         $this->assertEquals(2, $article->author_id);
+        $this->assertFalse($article->isNew());
     }
 
     /**