[IMP] util/fields: Convert to company dependent without company field

The current implementation of the function `_convert_field_to_company_dependent`
requires a field that stores the company value, but this field is not always
available.

This change aims to allow that case by passing a falsy value as company_field,
then it creates a json with the current value for all companies as the new
value in the column

Author: filisbits

src/base/tests/test_util.py

@@ -1131,6 +1131,49 @@ def test_invert_boolean_field(self):
         initial_repartition[False] += initial_repartition.pop(None, 0)
         self.assertEqual(back_repartition, initial_repartition)
 
+    @unittest.skipIf(not util.version_gte("saas~17.5"), "Company dependent fields are stored as jsonb since saas~17.5")
+    def test_convert_field_to_company_dependent(self):
+        cr = self.env.cr
+
+        partner_model = self.env["ir.model"].search([("model", "=", "res.partner")])
+        self.env["ir.model.fields"].create(
+            [
+                {
+                    "name": "x_test_cd_1",
+                    "ttype": "char",
+                    "model_id": partner_model.id,
+                },
+                {
+                    "name": "x_test_cd_2",
+                    "ttype": "char",
+                    "model_id": partner_model.id,
+                },
+            ]
+        )
+
+        c1 = self.env["res.company"].create({"name": "Flancrest"})
+
+        test_partners = self.env["res.partner"].create(
+            [
+                {"name": "Homer", "x_test_cd_1": "A", "x_test_cd_2": "A", "company_id": c1.id},
+                {"name": "Marjorie", "x_test_cd_1": "B", "x_test_cd_2": "B"},
+                {"name": "Bartholomew"},
+            ]
+        )
+        test_partners.invalidate_recordset(["x_test_cd_1", "x_test_cd_2"])
+
+        # Using company_id as default, only records with company set are updated
+        util.make_field_company_dependent(cr, "res.partner", "x_test_cd_1", "char")
+        self.assertEqual(test_partners[0].x_test_cd_1, {str(c1.id): "A"})
+        self.assertFalse(test_partners[1].x_test_cd_1)
+        self.assertFalse(test_partners[2].x_test_cd_1)
+
+        # Ignoring company field and composing value for all companies
+        util.make_field_company_dependent(cr, "res.partner", "x_test_cd_2", "char", company_field=False)
+        self.assertEqual(test_partners[0].x_test_cd_2, {str(id): "A" for id in self.env["res.company"].search([]).ids})
+        self.assertEqual(test_partners[1].x_test_cd_2, {str(id): "B" for id in self.env["res.company"].search([]).ids})
+        self.assertFalse(test_partners[2].x_test_cd_2)
+
 
 class TestHelpers(UnitTestCase):
     def test_model_table_conversion(self):

src/util/fields.py

@@ -845,9 +845,16 @@ def _convert_field_to_company_dependent(
         where_condition = "{0} IS NOT NULL"
     else:
         where_condition = cr.mogrify("{0} != %s", [default_value]).decode()
-    where_condition += format_query(cr, " AND {} IS NOT NULL", company_field)
 
-    using = format_query(cr, "jsonb_build_object({}, {{0}})", company_field)
+    if company_field:
+        where_condition += format_query(cr, " AND {} IS NOT NULL", company_field)
+        using = format_query(cr, "jsonb_build_object({}, {{0}})", company_field)
+    else:
+        using = format_query(
+            cr,
+            "(SELECT jsonb_object_agg(c.id::text, {}.{{0}}) FROM res_company c)",
+            table,
+        )
     alter_column_type(cr, table, field, "jsonb", using=using, where=where_condition)
 
     # delete all old default