[ADD] contribution files and image removed at 2ad092b

[FIX] website_sale: too tiny quantity input (if qty > 9)

[FIX] website: editor: ignore spurious changes for editable content

Changes to contentEditable or attributeEditable attributes
should not cause the corresponding section to be marked
as dirty (oe_dirty). This would otherwise cause an extra
editor save() for those, wrongly marking untouched
templates as `noupdate`, and possibly triggering access
right errors.

[FIX] website: repair compatibility with 8.0rc1 carousel (banner) snippets that may be present in existing pages

[FIX] account: recordsets are not sortable in-place

[FIX] correct typo introduce during forward-port in commit 388f1a1

[FIX] stock: allow the user to use the '!=' operator in domains for 'Qty on Hand' and 'Qty available'. Fixes #2763

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

[FIX] migration: allow debugging of migration scripts + avoid useless copies

[FIX] account: fixed broken constraint (due to the passage to the new API) to ensure the same partner is present on journal items we try to reconcile partially. Fixes #2527

[IMP] fields: allow sorting on inherited field in list views

[IMP] fields: split multi-purpose '_origin' into 'column' and 'inherited'

This makes it easier to determine when a field interfaces a column, and when it
implements an inherited field (with _inherits).

[REVERT] typo

[FIX] website_sale: checkout: do not reset address country every time

Once the value is set it can be modified by the user
(e.g. if geoip detection was wrong), so should not be reset.

[FIX] doc: branch name in configuration

The branch name is used in the version switcher, so the master branch should have a version of "master".

Maybe the release could be the revision hash? Not sure how to extract it from the repo.

[MERGE] forward port of branch 7.0 up to 9cc54dc

[FIX] product: name_search handles negative operators

[FIX] event: jump on the correct menu after installing
the event application (event kanban view).

Also added a missing description on event.type model.

[IMP] fields: split the default value from the compute function

This separation allows to set a default value on a related field, for instance.

[FIX] website*sale: product_id_change qty is 0 by default

Thus, need to pass the actual quantity to product_id_change in order to set values correctly

[FIX] sale_margin: stored field functions cannot trigger other stored field functions

the update of sale.order margin field was trigger with the margin stored field of sale.order.line: This cannot work with the old api

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

[MERGE] forward port of branch 7.0 up to 419c439

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

[FIX] fleet: contract onchange: ensure dict before executing `get`

[FIX] fields: copy origin to avoid sharing field objects between registries

Fix a bug introduced in revision f229974, where shared columns objects are
systematically reintroduced in registries.

[FIX] im_livechat: ir.config_parameter must be read as superuser

As ir.config_parameter is restricted to employees in ACL, for security reasons

Merge pull request #2796 from gurneyalex/8.0-improve_report_not_found_exception

[FIX] lookup_report: exception message when the report is not found

fix exception message in report seachr

when we get to the raise statement, `r` is `None`, so displaying it in the
exception message is useless. `name` is what we want to show.

[IMP] gamification_sale_crm: change date filed for definition "Logged Calls"

The field date_close is only computed when writing the state done. In v8, the phonecalls are always created in state done if the user is not in the group crm.group_scheduled_calls (displays menus to schedule calls.
As the field date_closed is then almost never set, change the date field by the one displayed in the form view. opw 614893

Merge pull request #2771 from odoo-dev/8.0-wmsstaging11-jco

8.0 wmsstaging11 jco

[FIX] purchase: do not reset price_unit on qty or uom change

only when the product is actually changed

[IMP] UoM corrections + Adapt tables in stock doc

[IMP] Float_compare before splitting moves in mrp

[IMP] Better demo data + property comment + recheck

[IMP] Recheck should be type object and procure_method read only when not in draft

[FIX] Inversion of moves in the correct way and assigning production_id

As the moves are split the other way, the original move needs to be done.  Also the production_id for linking the
new to be produced moves and the production order must be written on those.

[IMP] Clean

[FIX] mrp: solve bom structures views

A previous refactoring brought a bom_line_ids field on the mrp.bom, thus
deprecating the _child_compute method. But the previous refactoring did
not go through all the views, breaking everything that relied on the
_child_compute (tree view, report). As the bom_line_ids refers to the
mrp.bom.line model (introduced by this previous refactoring, note:
_child_compute returned mrp.bom record) and that we can't make a treeview
showing different model, this patch introduce a function field _get_child_bom_lines
on the mrp.line model, allowing to go through the bom_line_ids of a mrp.bom.line
if this mrp.bom.line refers to a mrp.bom.

[IMP] Rename bom_line_ids to child_line_ids in mrp_bom_line to avoid confusion

[IMP] Update module descriptions of procurement and product_extended + _bom_find and properties

Module description of procurement was deprecated (talking about mrp, ...) and in product_extended
it described things not implemented in the module.

In _bom_find, we passed a UoM which was not used in Saas-4 and it would not be logical that you
need to select a BoM that matches the UoM, so I removed it.

In the demo data, there was still a push rule which triggered a move from output to pack.  The copy=False
is correct for production_id when you would have these push rules.

For the properties: we want to allow to take a bom which has no properties, but only when there is no other
BoM matching the properties we pass.

Update module descriptions

[IMP] production_id copy + no round

[IMP] _bom_find without uom, property correction

[IMP] Change documentation + new action_consume/qty/origin corrections

Simplify the action_consume of the consumption lines after the corrections
by Kevin Wang.  Also the UoMs are revised as the action_consume uses the default UoM
of the product.

We have to avoid circular boms where a child bom should not contain the product that
represents the parent bom, but it is possible for example to use another product of the parent bom in
the child bom.

As the consume line move has no procurement rule, its origin will have no description.  So, when there is
none it will also check the description of the previous move (when passed to procurement for example) This way
the chained moves or purchase order for example will have the MO-number as origin and not nothing.

[IMP] Change assignation

[IMP] UoM changes continuation

[IMP] Make sure we can use 2 times the same product in a BoM

[IMP] Source document for consume lines to procurement

use warehouse._get_mto_route to get the MTO route

instead of using a custom lookup with a different implementation
closes #2608

pass context as a named argument

[FIX]keep primary move when split consume raw materials

[IMP] UoM in mrp + rereserve of partial

Merge pull request #2789 from erwin-bas-solutions/translation-timesheet-alert

[Fix] hr_timesheet: Make alert translatable

[Fix] hr_timesheet: When changing the date in timesheet activities gives alert, but alert is not translatable.

[FIX] event: added missing description on event.event model, leading to event model having Email Thread has description due to inheritance.

[MERGE] Forward-port of 7.0 up to 9b1a9c9

[FIX] tools: remove useless attributes

[FIX] account: default product taxes on product.template

As product.template is now most commonly used than product.product

[FIX] Non printable character 0x07h in website's german .po (Fixes #2688)

[FIX] tools: remove useless attributes

[FIX] bus, im_chat : set bus and im_chat in auto install to set im_odoo_support in auto install too.

Merge pull request #2620 from savoirfairelinux/8.0-http-unicode

Fix two possible UnicodeDecodeError when an exception occurs

Fix two possible AsciiErrors when an exception occurs

Signed-off-by: Sandy Carter <sandy.carter@savoirfairelinux.com>

[FIX] website_mail: t-field replace by t-esc as html_sanitize applied on the field

[FIX] website_mail: sanitize email templates when displaying templates selection

In cases the templates is not html syntax valid

[FIX] doc readme: the list of versions uses commas

it's not space-separated

Merge pull request #2766 from Nick-OpusVL/patch-1

Fix spelling error of Human Resources

Fix spelling error of Human Resources

[FIX] package: last fedora epel repo

[IMP] website_sale: confirm quotation only when the payment is done, not directly when hitting 'pay now'.

[FIX] purchase: payment_term_id twice in the view

+ replace widget selection by options no_open no_create

[FIX] website: restore compatibility with RC1 templates for commit 77c41cc5cb64a951789d251b445f05e42cf1b997

[REVERT] tests added in  forward port e2e60bf4eb89d92e883e8520771523c1c0377fd3

This is related to the revert 333852e19dd3d5174e07e29f9d8f6c087b2c09f6

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

[MERGE] forward port of branch 7.0 up to 333852e

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] website: 'Display Editor Bar on Website' group stopped working at commit a2ef044c477

The Edit button never appeared anymore for these users.
The idea was that they should see an edit button with
limited editing capabilities depending on their other
access rights.
For example, someone with only Sales Manager access and
'Display Editor Bar on Website'
would be able to edit online quotes from the website_quote
module, but not change the actual website pages or menus,
for instance.

[FIX] website_sale: avoid subscribing the website user (Public) to all shop orders

Especially as this user does not have a valid email, which
could prevent notifications to reach other followers.

[MERGE] forward port of branch 7.0 up to cac6a26

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

[FIX] website_sale: auto-confirm free orders upon cart validation

Orders are normally confirmed when the payment transaction
is processed, but there is no transaction for free orders.
This caused them to stay in draft until manually cancelled.

[FIX] im_odoo_support : fix event triggered to bind event on dropdown usermenu item.

[FIX] base: correct non-copy of `login_date` + adapt tests

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

[MERGE] forward port of branch 7.0 up to af37b30

[ADD] im_odoo_support : add the module for the live odoo support from all instances. This sets the bus and im_chat module auto installable.

[FIX] im_chat, im_livechat : fix the loading history for anonymous user and remove useless search_user_status rpc call.

[FIX] fix context problem in graph view

The group_by query expects the context to have group_by_no_leaf = true,
so we can not just blindly forward the context to the groupby query.
This is a defensive way to fix the problem, to avoid other possible
crashes.  But the context shouldn't have group_by_no_leaf anyway,
it does not make sense to explicitely do that in the action

[FIX] cron: doall false by default

As in most cases, we do not want this doall

Modify General Ledger report to be printed on landscape if set in wizard.

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

[FIX] payment: transaction formview display issue

Using group without defining the col attribute will let the form
renderer choose the right display to avoid screwing up the layout

[FIX] Error when installing module document

[FIX] website_sale: free carts were not confirmed

[FIX] account_voucher: use resolve_2many_commands() instead of buggy-hand-written function

[FIX] account_voucher: do not unlink records in onchange method

The method recompute_voucher_lines() is used by several onchange methods of
account.voucher.  It used to unlink() the voucher lines before recomputing new
voucher lines.  As a side effect, if you edit a voucher and discard your
changes, your voucher has lost its lines, although you did not save your
changes!  Moreover, the call to unlink() is invalidating the record cache,
which screws up completely the onchange() method.

So, instead of unlinking the existing lines, the onchange generates commands
[(2, id), ...] to remove the existing lines only when saving the changes.

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

[FIX] payment_ogone: last commit

[FIX] paymnet_ogon: add missing SHA OUT keys

The SHA OUT list was incomplete and therefore SHASIGN was not matching anymore
List of keys from https://viveum.v-psp.com/Ncol/Viveum_e-Com-BAS_EN.pdf

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

[FIX] portal_sale: force empty context

The context was removed by 1933e926.

[IMP] website_sale : avoid duplicated order lines

Add line id to while calling _cart_update() from sale_get_order()
The missing line_id parametre was making the _cart_find_product_line() call to fail as it was linked to an option while searching for lines without options (making the method recreate new lines).
Fixes #2573

[FIX] website: prevent t-field on selection fields

This quick fix avoids a buggy behaviour in version 8.0 that could
confuse users.

A future version should implement properly selection fields in t-field.

(closes #2490)

(cherry picked from commit fe3cac30e4c5c132da1de02576d4aa325979ccd9)

[IMP] models: when checking for data in a table, do not use column 'id'

Replace the query "SELECT min(id) FROM xxx" by "SELECT 1 FROM xxx LIMIT 1".
Both requests are as efficient, and the second one does not crash if column
'id' is missing.

[FIX] fields: a function field with fnct_search should be searchable

[FIX] css: overflow-x: auto should be on modal-body

[FIX] Report: attachment_use need attachment

[IMP] crm: remove TODO posted by past me

[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