Changeset 6991 for main/waeup.sirp/trunk/src/waeup/sirp

Timestamp:

3 Nov 2011, 15:33:53 (13 years ago)

Author:

Henrik Bettermann

Message:

Add comment from Martijn why 'hostel._p_changed = True' might be necessary.

File:

• main/waeup.sirp/trunk/src/waeup/sirp/hostels/hostel.py (modified) (1 diff)

main/waeup.sirp/trunk/src/waeup/sirp/hostels/hostel.py

@@ -148 +148 @@
             bt = u'%s_%s_reserved' % (sh, sex)
             hostel.beds_reserved.append(bed_string)
+            # Comment of Martijn:
+            # If you have a non-Persistent subobject (like a list) and you change it,
+            # you need to manually flag the persistence machinery on the object that
+            # its subobject changed, with _p_changed. This is only necessary if some
+            # of the objects are not sublclasses of Persistent. For common built-in
+            # collections in Python such as list and dictionary there are replacements
+            # (PersistentList, PersistentMapping), and more advanced building blocks
+            # for indexes (BTrees), that don't have this issue.
             hostel._p_changed = True
             message = u'reserved'