summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorNicolas Martinelli <nim@odoo.com>2019-01-16 07:45:45 +0000
committerNicolas Martinelli <nim@odoo.com>2019-01-16 07:51:53 +0000
commitb876cb397dbd8fe81a9a0ec0f6efbdc73178b9e7 (patch)
treeb2ca441ef6da7b9ee6e60a29c6db8814f438e5f4
parentae6545ccd5a97b336ae0a6a2bcea021f0f1acf7e (diff)
[FIX] website: null type redirection
Commit 68e9d5504cb293643 makes the redirection type mandatory, which is logical. Since we can't add such a constraint in stable, we make the field required in the view and assign a default value (in case the module is not upgraded, which is very likely to be the case). opw-1924983 closes odoo/odoo#30252
-rw-r--r--addons/website/models/website.py2
-rw-r--r--addons/website/views/website_views.xml6
2 files changed, 4 insertions, 4 deletions
diff --git a/addons/website/models/website.py b/addons/website/models/website.py
index 7072dd954f1..95565febaee 100644
--- a/addons/website/models/website.py
+++ b/addons/website/models/website.py
@@ -910,7 +910,7 @@ class WebsiteRedirect(models.Model):
_order = "sequence, id"
_rec_name = 'url_from'
- type = fields.Selection([('301', 'Moved permanently'), ('302', 'Moved temporarily')], string='Redirection Type')
+ type = fields.Selection([('301', 'Moved permanently'), ('302', 'Moved temporarily')], string='Redirection Type', default='301')
url_from = fields.Char('Redirect From')
url_to = fields.Char('Redirect To')
website_id = fields.Many2one('website', 'Website')
diff --git a/addons/website/views/website_views.xml b/addons/website/views/website_views.xml
index 97fe532e745..5e14ce2378c 100644
--- a/addons/website/views/website_views.xml
+++ b/addons/website/views/website_views.xml
@@ -70,9 +70,9 @@
<sheet>
<group>
<group>
- <field name="type"/>
- <field name="url_from"/>
- <field name="url_to"/>
+ <field name="type" required="1"/>
+ <field name="url_from" required="1"/>
+ <field name="url_to" required="1"/>
</group>
<group>
<field name="website_id"/>