[FIX] account_payment: lin2bank correct fallback condition

[FIX] web: chain close action on wizard confirm

If a wizard has several steps, or laucnh a second wizard, the view from where has been loaded the initial wizard is finally reloaded

[FIX] mail: more robust parsing of In-Reply-To/References (OPW 608919)

When parsing incoming messages, ignore white-space around
In-Reply-To headers, and extract message-id items inside
the References header using a regex.
This actually serves as a workaround for broken MTAs
mangling References (such as outlook.com nesting past ones
with commas, violating RFC2822).

Closes #516 as a manual rebase.

"[REVERT][FIX] point_of_sale: field date of report.pos.order should be a datetime"

This reverts commit 97d097a2af8c3992d5108bd75b82fa8479d196cc.

As explained in the commit comments (on Github), this patch leads to an infinite loop in 7.0, the filter of the pos orders report using the '=' operator in its domain, which is not available for datetime fields, but is for date fields.

This should not be forward ported to newer release (saas-3)

[FIX] l10n_ma: proper chart of account name

[FIX] email_template: remove group restriction on field mail.compose.message.template_id

This was added as a security check, but access to this field is need by the wizard itself.
As non employee users can't read/create email templates, this will not be problematic

[FIX] base: security: employee must have access to res.groups

[IMP] report_sxw: missing parameter without_company for display_address

Fixes #811

[FIX] account: avoid date and cache issues

Explicitely refresh invoice browse_record(...) in order to have correct 'date' in account.move.
Use context_today() date instead of time.strftime() for date_invoice. (opw 611210)

[FIX] resource: security, resource not readable by externals

[FIX] email_template: security, email_template no rights for externals

[FIX] security: disallow read on ir.sequence for externals

[FIX] security: remove read access on company critical fields

[FIX] security: res.groups should be readable by admins only

[FIX] security: externals should be able to read attachements

without having the rights to read ir.config_parameter

[FIX] security: ir.config_parameter should not be readable by externals

[FIX] sale_order_line: use order date to compute currency instead of current date

[FIX] product_visible_discount: use order date to compute currency instead of current date

[FIX] hr_holidays: employees cannot approve their holidays

Nor modify once approved

It wasn't possible for employees to approve their holidays themself, thanks to the GUI, but this was possible through xmlrpc calls, or when altering the html directly in the browser.
Besides, this was also possible to edit the holiday through the same trick once the holiday validated

[FIX] board: Do not offer to create dashboard when there is no action

In case we have no defined action (e.g. redirection by python code), adding a dashboard should not be possible as the rendering based on the action.
This patch hides the 'Add to Dashboard' button in search view (opw 611288)

[FIX] point_of_sale: generate bank statement with commerical partner

When a pos session is closed & confirmed, the account.move were generated with the commercial partner except for the bank statement which prevented automatic reconciliation.
This patch uses the commercial partner also for bank statement.
Fixes #1558, #1764

[FIX] fetchmail: do not delete a failing email when using POP

In case of IMAP, the email is simply set as read but in case of POP, the email is deleted. To avoid loosing data (e.g. misconfiguration of server), the email is kept for POP.
Bug lp:1296724, opw 605667

[FIX] ir_model: invalidate cache when adding new fields

When a new ir.model.field is created, add the new field in the fields_by_model (cache of custom fields). This is required as the __init__ method would not retrieve the new field if fields_by_model is already set.
Otherwise, the _columns would not contain the new fields and we could not access it without restarting the server (e.g. the installation of a module adds ir.model.fields and use it in the a view.

[FIX] note: small bugs

[FIX] base: wrong model change check

Comparing an id and a browse record will always fail so the exception would have always been raised when changing a model (e.g. updating a module with custom fields).

[IMP] account_budget: added date filter in print budget report

Fixes #1743, opw 611307

[FIX] point_of_sale: report, product_id of line is product_product

product_id column of pos_order_line is a product_product
the left join of l.product_id was done on product_template, instead of product_product

It worked as long as the ids product_product were the same as product_template. Meaning that, if you used variants, this report view was screwed.

[FIX] purchase: skip move if not linked to a purchase order

When this situation happens. the 'cost' is None and the web interface cannot handle this value, provoking a JS error. Thus, prefer to fallback on the standard way to get the cost: based on the current standard price of the product.
Fixes #1032

[FIX] base: python 2.6 incompatibility for dictionary comprehension

[IMP] auth_oauth: set default 'zocial' css_class

The buttons for new providers needs at least the zocial css class to be seen as button. As the field css_class is not in the default form view, add it by default.

[FIX] web: ignored readonly attribute on image widget

[FIX] account_voucher: wrong type in context

The type 'general' no longer exists on a account.voucher. As in general context, removed the default value on this menu.
Fixes #1261, it produces a traceback in 8.0 where the selection fields are less fault-tolerant.

[IMP] account_voucher: make the field tax_amount always readonly

The field tax_amount is fieled with onchanges and the compute_tax method. Setting a different value than the one computed by the system may lead to unbalanced move (which is obviously wrong).
In the future, handeling these operations by setting the correct value to the tax accounts would be better.

[FIX] account_voucher: avoid error if no partner

The field partner_id is not required on an account.voucher but the validation was failing if none was set (opw 611663).
This patch makes a fallback on the account of the voucher if neither a partner nor a writeoff account is specified.

[FIX] product: make sure unlinked product still exists

When uninstalling/updating a module, we may execute unlink method on product.template before product.product. In such cases, the product is already removed after removeing the template (_inherits) and the chained unlink of the product would fail (traceback when browsing).

[FIX] point_of_sale: field date of report.pos.order should be a datetime

The date_order field of the pos.order is a datetime, not a date
As, in report.pos.order, the date field is related to date_order of the pos.order

We usually do not commit fixes altering the model fields structure in 7.0, but this one is retro-compatible, as the database structure won't change

[FIX] web: do not focus if the focus_field is not a field

Sometimes, focus_field could be a button, and in such cases, we should not focus on it

opw-612045

[FIX] stock: inventory lines with no production lot: compare with correct stock level

Inventory lines without production lot number should
be compared with the stock level of products without
a production lot. Otherwise the final result of a
validated inventory is wrong as soon as there are
lines with and without production lot for the same
product.

Adds corresponding regression test.

Fixes https://bugs.launchpad.net/openobject-addons/+bug/1008099
Manual rebase of #1658, courtesy of Numerigraphe

[FIX] account: reconcile: no useless revalidation

Avoid revalidating the complete account moves
that contain the lines being reconciled.
The reconciliation does not change the validity
of those moves anyway.
This represents a very important speed up of
reconciliation when moves with several hundred
lines are involved.

[IMP] base: no default domain for partner's title

The title field should not have a domain by default.
When a new record is created or the 'is company' field is changed, the onchange_type is triggered to restrict the domain accordingly to is_company field.
When an existing record is modified, there will have no domain on title field. It's not ideal but at least companies will be able to change the title without the need to modify twice the field is_company.
Fixes #1713

[I18N] Update 7.0 translations with latest changes on Launchpad

See also https://github.com/odoo/odoo/wiki/GitHub-Transition#translations

[I18N] Update all 7.0 translation templates with latest terms and annotations

Total new terms: 168
Total deleted terms: 95
Total identical terms: 16329
(Some modules skipped, typically all l10n_* modules)

[FIX] tools.translate: addons path resolution failed with duplicates

When the addons_path config contained the
default path, the system was working with
an empty addons_path in the middle of the
other paths. This empty one matched for all
files, making all files appear to belong to
the root path component, e.g. `home` if the
root path is in /home.
Refactored a bit to avoid duplicates and
redundant path calculations.

[FIX] tools.translate: when loading entries from a PO file, use also the POT.

On Launchpad, as commented on bug 933496, PO entries (and their comments) are
shared between series. This mean that e.g. the 7.0 series can have the wrong
`reference` comments (those beginning with #:) as they would be copied from say
the trunk series. Those `reference` comments are used to import translations
and look them up.

This patch adds a few lines of code to tools.translate so that targets
defined in the POT `reference` comments are used in addition to those from the
PO file.

Also adds a test module to validate the new behavior.

This patch stems from:
 - the 6.1 branch by Vo Minh Thu:
    https://code.launchpad.net/+branch/~openerp-dev/openobject-server/6.1-fix-po-targets-933496-vmt
 - the 7.0 port by Numerigraphe:
    https://code.launchpad.net/~numerigraphe-team/openobject-server/7.0-fix-po-targets-933496-vmt

[FIX] portal_sale: remove more access rights, add pricelist access

Pricelist access is necessary for being able to read
the unit price of sale order lines, and does not
pose major security risks, as it does not permit
reading the actual pricelist definition.

[FIX] portal_sale: remove un-necessary access rights, improve rules

Portal access rights need to be associated with
security rules, or not granted at all, as soon
as they grant access to non-trivial data.

[FIX] base: revert 19e401a for title on res.partner form

The title of the partner is restricted with a context that will change in the onchange_type. Having twice the same field will not work well.
Remove the invisible attribute as the domain is handled by the domain.

[IMP] calculating nextcall datetime of a cron SHOULD use the cron's user

[FIX] context_timestamp MUST return a "timezone aware" timestamp

[FIX] export: add the context in the export

In some cases, the data dispalyed in a list depends on the context
This context needs to be passed to the export method, so the exported data reflects correctly the data from the list view

[FIX] account.invoice: payments can never be shown to portal users, but residual might be needed

The residual amount is typically needed to render the
online payment forms (payment acquirers).

Payments on the other hand rely on account.move.line,
something that portal users should never be allowed to read.
Removing the field from the view by setting a model-level
group permission ensures they will not see an error.

[FIX] portal_sale: browse data for payment acquirer form with superuser ID

The payment form needs to be rendered for non-employee
users, and may require accessing extra financial data
not available for those users.

[FIX] account_followup: avoid crashing when a move line has no ref

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

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

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

[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

[FIX] account_voucher: fix typo in journal_id type

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

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

[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] product_visible_discount: no crash when no pricelist passed or product without company

Fixes an error introduced in f6fb2b6

[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] account: unit arg of on_change_unit_amount can be False

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

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