diff --git a/addons/website/models/website_form.py b/addons/website/models/website_form.py index 896568595ef..cb4b4b141b5 100644 --- a/addons/website/models/website_form.py +++ b/addons/website/models/website_form.py @@ -1,9 +1,13 @@ # -*- coding: utf-8 -*- # Part of Odoo. See LICENSE file for full copyright and licensing details. +import logging + from odoo import models, fields, api, SUPERUSER_ID from odoo.http import request +_logger = logging.getLogger(__name__) + class website_form_config(models.Model): _inherit = 'website' @@ -116,6 +120,11 @@ class website_form_model_fields(models.Model): if not self.env['res.users'].has_group('website.group_website_designer'): return False + unexisting_fields = [field for field in fields if field not in self.env[model]._fields.keys()] + if unexisting_fields: + # TODO in master: Raise instead of log + _logger.error("Unable to whitelist field(s) %r for model %r.", unexisting_fields, model) + # the ORM only allows writing on custom fields and will trigger a # registry reload once that's happened. We want to be able to # whitelist non-custom fields and the registry reload absolutely diff --git a/addons/website_form_project/data/website_form_project_data.xml b/addons/website_form_project/data/website_form_project_data.xml index 1a3052472c3..df4d340a0eb 100644 --- a/addons/website_form_project/data/website_form_project_data.xml +++ b/addons/website_form_project/data/website_form_project_data.xml @@ -11,7 +11,6 @@ project.task diff --git a/addons/website_form_project/static/src/js/website_form_project_editor.js b/addons/website_form_project/static/src/js/website_form_project_editor.js index 5c137b0254c..4ba1aeedbc9 100644 --- a/addons/website_form_project/static/src/js/website_form_project_editor.js +++ b/addons/website_form_project/static/src/js/website_form_project_editor.js @@ -11,7 +11,8 @@ FormEditorRegistry.add('create_task', { string: _t('Task Title'), }, { type: 'email', - modelRequired: true, + custom: true, + required: true, fillWith: 'email', name: 'email_from', string: _t('Your Email'),