Commit cc266648 authored by Jim Fulton's avatar Jim Fulton

Bugs Fixed

----------

- The helper function ZODB.blob.is_blob_record didn't handle having
  None passed to it, but database "delete" records have None for their
  data.
parent 86a2eab6
...@@ -2,6 +2,17 @@ ...@@ -2,6 +2,17 @@
Change History Change History
================ ================
3.9.0c3 (2009-09-01)
====================
Bugs Fixed
----------
- The helper function ZODB.blob.is_blob_record didn't handle having
None passed to it, but database "delete" records have None for their
data.
3.9.0c2 (2009-09-01) 3.9.0c2 (2009-09-01)
==================== ====================
......
...@@ -946,7 +946,7 @@ def is_blob_record(record): ...@@ -946,7 +946,7 @@ def is_blob_record(record):
storage to another. storage to another.
""" """
if 'ZODB.blob' in record: if record and ('ZODB.blob' in record):
unpickler = cPickle.Unpickler(cStringIO.StringIO(record)) unpickler = cPickle.Unpickler(cStringIO.StringIO(record))
unpickler.find_global = find_global_Blob unpickler.find_global = find_global_Blob
......
...@@ -511,6 +511,11 @@ def is_blob_record(): ...@@ -511,6 +511,11 @@ def is_blob_record():
>>> ZODB.blob.is_blob_record('cWaaaa\nC\nq\x01.') >>> ZODB.blob.is_blob_record('cWaaaa\nC\nq\x01.')
False False
As does None, which may occur in delete records:
>>> ZODB.blob.is_blob_record(None)
False
>>> db.close() >>> db.close()
""" """
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment