[FIX] mrp: only show produce button when not done, product readonly in produce wizard and move_id necessary to make onchanges work

The labels of the fields say that they store hours, but actually they store minutes. Correct the code to store hours.
Add digits to fload field for a better display in the webclient

date_planned_start should also be Start Deadline instead of Expected Start Date.
We don't want the Expected dates to function as a planning tool, but as
a priority for the planning of the workorders.

If the user has not filled in the origin in the scrap object,
we set it to be its associated picking or production order if any.

amend to d78256e7

We wish to keep the email address if no partner is assigned.

the onchange changed the partners of the ticket when the ticket changed project
to the project's partner, but if the ticket already has a partner it shouldn't
change.

* web_editor, website_blog

Since recent refactoring, web editor dialogs automatically trigger
an event when closed (saved or cancel). The problem is that there
still was manual trigger of those events or variants ("save" instead
of "saved").

To prevent multi-saving and be consistent, all save events are now
automatically triggered on save/close (no more manual) and the event
name is "save".

a random ensure_one() was added during the migration causing an error when
modifying portal access management for partners with more than one contact.

Removing it.

This is a backport of saas-13 commit a9bd9ab1.

mail/view controller is a generic controller that redirects the user to a
given view, depending on the record and the user. Users may be redirected
to the backend form view, or to a website view. This is done notably by
calling get_access_action method that gives the action to perform (act_window
or url).

Previously this method was called using SUPERUSER. However it was therefore
impossible to know who the user was. This method is now called using the
current user. Various overrides of get_access_action have been updated to
add some logic and access rights check directly in the method allowing
more fine-grain behavior of the access action.

Since saas-12, when deleting contents, parents are deleted too if they
become empty (while it was not possible to delete contents in that case
before). The problem here was that the history was recorded multiple
times if one deletion made multiple snippets be deleted and this
sometimes could crash.

Now the history is recorded only on first deletion when the user ask
for it.

Following 5c5d0865, apparently unused records
are simply deleted after updating existing ones.  This causes an issue when a
module is installed or upgraded.  The inspection of `ir.model.fields` records
is based on a *partial* registry (not all modules loaded); therefore records
will be deleted and recreated afterwards.  This triggers inconsistencies in
other models that refer to `ir.model.fields`, like server actions, base action
rules, mail templates, etc.: many2one fields will be set to null!

An issue originally occurs in the Event module.
- Go to Events >> Reports >> Events
- Add a "Group By" in the search view: "Event Month"
- A mighty traceback pops up

This is because the rows and the columns are grouped by the same field
(`event_date:month`).

In method `_read_group_format_result`, a group_by of type "date" or
"datetime" is replaced by a "(range_start/range_end, label)" tuple. The
first element of the loop on `annotated_groupbys` is handled correctly,
but the next identical element will crash on `value + gb['interval']`,
since `value` is now a tuple and not a date/datetime anymore.

opw-686242

There is a hackish feature in the web_editor which forces the
contenteditable to true on links which are clicked and removes it
anywhere else in the page so that only the link is editable after
click. This allows to add text at the end of a link without adding it
outside of the link by mistake.

The feature was badly implemented because, if the link was editable
without having any parent which is editable, the contenteditable was
forced to true, then immediatly forced to false when forcing to false
every page element. This was the case for lots of t-field. By example
the product categories in eCommerce are t-field links which are in non
editable <li/> elements...

When we are on the page:

 /blog/myblog-1/page/2

the link to page 3 is:

 /blog/myblog-1/page/3

So we need to give /blog/our-blog-1 as source url of pager.

Previously in this instance, since 4faed0b7 the page 3 url
in this scenario would erroneously be:

 /blog/myblog-1/page/2/page/3

opw-688681

This token will be added automatically at posting (by ajax.js from framework)

Avoid traceback "'int' object has no attribute 'partner_id'"
uid is the id and not the browser record of 'user'

Avoid traceback: Database fetch misses ids (u'<id>') and has extra ids (<id>),
 may be caused by a type incoherence in a previous request.
JS call with id as string.

Avoid traceback: Invalid value res.partner(x,y,z) in domain term ('partner_id', 'in', res.partner(x,y,z))
Search in new api return browseRecord now

Avoid to write on a lead without the right on it.

Tree view was removed to hide the different view types display in enterprise as it was "cleaner" for a dashboard. Re-adding for import button.

Future versions will not display departments as dashboard but employees anyway.

These fields are no longer in the model `._columns`
since their migration to the new API.

Courtesy of Daniel Dico
https://github.com/ddico

opw-688768

Since the below revision:
32b5515a

It was no longer possible to create a payment
for a pos order without picking.

Indeed, if `picking_type` is not set,
`picking_id`, which is initialized to `False`,
remains as `False`, and you therefore can't
ask it's `.id` (`picking_id.id`).

opw-687597

If the title was 'true' or 'false', the export failed, because we are tying to
concat bool and str. TypeError: cannot concatenate 'str' and 'bool' objects

How to reproduce, install sale, be sure to have one order with delivered = True
In Reporting / Sale analysis, add a group by 'shipped' filed to have 'true' as
title.

This commit closes issue odoo/odoo#13425

Changes f -> FALSE and t -> TRUE, otherwise when installing the l10n_lu
COA, the reconcile flag is wrong.

Closes #13282
opw-688957

name_get function needs to ensure that read passes the required
ids as a list, not a single value.  This is very important if
the method is inherited.

When a custom field is created, it now appears in `ir.model.fields` on the
model's children models (by `_inherits`).  This makes the reflection more
accurate (inherited custom fields are visible), although not perfect (they
appear as "base" fields and thus cannot be overridden).

The reflection of fields in `ir.model.fields` has been improved to deal with
custom field creation, modification and suppression.

Note that custom fields are still not inherited through `_inherit` relations.

opw-687626
opw-645065

Issue: Missing Menu on E-commerce because the user click on the wrong menu
opw-681201

The checkbox was there to activate autoplay... but was not doing
anything since the web_editor/website split.

Before migration:
  'phone': customer and customer.phone or lead.phone,
After migration:
  'phone': customer.phone if customer else self.phone,

So if a customer exists and that it has no phone, the phone
on the lead need to be preserved during convertion in opp.