diff options
| author | Martin Trigaux <mat@odoo.com> | 2020-02-14 11:22:15 +0100 |
|---|---|---|
| committer | Martin Trigaux <mat@odoo.com> | 2020-05-14 13:59:10 +0200 |
| commit | d9287caf9408bb4ff85ad734cbca08e44321e205 (patch) | |
| tree | 0d3b7c5705e5172176a7d76bf9098e381190bf4b /addons/website/tests | |
| parent | 56a8c9e4311d24c33b7ad08b8230e4be345a85b8 (diff) | |
[IMP] *: convert to private methods
render, render_template, load, activity_schedule_with_view,
get_website_pages should all be private:
It should not be possible to render an aribtrary template only with
its name or id
Still need to render some qweb views from js so the method
render_template is kept public.
This explains why the website editor still need read access on
ir.ui.view as we want to allow any snippet to be rendered.
Diffstat (limited to 'addons/website/tests')
| -rw-r--r-- | addons/website/tests/test_qweb.py | 2 | ||||
| -rw-r--r-- | addons/website/tests/test_views.py | 4 |
2 files changed, 3 insertions, 3 deletions
diff --git a/addons/website/tests/test_qweb.py b/addons/website/tests/test_qweb.py index 99f53078310..7b7d08ded8d 100644 --- a/addons/website/tests/test_qweb.py +++ b/addons/website/tests/test_qweb.py @@ -32,7 +32,7 @@ class TestQweb(TransactionCaseWithUserDemo): demo_env = self.env(user=demo) - html = demo_env['ir.qweb'].render('website.test_template', {"user": demo}, website_id= website.id) + html = demo_env['ir.qweb']._render('website.test_template', {"user": demo}, website_id= website.id) html = html.strip().decode('utf8') html = re.sub(r'\?unique=[^"]+', '', html).encode('utf8') diff --git a/addons/website/tests/test_views.py b/addons/website/tests/test_views.py index 9f19cdcc303..c662416756c 100644 --- a/addons/website/tests/test_views.py +++ b/addons/website/tests/test_views.py @@ -204,7 +204,7 @@ class TestViewSaving(common.TransactionCase): ) self.assertIn( replacement, - view.render().decode('utf-8'), + view._render().decode('utf-8'), 'inline script should not be escaped when rendering' ) # common text nodes should be be escaped client side @@ -213,7 +213,7 @@ class TestViewSaving(common.TransactionCase): self.assertIn(replacement, view.arch, 'common text node should not be escaped server side') self.assertIn( replacement, - view.render().decode('utf-8').replace(u'&', u'&'), + view._render().decode('utf-8').replace(u'&', u'&'), 'text node characters wrongly unescaped when rendering' ) |