[FIX] calendar: onchange_dates: protect against void dates (for example when creating a meeting through the form view)

[FIX] website: slug: fallback to positive ID when slug appears to contain a missing negative ID

In some rare cases database records have negative IDs,
so the slug URL could look like /foo--20. This could
be mistaken for a slug ending with a `-` and a positive ID.
The latter is not supposed to happned as final hyphens
are stripped by slugify, but has been used in the past
and may be used in old links.

[FIX] web: multiple opened modal need fixing since bootstrap lib update

[MERGE] Foward-port saas-5 up to ee4df1e

[FIX] mail: bad display of join button on mail groups.

[MERGE] Forward-port saas-4 up to 8b15482

[MERGE] Forward-port saas-3 up to e79a367

[FIX] orm.search_count: ignore `limit`, `offset` and most importantly `order`

These parameters are (or should be) irrelevant for
a search_count(), and they could actually break the
result or make it significantly slower (e.g applying
`order` on large tables).
This fixes a performance regression introduced by
0f43032b.

We could also raise an error offset/limit are
passed in combination with count, but that seems
unnecessary.

Also switched to "SELECT count(1)" for the count
query, as it is simpler and just as fast.
We'd get the same perf with * or any constant value,
as in "SELECT count('me in')", but let's keep it
simple ;-)

[FIX] web: remove unneeded overflow-y on modal-body class

Will fix problem where opened dropdown list were hidden behind the modal footer

[FIX] mass_mailing: allow to add a user/email to a specific mailing list and not always the last as previously.

[FIX] event: registration and confirmation template now have a domain on event.registration.
Otherwise all templates are displayed and this will lead to some issues.

Also added a description on event.registration, because currently its
displayed name is Email Thread.

[FIX] account: avoid to propagate context in send by email button

There is no reason to propagate the context in those buttons.

Besides, it leads to issues concerning the email template, rendering the wrong res_id because the active_id was wrongly propagated

[FIX] mail: message-id not parsed properly in message_route

Same as for rev 4bad513, the references were not correctly parsed, added missing regex

[FIX] crm: base_partner_merge, search with lowercase for name,email

and without spaces for vat

[MERGE] forward port of branch 7.0 up to 7c5bf67a5f9378fe4ae01eeffff24889f7bdcdc5

[FIX] account: unit arg of on_change_unit_amount can be False

[MERGE] forward port of branch 7.0 up to cd31cbf556cc11950ce6188041cf5c2fcd89e0f8

[FIX] account: on_change_unit_amount do not reset uom to default

[FIX] web: fix tooltip that were stuck visible

tooltip stuck visible should be removed on click + remove tooltip container except for modal in order to prevent them for staying visible in some rare occasion

[FIX] web: css tweaks

[ADD] crm: use company/contact name for fill partner_info in chatter

Sometimes, when you send an email in the chatter, a pop-up is displayed to fill the partner details
This case happens when the email address you send the email is not associated to an existing partner
In such cases, in lead, we use the company name and contact name to fill the partner name

[FIX] website_customer: check that country id exists before access the name, else traceback when we come from a non existing id via the url/controller

[FIX] account_analytic_plans: _convert_ref takes only 1 arg

Since the convert of account_invoice to the new api.
_convert_ref is defined as a staticmethod in account_invoice.py

[FIX] account: _get_analytic_lines needs the date_invoice

During the conversion of account_invoice.py to the new API, the date_invoice written on the invoice has been moved in the end of the action_move_create method.

But, the method _get_analytic_lines needs it sooner.

[FIX] base_action_rule: define patched methods inside a function to fix their closure.

[MERGE] forward port of branch saas-5 up to c86f217

[FIX] stock: production lot as no longer a company_id field

[MERGE] forward port of branch saas-5 up to e4cb520

[MERGE] forward port of branch saas-4 up to fa07bc8

[MERGE] forward port of branch saas-3 up to 310d3fe

[FIX] ir.actions.server: ir.values ref (`More` menu entry) should not be copied

[MERGE] forward port of branch 7.0 up to 39f3e40

[FIX] sale: copy_quotation must propagate the context

Passed default type while creating document page category from document page

[FIX] project_long_term: avoid bad duplication of phases

When we duplicate a project, the related phases need to be duplicated as well but keeping references between each other.
e.g. 'project 1' with 'phase A' follwed by 'phase B' should become 'project 1 (copy)' with 'phase A (copy)' followed by 'phase B (copy)'.
Fixing bug lp:1212860, opw 607062

[FIX] account_voucher: avoid trying unreconcile a move twice

When we cancel a voucher, we may be trying to unlink a reconciliation that was already removed on another move (just looking at the version in cache). In such cases, the unlink would fail with traceback. opw 610287

[FIX] cli/server: fixes #1307, add missing Environment management blocks

[FIX] sale: config, timesheet if module account_analytic_analysis

In Settings > Sales, the onchange_timesheet ensure that if you check timesheet, it checks module_account_analytic_analysis.

Therefore, if module_account_analytic_analysis is installed, timesheet should be checked by default
Until now, it worked "luckily", because the onchange_timesheet of the field module_account_analytic_analysis was triggered before the onchange_timesheet of the timesheet field. Nevertheless, we shouldn't trust the onchange calls order.

[IMP] sale, crm: remove remaining interval filters, continuing 366a30d

[FIX] addon web: prefix css dropdown-submenu

to prevent possible future conflicts

[FIX] correct various date issues in reporting

* remove old 'day', 'month', 'field' and replace them by the actual
date/datetime field
* remove weird cast to char when creating the view to prevent crash
when grouping on them
* remove duplicates (such as 'creation_date' and 'create_date')
* fix typing errors (field type date defined as a datetime in the
postgres view)
* fix search view definition

[FIX] correctly type date field in crm_lead_report

opening_date and date_closed were cast to char but are supposed to be
dates.  When you try to group crm_lead_report by one of them, postgres
was not happy.  Now, it properly tells postgres that they are of type
date.

[IMP] improve graph view layout and groupbys

* add a 'total' foldable button to allow the user to control the column
in pivot table view
* in the dropdown menu, add all fields from the model instead of only
those from the search view groupbys

[FIX] website: do not restore page views

Merge pull request #1284 from sebalix/8.0-fix-xmlrpc-marshall-none

[FIX] read() method returns None values instead of False, incompatible with XML-RPC

[FIX] models: fixes #1017; do not update list in place in the construction of _depends on models

[IMP] website: disable translations for example URLs

[IMP] website: unused imports

[IMP] website: update sample social media accounts

[ADD] safe_eval: allow YIELD_VALUE, used e.g. for generator comprehension

[FIX] snippets: avoid orphan full stop (.)

[FIX] website.menu: url field should not be translated

Pages are made translatable using a dedicated mechanism,
and there is no UI to translate menu URLs anyway.

[IMP] website: disable translation exports for test models

Avoids noise in POT files

[IMP] BaseModel: new `_translate` attribute to disable translations

Can be defined to False in any model to completely
disable translations for this model, when they are
irrelevant. This is useful e.g. for test classes
that use attributes that would normally be translatable.

[IMP] tools.translate: unused import

[FIX] google_calendar: cleanup/spellcheck translatable terms

[REM] fields: remove fields.Any, temporary artifact for ill-typed fields

This was added in master-apiculture at f1f16a8 to
permit special function fields that return
structured JSON-like data.
This is unnecessary and caused typing problems, for
example for the type field of ir.model.fields, or
when you decide to store them.

It is simpler to explicitly declare these fields
as fields.Char and have them serialize their results
to JSON strings, or to declate them as fields.Binary
and return any opaque data they want.

[FIX] base: ir.qweb: ensure element.tail is correctly encoded

element.{text,tail} are either ascii-compatible `str`, or `unicode`
when non-ascii-compatible. This could force the implicit decoding
of utf-8 encoded contents when joining template bits, breaking
the rendering.

Fixes #1085, and related to #1130

[FIX] web: fix multiple modal closing (e.g. set defaults)

move call trigger('closing') after the modal('hide') call to prevent element being destroyed before we could do that call leaving a backdrop visible

[FIX] web: return _super deferred in start method

Forget to include the _super method in the deferred returned by the start function of the search view
See previous rev. 70605240a9e4e0279bd339c3d80cab3862a58abd

[FIX] web: headless search views are ready from start

function start, line 783, filters_ready waits for fields_view_get deferred to be resolved before calling prepare_filters, which prepare the view filters.

The thing is that, at line 416, if this.headless boolean is true, the searchview is marked as ready (by resolving the this.ready deferred), and the fields_view_get deferred will never be resolved, as it is resolved in the else case, where the this.headless boolean is false.

Therefore, in this.headless true case, this prevents the deferred $.when(this._super(), filters_ready) to be resolved, as the deferred filters_ready will never be resolved. Therefore, the deferred returned by the start function is never resolved, and the searchview will never be regarded as ready neither.

In views.js, the ViewManager is marked as ready when the deferred manager_ready = $.when(searchview_loaded, main_view_loaded, this.view_completely_inited) is resolved. In the this.headless true case, as the searchview is never marked as ready, the manager isn't neither.

This leaded to some issues, like the action buttons in form views being disabled on click, and never re-enabled once the action/wizard executed. See opw-610723

[REF] crm analytics: refactor - lint

[FIX] stock: partial deliveries switch on delivered picking

Users don't care for the backorder picking precisely because they can't process it, whereas they may have to do some more things on the picking they processed: invoice it, print delivery orders or transportation stickers..
Refresh the browse record after changing the name to avoid the need to rebrowse.
Fixes #1372

[IMP] crm: Add Campaign source and medium in group by + fix a bug when name_search return a tuple and not only the id.

[FIX] mail: reset of default_type in context

When creating a new message, we need to reset the default_type key to avoid context propagation (e.g. invoice menu).
However we used to compare the key in context (char) with the list of selections (tuple) so it was never matching and always reset.

Merge pull request #1391 from savoirfairelinux/7.0-translate-trans_generate_debug_print

[FIX] translate: Fix debug print when parse path list to translate.

[IMP] website_blog: group the get_discussion in one request(get_discussionS), that will avoid to make one request by paragraph in blog.

[MERGE] forward port of branch 7.0 up to bbb8d8f25a7a81c26965aa76b5085961a13132d6

[MERGE] forward port of branch 7.0 up to 767d2ea9a63ceb6d8473766f52eb06f69b281ca1

[MERGE] forward port of branch 7.0 up to b91a75f7eff85edd5d51a91d10d6e3be8a409030

Conflicts:
addons/base_vat/base_vat.py
openerp/tools/safe_eval.py

[FIX] website_event: display unconfirmed events too

[FIX] web: on refresh, do search_read with fields passed

[FIX] wek_kanban_gauge: use field raw_value to avoid thousands separator etc.

[FIX] {base_action_rule,calendar,hr,mail,project_issue}: decorate the method `message_post` to specify signature

[IMP] base_vat: differenciate VIES and classical VAT error

Display a different error message so that users are aware if the test failed with VIES enabled or not (opw 609823)

Forward port of branch saas-5 up to eda2f06

[IMP] website_crm: allow to specify a view_from and view_callback for a form. That will allow to redirect the user on another thanks page depending form the form. We add also a preRenderThanks method with allow to updates the values used by the thanks page rendering.

Revert 3dec090 "[FIX] ir_values: fallback when no condition"

An ir.value without condition should not match when searching with a condition.
When a field with change_default on it is modified, the method get_defaults is called with the new value. This means that manually modifying a field with this trigger would put back the default value (opw 611193).

[FIX] account: no journal items on consolidation accounts

As for the view accounts, the consolidation accounts should not allow to create journal items.
Fixes #1090

[FIX] stock: backport of rev 0a6d63b

[FIX] translate: Fix debug print when parse path list to translate.

[FIX] res.users: safer read() when computing default company value

The goal is to avoid any kind of prefetching
of other fields when copmuting the default
company, as this will sometimes happen in the
middle of a database update, when the default
company is used for setting the default value
of new columns. In that case the db schema
may not fully match the in-memory model, leading
to an SQL error.

[FIX] survey: display comments in results

[FIX] hr_holidays: do not allow reduce remaining leaves

In the Employee form, a remaining legal leaves field is available, which shows remaining validated leaves from allocation requests
As this is not allowed to delete a validated allocation requets, it shouldn't be possible to reduce the validated remaining leaves of an employee.
To reduce the remaining leaves of an employee, the user should cancel and remove the allocation request

[IMP] account.invoice: add extra @id on `smart button` to make inheritance easier

[FIX] models: display_name and name_get mismatch

- display_name uses name_get and not the other way around:
name_get should not call _compute_display_name, _compute_display_name should call name_get.
The previous behaviour was not backward-compatible with the old api.
All the models redefining name_get would have 2 different behaviors between name_get and display_name.

- Do not set an inverse function to display_name:
In most cases, writing on display_name writes on _rec_name (if any, not mandatory).
If the display_name computation is redefined, we need to redefine as well the inverse method to avoid unexpected behaviour
This required to also modify tests in base_import as readonly fields are avoided.

- Remove search method on display_name:
For the same reason as for the first point, it could be good that searching on display_name use name_search (and not the other way around).
However doing this would be very inefficiant (need to do the search, without limit, extract the ids of the name_get result just to generate
a subdomain ('id', 'in', [...]). As in most cases it would anyway mean to search on the _rec_name it's better to directly do so.

- Changing label to avoid mismatch:
In view displaying the list of fields or when a match is made on the label of a field (e.g. when importing csv file,
matching is made on both label and technical name), the fact that display_name field has '
Calling it 'Display Name' will avoid most errors.

- remove display_name definition from website_forum_doc,ir_model:
These fields are doing the same thing as the display_name of the new api, we can remove them.
We need to keep the one for res.partner as it's a stored field.

[FIX] survey: correct counting of finished answers

[FIX] survey: comments are properly recorded now

[DOC] survey: retroengineering

A bit of commenting in order to help future debugging and unf**king of
the survey reporting tool.

[FIX] survey: remove duplicate browse record

The `prepare_result` dict shouldn't keep a browse record of a survey, as it
is explicitely passed to the template by the controller
`survey_reporting`

[FIX] survey: minor view fixes

[ADD] sale: validity date in sale order report

[ADD] safe_eval: UNPACK_SEQUENCE and Exception

Allowing UNPACK_SEQUENCE allows the use of the "multi" assignation:
a, b = [1,2]
for a,b in items

[REVERT] 8b41906: disabled all on_change on new record form

This reverts commit 8b41906bf8a22e3f00d3ea334f89b7f1aec27442.

[FIX] ir_fields: allow accent on selection import

If the selection label (not value) had accents, it wasn't possible to import it using the label having accents

[ADD] base_vat: add VAT validation for Peruvian localization.

[FIX] website: more robust inherit between footer_custom and footer_default

[FIX] website_membership: access rules fixes

When searching on memberships, we use domain clauses in the format 'partner.x = y' where partner is a many2one to res.partner. The object res.partner has strict security rules for public users and this search will return zero result if not done with SUPERUSER_ID.
In addition, we need to access the list of products (membership_ids) in the domain to be sure we will retrieve only published membership (otherwise it would crash in the sort below).

[IMP] ir.logging: add index on dbname/type, important with high number of log lines

[FIX] website: iframe to google maps need the trailing slash, otherwise the path and the id are not separated (e.g.: '/members42')