move import match to base

[emjay0921]

No description provided.

[nhatnm0612]

Suggested change

	@api.onchange("field_id")
	def _onchange_field_id(self):
	for rec in self:
	field_id = rec.field_id.id
	fields_list = []
	for field in rec.match_id.field_ids:
	new_id_str = str(field.id)
	if not new_id_str.find("NewId_'virtual"):
	fields_list.append(field.field_id.id)
	duplicate_counter = 0
	for duplicate_field in fields_list:
	if duplicate_field == field_id:
	duplicate_counter += 1
	if duplicate_counter > 1:
	raise ValidationError(_("Field '%s', already exists!") % rec.field_id.field_description)
	_sql_constraints = [
	(
	"field_id_uniq",
	"UNIQUE(field_id, match_id)",
	"Field should be unique for each match",
	)
	]

[emjay0921]

This is edited, sql_constraints is not good here is we should allow different sub-fields for a one2many field.

[nhatnm0612]

Suggested change

	_sql_constraints = [
	(
	"model_id_uniq",
	"UNIQUE(model_id)",
	"Model should be unique!",
	)
	]

[emjay0921]

This will be removed, as we can have multiple Import Match Configuration for a single model.

[nhatnm0612]

Suggested change

	available = self.search([("model_name", "=", model_name)])
	field_to_match = []
	for record in available:
	field_to_match.append(record.field_ids.mapped("name"))
	for f in record.field_ids:
	if f.name in fields or f.field_id.name in fields:
	result |= record
	available = self.search([("model_name", "=", model_name)], limit=1)
	if not available:
	return [], []
	field_to_match = []
	for f in available.field_ids:
	if f.name in fields or f.field_id.name in fields:
	result |= available
	field_to_match.append(f.name)

[emjay0921]

Added a boolean field in import match to have an option if matched data will overwitten or ignored.

[gonzalesedwin1123]

Fix the errors in the automated test