From 3c7a6733bacc974e80e78f8078d4c951ce75f680 Mon Sep 17 00:00:00 2001 From: Denis Roussel Date: Tue, 24 Sep 2024 16:54:39 +0200 Subject: [PATCH] [IMP] stock_package_type_category: Display category code also in Many2one selections --- .../models/stock_package_type.py | 23 +++++++++---------- .../tests/test_stock_package_type_category.py | 2 +- 2 files changed, 12 insertions(+), 13 deletions(-) diff --git a/stock_package_type_category/models/stock_package_type.py b/stock_package_type_category/models/stock_package_type.py index d3bb206ee..1cf24b797 100644 --- a/stock_package_type_category/models/stock_package_type.py +++ b/stock_package_type_category/models/stock_package_type.py @@ -1,6 +1,6 @@ # Copyright 2024 ACSONE SA/NV # License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl). -from odoo import api, fields, models +from odoo import fields, models class StockPackageType(models.Model): @@ -14,18 +14,17 @@ class StockPackageType(models.Model): index="btree_not_null", ) - @api.depends("category_id", "category_id.code") - def _compute_display_name(self): - res = super()._compute_display_name() - for package_type in self: - if package_type.category_id: - package_type.display_name = " ".join( - [ - package_type.display_name, - str("(" + package_type.category_id.code + ")"), - ] + def name_get(self): + result = [] + for rec in self: + name = rec.name + if rec.category_id: + name = "{name} ({category})".format( + name=rec.name, category=rec.category_id.code ) - return res + result.append((rec.id, name)) + + return result @property def _rec_names_search(self): diff --git a/stock_package_type_category/tests/test_stock_package_type_category.py b/stock_package_type_category/tests/test_stock_package_type_category.py index 7ec8b5d3f..42e055e59 100644 --- a/stock_package_type_category/tests/test_stock_package_type_category.py +++ b/stock_package_type_category/tests/test_stock_package_type_category.py @@ -41,4 +41,4 @@ class TestPackageCategory(BaseCommon): package_search = self.env["stock.package.type"].name_search("TEST") - self.assertEqual([(package_type.id, package_type.name)], package_search) + self.assertEqual([(package_type.id, package_type.display_name)], package_search)