addressed issue 916 and reverted some changes but not convinced this is always the right solution
This commit is contained in:
@@ -1 +1 @@
|
|||||||
Version 2.00.0 (2012-08-06 20:54:15) dev
|
Version 2.00.0 (2012-08-06 21:24:31) dev
|
||||||
|
|||||||
+4
-1
@@ -7487,7 +7487,10 @@ class Table(dict):
|
|||||||
new_fields.append((ofield,ofield.update))
|
new_fields.append((ofield,ofield.update))
|
||||||
for ofield in self:
|
for ofield in self:
|
||||||
if ofield.compute:
|
if ofield.compute:
|
||||||
new_fields.append((ofield,ofield.compute(Row(fields))))
|
try:
|
||||||
|
new_fields.append((ofield,ofield.compute(Row(fields))))
|
||||||
|
except KeyError:
|
||||||
|
pass
|
||||||
if not update and ofield.required and not ofield.name in new_fields_names:
|
if not update and ofield.required and not ofield.name in new_fields_names:
|
||||||
raise SyntaxError,'Table: missing required field: %s' % ofield.name
|
raise SyntaxError,'Table: missing required field: %s' % ofield.name
|
||||||
return new_fields
|
return new_fields
|
||||||
|
|||||||
Reference in New Issue
Block a user