[FIX] gamification: prohibitive record rule processing with many users/goals

Due to the multi-company record rule on gamification.goal,
each access to the Goals menu and each opening of the
Messaging menu (thus calling get_serialised_gamification_summary())
is extremely slow (with several thousands goals/users).

Adding auto_join to the user_id FK on goals makes it much
faster. However it causes crashes when reading the table
because the _order of gamification.goal uses `create_date`,
which becomes ambiguous after the auto_join with res_users.

Solving this can be done by re-implementing _read_flat()
in the ORM using the internal Query object, as in search(),
which takes care of fully-qualifying all column names.

Until this is fixed, a simple workaround is to use
start_date in the _order instead of collision-prone `create_date`.

[FIX] pos: the widget.pos.cashier is not always defined

[FIX] fleet: creation of suppliers is now correct

[MERGE] forward port of branch 7.0 up to 60ab6f1

[FIX] web: on switch form (from list & kanban) do not load record if empty dataset

[FIX] account_anglo_saxon: avoid duplication of price difference lines

When computing the price difference lines, in move_line_get of account_anglo_saxon, we loop on the result of super call for each lines (n * n times) to compute the price difference.
The product_id was used to match the returned line and the original invoice line. This was wrong as we could get several lines with the same product_id (and then get n * n price difference lines).
This patch adds the line id to the result of move_line_get (from account) so that account_anglo_saxon can filter more efficiently and only get one price difference per invoice line.
Fixes #704

[FIX] web: missing self initialisation

The self is needed in the _value() method below, opw 608878

[FIX] website_customer: pager with country filter

[FIX] mail: forward-port of rev 245bb4ebdfeccb05134b3ff943c9fd0fb01173c2

[MERGE] forward port of branch 7.0 up to 245bb4e

[FIX] portal_claim: create claim with no partner_id

If partner_id is False, do not try to name_get

This fix is related to the rev 095be21ab17f7f731877d61c99b48b51119ff39a

[FIX] mail: to/from/reply-to addresses formated according to RFC2822

Use formataddr method from email.utils lib, which do the job correctly

[FIX] l10n_be_coda: set a meaningful filename default value

[FIX] procurement: refactor scheduler so it continues even if a procurement failed

If the mrp scheduler fails during the treatment of a procurement (due to a postgresql error, like an update lock), we let it try the other procurements, instead of stopping at the first procurement which fails.
Besides, we retry to treat the procurements which failed, until the list of the remaining procurements is always the same (meaning something is wrong and block the procurements treatment).

[IMP] payment_paypal: better label for paypal_seller_account

Rename the paypal_seller_account field as 'Merchant ID' instead of 'Seller ID' to match paypal terminology (opw 611765)

[IMP] calendar: use registry of request in notify controllers

[FIX] website: google_map remove trailing slash and add an attribut for disable scrolling

[FIX] website_event: show online in event in all country. Add filter online event. Keep the date between filter.

[MERGE] forward port of branch 7.0 up to 3632949

[FIX] web: image fields re-render on next/previous

By default, on binary images read, the server returns the binary size
This is possible that two images have the exact same size
Therefore we trigger the change in case the image value hasn't changed
So the image is re-rendered correctly

[FIX] yaml tests: Set on_change keys not in view logger as debug instead of warning

This is related to commit d31faceb67fd87acaca00365dcfc28925b08a37f
(This is to avoid the runbot being yellow if on_change methods have extra keys in the returned value, this is not wrong, the web client will simply ignore them).

[FIX] web_kanban : resequence column and content by drag and dropping

[FIX] website: seo, url encode params, + handle connection breaks

[FIX] website: use internal route for SEO suggest keyword, old server is down

[FIX] Calculate date interval using super user time zone

When you set the date of a cron the July 1st at midnight, if the user
time zone has a positive offset, then the converted UTC date is the
June 30th and adding 1 month will end up on July 30th translating to
July 31th instead of September 1st.

To solve this issue we use the super user time zone for the date
calculation.

[MERGE] forward port of branch 7.0 up to 5f696ba

[FIX] pos: display cashier name on receipt, not the user

[FIX] account_anglo_savon: round prices according to account precision to avoid getting unbalanced move

The standard move_line_get from account.invoice returns prices rounded (price_subtotal field as account precision) while the anglo-saxon module could return dict with non-rounded prices.
With amounts with bigger precision, we could get a difference of a few cents when comparing the debit/credits and then not be able to balance the invoice (opw 597208).

[FIX] yaml tests: extra key in on_change result isn't that bad.

In the return result of an on_change, having keys which are not ine the view is not that bad, it is not an actual problem.

Display warning instead of asserting all the keys are in the views.

[FIX] account_analytic_plan: correctly check context

The id of the journal is passed in the context. Avoid trying to browse on jounral_id=False
Fixes #821

[FIX] project_timesheet: test used fields added not in dependencies

The fields procure_method and supply_method are not present if we just install project_timesheet which makes the tests fail (opw 610590).

[MERGE] forward port of branch 7.0 up to a86dfb8

[FIX] mail: avoid duplication followers when merging partners

[IMP] crm: clean after partner merge

When trying to merge partners, an sql error may be violated (e.g. unique constraints).
In this case, catch the error and delete the problematic record as it will no longer be relevant with the old partner and left unconsistant data in the database.

[FIX] website_sale: avoid displaying inactive products in shops

The domains in the form ('o2m_field', operator, False) do not use the orm but convert the domain to ('id', 'invert operator', [list of ids]). This means that the orm is not used and implicit filter (active=True) or access rights are not checked.
A proper fix in master should be done to use the orm instead of an SQL query.
This patch force a search to be made on product.product and then exclude the products where active=False (opw 607602).

[FIX] web_kanban: in non-grouped kanban, on switch to form, display first record

in view_form.js, in do_show method, if the dataset index is not set, the on_button_new method is triggered, opening a new record
In regular kanbans (non-grouped kanban), the dataset index wasn't set, and, therefore, when switching to the view mode, it opened a form for a new record instead of opening the first record of the list, like this is the case for the view list and the grouped kanban

[IMP] email_template: add a lambda function to allow a user to format a date from a template mail into a specific timezone, and/or in a specific format.

Eg: ${format_tz(object.name, object.employee_id.address_id.tz,'%m/%d/%Y %H:%M')}

[FIX] account_voucher: fix typo in journal_id type

[FIX] report: allow using 0 decimal precision in rml reports

[FIX] calendar/crm: Add active_id (if in model partner_id) in defaults values from partner_ids when we creates a new meeting.

Bug was that when you click on meeting button from a res_partner form (Customers in menu), the button overwrites the context to use the active partner as a default in the calendar meeting. By consequence, the context overwritte the default partner who are the creator. Now the context could be removed from action and that is in get_default for partner_ids (from model calendar_event) that we add the creator AND the active_id if from a model res_partner.

[FIX] payment_paypal: SUPERUSER_ID removed by inadvertence in 1826f6d4eca499754a1f4bbe7179987eab473a55

[FIX] payment_paypal: use the validate url according to the acquirer env

[FIX] account: child_of operator in invoice analysis report

When displaying invoice analysis of a company, include the invoices of the members of this company (opw 611084)

[FIX] account: payment_ids should list payments of same account than the invoice account

[FIX] Account: Consolidation Account should not be carried to next fiscalyear

[FIX] l10n_be: spelling error in financial reports

[FIX] account_voucher: validation of refund in multicurrency

Fixing several issues for refunds in multicurrency mode that prevented the moves to be balanced.
The amount_currency needs to use the absolute value, as a refund will have a negative amount.
The sign for currency_rate_difference needs to use the line instead of the voucher as they can have different value. e.g. a voucher of 1000$ with invoice of 1200$ and refund of 200$ will have two lines and their currency_rate_difference should have different signs.
Avoids doubling the value in foreign_currency_diff
Fixes #1490, opw 607118 & 611580

[FIX] mail: add tooltip to send a message and log a note links

[FIX] portal_sale: inadvertence during the forward port 978fd47261d3a6cb6e39053892fb291bf61b511c

[FIX] crm_claim: set options no_create, no_open instead of selection

To avoid access denied errors for portal users

[MERGE] forward port of branch 7.0 up to 095be21

[FIX] crm_claim: allow to create claim as portal user

It wasn't possible to create claims as a portal user when the portal user had a parent_id

[FIX] point_of_sale: clear tag in pos order sheet layout

Without clear tag total amounts breaks sheet layout.

[FIX] website_quote: add description sale to quote line description

[FIX] stock: onchange method should take the description

The onchange_product_id method used to only change the description if the stock.move is not saved yet. That does not make much sense.
opw 607347, bug lp:1314700

[FIX] analytic: name_search on accounts

The name_search was hardcoding ilike operator while it could be different. This means that a name_search "name != Agrolait" would return accounts with agrolait.
A side effect of this was the impossibility to import account_analytic_account whose parent is a substring of another - opw 607315

[FIX] sale: quote character in company name is converted to html entity

Avoid to get company names such as "L'abc" to be transformed as "L'abc" opw 607221

[MERGE] forward port of branch 7.0 up to e07bc5d

[ADD] res_partner: test case for fix rev 4a278809741d4e961ac13d3477a046d43b32cfa7

[FIX] survey: missing timezone conversions

As the survey hardcode lot's of views and reports in python, we do not beneficiate from automatic timezone conversions in the webclient.
Hopefully this is fixed in v8 after refactoring of the survey module. opw 608786

[FIX] res_partner: commercial partner sync

On child partner creation of a parent partner, the commercial_partner_id is not computed (and stored) in the create method.
Therefore, we should compute the commercial_partner_id before trying to sync his data

[FIX] payment*: add the customer state to payment (paypal) transactions

[FIX] payment_paypal: paypal IPN, receiver_id is not always passed

The method  _paypal_form_get_invalid_parameters checks that the params passed by the paypal feedback coroborate with the transaction requested by Odoo.

It looks like the 'receiver_id' is not always passed by the paypal feedback. Therefore, we check this param only if it is available in the paypal feeback.

This issue had as side-effect to not validate the paypal transaction, which remained 'draft' permanently.

[MERGE] Forward-port 7.0 up to 2493395

[FIX] product_visible_discount: no crash when no pricelist passed or product without company

Fixes an error introduced in f6fb2b6

[FIX] website: OpenERP->Odoo in website footer

[MERGE] Forward-port 7.0 up to 9411a2da

[FIX] Model.load(): extra error checking while importing data

If any missing or partially incorrect values cause
an exception other than a psycopg2 error, we should
still catch it, rollback that record and report
the error, rather than letting bubble and fail
without any feedback to the user.

Fixes #1485

[IMP] base: show possible titles for companies (opw 608243)

[FIX] product_visible_discount: multicurrency pricelists

When we compute the discount of a product to display (result of product_id_change), we compare prices in the currency of the product while we expect prices in the currency of the pricelist. opw 606188

[FIX] orm: set default before removing magic fields

When a record is created, the magic fields (id, create_date,...) are first removed from the vals as the user should not set a value for these.
However if a value for this is given in default value (e.g. defined in an ir.value), the creation would crash (sql error : column specified more than once) as the magic column would be added again.

[FIX] website: update url in the footer

[FIX] website_sale: confirm sale order on payment_confirmation

Same behavior than in saas-5

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

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

[FIX] website: do not restore page views

[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

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

[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] web: on refresh, do search_read with fields passed

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

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] translate: Fix debug print when parse path list to translate.

[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

[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

[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