[FIX] product: correct name_get for products with resellers.

Error introduced during forward-port commit 14f3085

[FIX] web: even if there is no view, the field has to be rendered

This is related to ÿ07f3f4d9206d4c1a3e484eb89f09d2258977a

[FIX] mail_thread: allow to pass mail_track_log_only in the context to remove the subtype of the notification. By this way, no mails will be sent because partners_to_notify will be empty.

[FIX] web: push render value deferred only if the field is in a form view

[FIX] website: editor: preserve label and color for link buttons targeting a page

When selecting a page target rather than a static
URL the style and label information were ignored/lost.

[FIX] implementation of t-call's 0-index in qweb-js

Had expected regularity, had not noticed the Python version is really an
ignoble hack where the string "0" (exactly) is interpreted in a specific
manner when used in t-raw and format strings, *and nowhere else*.

Had special-cased the expression everywhere which broke e.g.
<t-set="xxx" t-value="0"/>. Reproduce Python behavior more closely,
and strongly consider swapping out both for something different e.g.
[[content]] or something like that, which doesn't cause problems in
Python but doesn't have to be so specially cased either.

[FIX] models: do not look up the registry class when building cls._fields

Merge pull request #3001 from Bubbles-IT/8.0-fix-pos-tax-label

[FIX] point_of_sale: printing of tax name on hardware was missing name

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

[MERGE] forward port of branch 7.0 up to 75d3ea6

[FIX] point_of_sale: fixed display of untaxed amount in sales lines report

[FIX] point_of_sale: fixed display of untaxed amount in sales lines report

[FIX] email_template: do not try to add user signature if none set

append_content_to_html expects two strings. If the user has no signature, it returns user.signature returns False

[FIX] product: use new api to retrieve product standard price as SUPERUSER

As it looks like self is a product.template, and product can be a product.product. Using the new api avoid to have this problem.

[FIX] product: redo forward port 2832d2d for fix 12eb157 lost with 43977de

[FIX] models: in _add_field(), set the field as an attr before setting it up

In the case of custom fields, the field's parameters were set up without the
field being present in the class hierarchy.  Because of this, the parameter
inheritance mechanism was missing the field itself.  As a consequence, custom
selection fields ended up without selection, for instance :-/

[ADD] doc: document domain operators, fix typos

* Add description for all allowed operators.
* Improve wording of domain documentation and fix spelling.

[REM] undocument removed method add_default_value

[FIX] website.snippet: can't edit content of the banner. (+ better fix for backward compatibility)

Merge pull request #3061 from odoo-dev/8.0-wmspocurrency-jco

[FIX] Use currency of pricelist when creating PO from procurement

[FIX] Use currency of pricelist when creating PO from procurement

[FIX] read_group: date format: use natural year along with month names, do not mix with ISO week-year

The ISO week-year notation can produce confusing values
when the first week of the year is so short that it
becomes week 0 and is considered the last week of the
previous year, depending on the locale.

For instance, using ISO notation:
  'W53 2015' == dates.format_date(
      date(2015,1,1), format="'W'w YYYY", locale='en_GB')
  'W53 2005' == dates.format_date(
      date(2006,1,1), format="'W'w YYYY", locale='de_DE')

This is surprising but actually valid.

However it definitely yields wrong output when combined with
months formats:
  'January 2014' == dates.format_date(
      date(2015,1,1), format="MMMM YYYY", locale='en_GB')

As a result we must always use `y` to denote the year in
any date format, *except* when it is combined with the
week number `w`, in which case we must use `Y`.

See the documentation at:
   http://babel.pocoo.org/docs/dates/#date-fields

[FIX] web: wait for deferreds when actualizing mode

In edit mode, in a *2many with many2many_tags, when adding a new tag when none was set, discarding the form let the new tags displayed while it shouldn't (only a display issue, the tag wasn't added in database)

Fixes #2926

[FIX] GeoIP - allow to specify the path to GeoIP in the server config.

--geoip_db='the_path'  which can be used via openerp.tool.config['geoip_database']

[FIX] mail: followers widget was broken

This was probably due to 35f5fb46e78aef1feda4d9cfddd93bd356bdca28.

[FIX] stock_picking_wave: removed obsolete many2many widget from stock_picking_wave_view

[FIX] web: oe_navbar compatibility on IE11

[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] website_membership: crash when member has no country

in memberships countries list

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

[MERGE] Forward-port 7.0 up to 1b49a87

[FIX] ir.mail.server: restore parsing of multiple RFC2822 addresses including non-ASCII chars

Rev f2cf6ced1 modified RFC2822 parsing in order to better support
unicode characters inside the Name part of an address header.
However the patch broke handling of multiple addresses (comma
separated) - silently discarding all recipients except the
first one, as soon as any non-ASCII character was present.

This patch restores the functionality while preserving the
fix from f2cf6ced1, and simplifies the code using email.utils
utility functions.

Fixes (again) lp:1272610, OPW 607683

[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

[FIX] account_invoice: using has_group to determine if user has group group_supplier_inv_check_total

has_group do not care about the fact res.groups is access restricted

[FIX] res.users: exactly match login when operator in ilike, equal

[FIX] point_of_sale: printing of tax name on hardware was missing data 'name', also made the regular print use the same value for consistency.

[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

[IMP] mail: parsing emails with several html parts

If an email contains several text/html parts inside a multipart email, the previous code was only keeping the last content part.
The Content-Type: multipart/mixed allows several independent part (RFC1341 7.2.2), so two html is technically valid.
With this patch, the two parts are concatenated. (opw 614755)

Modify append_content_to_html regex to make sure the regex keeps the content of the html instead of removing it.
e.g.: "123 <html> 456 </html> 789" used to be stripped to "123  789" while we expect "123 456 789"

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

[MERGE] forward port of branch 7.0 up to 03df412f

[FIX] account: precision_get returns a number of digits

In other words, it returnes a precision digits, not a precision rounding

[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

[FIX] ir.ui.view: creating custom views in backend did not compute view type

When creating new records in backend all field values are
sent to the server, including empty fields.

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

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

[FIX] gamification: on delete badge user, cascade

Otherwise, this is no more possible to delete users having badges

[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

[FIX] account: self typo