[15.0] [FIX] mrp_bom_attribute_match: structure and cost report

[ivantodorovich]

This PR fixes two issues with this module:

1. An issue with mrp_account's button "Compute Price from Bom"
2. An issue with the Structure and Cost smart button

In both cases, when called on a bom that has a component template and no product_id set, a very similar traceback was raised:

  File "/odoo/src/addons/mrp_account/models/product.py", line 95, in _compute_bom_price
    total += line.product_id.uom_id._compute_price(line.product_id.standard_price, line.product_uom_id) * line.product_qty
  File "/odoo/src/addons/uom/models/uom_uom.py", line 234, in _compute_price
    self.ensure_one()
  File "/odoo/src/odoo/models.py", line 5207, in ensure_one
    raise ValueError("Expected singleton: %s" % self)
ExceptionThe above exception was the direct cause of the following exception:Traceback (most recent call last):
  File "/odoo/src/odoo/http.py", line 643, in _handle_exception
    return super(JsonRequest, self)._handle_exception(exception)
  File "/odoo/src/odoo/http.py", line 301, in _handle_exception
    raise exception.with_traceback(None) from new_cause
ValueError: Expected singleton: uom.uom()

NOTE: It's easier to review commit by commit

[yankinmax]

Not familiar with the module: code seems ok.

[jcoux]

Technically seems good 👍

[ilyasProgrammer]

LGTM

[francesco-ooops]

@OCA/manufacturing-maintainers could this be merged? :)

[francesco-ooops]

@pedrobaeza what do you think?

[pedrobaeza]

Please squash a bit the commit history.

[ivantodorovich]

Hello @pedrobaeza !

As it is right now, each commit is atomic, taking care of a specific concern.
Squashing would make the history dificult to read.

[pedrobaeza]

OK, as I haven't followed the thread, just do the administrative task given the reviews:

/ocabot merge minor

[OCA-git-bot]

This PR looks fantastic, let's merge it!
Prepared branch 15.0-ocabot-merge-pr-958-by-pedrobaeza-bump-minor, awaiting test results.

[ivantodorovich]

Thanks 🙏🏻 !!

[OCA-git-bot]

Congratulations, your PR was merged at b963089. Thanks a lot for contributing to OCA. ❤️

[francesco-ooops]

@ivantodorovich seems mrp_account_bom_attribute_match has corrupt readme, could you fix?

[ivantodorovich]

Oops,. thanks for noticing @francesco-ooops !
Fixed here: #983