Merge pull request #611 from odoo-dev/saas-3-really-fix-session-expired-fme

[MERGE]  http routing: fix handle_exception logic and _authenticate exceptions

* Better separate JsonRequest and HttpRequest handling
for exceptions, so each type of request handles exception
the right way. Previously HttpRequest would supersede
JsonRequest in some cases and generate pure HTML
responses where a JSON result was expected.
* Also ensure that ir.http._authenticate() only raises
two possible exception types, hiding any other kind
of internal errors:
 - openerp.exceptions.AccessDenied
 - openerp.http.SessionExpiredException

Merge pull request #325 from odoo-dev/saas-3-opw-608186-acl

[FIX] sale: cannot filter on paid = False (opw 608186)

[MERGE] forward port of branch 7.0 up to e5533d0

Merge pull request #654 from nel-odoo/patch-3

Production : Use the source location of the MO instead of the one of the BOM

[FIX] web: focused field in view editable list

When adding an item to the editable list, the focused field was no longer the first visible field
This is related to revision 4a508885ac043b25465aa5741955082aa66ce949
visible_columns list is not ordered

Production : Source location of the BOM

Use the source location defined on the current MO's routing not directly on the BOM

[FIX] website: correct selector

The selector to toggle the theme was triggered too often (on almost each button) but had no effect because no view-id was present.
Patch fix #627

Merge pull request #648 from odoo-dev/7.0-fix-searchbar-navigation-ged

[FIX] fixes searchbar navigation (web client)

[FIX] Fix attach icon that was under the upload label.

[FIX] fixes searchbar navigation (web client)

the code handling the keydown events was moved, but the variable this
was not adjusted accordingly, resulting in a broken navigation.

It is now possible to press LEFT and RIGHT again to move the focus
between facets.

[FIX] web_calendar: check for edit access rights + use correct display mode (view or edit) when going into a document form view from the calendar.

Merge pull request #138 from odoo-dev/saas-3-bug_1307341-ptr

[FIX] kanban: ie9: fixed the display of 'show more' button

Fix wrong partner_id refs in Project demo data

Removed widge=selection from account_id field in line in Purchase Receipt, as it only displays limited records(100) from the limit set in the name_search

Merge pull request #616 from odoo-dev/7.0-opw-608485-rgo

[FIX] l10n_be_invoice_bba: random generation do not use sequence, so should be 10 char long during padding

git commit -a -mm

Bring back exception conversion to Forbidden

[FIX] auth_crypt: encrypt all passwords at installation

When `base_crypt` was updated for v7, the auto-encryption
at installation was dropped, with user passwords only
encrypted on-demand whenever the user would connect.

It is important to encrypt all passwords immediately to
prevent password compromission for user who do not
login often or even for deactivated users who are not
allowed to login anymore.

Fixes https://bugs.launchpad.net/openobject-addons/+bug/1280152

Based on LP merge proposal by Nicolas Bessi (Camptocamp):
https://code.launchpad.net/~camptocamp/openobject-addons/improve_auth_crypt_3_please_launchpad_work-nbi/+merge/206476

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

Changed as per the recommandation of Sir Odo

[FIX] l10n_be_invoice_bba: Fixed the length of random generated code to ensure it generates 10 digit code so that it allows validation of invoice. (Maintenance Case: 608485)

[FIX] web: allow using dates and datetimes without leading zeros

[FIX] Session Expired message in backend

[IMP] Ir Action Server - Fix the domain for the field 'Link Using field'.

[FIX] GAMIFICATION: Fixed the Attribute Error
    ' 'str' object has no attribute 'strftime' '

[FIX] web: save date(time) field value on key enter in editable list

In editable list, on keypress enter, the _next method is called, saving the current line and starting the edition of the next one
The _next is triggered before the date(time) field change event, and, therefore, the saved value of the date(time) field is the old one

[FIX] auth_oauth: url_unquote_plus does not handle boolean

If the redirect param 'r' is not in the state, do no try to unquote it

[FIX] auth_oauth: unquote redirect url from state

This is related to 2db6a0080f9eeac7030d6d2d856574f2192d8aa5. Looks like Firefox auto unquote, but not Chrome

[FIX] auth_oauth: quote redirect url in state

Otherwise, Firefox is splitting the state at the first &, which can potentially be located in the redict param of the state

[FIX] mail: missing .id during forward port 3c0292645fcb9604e3c4f08a37b7e8702d464065

[FIX] account_analytic_analysis: product_id_change, variable price referenced before assignment in some cases

[MERGE] Forward-port of 7.0 bugfixes up to 63ea0df73ffd6bf659b2ef29cdc3d3387bec92ac

[FIX] account: default domain on journal_id field for invoices

Prior to this fix, the domain was set in the onchange methods onchange_company_id. Therefore, if the onchange was not triggered, the domain wasnt apply (e.g. while editing an existing invoice).

[IMP] Website Sale - ribbon - html class from product.style is not required, so we need to display it as empty in website to avoid an internal 500

[IMP] Calendar - Add an explicit /web in url from invitation mail since it required to works

[FIX] Google Calendar - Split the last underscore because Google uses sometimes '_' in their own google_id. Retrieve the parent event_id to exclude when the exlusion has been done in google

Merge pull request #137 from odoo-dev/saas-3-bug_1312046-ptr

[FIX] event: avoid superposition of date and text in grouped kanban view

[FIX] event: improve name from previous merge

Merge pull request #111 from odoo-dev/7.0-bug_1287200-ptr

[FIX] event: change the field nbevent to make sense

The field nbevent was representing the number of registration with value of 1 for events without registrations (which makes no business sense). Fix to actually represent the number of registrations and display a better field name.

Merge pull request #336 from odoo-dev/7.0-opw-608606-fka

[FIX] account_report_company: display_name field is not computed for non-active partners

Merge pull request #471 from odoo-dev/7.0-opw-606994-rgo

[MERGE] mail: Speed improvement, backport of rev ed116d6b6b5ab5e4436cce5b6da078417e0a9efc

[FIX] hr_holidays: date_from, date_to required in form view only for leave requests, not for allocation requests

[FIX] hr: working and home adresses should not be customers by default, only 'normal' adresses.

[FIX] hr_holidays: display float for leaves remaining / leaves taken in holiday type name get

[FIX] web: view_list_editable, on add an item, focus on the first cell of new row

Before, the focus was set on the first row, and, thererefore, if the list was enough long, it jumped out and the actual new row was hidden

[FIX] portal: typos in portal wall actions.

[FIX] hr: typo in a field definition

Merge pull request #506 from odoo-dev/7.0-read-duplicate-fields-mat

[MERGE] orm: avoids error when reading twice a field

[FIX] orm: avoir errors reading twice a field

_read_flat: remove duplicated fields in read call
get many2one: as False is instance of int, check the value of x first to avoid calling a name_get with a list of False

When we were reading twice a m2o field where at least one result is null, the first call to name_get would set the value to False instead of None and then accepted by the filter 'isinstance(x, (int,long))'

[MERGE] Forward-port of 7.0 bugfixes up to 4d3a5df

parse_value: interger != float

Wwhen parsing a integer field, do not accept float values. '1' or '1.0' is ok but not '1.1'. (opw 608544)

[FIX] Account, l10nbe - Respect name_search operator for account.account, so = really work as expected.

Also fix data file in l10nbe which was relying on the old implementation.

[FIX] l10n-be data - Be more accurate in the belgium data. All accounts have by default a right padding of 0 to be on 6 char, so the csv need to have the accounts on 6 char since with have removed the default replacement of "=" by "=like" to allow the manual import

[FIX] Import - Use the right comparator to allow import with accounts_ids

[FIX] sale_order: set state field track_visibility

sale_order module override state field of sale.oder (sale.sale.py) and forgot to reset the track_visibility, therefore sale orders state changes are no longer tracked once this module installed.

[FIX] base_action_rule: automatic commit during background processing of rules

This will make sure already processed rules are persisted
in case a timeout occurs while processing the next ones,
ensuring that *some* progress can be made in most cases.

[FIX] base_action_rule: prevent recursive rule trigger also when processing time-based rules

A safety guard already in place prevents the recursive
trigger of rules while another rule is being executed,
as a simple measure to prevent endless loops.
This guard was missing when processing time-based rules.

[FIX] base_action_rule: do not repeatedly trigger already-executed rules based on time condition

A programming error introduced at rco@openerp.com-20121220142445-emzzvhlw400q37c9
fails to properly check when a rule has already been executed in the past.
Time-based rules should only be executed if the trigger date is in the past and:
 - either they never executed before
 - or the last execution date is older than the trigger date

[FIX] base_action_rule: do not force registry reload when writing on a rule unless really needed

Executing a rule always updates the `last_run` value, which would
always force a useless registry reload to occur. Only signal the
change if the rule applies to a model that was not previoulsy
monitored for base action rules.

Merge pull request #417 from gurneyalex/7.0-fix_1311004_account_move_line_index-afe

[IMP] account.move.line: extra index on (date, id) corresponding to the default _order, courtesy of @gurneyalex.

 Fixes https://bugs.launchpad.net/openobject-addons/+bug/1311004

Merge pull request #413 from gurneyalex/7.0-fix_lp1025703-afe

[FIX] delivery: add missing information in chained pickings

Merge pull request #411 from acsone/lp-1200611_lga

[FIX] hr_timesheet_sheet: correctly set the employee on the attendance

[FIX] account: move translatable tag

The field note on account.fiscal.position and account.fiscal.position.template should not be translatable in classic modules.
The l10n_multilang module is intendend to make chart of accounts multilang and is the place to set translate=True

[MERGE] forward port of branch 7.0 up to 100eba8

[FIX] fields.py - avoid dict comprehension inside a dict comprehension

Merge pull request #415 from gurneyalex/7.0-fix_lp1192115-595240-opw-skh

[FIX] delivery: do not remove the delivery method on the sale.order when adding it as an order line (useful information)

[FIX] account_voucher: better pay invoice wizard

If the writeoff_amount is above 0, the fields writeoff_acc_id can be displayed and is required depending of the payment option. If the writeoff_amount is reset to 0, this field should no longer be required.

gamification: when changing the domain on a goal definition give a simple check to avoid getting a traceback when the challenge is reloaded

[MERGE] forward port of branch 7.0 up to 6fdb783

Merge pull request #56 from yvaucher/7.0-fix-1319109

[FIX] report_webkit: avoid sharing parser instance globally and make it thread safe

[FIX] mail: Speep improvement when sending email using 'Send by Email' wizard. (Maintenance Case: 606994)

[IMP] add an index on account_move_line(date, id)

fixes https://launchpad.net/bugs/1311004

[FIX]Delivery: carrier information not propagated from sale order to pickings (Case: ref 595240)

closes lp:1192115

[FIX] delivery: propagate additional fields in chained pickings

closes #367

Merge pull request #404 from odoo-dev/7.0-norecomputestoredm2ofuncfields-chs

[FIX] orm: do not recompute m2o stored function field at read.

hr_timesheet_sheet: for attendance, set default value for employee_id in the context

[FIX] orm: do not recompute m2o stored function field at read.

The get() method of m2o function fields is used for 2 different things:
 - call the function defining the m2o
 - get the name_get representation of the value

Until this pathc, only the first case was handled, resulting to a useless
recomputation of the field when reading it.

[FIX] orm: do not resize unlimited char fields

[FIX] ir_cron: use the same search criteria as when listing job to excecyte when acquiring the ock on the job before its execution to prevent running already executed job

Backport of fix made in trunk

[FIX] website: on adding a slide to slider, new slide is last on save

Before, the new slide was set as first slide
Moreover, the indicators were not correct neither

[IMP] stock: when checking the state of the product, do not recompute the quantity on every move but only the one currently browsing (optiomisation)

[FIX] website_event_track: remove trailing comma in speakers name in event tracks

[FIX] document_page: allow other users than administrator to create menu to a static page

User should be at least have Configuration settings group to be allowed to create actions. Basic document page users do not have this rights, therefore we temporary give superuser_id to create the action

This is related to rev 133c5033a092845e462b894c0d7739191480c709

[FIX] project_timesheet: don't trigger write if there is nothing to write

Triggering an empty write could break constraint (can not update past timesheet) (opw 606963)

[FIX] web: many2many field handle "no_create" option

uninstall: avoid errors during uninstall

When uninstalling a module, remove the ir.model.constraint after removing the non-model records and before fields and model definition.
Without this fix, some constraint would be removed too early allowing to have broken relations and data left from removed module.

[FIX] account_analytic_analysis: Only allow to select sale pricelist for contracts

[FIX] account_analytic_analysis: Recurring invoice lines price if pricelist does not match

If the price according to price list is False, use default sale price in the recurring invoice lines

[FIX] website: prevent to split google analytics script

[FIX] point_of_sale: correct call to default value

The default value should be called with a lambda otherwise the payment_date is set to the date the server was started (opw 606715)

[FIX] kanban: avoid wrong placement of button on ie9

On ie9, the 'read more' button in kanban view was placed in the middle of the window instead of the bottom

[FIX] stock: use correct vals in grouped invoice (opw 606535)

When creating a grouped invoice, the invoice_vals value is used to create the invoice line.
The value was not reset for grouped invoice and we reused the values of previous line.

[MERGE] forward port of branch 7.0 up to 0c4bc1c

[FIX] sale: keep customer reference in grouped invoice (opw 606523)

[FIX] delivery: allow to add delivery method in quotation if quotation in quotation sent stage

opw-607920

[FIX] sale: cannot filter on paid = False

[FIX] display_name is not compute for non-active partner

[FIX] event_sale: stored seats_max field

seats_max is a stored field in event, and should remain stored when overrided in event_sale

[FIX] calendar: avoid double popup, opw 606297

Some browsers (e.g. chrome) trigger onEmptyClick as well as
onBeforeLightbox during drag&drop which calls two slow_create
calls (and two popups). Workaround to kill the second one.

Merge pull request #286 from jbq/bugfix

[FIX] mail: detection of MIME type

When parsing the mail headers, the content-type may has a 'type' attribute for Multipart/Related  objects (rfc2387).
Previous check would match on attached files of type text instead of real `content-type: text/`.