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

[MERGE] forward port of branch 7.0 up to 59b76ee

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

[MERGE] forward port of branch 7.0 up to 5872c24

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

[IMP] crm_partner_assign: remove inexistant onchange

This field is readonly so a onchange is useless anyway (opw 612593)

[MERGE] forward port of branch 7.0 up to 7307227

[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

[FIX] website_forum_doc: forum.documentation.stage readable by all

[FIX] payment_paypal: avoid json decode error

with post.pop('custom', '{}'), if the key 'custom' is not there, it will return '{}', but if the key is there, but the value is equivalent to False, the json decode will fail.

[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

[IMP] Optimize the schedulers of the calendar, fetchmail and base_action_rules modules

Disable the the crons when they'd have nothing to do, re-enable them when a
task is (probably) introduced

[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

Forward port of branch saas-3 up to c3f5289

[IMP] gamification: improve 40e8437 to make only one global request

[FIX] gamification: performance improvement

On a large database, thousands of badges may have been granted. This code used to prefetch all fields of the users and badge users. This was highly inefficient. Replace this code by SQL query.

[FIX] web: no_create, if loose focus do not open quick_create

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

[MERGE] forward port of branch 7.0 up to 18d365c

"[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] stock: display product variant name in the slip

[FIX] l10n_ma: proper chart of account name

[FIX] website_hr: give public group the right to read resource.resource records

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

[MERGE] forward port of branch 7.0 up to 15ee4d8

[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] mail: read ir.config_parameter with SUPERUSER_ID

[MERGE] forward port of branch 7.0 up to f65c913

[FIX] base: security: employee must have access to res.groups

[FIX] account: use method has_group() instead of browsing the group directly.

[FIX] hr_holidays: remove now invalid xml attribute "model" (introduced by last forward-port)

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

[MERGE] forward port of branch 7.0 up to 3509e15

[MERGE] last forward port of branch saas-4 up to a65a95f

[FIX] website_blog: controllers blog expects a date, not a datetime

[FIX] stock: missing quantity parameter from get_price_from_picking

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

[IMP] stock: filter for procurement.type view

Shouldn't allow to select view in the location field. Otherwise you'll end having pickings composed of stock moves using view stock locations.

[FIX] sale: on create, set taxes if product_id_change called

[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] web: fix blur problem in FieldMany2One widget when clicking on autocomplete option inside a wizard

Revert "[FIX] web: grid headers should be aligned properly"

This reverts commit 4c335588a1f5d6d1f03e17aa20e37ffe4197447c.
It leads to table widths inconsistencies: The tables take more width then before, and overflow much more than before. This can be seen in "Accounting" > "Journal Entries", in the list of the form view

[FIX] hw_escpos:  Replaced "tab" by space

call to self.open() had a tab instead of spaces

[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] website_sale: pager in product list

Keep the reference to category and search view in pager.
Update the search template to include form and keep correct reference to the category (as a slug and not a parameter).
Fixes #688

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

[IMP] stock: stock incoterm converted to uppercase

[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] web: grid headers should be aligned properly

Forward port of branch saas-4 up to rev 15a3a2a

Forward port of branch saas-3 up to 7bcefaf

Forward port of branch 7.0 up to 0ab88f5

[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] base: correct view branding tests after c013210

[MERGE] forward port of branch saas-4 up to ddef2dd

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

[MERGE] forward port of branch 7.0 up to d0a0b7d

[FIX] web: correct /web/proxy/load to set correct base_url

[FIX] website translation bug

Sometimes a node can't be translated using the website Translate
mode. The translation is added to the Application term list but
the id of the view is not correct.

This happen when a translatable node is a children of a inherited
node whose branding could not be kept. data-oe-source-id was left
over because it was not registered in MOVABLE_BRANDING

[FIX] website_forum: portal user are allowed to delete forum post

If a portal user has engouh karma, this is possible for him, at least graphically, to delete forum posts, even if is not the author of it.
So, he should be allowed to delete forum posts in the ACL

[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

[IMP] product: filter and group by category_id on product.template

Was removed (for unknown reason) during refactoring of product and template (opw 611699)

[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] account_anglo_saxon: variable 'standard_price' referenced before assignment

Due to forward-port, the variable standard_price was reintroduced instead of valuation_price_unit
Move the change of currency after the fifo/lifo computation.
Fixes #1801

[FIX] web: ignored readonly attribute on image widget

[FIX] web: 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).
Replaced by auto, the default value adviced by CLEditor.

[IMP] product: specify a view for 'Variant Values' action

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

[FIX] sale: define sale_note field before using it as a default value for sale.order

When uninstalling/installing sale module, this can be problematic

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

[FIX] account_analytic_analysis: quantity overdue on quantity exceeded, not reached

[FIX] website_twitter: IE does not accept twitter date format

Datetime like Tue Jul 13 23:18:36 +0000 2010 is not accepted by IE and return "Invalid date".
Therefore, We parse the date to a format known by IE (and other browsers)
Trick took from http://stackoverflow.com/questions/3243546/problem-with-javascript-date-function-in-ie-7-returns-nan

[FIX] website_customer: the extra / for url pager is not need in saas-4

[FIX] product: make sure unlinked product still exists

When uninstalling/updating a module, we may execute unlink method on product.template before product.product. In such cases, the product is already removed after removeing the template (_inherits) and the chained unlink of the product would fail (traceback when browsing).

[FIX] point_of_sale: field date of report.pos.order should be a datetime

The date_order field of the pos.order is a datetime, not a date
As, in report.pos.order, the date field is related to date_order of the pos.order

We usually do not commit fixes altering the model fields structure in 7.0, but this one is retro-compatible, as the database structure won't change

[FIX] web: do not focus if the focus_field is not a field

Sometimes, focus_field could be a button, and in such cases, we should not focus on it

opw-612045

[FIX] stock: inventory lines with no production lot: compare with correct stock level

Inventory lines without production lot number should
be compared with the stock level of products without
a production lot. Otherwise the final result of a
validated inventory is wrong as soon as there are
lines with and without production lot for the same
product.

Adds corresponding regression test.

Fixes https://bugs.launchpad.net/openobject-addons/+bug/1008099
Manual rebase of #1658, courtesy of Numerigraphe

[FIX] account: reconcile: no useless revalidation

Avoid revalidating the complete account moves
that contain the lines being reconciled.
The reconciliation does not change the validity
of those moves anyway.
This represents a very important speed up of
reconciliation when moves with several hundred
lines are involved.

[FIX] Fixed the issue of calendar view reload events inconsistency, when the slow create popup creates event through any form button which is kind of object, action or workflow at that time events are reloaded but still button action is not called like workflow is not triggered and hence state of the event is still not changed and due to domain newly created event is filtered, note that events are refetched when create_completed is triggered and create_completed is triggered right after creation of record, but there are case where some form button changes state of the record and it requires to refetched for example we have scenario in mrp order planning where Confirm Production creates event but before workflow is triggered and state of the event is changed, events are reloaded and newly created event will not returned because of domain.

[MERGE] Forward-port saas-4 up to 9a84d55

[MERGE] Forward-port saas-3 up to 9b55532

[FIX] export: in saas-3, req becomes request.

Error during forward port b9275da8a52255d0fc3f914de9d406612e7dde50