Commit e1bfcdec authored by Nicolas Dumazet's avatar Nicolas Dumazet

if one coordinate is out of bounds, the whole cell is out of bounds

* No need to examine further following coordinates
* No need to duplicate the same error message
* No need to no-op using set.add(alread_added_item)


git-svn-id: https://svn.erp5.org/repos/public/erp5/trunk@32816 20353a03-c40f-0410-a6d1-a30d3c3de9de
parent e5e60f6c
...@@ -681,6 +681,7 @@ class XMLMatrix(Folder): ...@@ -681,6 +681,7 @@ class XMLMatrix(Folder):
if coordinate >= len(base_item[i]): if coordinate >= len(base_item[i]):
addError("Cell %s is out of bound" % object_id) addError("Cell %s is out of bound" % object_id)
to_delete_set.add(object_id) to_delete_set.add(object_id)
break
if fixit and len(to_delete_set) > 0: if fixit and len(to_delete_set) > 0:
self.manage_delObjects(list(to_delete_set)) self.manage_delObjects(list(to_delete_set))
......
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