[FIX] models: on update, call inverse function on field even when it is stored

The inverse function of a stored computed field was not called when creating or
writing on a record with such a field.

[IMP] models: do not use compute methods to determine default values anymore

Compute methods could give results that should not be considered as default
values.  For instance, a related field usually defaults to a null value, which
is then set to the field with its inverse method by create().  This may violate
a non-null constraint if the original field is required.  Therefore, compute
methods are no longer used to determine default values.

[FIX] models: do not prefetch fields to recompute, and recompute once only

The method _prefetch_field() was accidentally prefetching fields to recompute;
which was skipping the actual recomputation, since a value was put in cache.
But sometimes the field's value was fixed by an extra recomputation of the
field.  Here we remove the extra recomputation and fix the cache corruption.

[FIX] CONTRIBUTING: dead link

[FIX] README: grammar

fix typo

[FIX] incorrect qweb interpolations

unnecessary, worked by accident and broken by change to interpolation regex in 35f5fb46

[FIX] website_sale: better fiscal position detection + support update during checkout

The customer can change the country and tax
number in the billing information during
checkout, and the taxes should be properly
updated according to the re-detected fiscal
position.

The fiscal position detection also depends
on the `vat_subjected` flag, which we now assume
to be implicit as soon as the customer filled
in a valid Tax Identification Number.

[FIX] web: re-enable test_menu.py without the mock-menu test

[FIX] packaging: include RML & xsl in manifest.in

[IMP] ir_qweb: remove the option 'only_date' introducted in cefc9ad8e3e3b0a14804da6c40bfd71e16d4175a. Now we can use widget='date' to do the same thing. That keeps the same behaviour that in the backend.

    <span t-field=model.datetime_field t-field-options="{'widget': 'date'}"/>

[FIX] web: revert c1d48362d3a38606ffb99d569fdc6147dcd0c775 and consider there is a previous dialog only if isDestroyed returns False

[IMP] fields: do not force required=True on related fields

This may cause issues if the field is stored: non-null constraint will prevent
saving records when no value is provided.

[FIX] models: in _create() and _write(), mark fields to recompute earlier

An issue occurs when a constraint is checked before computed fields are marked
for recomputation: the constraint will read the field's current value, which
may be wrong.  If the field is marked soon enough, the constraint will trigger
the recomputation and use a correct value.

[FIX] web: destroy modal on cross click or escape

Instead of just hidding it
Otherwise, when opening a new dialog, it is considered that there is a previous opened dialog, and this can lead to refresh the record which opened the hidden/previous dialog

This is related to rev. 57b48602fb7afa5d0539e8d0caa8d1e10c3910e5

[IMP] doc: add testing reference and improve docstrings

[IMP] fields: rename attribute 'copyable' into 'copy'

[IMP] fields: do not copy field objects anymore, but make new instances instead

Because of the parameter overriding mechanism implemented by fields, it is no
longer necessary to copy field objects.  It is even better to no copy them in
the case of related fields.

[FIX] barcode_interface: when loading widget, picking_type_id was sometime not correctly set

[ADD] doc: workflow doc from v7+ (saas-5)

I tried starting a better one, but the workflow code is a complete mess so I
have no idea how this thing works without significant study and testing.

Thus, not trying to write a brand new and complete doc, maybe later.

[IMP] fields: add assertions to check parameters (comodel_name, size, digits)

[IMP] fields: set the default value to the closest field.default or _defaults

This solves a subtle issue: in the following case, the class Bar should
override the default value set by Foo.  But in practice it was not working,
because _defaults is looked up before field.default.

    class Foo(models.Model):
        _name = 'foo'
        _columns = {
            'foo': fields.char('Foo'),
        }
        _defaults = {
            'foo': "Foo",
        }

    class Bar(models.Model):
        _inherit = 'foo'
        foo = fields.Char(default="Bar")

The change makes field.default and the model's _defaults consistent with each
other.

[IMP] fields: make attribute 'default' callable

[I18N] Update translations from Launchpad 8.0 branches

[IMP] new-api-ify 'trigger' server action

[FIX] actions: match the behavior of multi actions with its help text

also fix the corresponding text and add explicit sequence number because I
don't understand what the bloody hell it does without that, except that it's
not the right thing. At all.

[ADD] actions documentation

[FIX] purchase: do not pass uom in product_id_change when changing of product

To allow the on change to set the product default unit of measure when changing of product, and only in this case (not when changing price or quantity)

[FIX] fields: in to_column(), returning self.column is generally not correct

Consider the following example:

    class Foo(models.Model):
        _name = 'foo'
        _columns = {
            'state': fields.selection([('a', 'A')]),
        }

    class Bar(models.Model):
        _inherit = 'foo'
        state = fields.Selection(selection_add=[('b', 'B')])

The attribute 'column' of the field does not have the full selection list,
therefore the column object cannot not be reused, even a copy of it.  The
solution is to systematically recreate the column from the field's final
specification, except for function fields that have no sensible way for being
recreated.

[FIX] openerp/osv/fields: fix method to_field() to pass some falsy parameters.

For instance, the attribute copy=False was not passed when converting the
_column to a Field.  Simply make sure those parameter are always passed.

[FIX] website_sale*: handle no delivery methods in website

Display a user friendly message instead of crashing when there is not delivery method available for a specific order

Revert "[FIX] Error when installing module document"

This reverts commit 8dd4c83d9cf6f45ca39f120b296c1c69079154a4.
It does not fix the issue, but hides it instead.
An actual fix has been made since.

[FIX] pos_restaurant: Issue 614981: product renaming from name to display_name was not done in the pos_restaurant module, resulting in empty product names in various places

[FIX] dict notation for js reserved words in options (Fixes #2957)

[FIX] kanban: remove container=element for tooltips on kanban

[IMP] website_event: add mail subtypes

Add published and unpublised mail subtypes, to track events publication, and to be able to follow these subtypes

[FIX] website_sale: save the company in street and the street in street2.

The old behaviour was not better, because when we print the invoice, the order was ugly:
    Name
    Street
    Company name
    Country

Now we will have:
    Name
    Company name
    Street
    Country

This patch is not retro-compatible:
   Old partners will see the address in company name and vice-versa.
   Need to update view and switch street field and street2 field

[FIX] point_of_sale: prevent empty order creation as they are not accepted by the backend

[IMP] gamification: avoid sending twice emails in cron

When the cron is running on a database with a large number of goals (e.g. website_forum with thousands of users), it's possible the CPU time is exceeded and we may have a rollback after sending some emails (for granted badges).
To avoid sending twice emails, commit in cron mode after each reward.

[FIX] ir_actions: ids of unlink method can either be an ID list or one ID

Merge pull request #2955 from odoo-dev/8.0-wmsmrpdatefix-jco

[FIX] Can not execute line when stop date is EARLIER than current date

[FIX] migration: test if package is installed using its state.

checking if there is an installed version set is not enough because this field is not reset when the module is uninstalled

[FIX] Can not execute line when stop date is EARLIER than current date

Merge pull request #2375 from odoo-dev/8.0-webclient-doc-xmo

Port web client tutorial fully (-ish), improve JS doc.

[FIX] JS tutorial

[FIX] sphinx 'project' value

The project name automatically gets the release and the literal string
"documentation" appended by default (and "html_title" can be set to generate a
title differently), so having "documentation" set in the project variable
duplicates it in the page title.

[ADD] port webclient RPC doc from web/doc

[ADD] JS widgets reference documentation

Also fixed Widget#alive's behavior and tested it

[MERGE] forward port of branch saas-3 up to 56f678c

[FIX] gamification: on delete goal user, cascade

Otherwise, this is no more possible to delete users with gamification goals

Merge pull request #2269 from odoo-dev/8.0-qweb-doc-and-fixes-xmo

QWeb documentation, fixes and improvements

[ADD] pyqweb-specific stuff, pyqweb APIDoc

[IMP] qweb: foreach handling

* fix mapping handling to match JS impl: current value set as _value instead
  of being lost
* add handling of integer parameter
* only set _size and _last if current iterable is sized

[ADD] qweb: call directive's body

* __content__ can't be used in Python implementation because safe_eval, so use
  ``0`` from Python implementation instead
* remove postfix from t-call tests because due to implementation details all
  whitespace crap following a t-name is added to rendered template in Python
  impl, and don't want to normalize whitespace.

[ADD] qweb: handling of t-att=mapping

Changed render_att_att to return an iterable of pairs instead of a pair, and
dispatched t-att on whether its result is a Mapping.

Also changed qweb test runner so it uses ordereddict for JSON mapping in
params, otherwise iteration order (and thus order of attributes in output) is
unpredictable and results don't/can't match expectations (as both are
strings).

Note that this relies on JS implementation details wrt iteration order of
mappings. Tests would probably be somewhat less brittle if rendering output
was parsed to XML... if that's possible (?)

[ADD] running of XML cases to python qweb

[IMP] qweb doc, tests

* document qweb based (mostly) on JS version
* convert JS qweb tests to (mostly) language-independent XML so they can be
  used for JS and Python implementations
* add some more tests (e.g. precedence between t-value and body in t-set)
* remove ``t-import``
* fix parity in foreach(dict) (and rename some variables to make array and
  object versions more similar)

[ADD] qweb-js: escf, rawf for parity with Python version

[ADD] qweb-js: jinja-style interpolation pattern

[IMP] qweb-js: reimplement string interpolation compilation as a single pass

[IMP] update qweb-js tests

* latest qunit
* template loading (handle async)
* add format tests

Forward port of branch saas-3 up to fc9fc3e

[FIX] account: res_config: gain/loss exch. account onchange

Commit f4e350ca3 defined an `onchange_company_id` method, never executed
because another `onchange_company_id` was defined below. Merged the first
in the second.

[FIX] website_quote: forbid to define a quote template on a sale order

Defining a quote template may change the product list. The field template_id is now editable
only when the sale.order is in draft or sent state.

[IMP] website_crm: allow the response from crm thanks page to be overridable

[FIX] google_calendar: add missing parameter to logger_info

[FIX] common.py: missing import

It was not possible to call "about" function via remote control
(XML-RPC), because it contains string translation without import.

[FIX] models: fields_get() shall not return info about fields not set up yet

When processing data files during a module installation/upgrade, not all fields
are set up yet, in particular relational custom fields.  Make fields_get()
ignore those fields, so that views can be created/updated and validated,
provided they do not refer to those fields...

[FIX] project: typo in protect.task create

that typo will cause project task create error when set date_end before the time right now

Merge pull request #2918 from odoo-dev/8.0-wmsextramoves-jco

[IMP] Make sure invoice_state of picking is used when creating extra moves

Merge pull request #2827 from odoo-dev/8.0-eanproductviews-2772-jco

8.0 eanproductviews 2772 jco

[FIX] Fix inheritance in point_of_sale views + ean visible on creation + landed costs views

[IMP] Set custom EAN not visible when multiple variants

[IMP] Change landed costs views + make ean visible upon creation

[FIX] Adapt product variants in order to show internal code and ean13 #2772

[IMP] Typo

[IMP] Make sure invoice_state of picking is used when creating extra moves and take price of related move if there is one

When we would copy, it gives problems as it won't be invoiced the correct way on sale, so we provide for a simpler solution.

[FIX] website_crm: move the input hidden generated with kwargs at bottom of the form. Else, if the args is a field displayed later, form never look about the new value and error remains. When we post a form with 2 fields with the same name, we got only the first one into kwargs from controller.

[IMP] hr_holiday/calendar : allow to pass no_email into context when we create a meeting to avoid to send a mail to attendees.

[IMP] google_calendar: add missing file google_calendar.xml

[IMP] Google calendar - add function to allow the cron to run google calendar synchronization

[FIX] account_anglo_saxon: avoid two box purchase_ok field

as account_anglo_saxon depends from purchase and purchase already adds the field purchase_ok
Fixes #2630

[FIX] auth_ldap: remove insignificant spaces in login

strip spaces from user name as they will probably be ignored by ldap's search (rfc4518)
Fixes #2865

Merge pull request #2894 from erwin-bas-solutions/translation-reconcile

[FIX] Account. Missing translation on write-off comment

[FIX] Account. Missing translation on write-off comment

Forward port of branch 7.0 up to b3485fb

[FIX] event: double context

[FIX] event: missing context

This fixes some translation issues (templates not translated at event confirmation)

[FIX] website_quote: use dedicated email template if a quote template exists

[Revert] cb30783aba642627c6bf71d4aa812de00e7d536d & 6349048ba0e5e74c74e783ffce8c3c0083d2f060

[FIX] web: avoid force_reload in list editable
[FIX] web: force load record after reload page

These fixes prevent to open existing record form

[FIX] account_anglo_saxon: price difference, discount and taxes

When computing the price difference amount do not integrate the eventual discount and taxes included in the price.
Otherwise the total of the generated accounting enty would be higher than the total of the invoice. opw 611350

[MERGE] forward port of branch 7.0 up to 9066da3

[FIX] point_of_sale: do not display False as currency if symbol is not set

[FIX] ir_translation: apply tools.ustr on the trad itself

tools.ustr(None) returns u'None', res[0] can be None.

[FIX] web: avoid force_reload in list editable

When creating a new record in list editable, due to previous commit 6349048, the load_record was called twice and the first record of the current list view (self.dataset.index) was used to fill the new record.
With this, we make sure a new record is indeed created.
Fix the web test to have a default_get call in mock models and increase the number of default_get assertions (for creations in list editable, the default_get is then called twice, not optimal but due to the absence of distinction between empty datarecord and filled with default values).

[MERGE] forward port of branch 7.0 up to 6d4e1cc

[FIX] ir_translation: remove control characters from translations

This is possible that control characters (such as line returns) are inserted wrongly in translations
These should not influence on the web interface

[FIX] account: return format of function field

When no result is found on the function field 'invoice' (account.move.line), instead of returning {move_id: (False, '')}, return {move_id: False} (expected for m2o fields)
Fixes #2138, opw 613096

[FIX] web: force load record after reload page

When reloading a page, the _actualize_mode is called before the record is loaded and the form is displayed partially in edit mode (o2m fields with delete/add icons).
This patch forces to trigger a load_record when the datarecord has not been loaded yet in the do_show merthod (opw 607910)

[REF] website*: set spinner style in class, instead of inline

This fix is related to:
8298cf6510db906006c43f8c96f712adb2740633
51e399d9acbda42a7ae59226f4302fb7c2a3aed7
9056c58cb8d12d17080ed4e5ede7a6466a0e1f75

[FIX] website_sale: shopping cart quantity width (qty > 9)

[FIX] website_sale: too tiny quantity input (if qty > 9) for all occurences

[FIX] api: clear environments on module installation