[FIX] Image resizing bug

Merge pull request #2352 from odoo-dev/8.0-imp-bank-statement-reconciliation-widget-ama

fixes and improvements in the new bank statement reconciliation widget.

[FIX] account: bank statement reconciliation: when creating a new move line to reconcile an existing one, use the existing move line's partner or, if there's none, the statement line's partner

[IMP] point_of_sale: opening the cashbox only if paying with cash was problematic as you sometimes need to use the cashbox with bank payment methods such as ticket meals

[IMP] point_of_sale: partner edition ! -> create partners, edit partners, assign them barcodes, take pictures, etc.

[WIP] point_of_sale: qweb template for partner edition

[IMP] Brought back /website/image route w/filename hinted in headers

Also added support for this route in widget Image#from_html()

[MERGE] Forward-port saas-5 up to f9bcd67

[FIX] purchase: keep reference to purchase order line

When generating an invoice from a stock.picking, the reference to the purchase.order.line needs to be kept (e.g. this is needed by anglo-saxon for price valuation). (opw 600767)

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

[FIX] orm: typo during forward port 60a82133cc0e0d7c4879b6c518054cbd44e20c0c

[REF] account: minor change made during the code review

[FIX] fields: make convert_to_cache() more robust with inverse one2many

When a one2many field uses an integer field as inverse, the onchange method on
the second model may receive a dictionary for the value of the integer field.
This is because the client expects that field to be a many2one.

[IMP] Refactor ir.ui.view's read_template cache invalidation

[FIX] analytic: contract manager_id defaults to the context value

[FIX] sale_stock: packaging was given a product instead of a product template id, leading to possible crash or incoherent values.

[MERGE] Forward-port saas-3 up to 09ab2d4

[MERGE] Forward-port 7.0 up to 938502a

[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

[IMP] account: bank statement reconciliation: allow to filter on move line name

[FIX] Bundlification remains (Fixes #2463)

[IMP] base: show local modules in Settings until apps.odoo.com is migrated for 8.0

[IMP] broaden scope of search of bank statement reconciliation candidates

Looking for accounts with reconcile=True is enough.
Restricting on payable/receivable account types narrows the search
to much and makes it difficult to implement transfer account holding
the payment while they are in transit at the bank.

[IMP] mail: app specific notifications

[FIX] account, bank reconciliation widget: fixed the method that manage the pager in case of partial reconciliations

[FIX] ir_ui_view: Active (boolean) should not be required

If so, impossible to uncheck the box in the webclient

[FIX] account: bank statement reconciliation: when fetching moves lines for reconciliation, if lines from the same partial reconciliation are filtered out, keep fetching to returne the correct number of lines

[MERGE] forward port of branch 7.0 up to f5f7609

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

opw-613772

[FIX] account: bank statement reconciliation widget: wait for all reconciliations to be displayed before to load matches, so moves lines in reconciliation proposition are correctly excluded

[FIX] account: bank statement reconciliation widget: when filtering move lines by string, for a statement line with no partner, do a or on varchar fields, not a and

[IMP] website_sale: do not show transaction on sale order

This object is quite technical and has not purpose to be displayed on a sale order.

Forward port of branch saas-3 up to rev 2ee1843

[FIX] website_sale: retrieve transactions as superuser

Due to additional security rules, the transactions made as public user will have a new partner_id. The transaction needs to be retrieved as admin to be set in the context.
The operations in payment_get_status are made as superuser but the session_id is checked in the assert above to avoid url manipulation.

[FIX] account: bank statement reconciliation widget: do not set partner of a statement line with no partner when selecting a move line

[FIX] account: bank statement reconciliation: allow to match move lines when no partner

[FIX] survey: mess with JS loading

Since yesterday, JS bundles are now loaded at the end of the webpage.
Old-style modules that hardlink js code in the <head> section are now
broken since they use jQuery's $.

[REF] account: minor change made during code review

[IMP] payment: simplify rev d99835e

The group public is defined in base so no need to add security rule in website_payment module (same as for portal)

[FIX] account: wrong conversion way to foreign currency in the bank statement reconciliation widget

[FIX] hr_recruitment: possibility to group application by creation date

[REM] unused class attribute

[MERGE] Miscellaneous SEO improvments

- Compress images using pngquant
- Added http.send_file() helper
- Move javascripts in the bottom of the page (better google pagespeed ranking)
- Added website.get_image_url() helper that uses a hash allowing the usage of
  a more aggressive cache
- Misc. tools/image improvements

[FIX] website.image_url() access rights

[FIX] double html escaping in qweb

[IMP] add `url_for` hook to bundle urls

[ADD] image_save_for_web() format argument

[ADD] ir.qweb render_attribute() hook for postprocessing

[ADD] Compress uploaded image by default

Added an opt-out option available from the button's dropdown

[ADD] tools.image_save_for_web() helper for image compression

[IMP] Convert product demo png's to 8bit

[IMP] Compress resized /website/image's

Refactored and fixed tools.image_resize_image() that converted to RGBA
after making thumbnails, resulting in bad looking picture in case the
source is in 'P' mode (indexed palette)

[ADD] Helper for /website/image (allows to use aggressive cache)

[IMP] Move website_mail_group floating assets in frontend bundle

[IMP] Move website_sale_tracking.js in front-end bundle

[WIP] Testing javascripts in the bottom of the page

[IMP] use http.send_file() for /logo.png

[ADD] http.send_file() helper

[IMP] compress *logo*.png with pngquant

[FIX] payment_paypal: paypal account migration: correct search of existing providers

[FIX] payment_paypal: create account with at least required fields

[FIX] account: invoice view - useless trailing spaces in labels

[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] web_calendar: make UI string translatable

Fixes issue #1755 partially, when the new term
will be translated

[FIX] ir.ui.view: match translated terms without heading/trailing whitespace

As the whitespace is stripped when exporting terms
to translate, it must be done as well when matching
translations at runtime.

Fixes issue #1755 partly

[FIX] ir.translation: translated terms within view definitions were not loaded

The changes for QWeb template translation have introduced
res_id values in PO comments for all `#: view:` terms, as
the real database id needs to be resolved when loading them.

For non-QWeb views this is not necessary and actually caused
the terms to be dropped when the res_id could not be resolved.

Rather than having a different PO comment format for QWeb and
non-QWeb views, we can extend the QWeb hack to force the
res_id to 0 for non-QWeb views.

Fixes issue #1755

[FIX] share/wizard: fix new-api method call incorrectly passing context

[IMP] account: bank statement reconciliation widget: show bank statement line name AND ref

[IMP] account: bank statement reconciliation widget: ctrl-enter makes only 1 RPC call

[IMP] account: bank statement reconciliation: correctly find reconciliation proposition based on amount match

[IMP] sale: quotation confirmed mail subtype defaults to True

[FIX] payment: access rights limitation

Do not allow everybody to access account.transactions.
Restrict by default to readonly and even restrict the access with a record rule, give access to salesman.

[FIX] models: make field inheritance work when source field is defined in old api

[REM] report: remove openerp/report/pyPdf

Distributing our version of pyPdf is not required anymore since we
introduced the new reporting tool.

In order to keep reports working, the standard python library
`python-pypdf` has to be installed instead (through pip or your
distribution's package manager)

Update README.md

[FIX] portal_sale: security rules matching the one of public user

The portal users had no access to the attributes objects (while the public user does) and was not allowed to render the shop page.
Fixes #1961

[FIX] request.website browse record bad context

[FIX] account: account_invoice's onchange_partner_id uses the first partner bank account if several are defined. Merge of #2280

[IMP] account: bank statement reconciliation: better animations

[FIX] fields: convert_to_read, handle missing many2one foreign keys

[IMP] account: bank statement reconciliation: for a partially reconciled move line, display total and residual amount

[FIX] packaging: windows: passlib dependency was not correctly set

[FIX] web: traceback in search view when no field

[FIX] website_hr_recruitement: Don't use GeoIp if all countries is selected

[IMP] account: bank statement reconciliation widget: display partner name found in statement import in change partner field ; do not set the partner to false before changing the partner

Merge branch 'acsone-8.0-onchange-one2many-test': add test for onchange on one2many field

[FIX] display problem in graph view in some cases

Recently, the graph view was changed to prevent changing groupbys
for active custom filters.  Unfortunately, I did not take into account
the fact that pivot table can be drawn in part when expanding rows.

The "frozen" parameter was then undefined and caused display problems,
this patch should fix the issue.

[FIX] models: improve implementation of _compute_display_name()

The method was expecting that name_get() returns complete and in-order values.
Because of this, some records in the recordset could end up without a value.

[IMP] mail,website: improve update_notification

[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] website_hr_recruitement: typo

[FIX] fields: in *2many.convert_to_cache, fix the effect of command "1"

[FIX] website_hr_recruitment: add param to force all countries else it use GeoIP, check that country returned by GeoIP have jobs availables else fallback on all countries

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

[MERGE] forward port of branch saas-5 up to 9699f62

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

[FIX] web: do not check if db in session match filter if no db

[IMP] api: compute methods cannot depend on field 'id'

This feature hasn't been implemented because of complexity problems.
Therefore, we should not allow the developer to use it.

[MERGE] forward port of branch saas-5 up to 163d7f5