[17.0][MIG] product_configurator

product_configurator/models/product_config.py

@@ -40,7 +40,7 @@
  continue
  for line in lines[:-1]:
  if line.operator == "or":
  computed_domain.append("|")
  computed_domain.append(
  (line.attribute_id.id, line.condition, line.value_ids.ids)
  )
@@ -96,25 +96,25 @@
  @api.depends("attribute_id")
  def _compute_template_attribute_value_ids(self):
  for domain in self:
  domain.template_attribute_value_ids = (
  domain._get_allowed_attribute_value_ids()
  )
 
  def _compute_attribute_id_domain(self):
  if "product_attribute_ids" in self.env.context:
  return [("id", "in", self.env.context["product_attribute_ids"][0][2])]
  return []
 
  def _get_allowed_attribute_value_ids(self):
  self.ensure_one()
  product_template = self.env["product.template"]
  if self.env.context.get("product_tmpl_id"):
  product_template = product_template.browse(
  self.env.context.get("product_tmpl_id")
  )
  template_lines = product_template.attribute_line_ids
  attribute_values = self.attribute_id.value_ids
  return (
  product_template
  and (template_lines.mapped("value_ids") & attribute_values)
  or attribute_values
@@ -177,9 +177,9 @@
  )
  def _compute_template_attribute_ids(self):
  for config_line in self:
  product_template = config_line.product_tmpl_id
  attribute_line_ids = product_template.attribute_line_ids
  config_line.template_attribute_ids = attribute_line_ids.mapped(
  "attribute_id"
  )
 
@@ -266,8 +266,8 @@
  product_tmpl_id=cfg_img.product_tmpl_id.id,
  final=False,
  )
  except ValidationError as exc:
  raise ValidationError(
  _(
  "Values entered for line '%s' generate "
  "a incompatible configuration"
@@ -321,7 +321,7 @@
  lambda line: line != config_step
  ).mapped("config_step_id")
  if config_step.config_step_id in cfg_steps:
  raise ValidationError(
  _("Cannot have a configuration step defined twice.")
  )
 
@@ -369,14 +369,14 @@
  def _compute_config_step_name(self):
  """Get the config.step.line name using the string stored in config_step
  field of the session"""
  cfg_step_line_obj = self.env["product.config.step.line"]
  cfg_session_step_lines = self.mapped("config_step")
  cfg_step_line_ids = set()
  for step in cfg_session_step_lines:
  try:
  cfg_step_line_ids.add(int(step))
  except ValueError:
  _logger.debug("Step from session not valid")
  cfg_step_lines = cfg_step_line_obj.browse(cfg_step_line_ids)
  for session in self:
  try:
@@ -386,7 +386,7 @@
  )
  session.config_step_name = config_step_line.name
  except Exception:
  _logger.debug("Invalid session data ignored")
  if not session.config_step_name:
  session.config_step_name = session.config_step
 
@@ -407,20 +407,20 @@
 
  product_tmpl = self.product_tmpl_id
 
  self = self.with_context(active_id=product_tmpl.id)
 
  value_ids = self.flatten_val_ids(value_ids)
 
  weight_extra = 0.0
  product_attr_val_obj = self.env["product.template.attribute.value"]
  product_tmpl_attr_values = product_attr_val_obj.search(
  [
  ("product_tmpl_id", "in", product_tmpl.ids),
  ("product_attribute_value_id", "in", value_ids),
  ]
  )
  for product_tmpl_attr_val in product_tmpl_attr_values:
  weight_extra += product_tmpl_attr_val.weight_extra
 
  return product_tmpl.weight + weight_extra
 
@@ -437,10 +437,10 @@
  cfg_session.weight = cfg_session.get_cfg_weight()
 
  def _compute_currency_id(self):
  main_company = self.env["res.company"]._get_main_company()
  for session in self:
  template = session.product_tmpl_id
  session.currency_id = (
  template.company_id.sudo().currency_id.id or main_company.currency_id.id
  )
 
@@ -507,7 +507,7 @@
  def _check_product_id(self):
  for session in self.filtered(lambda s: s.state == "done"):
  if not session.product_id:
  raise ValidationError(
  _("Finished configuration session must have a " "product_id linked")
  )
 
@@ -553,14 +553,14 @@
  if field_vals[0] == 6:
  field_val += field_vals[2] or []
  elif field_vals[0] == 4:
  field_val.append(field_vals[1])
  # field_val = [
  # i[1] for i in vals[field_name] if vals[field_name][0]
  # ] or vals[field_name][0][1]
  elif not attr_line.multi and isinstance(vals[field_name], int):
  field_val = vals[field_name]
  else:
  raise UserError(
  _("An error occurred while parsing value for attribute %s")
  % attr_line.attribute_id.name
  )
@@ -573,8 +573,8 @@
  val = vals.get(custom_field_name, False)
  if attr_line.attribute_id.custom_type == "binary":
  # TODO: Add widget that enables multiple file uploads
  val = [{"name": "custom", "datas": vals[custom_field_name]}]
  custom_val_dict.update({attr_id: val})
  # Ensure there is no standard value stored if we have switched
  # from selected value to custom value.
  attr_val_dict.update({attr_id: False})
@@ -634,7 +634,7 @@
  ).unlink()
 
  if custom_val_dict:
  binary_field_ids = (
  self.env["product.attribute"]
  .search(
  [
@@ -652,9 +652,9 @@
  continue
 
  if "custom_value_ids" not in update_vals:
  update_vals["custom_value_ids"] = []
 
  custom_vals = {"attribute_id": attr_id}
 
  if attr_id in binary_field_ids:
  attachments = [
@@ -665,11 +665,11 @@
  )
  for val in vals
  ]
  custom_vals.update({"attachment_ids": attachments})
  else:
  custom_vals.update({"value": vals})
 
  update_vals["custom_value_ids"].append((0, 0, custom_vals))
  self.write(update_vals)
 
  def write(self, vals):
@@ -677,7 +677,7 @@
  # TODO: Issue warning when writing to value_ids or custom_val_ids
  res = super().write(vals)
  if not self.product_tmpl_id:
  return res
  value_ids = self.value_ids.ids
  avail_val_ids = self.values_available(value_ids)
  if set(value_ids) - set(avail_val_ids):
@@ -685,9 +685,9 @@
  try:
  self.validate_configuration(final=False)
  except ValidationError as exc:
  raise ValidationError(_("{}".format(exc)))
  except Exception as exc:
  raise ValidationError(_("Invalid Configuration")) from exc
  return res
 
  @api.model_create_multi
@@ -711,7 +711,7 @@
  )
  value_ids = vals.get("value_ids")
  if value_ids:
  default_val_ids += value_ids[0][2]
  try:
  self.validate_configuration(
  value_ids=default_val_ids,
@@ -721,9 +721,9 @@
  # TODO: Remove if cond when PR with
  # raise error on github is merged
  except ValidationError as exc:
  raise ValidationError(_("%s") % exc.name) from exc
  except Exception as exc:
  raise ValidationError(
  _(
  "Default values provided generate an invalid "
  "configuration"
@@ -754,9 +754,9 @@
  try:
  self.validate_configuration()
  except ValidationError as exc:
  raise ValidationError(_("%s") % exc.name) from exc
  except Exception as exc:
  raise ValidationError(_("Invalid Configuration")) from exc
 
  duplicates = self.search_variant(
  value_ids=value_ids, product_tmpl_id=self.product_tmpl_id
@@ -783,7 +783,7 @@
  if value_ids is None:
  value_ids = self.value_ids.ids
 
  value_obj = self.env["product.attribute.value"].with_context(
  pricelist=pricelist.id
  )
  values = (
@@ -798,14 +798,14 @@
  value_ids = self.value_ids.ids
  vals = self._get_option_values(pricelist, value_ids)
  for val in vals:
  prices["vals"].append(
  (
  val.attribute_id.name,
  val.product_id.name,
  val.product_id.price,
  )
  )
  product = val.product_id.with_context(pricelist=pricelist.id)
  product_prices = product.taxes_id.sudo().compute_all(
  price_unit=product.price,
  currency=pricelist.currency_id,
@@ -814,10 +814,10 @@
  partner=self.env.user.partner_id,
  )
 
  total_included = product_prices["total_included"]
  taxes = total_included - product_prices["total_excluded"]
  prices["taxes"] += taxes
  prices["total"] += total_included
  return prices
 
  @api.model
@@ -929,7 +929,7 @@
  ("state", "=", state),
  ]
  if parent_id:
  domain.append(("parent_id", "=", parent_id))
  return domain
 
  def get_session_vals(self, product_tmpl_id, parent_id=None, user_id=None):
@@ -967,7 +967,7 @@
  if (value_ids or custom_value_ids) and state != "select":
  return False
  elif not (value_ids or custom_value_ids) and state != "select":
  raise UserError(
  _(
  "You must select at least one "
  "attribute in order to configure a product"
@@ -994,7 +994,7 @@
  if next_step:
  pass
  elif not (value_ids or custom_value_ids):
  raise UserError(
  _(
  "You must select at least one "
  "attribute in order to configure a product"
@@ -1140,7 +1140,7 @@
  step_to_open = step
  break
  if step_to_open:
  return "%s" % (step_to_open.id)
  return False
 
  @api.model
@@ -1271,7 +1271,7 @@
  # Validate custom values
  attr = line.attribute_id
  if attr.id in custom_vals:
  attr.validate_custom_val(custom_vals[attr.id])
  if final:
  common_vals = set(value_ids) & set(line.value_ids.ids)
  custom_val = custom_vals.get(attr.id)
@@ -1287,7 +1287,7 @@
  and not custom_val
  ):
  # TODO: Verify custom value type to be correct
  raise ValidationError(
  _("Required attribute '%s' is empty") % (attr.name)
  )
 
@@ -1364,7 +1364,7 @@
  custom_attrs_with_error = list(
  set(custom_vals.keys()) - set(custom_attr_ids)
  )
  custom_attrs_with_error = self.env["product.attribute"].browse(
  custom_attrs_with_error
  )
  error_message = _(
@@ -1377,7 +1377,7 @@
  )
  message_vals = ""
  for attr_id in custom_attrs_with_error:
  message_vals += f"\n {attr_id.name}: {custom_vals.get(attr_id.id)}"
  raise ValidationError(error_message % (message_vals))
 
  # Check if there are multiple values passed for non-multi attributes
@@ -1387,7 +1387,7 @@
  attrs_with_error = {}
  for line in mono_attr_lines:
  if len(set(line.value_ids.ids) & set(value_ids)) > 1:
  wrong_vals = self.env["product.attribute.value"].browse(
  set(line.value_ids.ids) & set(value_ids)
  )
  attrs_with_error[line.attribute_id] = wrong_vals
@@ -1402,7 +1402,7 @@
  )
  message_vals = ""
  for attr_id, vals in attrs_with_error.items():
  message_vals += "\n {}: {}".format(
  attr_id.name, ", ".join(vals.mapped("name"))
  )
  raise ValidationError(error_message % (message_vals))
@@ -1425,9 +1425,9 @@
  value_ids = list(set(value_ids) - set(custom_value_id.ids))
 
  if not product_tmpl_id:
  product_tmpl_id = self.product_tmpl_id
  if not product_tmpl_id:
  raise ValidationError(
  _(
  "Cannot conduct search on an empty config session "
  "without product_tmpl_id kwarg"
@@ -1485,9 +1485,9 @@
  if value_ids and value_ids[0]:
  for val in value_ids:
  if not val:
  continue
  if isinstance(val, list):
  flat_val_ids.add(val[1])
  elif isinstance(val, int):
  flat_val_ids.add(val)
  return list(flat_val_ids)
@@ -1496,8 +1496,8 @@
  if prices is None:
  prices = {}
  dp = None
  prices["taxes"] = formatLang(self.env, prices["taxes"], monetary=True, dp=dp)
  prices["total"] = formatLang(self.env, prices["total"], monetary=True, dp=dp)
  prices["vals"] = [
  (v[0], v[1], formatLang(self.env, v[2], monetary=True, dp=dp))
  for v in prices["vals"]
@@ -1510,19 +1510,19 @@
  :param custom_vals: dict {product.attribute.id: custom_value}
  :returns: list of custom values compatible with write and create
  """
  attr_obj = self.env["product.attribute"]
  binary_attribute_ids = attr_obj.search([("custom_type", "=", "binary")]).ids
  custom_lines = []
 
  for key, val in custom_vals.items():
  custom_vals = {"attribute_id": key}
  # TODO: Is this extra check neccesairy as we already make
  # the check in validate_configuration?
  attr_obj.browse(key).validate_custom_val(val)
  if key in binary_attribute_ids:
  custom_vals.update({"attachment_ids": [(6, 0, val.ids)]})
  else:
  custom_vals.update({"value": val})
  custom_lines.append((0, 0, custom_vals))
  return custom_lines
 
@@ -1530,12 +1530,12 @@
  def get_child_specification(self, model, parent):
  """return dictiory of onchange specification by
  appending parent before each key"""
  model_obj = self.env[model]
  specs = model_obj._onchange_spec()
  new_specs = {}
  for key, val in specs.items():
  new_specs[f"{parent}.{key}"] = val
  return new_specs
 
  @api.model
  def get_onchange_specifications(self, model):
@@ -1544,16 +1544,19 @@
  - needed this method because odoo don't add specification for fields
  one2many or many2many there is view-reference(using : tree_view_ref)
  intead of view in that field"""
  model_obj = self.env[model]
  specs = model_obj._onchange_spec()
- for name, field in model_obj._fields.items():
- if field.type not in ["one2many", "many2many"]:
- continue
- ch_specs = self.get_child_specification(
- model=field.comodel_name, parent=name
- )
- specs.update(ch_specs)
+
+ # TODO :- Commented a code and ths code already base in a odoo base modules.
+ # for name, field in model_obj._fields.items():
+ # if field.type not in ["one2many", "many2many"]:
+ # continue
+ # ch_specs = self.get_child_specification(
+ # model=field.comodel_name, parent=name
+ # )
+ # specs.update(ch_specs)
+
  return specs
 
  @api.model
  def get_vals_to_write(self, values, model):
@@ -1561,24 +1564,24 @@
  - same functionality by _convert_to_write
  - needed this method because odoo don't call convert to write
  for the many2many/one2many fields"""
  model_obj = self.env[model]
  values = model_obj._convert_to_write(values)
  fields = model_obj._fields
  for key, vals in values.items():
  if not isinstance(vals, list):
  continue
  new_lst = []
  for line in vals:
  new_line = line
  if line and isinstance(line[-1], dict):
  new_line = line[:-1] + (
  self.get_vals_to_write(
  values=line[-1], model=fields[key].comodel_name
  ),
  )
  new_lst.append(new_line)
  values[key] = new_lst
  return values
 
 
 class ProductConfigSessionCustomValue(models.Model):
@@ -1589,8 +1592,8 @@
  @api.depends("attribute_id", "attribute_id.uom_id")
  def _compute_val_name(self):
  for attr_val_custom in self:
  uom = attr_val_custom.attribute_id.uom_id.name
  attr_val_custom.name = "{}{}".format(
  attr_val_custom.value,
  (" %s" % uom) or "",
  )
@@ -1618,7 +1621,7 @@
  """Return custom value evaluated using the related custom field type"""
  field_type = self.attribute_id.custom_type
  if field_type == "binary":
  vals = self.attachment_ids.mapped("datas")
  if len(vals) == 1:
  return vals[0]
  return vals
@@ -1639,7 +1642,7 @@
  )
  > 1
  ):
  raise ValidationError(
  _("Configuration cannot have the " "same value inserted twice")
  )
 
@@ -1657,16 +1660,16 @@
  @api.constrains("attachment_ids", "value")
  def check_custom_type(self):
  for custom_val in self:
  custom_type = custom_val.attribute_id.custom_type
  if custom_val.value and custom_type == "binary":
  raise ValidationError(
  _(
  "Attribute custom type is binary, attachments are the "
  "only accepted values with this custom field type"
  )
  )
  if custom_val.attachment_ids and custom_type != "binary":
  raise ValidationError(
  _(
  "Attribute custom type must be 'binary' for saving "
  "attachments to custom value"