[FIX] utm, *: allow quick creation of campaigns
Purpose: ======== Since the introduction of the required "title" field in the utm_campaign model in commit [1], quick creation of campaigns from many2ones is not possible anymore (the slow creation modal is opened when clicking on "Create"), and the name value entered is not copied into the name field of the creation form. This is because the model now has a field "name", and a field "title", which are both required. The creation using the many2one widget tries to create a record using the entered value as title and no name, which fails. It fails because the create method replaces this empty name by a False value, since it could not generate a unique name given no name. This commit makes sure that if a title but no name is given when creating a new campaign record, one does not set the name value to False anymore. It also makes sure that the value entered before clicking on the "create and edit" button is copied in the form, by adding the "create_name_field" option on the many2one fields (the form uses the title field, not the name). The "always_reload" option has also been added because without it, the many2one shows the name of the created record instead of showing the title. [1]: https://github.com/odoo/odoo/commit/4dbcefb5e5b1878e81fe9be9fe48a785f813334f Task-3113954 closes odoo/odoo#110474 X-original-commit: 869208cd09b6ceb5dcda55b6fe4131d7cc8d43e8 Signed-off-by: Warnon Aurélien (awa) <awa@odoo.com>
This commit is contained in:
@@ -273,7 +273,7 @@
|
||||
<field name="company_id"
|
||||
groups="base.group_multi_company"
|
||||
options="{'no_create': True}"/>
|
||||
<field name="campaign_id" />
|
||||
<field name="campaign_id" options="{'create_name_field': 'title', 'always_reload': True}"/>
|
||||
<field name="medium_id"/>
|
||||
<field name="source_id"/>
|
||||
<field name="referred"/>
|
||||
@@ -319,7 +319,7 @@
|
||||
</div>
|
||||
</group>
|
||||
<group string="Marketing">
|
||||
<field name="campaign_id" />
|
||||
<field name="campaign_id" options="{'create_name_field': 'title', 'always_reload': True}"/>
|
||||
<field name="medium_id" />
|
||||
<field name="source_id" />
|
||||
<field name="referred"/>
|
||||
|
||||
@@ -44,7 +44,7 @@
|
||||
<field name="short_url"/>
|
||||
</group>
|
||||
<group name="utm" string="UTM">
|
||||
<field name="campaign_id"/>
|
||||
<field name="campaign_id" options="{'create_name_field': 'title', 'always_reload': True}"/>
|
||||
<field name="medium_id"/>
|
||||
<field name="source_id"/>
|
||||
</group>
|
||||
|
||||
@@ -341,6 +341,7 @@
|
||||
<field name="campaign_id"
|
||||
string="Campaign"
|
||||
groups="mass_mailing.group_mass_mailing_campaign"
|
||||
options="{'create_name_field': 'title', 'always_reload': True}"
|
||||
attrs="{'readonly': [('state', 'in', ('sending', 'done'))]}"/>
|
||||
<field name="medium_id"
|
||||
string="Medium"
|
||||
|
||||
@@ -39,7 +39,7 @@
|
||||
name="utm_link"
|
||||
groups="base.group_no_one"
|
||||
attrs="{'invisible': [('move_type', 'not in', ('out_invoice', 'out_refund'))]}">
|
||||
<field name="campaign_id"/>
|
||||
<field name="campaign_id" options="{'create_name_field': 'title', 'always_reload': True}"/>
|
||||
<field name="medium_id"/>
|
||||
<field name="source_id"/>
|
||||
</group>
|
||||
|
||||
@@ -644,7 +644,7 @@
|
||||
<field name="origin"/>
|
||||
</group>
|
||||
<group name="utm_link" colspan="2" class="mt-0">
|
||||
<field name="campaign_id"/>
|
||||
<field name="campaign_id" options="{'create_name_field': 'title', 'always_reload': True}"/>
|
||||
<field name="medium_id"/>
|
||||
<field name="source_id"/>
|
||||
</group>
|
||||
|
||||
@@ -44,7 +44,8 @@ class UtmCampaign(models.Model):
|
||||
vals['title'] = vals['name']
|
||||
new_names = self.env['utm.mixin']._get_unique_names(self._name, [vals.get('name') for vals in vals_list])
|
||||
for vals, new_name in zip(vals_list, new_names):
|
||||
vals['name'] = new_name
|
||||
if new_name:
|
||||
vals['name'] = new_name
|
||||
return super().create(vals_list)
|
||||
|
||||
@api.model
|
||||
|
||||
@@ -61,3 +61,7 @@ class TestUtm(TestUTMCommon):
|
||||
|
||||
utm_8 = utm_7.copy()
|
||||
self.assertEqual(utm_8.name, 'UTM d [2]', msg='Must add a counter as suffix to ensure uniqueness')
|
||||
|
||||
# Test name uniqueness when creating a campaign using a title (quick creation)
|
||||
utm_9 = self.env['utm.campaign'].create({'title': 'UTM dup'})
|
||||
self.assertEqual(utm_9.name, 'UTM dup [6]', msg='Even if the record has been created using a title, the name must be unique')
|
||||
|
||||
Reference in New Issue
Block a user