[FIX] forum

[FIX] sorting data

[FIX] small bug

[IMP] cleaning

[IMP] name is name (and not URL in case of links

Merge branch 'master' into master-forum-extension-tpa

[IMP] layout

[IMP] misc layout improvements

[IMP] misc design elements

[IMP] better layout and options

[IMP] database manager: options on dump format

[CLEAN] event_sale: lint / cleaned module, migrated event models to the new API.

[IMP] web: nodeValue is deprecated

[CLEAN] website_event: migrate models to the new API + some code cleaning.

Updated some code in website_event_track, inheritance with new API.

Controllers need a major cleaning, but, hey, they are too creepy.

[IMP] im_chat, im_livechat : add reporting view, quit conversation, option dropdown menu, feedback for live support, better access rules and shortcode model.

[CLEAN] event: lint / remove dead code
- event.event: removed is_subscribed
- event.registration: removed log_ids
- res.partner: removed speaker boolean field, used nowhere
- linted the event module

[FIX] Package: Windows: service name in setup.nsi

[IMP] event: refactored event form view. Usability reviews
showed that the form view was complicated and difficult to use. Now
that stat button and edition options in the front-end exist, it is
possible to remove some of the numerous tabs as well as some options
from the form view.

Coming: further cleaning of the model, behavior and probably the various
event-related views.

[IMP][TEST] event: auto confirmation option, allowing to
automatically confirm events. In a simple flow it is usefull to automatically
confirm event and registrations.

[TEST] event: moved draft2done test into a unittest, easier to finely tune
in order to test the auto confirm option. Moreover this gives a basis to
further add python tests in the module. Hooray.

[IMP] mass_mailing: option to keep sent emails

[MERGE] forward port of branch 8.0 up to e883193

[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

[IMP] hr_timesheet_sheet: improve stat button in timesheet to show Number of hour

(cherry picked from commit fb0edc71c17df3224ba19c9193c58eca67be8ed0)
(closes #2617)

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

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

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

[IMP] event: new kanban view. New! Shiny! With popcorn!

Added two features in kanban view
- having a data-routing attribute on the oe_kanban_global_click
container allow to have a custom routing instead of only
'form view' / 'edit view'. This is used for example to go to
the website view of events.
- allow type=url along with object, action, ... to have some specific
urls for links.

[IMP] event: remove automatic date_end assignation based on
date_begin; also quickly fixed view of event locations.

[REV] website: new blog demo snippets

This reverts commit 7246b8105bdebdd175ca82d5018f68e15e60eb43.

[FIX] base_action_rule: typo when committing base action rule
update, evaluate the action filter_domain, not its non-existing context.

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

[FIX] product: name_search handles negative operators

[FIX] crm: field opportunity_ids contains only opportunities

The field opportunity_ids and opportunity_count should only contain opportunities (and not leads).
This was confusing on the partner kanbans where the field opportunity_count included leads.

[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] account: remove inexistant sequence get

There is no sequence for account.bank.statement.line to prefill the field name (Communication), better remove it.

[IMP] crm, project, hr_recruitment: star (priority) management: less possible values, too much stars are too much tears !

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

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

[IMP] base_action_rule: allow using the CharDomain widget
to set the filter and pre-update filter. Using ir.filters is still
possible, but is not the only way anymore of setting the domains
used in the base action rules.

[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

[ADD] website: new blog demo snippets

(cherry picked from commit 6c6df546eb5a99d00d62fc6b497b4b861431423e)
(closes #2631)

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.

[FIX] web_tip: handles `content changed` event in forms; minor fixes

* eval tip on form content changes
* prevent multiple tip trigerring in some cases
* use scrollTo jquery plugin

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

[IMP] survey: delete label change / fix

[IMP] crm: phonecall: logged calls tooltip

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