[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] website_forum: Preventing user from posting answers on [Close] and [Deleted] questions. Solving issue of Red background when question is deleted

[IMP] website_quote : add action to update quotation button

[IMP] res_partner: form view cleaning + new function field website_private, inverse of website_published, more user-friendly to use on partner form view

[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

Merge pull request #1381 from odoo-dev/master-sale-quotation-button-rename-avo

[IMP] website_quote: rename and move 'view quotation' button

Merge pull request #1383 from odoo-dev/master-quote-remove-editbutton-jas

[IMP] website_quote : remove "press edit button" on template

[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

Cut long names

[FIX] stock: backport of rev 0a6d63b

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

[WIP] Replace pyinotify by watchdog

Merge pull request #1387 from odoo-dev/master-disable-subcommand-discovery-fme

[FIX] Temporary disable addons subcommand auto discovery

The problem with the current approach is that it imports all modules in
order to discover which one exposes a subcommand by subclassing
openerp.cli.Command. At that time, the arguments are not parsed in order
to setup the config because a subcommand could potentialy alter the
command line arguments (which is the case of start)

This behavior causes the modules to be imported with the default config and all
the modules using some config keys at startup won't have to value they
are supposed to have when considering the shared command line switches.

Long story short:

$ odoo --workers=6

is ok, but

$ odoo start --workers=6

is not for code that checks the config value at startup (eg: report module)

As the automatic subcommands are not used yet, we temporary disable it
until as proper solution is found.

[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.

[IMP] Make read_template conditionally cached (--dev)

[ADD] @conditional decorator for a conditionally applied decorator

[FIX] Temporary disable addons subcommand auto discovery

The problem with the current approach is that it imports all modules in
order to discover which one exposes a subcommand by subclassing
openerp.cli.Command. At that time, the arguments are not parsed in order
to setup the config because a subcommand could potentialy alter the
command line arguments (which is the case of `start`)

This behavior causes the modules to be imported with the default config and all
the modules using some config keys at startup won't have to value they
are supposed to have when considering the shared command line switches.

Long story short:

$ odoo --workers=6

is ok, but

$ odoo start --workers=6

is not.

As the automatic subcommands are not used yet, we temporary disable it
until as proper solution is found.

[FIX] Temporary disable addons subcommand auto discovery

The problem with the current approach is that it imports all modules in
order to discover which one exposes a subcommand by subclassing
openerp.cli.Command. At that time, the arguments are not parsed in order
to setup the config because a subcommand could potentialy alter the
command line arguments (which is the case of `start`)

This behavior causes the modules to be imported with the default config and all
the modules using some config keys at startup won't have to value they
are supposed to have when considering the shared command line switches.

Long story short:

$ odoo --workers=6

is ok, but

$ odoo start --workers=6

is not.

As the automatic subcommands are not used yet, we temporary disable it
until as proper solution is found.

[FIX] register dev_mode command line switch

[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

[FIX] stored arch field is now arch_db

[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.

[WIP] ir.ui.view's arch field fetches disk version if --dev mode is used

[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] contact widget should return a really empty address

This fixes a bug in website_quote, because qweb tests whether address is
something that bool() casts to False or not, in order to display an icon (or not).

[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')

[FIX] stock: do not set user_id on prepare_invoice_group

The user_id is already set by the prepare_invoice method, which is called before the prepare_invoice_group (the user_id is already set, thus)
Besides, _prepare_invoice is overriden in sale_stock, to set the picking sale order salesman as user_id, and, without this correct, grouping invoicse by partner re-set the user_id to uid, which is wrong.

[FIX] project: remove state field from task analys

The state has been replaced by stage_id in the view, so the state is no longer used
Moreover, when doing an advanced search, the field state is visible and if used -> Traceback (because not available in the report sql table)
opw-609984

[FIX] website: catch exception on destroy of ckeditor to allow the modifications to be saved anyway

[IMP] proj: optionalize default alias generation

[IMP] account: do not offer closed analytic accounts on invoices

lp:1223922, opw 609656

[REF] Odooification

[FIX] models: check harder that default value is not NULL before setting it

When computing defaults we may end up with
a falsy value that is not None (e.g. '' or False)
That value will be cast to None when being
saved in the database, depending on the column type
(e.g. saving False on a many2one actually stores NULL).

Improve the test to consider the value being written
*after* that conversion, to *really* avoid nonsensical
and expensive queries such as:

    UPDATE table set col = NULL WHERE col IS NULL;

[FIX] web: wrong tooltip display for long menus

Due to the switch to Bootstrap 3.2 in this version, tooltip auto placement works erroneously (see https://github.com/twbs/bootstrap/issues/13897).
This is a workaround, waiting for a fix in Bootstrap assets.

(Closes #1341)

[FIX] web: avoid trying to access to deleted record message when change password

The onchange call can be made with empty ids list (which would fail). opw 610810

[FIX] ir_attachment: _filestore cache ignore uid

replace ormcache_context by ormcache: use the context in the cache key is useless
set skiparg=3 (default skiparg=2) so the uid is not used in the cache key: the filestore path is the same for all database users

[FIX] website_sale: fix error in tracking_last_order which was using the field is_delivery from module delivery

[FIX] purchase_analytic_plans: keep reference to analytic distribution

Purchase orders created with invoice policy 'Based on incoming shipments' were not keeping the reference to the account analytic distribution when invoiced. opw 607577

Merge pull request #1340 from odoo-dev/saas-3-fix-website-snippet-button-css-chm

[FIX] website.snippet: display snippet editor button inline insead of 2 lines for the small boxes

Merge pull request #1338 from odoo-dev/8.0-fix-website-snippet-button-css-chm

[FIX] website.snippet: display snippet editor button inline insead of 2 lines for the small boxes

[FIX] gamification: why is there a button here ? do you need a button ? I don't think so...

[FIX] share: reduce risk of slowdowns during res.groups changes

Databases with thousands of users would experience
poor performance during any change to a group during
an update/install, caused by the prefetching of
all database users when the browse cache contains
most of the groups in the system.

[FIX] website_blog: Add sanitize=False on content of blog to allow writer/author to insert some tricky code like form or JS

[MERGE] Forward-port saas-5 up to a5f7891

[FIX] website.snippet: display snippet editor button inline insead of 2 lines for the small boxes

Merge pull request #1336 from odoo-dev/master-fix-tour-wait-chm

[FIX] tour.js: wait must be used for the current step instead of the previous step. Remove setimeout delay for activate the element.

Merge pull request #1337 from odoo-dev/saas-5-fix-product-taxes-view-chm

[FIX] account:  the 'Customer taxes' field on the product is readonly when my product doesn't have any variant

[MERGE] forward port of branch 7.0 up to 67cca3f

[FIX] website.snippet: display snippet editor button inline insead of 2 lines for the small boxes

[IMP] website_crm - add kwargs into create_lead function. That allow to get some fields which will be already join in the description from values to custom the behavior from create_lead function

[FIX] account:  the 'Customer taxes' field on the product is readonly when my product doesn't have any variant

[FIX] mail: display translated model name at record creation

When a record is created, if it inherits from mail.thread, a message 'OBJECT created' is posted. 'created' is translated but the name of the model wasn't.
This fix uses the name of the linked ir.model which is already a translatable field.
lp:1262000, opw 611043

[FIX] tour.js: wait must be used for the current step instead of the previous step. Remove setimeout delay for activate the element.

[FIX] ir.module.module: create module xml id also when auto-discovered

It was only being created during database initialization

[IMP] point_of_sale: add product variants in reporting

Closes #1322

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

[FIX] web: correct rev 680f955

Missing backport of commit f652402 for buffered dataset
Again, should not be forwardported

[FIX] fields: do not revalidate field values unless they are being modified

In the previous implementation of the new API fields,
both fields.Selection and fields.Reference were performing
early validation of their `value` as soon as it entered
the cache, either by being read, written, or computed.
This is a source of trouble and performance problems,
and is unnecessary, as we should consider that the database
always contains valid values. If that is not the case it
means it was modified externally and is an exception that
should be handled externally as well.

Revalidating selection/reference values can be expensive
when the domain of values is dynamic and requires extra
database queries, with extra access rights control, etc.

This patch adds a `validate` parameter to `convert_to_cache`,
allowing to turn off the re-validation on demand. The ORM
will turn off validation whenever the value being converted
is supposed to be already validated, such as when reading it
from the database.
The parameter is currently ignored by all other fields,
and defaults to True so validation is performed in all other
caes.

[MERGE] forward port of branch saas-3 up to 474eae9a43ad7860f11f1d3c1cb8c587eeb98412

[FIX] website_sale: translated terms payment status

Payment status (website_sale confirmation page) inject a status message, which was not set to be translatable. It should be the case.

[FIX] account: Bank Statement reconcilation with different currency

When generating reconciled moves in bank statement, use the amount_currency field instead of amount for currency conversion.
Otherwise we would endup with moves with an amount of 0.

[IMP] website_crm: allow creation of the lead to be overrided. That avoid people to rewrite all the routine of the form if they want to change the default behavior at lead creation when form is submitted.

[FIX] stock: wrong target table and function for lot_ids in stock.move