This reverts commit 82f236d98e322efd84c873df54f8e728e1d2a1bf.

The fix is actually not solving all the issues, and also introduce
a behavior change. Actually, the expected result for filters like

<filter string="My Pipeline" name="my" domain="[('user_id', '=', uid)]"/>

will result into a evaluated filters, but invalid as the uid is set to
the person configuring the google configuration, not the user who will
use it.

As it is tricky (or even impossible) to solve the issue properly in all
the cases, we prefer to warn the user that the filter is invalid instead.

closes odoo/odoo#64144

X-original-commit: 35fff5d26f20ad5cfacffaf9335fc8580a7f4793
Signed-off-by: Yannick Tivisse (yti) <yti@odoo.com>

Before this commit:

While creating new templates it will give a traceback

After this commit:

While creating new templates it will not give a traceback

LINKS

Task-2334852

closes odoo/odoo#58016

Signed-off-by: Thibault Delavallee (tde) <tde@openerp.com>

When simply need to parse a domain, it is easier

Add .strip() on ir.ui.view as ast.literal_eval produces an syntax
error if the node starts with spaces (as done in the xpath of
hr_attendance.view_employee_form_inherit_hr_attendance)

closes odoo/odoo#43831

Related: odoo/enterprise#7894
Signed-off-by: Martin Trigaux (mat) <mat@odoo.com>

The ondelete policy must be explicitly set for m2o fields whose inverse
is a reflection model (ir.*)

When setting the authorization code, a refresh token is generated in
the background and stored in a system parameter.

However, when saving the general settings (res.config.settings)
without changing the authorization code, the refresh token is deleted.

The refresh token should only be deleted if the authorization was
removed.

This was not the case as the set_param is always called and a value is
only computed when the authorization code has changed. The current
value was never retrieved.

Set a ir.config_parameter only when it a value has changed.

closes odoo/odoo#35028

Signed-off-by: Jérome Maes (jem) <jem@openerp.com>

Multi is the default api for methods, it is not necessary to explicitly
decorate methods with it, adds clutter and most people use it because
they see that the rest of the code uses it.

Done with `find . -type f -name '*.py' | xargs sed -i '/@api.multi/d'`

Superuser mode implies `record.env.is_XXX()`.

Improve the configuration setting of google drive to make it more userfriendly
Display a message when google drive is enabled but no token have been generated

Task: 1894961

closes odoo/odoo#29562