[FIX] account: limit search for fields_view_get to one result

+ use ir.model.data get_object_reference method for better performance (cached result)

opw-616616

[FIX] mail: read more show original message

Instead of html cleaned html, to allow display blockquotes

opw-614729

[FIX] product: reference in test

[FIX] product_visible_discount: discount with different unit of measures

The computation of the price without pricelist should take care of the unit of measure.
e.g. if computing discount for objects in dozen (on a product with price in unit), returned unit price should be (price*12) where 12 is the factor to go from dozen to unit.
Otherwise the compared prices (with and without pricelist) would not use the same unit of measure and the comparaison would be inconsistent. (opw 599727)

[FIX] product: pricelist, uom and price_surcharge

The price_surcharge attribute must be computed based on the reference unit of measure (divided by the factor).
This is to make sure than 12 units and 1 dozen have the same price after pricelist computation (opw 599727).

Added test checking the correctness of pricelist computation based on unit of measures.

[IMP] config: Test that the config file is readable

Fixes #3237

cherry-pick of 8e72049

[FIX] When using "--stop-after-init", set the return code correctly. A non zero return code reflect the number of databases that fail to load/update

Backport of 2c4e370b766db6280d38a4d3535e07de604b1922 and
33ce0e73dba7c23d004a278f61943be8052e7f37

[FIX] web: correct display of dropdown arrows in ie11

[FIX] account_followup: use %s string formatting to avoid concat None values

[FIX] product.uom: safer handling of factor/factor_inv in UI

Add readonly attribute to avoid sending both factor and factor_inv value to the backend when saving.
This was possible if the user switched between uom_type to fill the two fields.

[FIX] product: remove digits_precision from uom factor fields

Remove the hardcoded precision of 12 on factor and factor_inv,
to use the complete natural precision of NUMERIC types,
preserving all significant digits.

e.g. a UoM with a factor_inv of 6.0 used to be computed as:
factor_inv: 6.0 -> factor: 0.166666666667 (1.0/6.0, rounded to 12 digits) -> factor_inv: 5.999999999988 (1.0/factor)
which could lead to errors such 12*0.166666666667 = 2.000000000004 instead of 2.0

Slightly changed the way the ORM handles float fields to allow setting `digits=0`
as a way to explicitly require a NUMERIC value but without enforcing/rounding
the values at the ORM level, i.e. a truly full-precision field.

NUMERIC type has unlimited precision but is less efficient so should not be
used as the default behaviour, which is why we keep float8 as an alternative.

Modified the view to display the product UOM factor with a 5 digits value by default.
This value is for usability purpose only, the field still accepts bigger precision, by
setting the `digits` option on the field in the form view.

This change is safe in a stable series, the `digits=0` alternative is
treated the same as the default `digits=None` everywhere in the framework,
except when creating the database field.

[FIX] base: support float rounding with rounding_method=UP (ceiling)

Add rounding_method parameter on float_round method to offer
HALF-UP (default, usual round) or UP (ceiling) rounding method.
Use the second method instead of math.ceil() for product
reservations.

For UP, the python math.ceil() method uses "torwards infinity"
rounding method while we want "away from zero".
Therefore we use the absolute value of normalized_value to make
sure than -1.8 is rounded to -2.0 and not -1.

Fixes #1125 #2793

This is a cherry-pick of d4972ff which was reverted at 333852e due
to remaining issue with negative values.

[FIX] report: page numbering rml reports

Save the NumberedCanvas state before doing a page reset.
The order of execution when rendering an rml report is the following:
1. init canevas (_pageNumber = 1)
2. render the page element
3. if still pages to render, afterPage method
4. if still pages to render, showPage method (_pageNumber += 1)
5. back to step 2 for each page
6. draw the ResetPage element (setting flag _doPageReset=True)
7. end the document build with afterPage & showPage method

The PageReset element should be executed at the end of the rendering of a story (subdocument) to reinitialize the page numbers to 0 (for new story) and insert the pageCount element for that story with the total number of pages (needed if want to use tag <pageCount/> in rml).

In case of NumberedCanvas (e.g. used in Trial Balance report), the numbering is generated at the end of the build using the _saved_page_states dict in the canevas.
To have an accurate _saved_page_states content, it needs to be saved before the pageReset.

Fixes #2225

[FIX] point_of_sale: no need to specify numeric digits

In the report pos order, average_price was set as a numeric(16,2), therefore, if the amount was too big, it led to a psql crash:
A field with precision 16, scale 2 must round to an absolute value less than 10^14.

[FIX] res_partner: add parent_name related field

In some cases (e.g. with record rules), the name_get might not have access
to the parent name. Therefore a parent_name related field solves the
issue (as it read with as superuser).

[FIX] l10n_fr_hr_payroll: bad copy-paste in 69c1333

[FIX] l10n_fr_hr_payroll: report "Fiche de paye"

Avoids getting False for the field end_date (Sortie) in case of no end date specified on the contract. opw 614624

[FIX] account_followup: do no duplicate partner's account.move.line

Duplicating a partner should not duplicate the content of the unreconciled_aml_ids field.

[FIX] mail: set the parent_id to the first message of type email

And fallback of any type if there is none of type email

This fix allow the communication between two mail thread from two different Odoo servers having message creation subtypes, like project issue or crm lead

[FIX] sale: translate advance message on advance invoice

[FIX] web: cherry-pick of 893e4268650ddd517e3044f41c582fd92de6edfe & 61207f3f4d9206d4c1a3e484eb89f09d2258977a

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

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

[FIX] account: precision_get returns a number of digits

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

[FIX] account: self typo

[ADD] l10n_lu: abbreviated financial reports

Abbreviated version of the p&l and balance sheet reports

[FIX] stock: groups mixup in views

Use group_production_lot for serial options, group_stock_packaging for packaging, use group_tracking_lot for pallet/parcel
Groups are removed completly from the view for stock.tracking as they render the view useless.

Always display weights on the product form
They really have nothing to do with the logistic units and we don't have another group to restrict them to.

Fixes #1443

[FIX] event: double context

[FIX] event: missing context

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

[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

[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] 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] ir_attachment: restrict access to orphan attachments to employees

[FIX] auth_oauth: fb and google oauth providers reset on general settings apply

 - The res_config.xml file was missing in the manifest (so couldn't check the use of FB and Google OAuth from the general settings)
 - The default value for these oauth configuration were not set

[FIX] account_analytic_analysis: invoiced field takes customer invoices only

Besides, it subtracts customer invoices with customer refund invoices

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