[PATCH] testbot: Document a quirk of Collection::Add().

Francois Gouget fgouget at codeweavers.com
Wed Dec 6 21:13:09 CST 2017


Not only does it not force loading from the database, it marks the
Collection as loaded. So if Add() is the first call on a Collection,
calling GetKeys() or GetItems() later on will only return the new
Item.

Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
---
 testbot/lib/ObjectModel/Collection.pm | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/testbot/lib/ObjectModel/Collection.pm b/testbot/lib/ObjectModel/Collection.pm
index 9d3d3aef..1b29ec16 100644
--- a/testbot/lib/ObjectModel/Collection.pm
+++ b/testbot/lib/ObjectModel/Collection.pm
@@ -168,6 +168,11 @@ sub Load($)
 
 Instantiates a new Item and adds it to the Collection.
 
+Note that not only does calling Add() not force loading from the database,
+it marks the Collection as loaded. So if Add() is the first call on a
+Collection, calling GetKeys() or GetItems() later on will only return the new
+Item.
+
 See Item->new() for details on the Item's default column values and for
 important information regarding its primary key.
 
-- 
2.15.0




More information about the wine-devel mailing list