vassili/seitime-frappe
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 4

Merge pull request #1224 from nabinhait/unique

Browse source 
[fix] Adding Unique Constraint
This commit is contained in:
Anand Doshi 2015-07-30 11:56:14 +05:30
commit 0f3d03e167
1 changed files with 7 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
frappe/model/db_schema.py
View file

@ -75,7 +75,6 @@ class DbTable:
self.add_index = []
self.drop_index = []
self.set_default = []
# load
@ -240,6 +239,9 @@ class DbTable:
# sanitize
if e.args[0]==1060:
frappe.throw(str(e))
elif e.args[0]==1062:
fieldname = str(e).split("'")[-2]
frappe.throw(_("{0} field cannot be set as unique, as there are non-unique existing values".format(fieldname)))
else:
raise e
@ -286,17 +288,14 @@ class DbColumn:
# type
if (current_def['type'] != column_def) or (self.unique and not current_def['unique'] \
and column_def in ('text', 'longtext')):
and column_def not in ('text', 'longtext')):
self.table.change_type.append(self)
else:
# index
if (current_def['index'] and not self.set_index):
self.table.drop_index.append(self)
if (current_def['unique'] and not self.unique) and not (column_def in ('text', 'longtext')):
self.table.drop_index.append(self)
if current_def['index'] and not self.set_index and not self.unique:
self.table.drop_index.append(self)
if (not current_def['index'] and self.set_index) and not (column_def in ('text', 'longtext')):
self.table.add_index.append(self)