Commit Graph

76620 Commits

Author SHA1 Message Date
Stéphane Bidoul e11490c3a1 [FIX] account: Analytic account domain in reconcile with writeoff wizard
The domain for the analytic account in the `reconcile with writeoff` wizard
should be based on the `type` field, which must be `view`,
not on the `parent_id` field, as it's done everywhere else
(e.g. in the supplier invoice form).

`[('parent_id', '!=', False)]`
and
`[('type', '!=', 'view')]`
is almost the same, but the second domain is more appropriate.

Closes #4562
2015-09-30 16:22:05 +02:00
Denis Ledoux 4c901f8ac6 [FIX] sale: use default_get to define the invoice journal
When creating the invoice of a sales order, from the sales order
(`Create invoice` button on the sale order),
the journal used for the invoice was forced
with a specific domain. Besides, the only reason the journal
is forced is to check there is a sale journal for the
quotation company, and raise a warning if not.
This check was added in 1578c2858d.

This prevented to use the user defined defaults,
(`Set defaults` in the `debug` menu)
to set a different default journal per user.

Using the `default_get` instead solves this issue,
as it uses first the user defined defaults. Besides,
if no user defined defaults are set, it then uses
the default value set in the field definition, which
in this case returns the same journal then the forced
domain mentioned above, the domain used being the same.

There is therefore no change of behavior, while giving
the possibility to use the user defined defaults.

Fixes #8786
2015-09-30 15:59:54 +02:00
Miku Laitinen 9a3b711989 [FIX] payment_transfer: 'Wire Transfer' provider translatable
Closes #8398
2015-09-30 15:21:20 +02:00
Goffin Simon 48ccb19937 [FIX] sale: currency in report sale analysis
In Sale Report Analysis, the price total must be in the currency
of the company.
Introduced by b5c1cb4298
opw:649781
2015-09-29 16:43:47 +02:00
Stefan Rijnhart 0333e4da02 [FIX] product: default category on UoM quick creation
This revision adds the possibility to use the key
`default_category_id` in the context to set
the UoM category when quick creating a new UoM,
like it's the case everywhere else with the
`default_*` keys passed in the context.

Closes #4407
2015-09-29 16:24:22 +02:00
Jairo Llopis 68eaeb684d [ADD] crm: test user notification
Test if the user gets notified for incoming contact requests.
2015-09-29 15:27:46 +02:00
Jairo Llopis ffa28488be [FIX] mail: subscribe to threads before sending first message
Users subscribed to creation of records (new leads in a sales team, new
tasks in a project, etc.) are never notified. The reason is that users
were subscribed after the record creation notification.

Introduced in 43915a8721
Closes #8723
2015-09-29 15:27:46 +02:00
Denis Ledoux 25d8955e98 [FIX] payment: make the payment.acquirer name translatable
To allow to translate "Wire transfer" in the ecommerce.

opw-650450
2015-09-29 15:09:23 +02:00
Denis Ledoux 2d3e9d21a7 [FIX] website_sale: ensure payment transaction is for current order
When ordering on the ecommerce,
if a payment transaction was found in the session,
this transaction was used as transaction for the current order.

Nevertheless, if the transction is no longer linked to
the current order, we should not use it.

This happened, for example, when the quotation
was deleted while the customer/user didn't close
its browser, and the transaction id was therefore
still in its session.

opw-650417
2015-09-29 12:10:04 +02:00
Jos De Graeve c88f0d926c [FIX] sale: company filtered taxes for fix_tax_included_price
This revision is related to:
 - 503820acb6
 - 3b02e3d63d

The taxes used to define the price must be the product taxes
of the company of the order.

Closes #8759
2015-09-29 10:40:33 +02:00
Goffin Simon 06fbf3a572 [FIX] website_quote: price unit for suggested products
In a SO, the unit price of the suggested products must be computed with
the pricelist linked to this SO. Inspired from function 'product_id_change'
in model 'sale.order.line'.

opw:650168
2015-09-29 09:59:53 +02:00
Nicolas Martinelli e11f50599d [FIX] web_graph: duplicates header in pivot
For report with duplicated named headers, several columns could get
squashed together.

This fix check if a value is a duplicate or not, and if it is a
duplicates postfix an number allowing to differentiate the two headers
(for the human and the javascript).

This is the exact same solution than #6722
Backport of 5cff8fbb7c70a372cb5116c2fb98fd622c4d4358
opw-650000
2015-09-29 08:33:36 +02:00
Denis Ledoux fa17b86a9f [FIX] project: multi-company security rules in Tasks Analysis
This change avoid to display the tasks
that a user is not permitted to see
in the reporting view 'Task analysis'

Closes #4399

Courtesy of jkei
https://github.com/jkei
2015-09-28 15:58:15 +02:00
Arnaud Pineux 44bfb2bf44 [FIX] base_import_module: special chars in error messages
use ustr() instead of str() to avoid traceback with translations

Closes #4272
2015-09-28 15:21:24 +02:00
Stefan Rijnhart 5f79cbd029 [FIX] mrp: display lot_id in production wizard if full tracking
Show lot_id on production wizard
if product is fully tracked
(but not marked specifically for production tracking

Closes #4223
2015-09-28 14:19:57 +02:00
Xavier Morel a2388c762e [FIX] gamification: incorrect format breaking challenge acceptation
Reported by @yelizariev

closes #8737
2015-09-28 11:37:46 +02:00
Odoo Translation Bot c764b703ca [I18N] Update translation terms from Transifex 2015-09-27 01:58:59 +02:00
Christophe Simonis 7d68fa0ee6 [MERGE] forward port of branch saas-3 up to 504a04c 2015-09-25 15:03:12 +02:00
Christophe Simonis 504a04c823 [MERGE] forward port of branch 7.0 up to 117b636 2015-09-25 14:56:58 +02:00
Christophe Simonis 117b636d3f [FIX] mail: catch database errors when sending mails 2015-09-25 14:43:00 +02:00
Alexis de Lattre fc11b58239 [FIX] stock: service lines duplication when grouping DO invoices
This is possible that changes happen
during the loop in the multiple pickings:
an update in a picking could update another
picking. The browse must therefore be done
inside the loop to update the pickings with
the latest changes.

Fixes #4201
2015-09-25 12:25:52 +02:00
Pierre Verkest be33912a25 [FIX] report_webkit: get webkit path as SUPERUSER_ID
`ir.config_parameter` is readable by employees only.
It could happen to print a webkit report
as a portal / public user.

Closes #4181
2015-09-24 18:11:42 +02:00
Laetitia Gangloff 34f79f3de7 [FIX] purchase_double_validation: do not update limit on module update
In the module `purchase_double_validation`,
you can change the limit to require a second approval
in the purchase settings.

If the module was updated, the limit was re-set to
its default value.

Closes #4183
2015-09-24 17:59:52 +02:00
Denis Ledoux d24ff706a9 [FIX] account_followup: check_description constraint must take into account the translation
When adding a `%` in the followup letter content,
the constraint checking there is no syntax error
is supposed to raise a warning, telling '%' must
be set as `%%` when you want to display this percentage
char.

This constraint did not take into account the translation,
so when being in another language (e.g. French),
and adding a `%`, the warning was not raised, while
it should have been.

Closes #8680
2015-09-24 17:28:00 +02:00
Laetitia Gangloff 273114204f [FIX] note: `Use fancy note` was re-checked on update
Set security group as not updatable. To be able to not use fancy note

Closes #4178
2015-09-24 16:41:10 +02:00
Goffin Simon ddd2a3e97c [FIX] account: prepare_move_lines_for_reconciliation_widget
In the case where target_currency == company_currency, the actual_debit or the
actual_credit must be equal to the amount_residual defined on the account.move.line
and expressed in the currency of the company. The actual_debit/actual_credit must
take into account the possible partial reconciliations and this is done by the
residual amount.

opw:648744
2015-09-24 12:54:49 +02:00
Christophe Simonis 0cddf87a77 [FIX] {website_,}membership: membership should start the same day of the invoice 2015-09-23 17:15:01 +02:00
Denis Ledoux b1bd0c6cfa [FIX] account_cancel: allow cancel of whole bank statement
This revision is related to 6f355623f0.

The above revision had as goal to not allow the partial cancellation
(line per line) of a confirmed bank statement, as it would
prevent the ability to reconcile the statement, as reconcilling
a confirmed statement it not possible.

Nevertheless, cancelling the whole bank statement should be allowed
(using the `Cancel Statement` button), as this cancel all lines
and re-set the statement in draft.

opw-649864
fixes #8568
2015-09-23 15:38:04 +02:00
Martin Trigaux 19f9328d0b [IMP] l10n_cr: outdated URL
Costa Rica localisation team has moved to GitHub
cf #8651
2015-09-23 14:46:42 +02:00
Goffin Simon bdcae86d67 [FIX] purchase: invoiced purchase order line
When invoice_method == 'picking', a po line is invoiced if all the lines
linked to the po line are in invoices with state not in ['draft', 'cancel'].
To keep the same logic than before commit 9b1ab76.

opw:649746
2015-09-23 14:42:23 +02:00
Martin Trigaux 9e53f04332 [FIX] l10n_hu: missing flag 2015-09-23 10:46:59 +02:00
Christophe Simonis d05ee26a72 [MERGE] forward port of branch saas-3 up to 24bcdb7 2015-09-22 15:40:28 +02:00
Christophe Simonis 24bcdb75fb [MERGE] forward port of branch 7.0 up to c2aff47 2015-09-22 15:30:22 +02:00
Goffin Simon 1bcc5b5797 X] hr_evaluation: _rec_name
To avoid warning when making a name_search on "hr_evaluation.evaluation".
2015-09-22 13:40:26 +02:00
Goffin Simon 14b060ed71 [FIX] purchase: journal_id in auto procurement
The journal_id used in a purchase.order created in auto procurement
(by the function make_po) must be the journal_id of the company_id
set on the procurement. The company_id set on the procurement is
the company_id set on the SO (function _prepare_order_line_procurement).

When creating the PO in function make_po, the company_id must be set in the
context to use the right journal_id with the function _get_journal in
"purchase.order" model.

opw:648600
2015-09-22 13:34:45 +02:00
eLBati 16fda32a0e [FIX] l10n_it: tax code for Iva al 10% indetraibile
The tax code of `Iva al 10% indetraibile (D)`
is `template_ivacode_pagata_10ind`, like its
sibbling `Iva al 10% indetraibile (I)`.

The fact it wasn't is due to a typo.

Something similar is done with the taxes
`Iva al 20% indetraibile (D)`
and
`Iva al 20% indetraibile (I)`
with the tax code `template_ivacode_pagata_20ind`

opw-649557
Closes #8047
2015-09-21 17:35:01 +02:00
Ruchir Shukla 61300ee7f1 [FIX] stock_landed_costs: modify quants as admin
Nobody has write access to stock.quants so only the admin was able to execute
this method.
Fix it by using SUPERUSER_ID
Closes #8544
2015-09-21 16:40:34 +02:00
Denis Ledoux aabbdc7339 [FIX] web: Safari, download of attachments with accents
The structure `filename*=UTF-8` works in Safari, from
release 6.2 (which is set as version 537 in the user_agent version).

This structure has the advantage to handle the UTF-8 encoding,
and therefore special characters, such as accents.

Therefore, from this Safari release, we use the same format
than other browser. For older releases, we use the format
`filename=`, without UTF-8, ASCII encoded, replacing
special characters by `?`.

opw-649863
2015-09-21 15:30:05 +02:00
Adrien Peiffer (ACSONE) 0b4bd7a6d1 [FIX] mail: allow to delete mail alias all employees
It was allowed for all employees to read, create and edit
mail aliases. Only the deletion was prevented.
Nevertheless, giving the possibility to rename a mail alias
is allmost seen as a deletion, as you can rename it to something
that just won't be used anymore. Therefore, we can consider
to give any employees the rights to delete mail aliases.

Besides, not allowing the unlink leads to issues when the
mail alias is associated to a record the user wants to delete.
He was able to create the record, and its mail aliases, but
he could not remove the record, as he was not allowed to remove
the mail alias.
For instance, an HR officer was able to create a job position,
with its mail alias, but couldn't remove the job position he created.

Closes #8466
2015-09-21 14:26:37 +02:00
Wolfgang Taferner 4f41c3327c [FIX] website_sale: make checkout fields customizable
[FIX] Valuable checkout hooks for website_sale
[FIX] Values might need to be changed for different field mappings
[FIX] Make definition of form fields inheritable within a function
[FIX] Make shipping_info values available for inherited module manipulation

Close #8490
2015-09-18 21:17:51 +02:00
Olivier Dony dbddc8246a [FIX] payment_ogone: move debug logging to the level it belongs to (may include sensitive data) 2015-09-18 19:52:26 +02:00
Olivier Dony 0f06e5f54f [FIX] tests: outdated resource paths in manual test files 2015-09-18 15:52:11 +02:00
Goffin Simon 75233860a2 [FIX] event: confirmation_event and confirmation_registration
-The lang of the partner linked to the "event_registration" record
must be used to send the registration/confirmation email.

-strftime cannot be used on a string.

opw:649510
2015-09-18 10:17:02 +02:00
Nicolas Lempereur 3c39b62bce [FIX] website: escaping and external iframe
jQuery has a special behaviour when using .contents() over an iframe
object. This caused an error for escaping when saving the page with an
iframe content of an external domain.

introduced by 8c77c711
opw-649570
2015-09-18 09:41:32 +02:00
Martin Trigaux e1163691e7 [IMP] l10n_sa: remove sequence on manifest
No reason to get Saudi Arabia CoA as the first module, use default sequence
2015-09-17 18:03:49 +02:00
Valencia Rodrigues Sah c2aff4772e [FIX] product: default value on required field
The qty fields has become computed in 6.1, and the value set by the user is on
min_qty. Set default value as it is required.

Closes #8561
2015-09-17 17:45:20 +02:00
Denis Ledoux 90dfaa7add [FIX] delivery: add the delivery after all order lines in SO
The delivery line in the SO was added among other lines,
in second most of the time, due to the fact
the sequence of the line was set after
all other order lines sequences.

opw-649629
2015-09-17 12:47:34 +02:00
Joren Van Onder a11a490d30 [IMP] hw_escpos: support Star TSP650II
In order for the printer to work we need to configure it in USB vendor
mode. This means it will not have a USB interfaceClass of 7. So when
we don't find any USB device that identifies as a printer, we search
for Star devices and use the first one of those we find.

This should be reasonably safe because Star does not manufacture other
devices apart from printers that one would attach to the posbox.
2015-09-17 09:26:33 +02:00
Joren Van Onder ad727be267 [FIX] hw_escpos: don't try too hard to close a printer
Although it might be a good idea to make sure a printer device is
properly closed, trying for >1 min to do so is too much. It can cause a
>1 min delay between receiving a receipt and actually printing. Also,
just because the device is not cleanly closed does not mean that it will
not work when we open it again.
2015-09-17 09:26:33 +02:00
Joren Van Onder 1440dc1c96 [IMP] hw_escpos: don't replace a USBError exception with some string
There's absolutely no point to hiding exceptions like this, it makes
debugging more difficult than it has to be.
2015-09-17 09:26:33 +02:00