[IMP] product: name_get matching on commercial_partner_id for suppliers

The name_get of a product will use some information (e.g. default_code) based on the supplier.
The matching of the supplier should use the commercial_partner_id in case the supplier info are on the company and the partner_id in the context belongs to the company (e.g. creates quotation with a contact of the company).
Fixes #1219

[FIX] account: avoid closing fiscalyear with unpost entries

As when closing fiscal periods, forbid to close a fiscal year with unposted entries.
Fixes #1194, opw 610784

[FIX] account: remove orphan analytic lines

If the analytic account is not present on an entry, still removes the Analytic Journal Items linked to it.
Fixes #1194, opw 610784

[FIX] product: name_search handles negative operators

[FIX] purchase: minmum purchase quantity and float precision (opw 610557)

[FIX] hr_payroll_account: do not create entries with amount at 0

If the salary compuation gives a salary of zero, skip the creation of the account.move.line (not good to have lines at 0).
Fixes lp:1298116, opw 605816

[FIX] hr_expense: computation of tax/base amount

When computing the amount for a tax (included) with children, accumulate the included taxes to avoid taking only the last one (opw 614867).

[FIX] account: sort partner ledger report by partner ref, name

[FIX] tools: remove useless attributes

Revert "[FIX] product,float_utils: perform ceiling via float_round with new rounding_method UP"

This reverts commit d4972ffdb6b9356a524eef1dbc11f455ff4473f2.

Seems to break some cases, at least in _product_reserve from stock/stock.py

Actual use case:

SELECT product_uom, sum(product_qty) AS product_qty FROM stock_move WHERE location_dest_id=%s AND location_id<>%s AND product_id=3645 AND state='done' GROUP BY product_uom;
returning 1 | 6

SELECT product_uom,-sum(product_qty) AS product_qty FROM stock_move WHERE location_id=%s AND location_dest_id<>%s AND product_id=%s AND state in ('done', 'assigned') GROUP BY product_uom;
returning 1 | -6

results += cr.dictfetchall()
    total = 0.0
    results2 = 0.0
    for r in results:
        amount = uom_obj._compute_qty(cr, uid, r['product_uom'], r['product_qty'], context.get('uom', False))
        results2 += amount
        total += amount
Total = 1, amount = -5

It should actually be
Total = 0, amount = -6

[IMP] account: use float_compare for taxes

Avoiding some python float representation errors

[FIX] project_mrp: task description duplicated + product name translation

[FIX] cron: doall false by default

As in most cases, we do not want this doall

[FIX] res_users: do not copy login_date when duplicating users

[FIX] crm: merge the phonecalls during an opportunities merge

[FIX] portal_sale: force empty context

The context was removed by 1933e926.

[FIX] product: _compute_qty: first round before ceiling, to avoid pathological cases

Fixes problem when we try to sell 12 units of a product and change it to 1 dozen,
the algorithm was then trying to recompute the original amount and was getting
12,0000048 as a result which was then passed to the ceiling method, getting 13.0!

See also previous commit and issue #1125, PR #1126

[FIX] product,float_utils: perform ceiling via float_round with new rounding_method UP

Modified product ceiling() to use float_round() with special mode
for rounding UP (away from zero), avoiding pathological cases where
float representations errors were ceiling to the superior unit.

Also added correspding tests for rounding_method=UP

Fixes issue #1125, and replaces PR #1126.

[INP] portal_sale: remove buggy context

Even if this should never be used in normal portal environment, if a user with enough access rights tries to create an invoice, this would crash. We don't need context here (opw 612591).

[IMP] stock: Optimize stock_move.setlast_tracking()

When setlast_tracking is called on a large number of moves in a picking
(e.g.  when splitting moves in a picking), the time to complete grows
exponentially.  The reason is that it loops over all the moves of
a picking, even if it keeps only the last tracking.

The method now uses a search() with a limit so it doesn't need to browse
all the moves.

Added test to check the behaviour of setlast_tracking
Fixes #2448

[FIX] crm: correct open lead or opportunity view from calls

When opening a lead/opportunity from the phonecalls view, we did not open the correct view (always the lead).
This will use the type of the crm.lead to determine which view should be used, opw 608493.

[IMP] stock: do not generate empty backorders

If every line of a partial delivery is at zero, do not generate empty backorders (opw 608680)

[FIX] web_calendar: reset dataset index if search result empty

[FIX] stock: error on stock.partial.move creation

The stock_partial_move wizard removes the required attribute for the field picking_id on a stock.partial.move. This means that we could get moves without picking_id and the previous line was failing ('NoneType' object has no attribute 'currency_id'). opw 614531

[FIX] widget html: rendering issue in mail composer on Firefox
The cleditor width does not include the margins. Setting 100% will make the editable area too large (104%) on Firefox (opw 611700).
This issue was already fixed in newer releases, from commit 9247c37de7d67beb479905e218656665dc12391f
Nevertheless, it introduced a new issue: while editing the form, the content of widgets html which were located in hidden notebook pages weren't displayed when displaying the page content.
opw-614448

[FIX] account_voucher: import invoices, sign according to transaction type

[FIX] auth_ldap: remove unused sql column

[FIX] account_voucher: convertion rate of purchase receipt

The conversion of currencies in vouchers are made based on payment_rate_currency_id field (instead of usual currency_id).
This field was not present in the purchase view and not correctly set when we change the journal (and thus the currency) which was leading to rates of 1. With a rate of 1, the amount is equal to the currency amount (which is rarely correct). opw 611254

[FIX] purchase: keep fallback behaviour that was removed at rev 8973217

If no valid invocie is found on the purchase order line, use the price and currency of the line (coherent with help message)

[IMP] stock: display translated selection value in message

When a user tried to delete a done or canceled picking, the error messages used to display the key of the selection field ('done' or 'cancel') which was surprising in other languages than English. This patch takes the string value of the selection field, keeping the context to get the translated value (opw 613068)

[FIX] event: translate templates into user language

When sending an email of the registration/confirmation, the context was lost and the tempate sticked in en_US.
With the patch, the template will use the current user's language.

[FIX] kanban: wrong condition to reset the dataset index

The dataset index should be reset either if the dataset is empty or the index is greater than the records size

[FIX] hr_timesheet_sheet: avoid error when sort attendances

When an attendance line is edited, the tuple has no key 'name' which lead to a traceback.
Fixes #546

[FIX] stock/purchase: dates and currency rates

Use the currency rate of the purchase date instead of the one of at the reception time (opw 610430)

[FIX] account_anglo_saxon: correctly set the accounts for returns

The generated moves on a return should be the reverse of the outgoing move.
Fixes #1817, opw 604417

[FIX] loading: always process auto-installed modules for new databases

If the server was started without -i or -u and
happened to initialize a fresh database,
auto-installed modules that depend
on `base` only would stay in status "to install"
without actually being installed (until the next
installation round was triggered).
This was of little consequence in 7.0, but causes
a crash in 8.0.

Fixes #953

[FIX] orm write: do not try to store computed & stored fields for relational records deleted by *2many fields

opw-613772

[FIX] auth_ldap: avoid to try twice the authentication to the ldap server

See #1982

[FIX] point_of_sale: set journal as pos payment method on setting journal in the payment methods of a pos config when none is set.

[FIX] sale: backport of rev 709e22c32aca

[FIX] sale: sale report view, wrong domain

Quotations are sale.order with state draft and sent
Sale orders are sale.order with state other than draft, sent and cancel

[FIX] orm: do not drop foreign keys of transient models

During the update of a module, the existing foreign keys are dropped if they have a different ondelete_rule than the one specified on the field.
The foreign keys for many2one transiant -> non-transiant are created with cascade rule by default (see `m2o_add_foreign_key_checked` method) so the check needs to be realised in the same conditions.

[FIX] web: backport of ebb826a and 2372d30

Fixing breadcrumb not being displayed after some action button (e.g. validate invoice, opw 612763)

[FIX] tools: restrict available attributes

[FIX] mrp_repair: invoice the actual invoice partner id

[FIX] l10n_ro: backport of fix done in res_partner.py from aa76885

PR #720

[FIX] web: sort with undefined value

In this awesome programming language called javascript, undefined is neither bigger nor smaller than a string. Previous code was then considering undefined values equal to any string.
This fix allows to sort in a column of a o2m field and group the undefined values together. opw 607704

[FIX] stock: return picking, pass serial number value when returning

[FIX] survey: access rights error when editing

The wizard 'Edit Survey' > 'Add Question' was trowing an access rights error as the id was passed as a string instead of a real id. opw 607854

[FIX] hr_expense: tax amount computation

The tax_amount field should contain the basic amount (without tax) for base tax code. (opw 613470)

[FIX] web: *.openerp.com to *.odoo.com

[FIX] mail: encoding of sender name with unicode

When sending an email, both formats 'Name <email>' or '"Name" <email>' can be used for fields 'From', 'To' and others. If the name contains unicode characters, a regex only matching '"Name" <email>' was used to encode the name with RFC2047. That meant that the name was not encoded and eventually dropped, using only the email part.
Instead of using a limited regex, use the parseaddr method from email library.
Fixes lp:1272610, opw 607683

[FIX] ir_attachment: attachements removed on record unlink

[FIX] orm, read_group: read_group_fill_results handle no_leaf

[FIX] hr_expense: tax included computation

When a tax included has a several childs, the source line should not be duplicated for each tax.
Without the patch, the actual product price (without taxes) would be (total - tax amount) for the last tax only (and not the sum of each tax). (opw 609104)

[FIX] web: missing backport of rev df3c6b750249

[FIX] web: do not try to close a destroyed dialog. trigger "closing" signal on close by top-right "X" button

[FIX] ir_attachment: ignore if model uninstalled

if the model of an attachement no longer exists (the according module have been uninstalled, for instance), ignore the security check

[IMP] css: width of readonly list view field

readonly float fields in list view were not correctly placed when the focus was on the line (seen espacially if field was large, e.g. nb_register on registration)

[FIX] stock: show the UoS quantity on picking lines

The pickings already show the UoS, but not the associated quantity, which makes it useless.

[FIX] web: do not allow to access a database if not matching db-filter

[FIX] mail: translate tracked fields name in chatter

Status tracking messages are not translated into user's language while updating any document with tracked fields (opw 612847)

[FIX] tools,mail: improve parsing of references from rev. 0bee015dd67cb809fd235215b43d331074824e42

Backport of 8.0 fix was not enough for 7.0, because it still fails for the legacy reference match (model+res_id), which is not used in 8.0

[FIX] sale: keep page when confirming a sale order

To keep the pager and view changer after confirmation of a sale order, do not reload the view (quotation and sale order view are the same anyway)
Fixes #250

[FIX] document: display Attachment(s) menu only on form view

The attachment menu (list and add) has no effect in tree view (not supporting multi-items upload and display), the menu should then only be displayed in these view (opw 612534)

[FIX] stock: on partial picking, do no open a new window action if not needed

This is related to rev. dd4d72d

Not opening a new window action allows to keep the current list of stock picking being threated, and to use the next an previous arrow of the forom view of the web client.

[FIX] mrp: do not reset back stock moves to confirm

For instance, setting a BOM Phantom with:
Finished product: stockable, MTO Manufacture
Components: stockable, MTS, Buy. Inventory set to 1000
Stock moves of components are directly set to assigned once the procurement confirmed thanks to JIT
The stock moves should not be set back to confirmed after they have been assigned

[IMP] web: extra comments to explain commit 57b4860

Added some cryptic comments so we remember a bit
why we have a complicated dance with on_close.
Basically we do not want to reload the
original form view until the last popup is closed,
in the case where several wizard (steps) are opened
one after the other.

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