Commit 472d8db6 authored by Kazuhiko Shiozaki's avatar Kazuhiko Shiozaki

compare by str not by its md5 hash. this is faster in same case, and much faster in different case.


git-svn-id: https://svn.erp5.org/repos/public/erp5/trunk@43844 20353a03-c40f-0410-a6d1-a30d3c3de9de
parent bb7019c0
...@@ -38,8 +38,6 @@ from OFS.Image import Pdata ...@@ -38,8 +38,6 @@ from OFS.Image import Pdata
import cStringIO import cStringIO
from Products.ERP5Type.Utils import deprecated from Products.ERP5Type.Utils import deprecated
import md5
def _unpackData(data): def _unpackData(data):
""" """
Unpack Pdata into string Unpack Pdata into string
...@@ -141,10 +139,7 @@ class File(Document, CMFFile): ...@@ -141,10 +139,7 @@ class File(Document, CMFFile):
def _setFile(self, data, precondition=None): def _setFile(self, data, precondition=None):
if data is not None and self.hasData() and \ if data is not None and self.hasData() and \
md5.md5(str(data.read())).digest() ==\ str(data.read()) == str(self.getData()):
md5.md5(str(self.getData())).digest():
# Compute md5 hash only if there is something to hash on both sides.
#
# Same data as previous, no need to change it's content # Same data as previous, no need to change it's content
return return
CMFFile._edit(self, precondition=precondition, file=data) CMFFile._edit(self, precondition=precondition, file=data)
......
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